556102 - Use :moz-locale-dir pseudoclass instead of chromedir attribute

Status: RESOLVED FIXED

(SeaMonkey :: General, defect)

Description

[Stefan [:stefanh]]

Attachment: Switch from chromedir to :moz-locale-dir

For the record, subscribe.xhtml, aboutSessionRestore.xhtml, certError.xhtml use the same entity and it still lives in brand.dtd. Modern's scrollbox.css (obviously ported from windtripe) looks wrong to me - In rtl mode, I get the right-arrow at the left side of the menus (I expected the left-arrow at the left side). You can try this by adding the "intl.uidirection.en-US" pref and set it to "rtl".

Comment 1

[[e-mail]]

Comment on attachment 436016 [details] [diff] [review]
Switch from chromedir to :moz-locale-dir

> #tabs > :first-child,
>-#tabs[chromedir=rtl] > :last-child {
>+#tabs:-moz-locale-dir(rtl) > :last-child {
>   -moz-border-radius-topleft: 2px;
>   -moz-border-radius-bottomleft: 2px;
>    border-left: none !important;
> }
> 
> #tabs > :last-child,
>-#tabs[chromedir=rtl] > :first-child {
>+#tabs:-moz-locale-dir(rtl) > :first-child {
>   -moz-border-radius-topright: 2px;
>   -moz-border-radius-bottomright: 2px;
> }
Doesn't this need ltr on the other rules, otherwise in an rtl environment both sets of rules will apply to both first and last tabs?

Comment 2

[[e-mail]]

(In reply to comment #0)
> Modern's scrollbox.css (obviously ported from windtripe) looks wrong to me -
> In rtl mode, I get the right-arrow at the left side of the menus
What does scrollbox.css have to do with menu arrows? It's only used e.g. when you have zillions of bookmarks in a single folder and they don't fit on screen.

Comment 3

[Stefan [:stefanh]]

(In reply to comment #2)
> (In reply to comment #0)
> > Modern's scrollbox.css (obviously ported from windtripe) looks wrong to me -
> > In rtl mode, I get the right-arrow at the left side of the menus
> What does scrollbox.css have to do with menu arrows? It's only used e.g. when
> you have zillions of bookmarks in a single folder and they don't fit on screen.

It has nothing to do with menu arrows, I don't know what I was thinking of :-(

Comment 4

[Stefan [:stefanh]]

Fixed with Neil's comment #1 addressed: http://hg.mozilla.org/comm-central/rev/b9be42774b92