Closed
Bug 427729
Opened 17 years ago
Closed 17 years ago
We should not mirror the Location Bar "Go" button for RTL
Categories
(Firefox :: Theme, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: tomer, Assigned: tomer)
References
(Blocks 2 open bugs)
Details
(Keywords: rtl, Whiteboard: [RC2+])
Attachments
(2 files)
(deleted),
patch
|
Gavin
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
beltzner
:
ui-review+
|
Details |
Since we keep the location bar in LTR even when the rest of the GUI is RTL (The bookmarks star never jumps to the left in RTL) we should also keep the GO button (green arrow) pointing to the right, much like the English interface.
Current en-US navigation bar: <back forward> () (X) [% http://example.com >] [G ]
Current RTL navigation bar: [ G] [% http://example.com <] (X) () <forward back>
Should be: [ G] [% http://example.com >] (X) () <forward back>
Assignee | ||
Updated•17 years ago
|
Assignee | ||
Comment 1•17 years ago
|
||
Please note it is also require to remove the image file go-arrow-rtl.png
Attachment #315120 -
Flags: review?
Updated•17 years ago
|
Blocks: fx35-l10n-fa
Updated•17 years ago
|
Attachment #315120 -
Flags: review? → review?(dtownsend)
Updated•17 years ago
|
Assignee: nobody → tomer
Updated•17 years ago
|
Version: unspecified → Trunk
Comment 2•17 years ago
|
||
Comment on attachment 315120 [details] [diff] [review]
[GTK and Win] removal of go-arrow-rtl.png (patch)
I am not an appropriate reviewer for this patch.
Additionally for the future a unified diff is the preferred form for patches.
Attachment #315120 -
Flags: review?(dtownsend)
Comment 3•17 years ago
|
||
Mano make a great RTL address-bar for fx2. Why not implement it that way again? A LTR Go button (a triangle points to right, read: back, previous, ...) in an RTL interface will be so annoying.
Assignee | ||
Comment 4•17 years ago
|
||
(In reply to comment #3)
> A LTR Go button (a triangle points to right, read: back, previous, ...)
> in an RTL interface will be so annoying.
>
You right, but as Firefox 3 location bar contain some widgets, it will make it worse than the previous one, as the location bar is LTR. I don't think we should put the site icon in the middle of the bar, and the RSS feed icon will still be placed in the right side, so the most natural solution is to LTR the whole bar.
I'm also not sure how many people will think the go button meant to be backward instead of go button, even as both are pointing to the same side.
Comment 5•17 years ago
|
||
Can we get a screenshot of what this fixes? Best before and after.
Assignee | ||
Comment 6•17 years ago
|
||
(In reply to comment #5)
> Can we get a screenshot of what this fixes? Best before and after.
>
Sure, no problem.
Comment 7•17 years ago
|
||
Comment on attachment 315814 [details]
screenshot
Mike, most of the awesomebar is LTR, thanks to URLs, the arrow, too?
Attachment #315814 -
Flags: ui-review?(beltzner)
Comment 8•17 years ago
|
||
Comment on attachment 315814 [details]
screenshot
I'm willing to take Tomer's lead, here. I don't think this blocks, but we'd take a patch.
Attachment #315814 -
Flags: ui-review?(beltzner) → ui-review+
Assignee | ||
Comment 9•17 years ago
|
||
Please note that this bug is still open for RC1. Can we patch the theme before going final?
Comment 10•17 years ago
|
||
Comment on attachment 315120 [details] [diff] [review]
[GTK and Win] removal of go-arrow-rtl.png (patch)
I guess it's too late (unless we respin for RC2) but anyway asking Gavin for review. Tomer: can you make a unified patch?
Attachment #315120 -
Flags: review?(gavin.sharp)
Updated•17 years ago
|
Flags: wanted1.9.0.x?
Whiteboard: [RC2?]
Comment 11•17 years ago
|
||
Another possibility (along the lines of what Dao suggested for the placement of the dropdown button in Arabic vs. others) is for each of the RTL locales that want this to put it in their intl.css
Comment 12•17 years ago
|
||
Comment on attachment 315120 [details] [diff] [review]
[GTK and Win] removal of go-arrow-rtl.png (patch)
I don't have any opinion about whether this is better or worse than the current behavior. If we do this we should also stop packaging the icon (remove the three jar.mn lines that refer to it, and remove the file itself).
Attachment #315120 -
Flags: review?(gavin.sharp) → review+
Updated•17 years ago
|
Whiteboard: [RC2?] → [RC2-]
Assignee | ||
Updated•17 years ago
|
Attachment #315120 -
Flags: approval1.9?
Comment 13•17 years ago
|
||
Comment on attachment 315120 [details] [diff] [review]
[GTK and Win] removal of go-arrow-rtl.png (patch)
a=beltzner, ceremonial last patch and demonstration of our dedication to non-en-US centrism
Attachment #315120 -
Flags: approval1.9? → approval1.9+
Comment 14•17 years ago
|
||
Checking in gnomestripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/gnomestripe/browser/browser.css,v <-- browser.css
new revision: 1.218; previous revision: 1.217
done
Checking in winstripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/winstripe/browser/browser.css,v <-- browser.css
new revision: 1.221; previous revision: 1.220
done
FIXED it is.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [RC2-] → [RC2+]
Comment 15•17 years ago
|
||
(In reply to comment #13)
> (From update of attachment 315120 [details] [diff] [review])
> a=beltzner, ceremonial last patch and demonstration of our dedication to
> non-en-US centrism
Wow, it's really nice to see the (hopefully) last patch for Firefox 3 is to fix
an RTL issue... :-) Thanks Mike, Tomer and Axel!
Clearing the wanted1.9.0.x flag...
Flags: wanted1.9.0.x?
Updated•17 years ago
|
Blocks: Persian-Fx3.5
Comment 16•17 years ago
|
||
Verified with Mozilla/5.0 (X11; U; Linux i686; he; rv:1.9) Gecko/2008052912 Firefox/3.0 (RC2).
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
No longer blocks: fx35-l10n-fa
You need to log in
before you can comment on or make changes to this bug.
Description
•