408661 - Places Organizer - Back & Forward buttons are reversed in RTL UI

Status: RESOLVED FIXED

(Firefox :: Theme, defect, P2)

Description

[Asaf Romano (gone)]

Places Organizer - Back & Forward buttons are reversed in RTL UI

Comment 1

[Asaf Romano (gone)]

Attachment: patch

Comment 2

[(not reading, please use [e-mail] instead)]

Comment on attachment 293496 [details] [diff] [review]
patch

mano, one question, is this added line necesssary:

+  list-style-image: url("chrome://browser/skin/Toolbar.png");

Don't we have that from

 #forward-button {
   list-style-image: url("chrome://browser/skin/Toolbar.png");
   -moz-image-region: rect(0px, 48px, 24px, 24px);
 }

Comment 3

[Asaf Romano (gone)]

right, copypaste mistake.

Comment 4

[Reed Loden [:reed]]

This only affects winstripe and not gnomestripe?

Comment 5

[Tony Chung [:tchung]]

can someone assign a priority for this bug?

Comment 7

[(no longer active)]

Is this ready to go?

Comment 8

[Tomer Cohen :tomer]

Works for me (b5pre). Can we close this bug?

Comment 9

[Asaf Romano (gone)]

Tomer: did you test this just on linux?

Comment 10

[Tomer Cohen :tomer]

Linux only. I might get access to a windows machine from work tommorrow.

Comment 11

[Asaf Romano (gone)]

Attachment: patch for winstripe

Comment 12

[Dietrich Ayala (:dietrich)]

Comment on attachment 308537 [details] [diff] [review]
patch for winstripe

r=me

Comment 13

[Asaf Romano (gone)]

mozilla/browser/themes/winstripe/browser/places/organizer.css 1.4

Comment 14

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).