350596 - [RTL] Don't flip the feed icon

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Behnam Esfahbod [:zwnj]]

On my last cvs check-out, I see both LTR and RTL Feed logos, which are more annoying than useful.  Please just make it consistant.  It's not so important to have them reversed.

I'm going to attach an screenshot.

Comment 1

[Behnam Esfahbod [:zwnj]]

Attachment: Main and Preferences windows [fa-IR]

Comment 2

[Tsahi Asher]

i agree, it's inconsistent.

Comment 3

[Asaf Romano (gone)]

Jay, we need a RTL version of the livebookmark(s) icon with just the dropmarker on the other side (i.e. without flipping the feed icon itself)

Comment 4

[Jay Goldman]

Attachment: Livemark Item RTL for Winstripe with the RSS icon not flipped

Here you go Mano!

Comment 5

[Mike Connor [:mconnor]]

Comment on attachment 237344 [details]
Livemark Item RTL for Winstripe with the RSS icon not flipped

r+a=me, Mano will land this later today

Comment 6

[:Gavin Sharp [email: [e-mail]]]

browser/themes/winstripe/browser/livemark-folder-rtl.png and
browser/themes/winstripe/browser/livemark-folder.png also need changing.

Comment 7

[Asaf Romano (gone)]

Guess so...

Comment 8

[Behnam Esfahbod [:zwnj]]

Haven't fixed in branch yet.

Screenshot: attachment 237841 [details]

Comment 9

[Jay Goldman]

Attachment: Winstripe: Livemark-Folder not flipped

Comment 10

[Asaf Romano (gone)]

Checking in browser/themes/winstripe/browser/jar.mn;
/cvsroot/mozilla/browser/themes/winstripe/browser/jar.mn,v  <--  jar.mn
new revision: 1.13.2.19; previous revision: 1.13.2.18
done
Checking in browser/themes/winstripe/browser/livemark-folder-rtl.png;
/cvsroot/mozilla/browser/themes/winstripe/browser/Attic/livemark-folder-rtl.png,
v  <--  livemark-folder-rtl.png
new revision: 1.1.2.2; previous revision: 1.1.2.1
done
Checking in browser/themes/winstripe/browser/livemark-item-rtl.png;
/cvsroot/mozilla/browser/themes/winstripe/browser/livemark-item-rtl.png,v  <--
livemark-item-rtl.png
new revision: 1.2.18.3; previous revision: 1.2.18.2
done

Comment 11

[(no longer active)]

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