Menu | RTL issues

[rasheed]

Please check attachment.

wp-admin/nav-menus.php

[SergeyBiryukov]

.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]

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​

[ryan]

In [20998]:

RTL fixes. Props SergeyBiryukov, helenyhou. fixes #20841

[azaozz]

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?

[nacin]

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.

[SergeyBiryukov]

20841.3.patch​ removes text-align.

[ryan]

In [20999]:

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