Page MenuHomePhabricator

Add a PDF document "rendering" engine
ClosedPublic

Authored by epriestley on Mar 23 2018, 12:10 PM.
Tags
None
Referenced Files
F12829702: D19252.id.diff
Thu, Mar 28, 11:41 AM
Unknown Object (File)
Tue, Mar 5, 10:56 AM
Unknown Object (File)
Fri, Mar 1, 3:42 AM
Unknown Object (File)
Feb 21 2024, 11:30 AM
Unknown Object (File)
Feb 16 2024, 9:38 AM
Unknown Object (File)
Feb 3 2024, 6:34 PM
Unknown Object (File)
Feb 3 2024, 6:34 PM
Unknown Object (File)
Feb 3 2024, 6:33 PM
Subscribers
None

Details

Summary

Depends on D19251. Ref T13105. This adds rendering engine support for PDFs.

It doesn't actually render them, it just renders a link which you can click to view them in a new window. This is much easier than actually rendering them inline and at least 95% as good most of the time (and probably more-than-100%-as-good some of the time).

This makes PDF a viewable MIME type by default and adds a narrow CSP exception for it. See also T13112.

Test Plan
  • Viewed PDFs in Files, got a link to view them in a new tab.
  • Clicked the link in Safari, Chrome, and Firefox; got inline PDFs.
  • Verified primary CSP is still object-src 'none' with curl ....
  • Interacted with the vanilla lightbox element to check that it still works.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 23 2018, 2:14 PM
This revision was automatically updated to reflect the committed changes.