Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#37286 closed defect (bug) (fixed)

Site Icon: Preview when cropping an image is broken in RTL

Reported by: ocean90 Owned by: ocean90
Priority: normal Milestone: 4.6
Component: Customize Version: 4.3
Severity: normal Keywords: has-patch
Cc: Focuses: rtl

Description

  • When cropping an image in the media modal the app icon preview doesn't get always updated. The image wrapper needs a direction: ltr wrapper.
  • Same for the favicon, but this is more fun. It updates the wrong preview. Instead of updating the one in the media modal it updates the one in the customizer panel because both are using the same ID.

Attachments (1)

37286.patch (501 bytes ) - added by ocean90 10 years ago.

Download all attachments as: .zip

Change History (4)

@ocean90
10 years ago

#1 @ocean90
10 years ago

  • Version 4.44.3

#2 @ocean90
10 years ago

  • Milestone Awaiting Review4.6

#3 @ocean90
10 years ago

  • Owner set to ocean90
  • Resolutionfixed
  • Status newclosed

In 37964:

Customize: Fix site icon preview in RTL.

  • Set the direction of previews to LTR.
  • Remove the ID attribute from the favicon preview in the customizer panel. The preview in the media modal uses the same ID which prevented updating the correct favicon preview.

Fixes #37286.

Note: See TracTickets for help on using tickets.