Make WordPress Core

Opened 8 years ago

Last modified 5 years ago

#35092 reviewing defect (bug)

Remove crop check in wp_prepare_attachment_for_js() for sizes

Reported by: markoheijnen's profile markoheijnen Owned by: joemcgill's profile joemcgill
Milestone: Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: has-patch reporter-feedback
Focuses: Cc:


Currently only cropped images generated on the fly are displayed. This due to a ! $downsize[3] check. This change was made in #22598 and with the patch supplied I deleted this check. then medium/large does show up.

Attachments (1)

35092.diff (516 bytes) - added by markoheijnen 8 years ago.

Download all attachments as: .zip

Change History (2)

8 years ago

#1 @joemcgill
8 years ago

  • Keywords has-patch reporter-feedback added
  • Milestone changed from Awaiting Review to Future Release
  • Owner set to joemcgill
  • Status changed from new to reviewing

Curious why @nacin decided to include this check in the first place as it's not obvious to me what is being avoided—full size images?

@markoheijnen could you describe a use case where this creates unexpected behavior?

Note: See TracTickets for help on using tickets.