Horizontal scrollbar in RTL media modal

[ramiy]

The new media tab needs a few RTL adjustments on the UI, check out the screenshot.

1. On the search "input" box, move the X to left.

2. Remove the horizontal and vertical scroll bars.

Replying to ramiy:

1. On the search "input" box, move the X to left.

Which browser is that? Tested in Firefox 17, Chrome 23, IE 8, Opera 12, didn't see the X at all: 22935.media-rtl.png​.

2. Remove the horizontal and vertical scroll bars.

Could not reproduce the scroll bars either.

However, I've noticed an overlapping icon in the Publish box in Opera: 22935.publish-opera.png​.

I can confirm that the X appears in Chrome 23 and Safari 5.1.7 for Windows. However, it only appears after you have started typing a search term in the search box. I could not reproduce in Firefox 17, Firefox 10 ESR, IE 9, or Opera 12.

However, I was unable to reproduce the scroll bars issue, as reported by the original reporter.

I did notice the overlapping icon in Opera as well.

Replying to SergeyBiryukov:

To reproduce the X on the search "input" box, start typing your search term, using Chrome 23.

I can now reproduce the scroll bars in Chrome by reducing the height of browser window a bit.

Replying to ramiy:

To reproduce the X on the search "input" box, start typing your search term, using Chrome 23.

Appears to be a Chromium bug, fixed two weeks ago: ​​https://codereview.chromium.org/11421088/.

Still, 22935.patch​ fixes that as well.

The X on the search input box happens everywhere we have type="search" for inputs in the admin. That's probably better addressed all at once, in a separate ticket, and not for 3.5.1.

For the scrollbars, the vertical one should probably stay. I can't reproduce the horizontal one in OSX, but if the source of that can be found, then we can look at that. Would be good to know if that's RTL-specific or not.

Replying to helen:

The X on the search input box happens everywhere we have type="search" for inputs in the admin. That's probably better addressed all at once, in a separate ticket, and not for 3.5.1.

I have checked a default install of 3.5 and of trunk (both regular LTR install and a RTL install with no patches applied), and there is no X in the search box in the add media when using Firefox 17, Firefox 10 ESR, IE 9, or Opera 12. There is a X in Chrome 23 and Safari 5. All testing was done on a fully updated Windows 7 Ultimate computer.

I'll have to find the other search boxes and test them to see if the same thing occurs in all search boxes, or just the search box in add media.

I just did further testing of the search boxes found on the All Posts, Categories, Tags, Media Library, Pages, Comments, Install Themes, Plugins, Add New Plugins, and Users pages and had the same results as I did when checking the Add Media that I posted in comment 9 above.

Right, it's a Webkit thing, and the direction issue with the X has been happening all along ever since the introduction of type="search" fields in 3.4, so it's not for 3.5.1 and should be addressed all together in a separate ticket.

Replying to helen:

Would be good to know if that's RTL-specific or not.

It is. LTR looks fine: 22935.media-rtl.2.png​. Same window height, RTL: 22935.media-rtl.3.png​.

So, needs a new patch?

Not serious, no patch, moving to 3.6.

22935.1.diff​ should do the trick.