WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#20841 closed defect (bug) (fixed)

Menu | RTL issues

Reported by: rasheed's profile rasheed Owned by: ryan's profile ryan
Milestone: 3.4 Priority: normal
Severity: major Version: 3.4
Component: RTL Keywords: has-patch
Focuses: Cc:

Description

Please check attachment.

wp-admin/nav-menus.php

Attachments (4)

menu-fix-rtl.gif​ (113.2 KB) - added by rasheed 14 years ago.
20841.patch​ (960 bytes) - added by SergeyBiryukov 14 years ago.
20841.2.diff​ (1.2 KB) - added by helenyhou 14 years ago.
20841.3.patch​ (808 bytes) - added by SergeyBiryukov 14 years ago.

Download all attachments as: .zip

Change History (13)

@rasheed
14 years ago

#1 @SergeyBiryukov
14 years ago

  • Milestone changed from Awaiting Review to 3.4

@SergeyBiryukov
14 years ago

#2 @SergeyBiryukov
14 years ago

  • Keywords has-patch added

.howto span was replaced with .nav-menus-php .howto span in [18577], but not for RTL.

input[type="search"] { text-align: left; } was introduced in [20168] (a regression from 3.3).

@helenyhou
14 years ago

#3 @helenyhou
14 years ago

Patch looks good. Found one more thing to adjust - the placement of the down arrow for expanding/collapsing an item to edit. 20841.2.diff​

#4 @ryan
14 years ago

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

In [20998]:

RTL fixes. Props SergeyBiryukov, helenyhou. fixes #20841

#5 @azaozz
14 years ago

Did some testing and don't see the need for text-align on the type=search fields. Perhaps that was needed for an older version of a browser?

#6 @nacin
14 years ago

Yeah, I brought this up — really no reason for text-align to be forced here. georgestephanis also doesn't know where it came from. Per IRC, it can be removed, for both LTR and RTL.

#7 @SergeyBiryukov
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

@SergeyBiryukov
14 years ago

#8 @SergeyBiryukov
14 years ago

20841.3.patch​ removes text-align.

#9 @ryan
14 years ago

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

In [20999]:

Remove text-align on search input. Props SergeyBiryukov. fixes #20841

Note: See TracTickets for help on using tickets.