Opened 12 years ago
Closed 12 years ago
#20841 closed defect (bug) (fixed)
Menu | RTL issues
Reported by: | rasheed | Owned by: | 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)
Change History (13)
#3
@
12 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
@
12 years ago
- Owner set to ryan
- Resolution set to fixed
- Status changed from new to closed
In [20998]:
#5
@
12 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
@
12 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.
#8
@
12 years ago
20841.3.patch removes text-align
.
Note: See
TracTickets for help on using
tickets.
.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).