WordPress.org

Make WordPress Core

Opened 12 months ago

Closed 7 months ago

#24149 closed defect (bug) (fixed)

Set RTL body class for iframes

Reported by: mitchoyoshitaka Owned by:
Milestone: 3.7 Priority: normal
Severity: minor Version:
Component: Administration Keywords: dev-feedback
Focuses: Cc:

Description

iframe_header() sets the current locale string in the body class, but not the RTL flag. Please add this.

This came up in MP6 development, where MP6 does not (at this moment) rely on a separate rtl stylesheet like in the current admin CSS, but instead relies more on the body having .rtl set.

Attachments (1)

24149.diff (649 bytes) - added by mitchoyoshitaka 12 months ago.
Patch

Download all attachments as: .zip

Change History (8)

mitchoyoshitaka12 months ago

Patch

comment:1 SergeyBiryukov12 months ago

  • Keywords commit 3.7-early added
  • Milestone changed from Awaiting Review to Future Release

comment:2 wonderboymusic9 months ago

  • Milestone changed from Future Release to 3.7

these are all marked 3.7-early

comment:3 nacin9 months ago

There is also wp_iframe() and probably a few other places where we have a <body> element and should also have a .rtl class.

editimage.html might be the one place where you won't be able to externally style.

comment:4 ryan9 months ago

In 24994:

Set RTL body class for iframes.

Props mitchoyoshitaka
see #24149

comment:5 nacin8 months ago

  • Keywords has-patch commit 3.7-early removed

Where else does it need to be added?

comment:6 DrewAPicture7 months ago

  • Keywords dev-feedback added

comment:7 nacin7 months ago

  • Resolution set to fixed
  • Status changed from new to closed

Calling this as fixed pending further issues/needs.

Note: See TracTickets for help on using tickets.