[wp-trac] [WordPress Trac] #31050: Better PDF Upload Management

[WordPress Trac]

#31050: Better PDF Upload Management
-------------------------------------------------+-------------------------
 Reporter:  celloexpressions                     |       Owner:
     Type:  feature request                      |  mikeschroder
 Priority:  normal                               |      Status:  assigned
Component:  Media                                |   Milestone:  4.7
 Severity:  normal                               |     Version:
 Keywords:  has-patch needs-unit-tests needs-    |  Resolution:
  testing                                        |     Focuses:  ui
-------------------------------------------------+-------------------------

Comment (by mikeschroder):

 There were some good details in the Slack conversation above.

 A few notes:
 - I've got a workaround working where we generate the thumbs with the
 initially created render, which fixes the `multi_resize`/black thumbnail
 problem, and allows us to create whatever other sizes we think best.
 - Imagick supports `readImage()` back to at least 2.0.0 alpha (WP supports
 2.2.0+).
 - @joemcgill notes that he'd like to see a different/smaller size used in
 the gallery, and I entirely agree.
 - It's an option to only commit the thumbs, and not the Attachment Details
 view, which would allow WP to only generate lower quality images (it's
 faster to render PDFs at a lower DPI)
 - @joemcgill notes that he'd like to explore the post thumbnail option
 before this gets committed (I'm going to dig into what this would mean,
 and connect with him and @markoheijnen about it)
 - @joemcgill notes that we'll want to address the issue of some PDFs
 timing out that wouldn't have previously, and had suggested a size limit
 for the thumbnail generation. I noted that a lower DPI would also help
 with this -- especially for image intensive PDFs.
 - @joemcgill would prefer if we didn’t have to hack around
 `image_downsize()`, which he thinks would be possible if we didn't need to
 read `$meta['sizes']` from PDF attachments.

 If I missed anything, please feel free to add additional comments :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31050#comment:82>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform