WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Last modified 3 months ago

#39630 assigned defect (bug)

PDF Thumbnails in Media Library Don't Fall Back to Full Size

Reported by: daleharrison's profile daleharrison Owned by: antpb's profile antpb
Milestone: 6.3 Priority: normal
Severity: normal Version: 4.7
Component: Media Keywords: needs-patch
Focuses: ui, administration Cc:

Description

When previewing uploaded files in the Media Library, image thumbnails fall back to displaying the full size image when the option medium_size_h is set to 0 (the 0 setting is used to stop WordPress generating thumbs at that size).

The same functionality does not work for uploaded PDFs, instead displaying wp-includes/images/media/document.png in their place. The full-size thumbnail is generated, but not used.

The Attachment Details modal window successfully falls back to the full size image when previewing a PDF under the same circumstances.

Attachments (1)

Screen Shot 2017-01-18 at 12.47.17 PM.png​ (92.7 KB) - added by daleharrison 6 years ago.
Example of PDF Thumbnail Compared to Image Thumbnails in Media Library

Download all attachments as: .zip

Change History (12)

@daleharrison
6 years ago

Example of PDF Thumbnail Compared to Image Thumbnails in Media Library

This ticket was mentioned in ​Slack in #core-media by joemcgill. ​View the logs.
6 years ago

This ticket was mentioned in ​Slack in #core-media by joedolson. ​View the logs.
8 months ago

#3 @joedolson
8 months ago

  • Milestone changed from Awaiting Review to 6.2
  • Owner set to joedolson
  • Status changed from new to accepted

#4 @antpb
8 months ago

This was discussed in a recent Media component meeting and we suspect that the fallback just needs one more step between medium and default pdf icon to include the full thumb after no medium found.

This ticket was mentioned in ​Slack in #core-media by joedolson. ​View the logs.
7 months ago

#6 @antpb
7 months ago

  • Owner changed from joedolson to antpb
  • Status changed from accepted to assigned

This ticket was mentioned in ​Slack in #core by mukeshpanchal27. ​View the logs.
3 months ago

#8 @mukesh27
3 months ago

This ticket was discussed during the recent bug scrub. It looks like it's unlikely that work will be done on this during the 6.2 cycle.

@antpb Is this still possible to land in 6.2, or should it be moved to Future Release for now?

#9 @mukesh27
3 months ago

  • Keywords needs-patch added

This ticket was mentioned in ​Slack in #core-media by antpb. ​View the logs.
3 months ago

#11 @joedolson
3 months ago

  • Milestone changed from 6.2 to 6.3

Moving this to 6.3; @antpb hasn't had the time to tackle it during this cycle, but we still think this should be fixed soon.

Note: See TracTickets for help on using tickets.