477701 - [RTL] Save button of Library search is cut-off

Status: VERIFIED FIXED

(Firefox :: Theme, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Save button is cut-off

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090209 Shiretoko/3.1b3pre Ubiquity/0.1.5 ID:20090209020514

After doing a search within the library, the search bar appears right beyond the toolbar. As you can see in the screenshot the save button is cut-off on the left side. I can see that with the Hebrew locale. No idea if that applies to all RTL locales or only occurs for Hebrew.

Comment 1

[(no longer active)]

Henrik: Does this happen with Force RTL as well?

Tomer: you're not doing anything fancy in he's intl.css which can have this effect on he only, right?

(In reply to comment #0)
> Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre)
> Gecko/20090209 Shiretoko/3.1b3pre Ubiquity/0.1.5 ID:20090209020514

BTW, is this build ID correct?  (It's en-US.)  Just trying to see if it happens on trunk or 1.9.1.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

You see the en-US locale string here because I've tested it with your add-on. Now I also run a test with the latest fa build on trunk and it still shows this problem.

Comment 3

[Tomer Cohen :tomer]

(In reply to comment #1)
> Tomer: you're not doing anything fancy in he's intl.css which can have this
> effect on he only, right?
Not that I know of, but we should pay attention for keeping all the RTL locale styles in sync with others.

Comment 4

[(no longer active)]

(In reply to comment #3)
> (In reply to comment #1)
> > Tomer: you're not doing anything fancy in he's intl.css which can have this
> > effect on he only, right?
> Not that I know of,

Never mind, Henrik already confirmed that this happens on fa as well.

> but we should pay attention for keeping all the RTL locale
> styles in sync with others.

Yes, definitely.  We should probably give this a look before the 3.1 release...

Comment 5

[(no longer active)]

Attachment: Patch (v1)

The problem was caused by setting the start padding of #saveSearch instead of its left padding.  This caused the left padding of the button in RTL mode to be 0 pixels, which didn't leave any room for the leftcap image.

I tested this patch to make sure it has the desired behavior on the Mac OS X theme in an RTL build.

Comment 6

[Dão Gottwald [:dao]]

round-button-leftcap.png is 8px wide, so that's probably what the padding should be. The 1px left padding on #saveSearch .button-text should be removed.

Comment 7

[(no longer active)]

Attachment: Patch (v2)

Addressed comments.

Comment 8

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/d3661c8e1597

Comment 9

[(no longer active)]

Comment on attachment 362264 [details] [diff] [review]
Patch (v2)

Requesting approval for this theme-only and low risk patch which fixes a display problem for RTL locales on Mac OS X.

Comment 10

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 362264 [details] [diff] [review]
Patch (v2)

a191=beltzner

Comment 11

[(no longer active)]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/292cfa2a4b80

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Verified on trunk and 1.9.1 branch:

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; fa; rv:1.9.2a1pre) Gecko/20090221 Minefield/3.2a1pre ID:20090221020633

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090221 Shiretoko/3.1b3pre ID:20090221020541

Comment 13

[(no longer active)]

Dão, any particular reason why you cleared in-litmus here?