Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#27779 closed defect (bug) (fixed)

RTL: Themes screen layout issue in Firefox

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: nacin's profile nacin
Milestone: 3.9 Priority: normal
Severity: normal Version: 3.8
Component: Themes Keywords: has-patch commit
Focuses: ui, rtl, administration Cc:

Description

See the screenshot.

Attachments (3)

27779.png (835.6 KB) - added by SergeyBiryukov 11 years ago.
27779.patch (421 bytes) - added by SergeyBiryukov 11 years ago.
27779.2.patch (397 bytes) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (10)

@SergeyBiryukov
11 years ago

#1 @ocean90
11 years ago

Can't reproduce this in Firefox 28 on OS X.

#2 @SergeyBiryukov
11 years ago

In case it's not clear from the screenshot and the patch, this is caused by .theme.active .theme-name being 1px higher than other .theme .theme-name elements. The "Active" word somehow triggers this difference.

The patch simply makes sure the elements have consistent height that doesn't interfere with floating blocks. .theme .theme-name would probably make more sense for that, see 27779.2.patch.

#3 @ocean90
11 years ago

The patch simply makes sure the elements have consistent height

Okay, at least it doesn't break anything. But still can't see this on a clean install in FF 28 in OS X and Windows 8.1, with only $text_direction = 'rtl'; or Arabic language. Height is always 18px.

#4 @SergeyBiryukov
11 years ago

Could not trigger this with Arabic language files either, only with he_IL.

#5 @ocean90
11 years ago

  • Keywords commit added

Reproduced when I set the height of the active theme name to 19px.

#6 @nacin
11 years ago

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

In 28104:

Themes screen: Fix RTL layout issue in Firefox.

props SergeyBiryukov.
fixes #27779.

#7 @ocean90
11 years ago

Related: #27803

Note: See TracTickets for help on using tickets.