Closed Bug 240485 Opened 19 years ago Closed 18 years ago

Bookmarks Toolbar items go under the 'chevron' icon, when interface is RTL

Categories

(Firefox :: Toolbars and Customization, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: linxspider, Assigned: mconnor)

Details

(Keywords: rtl, Whiteboard: fixed-aviary1.0)

Attachments

(2 files, 3 obsolete files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; he-IL; rv:1.6) Gecko/20040206 Firefox/0.8
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; he-IL; rv:1.6) Gecko/20040206 Firefox/0.8

When the interface direction is RTL as in Hebrew or Arabic, the bookmarks on the
'Bookmarks Toolbar Folder' go under the 'chevron' icon(the icon looks like this
'>>' ,This icon is actually used when they are Bookmarks that don't fit on the
toolbar).

Reproducible: Always
Steps to Reproduce:
1. Make sure they are a few bookmarks on the Bookmarks Toolbar Folder. Also make
sure the toolbar is displayed(if not go to view->toolbars->bookmarks toolbar).

2.Add this code to the userChrome.css file To chanage the direction to RTL(it's
under the chrome folder inside the user profile.

/*make UI RTL */

window,dialog,wizard,page { direction: rtl; }

menu { direction: rtl; }

outliner { direction: rtl; }

/*************************************
**  Fix address bar directionality  **
*************************************/

/*make address bar remain LTR */

#urlbar { direction: ltr !important; }

/* leave search from address bar in RTL */

#urlbar .autocomplete-search-engine {
direction: rtl !important;
}

/***** End address bar fix ******/

/* Align text in menulist pupop items to right  */

menulist > menupopup > menuitem {
  padding: 1px 2px 1px 30px !important;
}

/* Align autocomplete popups items in urlbar to left in firefox */

popup[type="autocomplete"],
.autocomplete-history-popup {
 direction: ltr !important;
}

3. Open the Firefox browser.

Actual Results:  
the Bookmarks Toolbar items are aligned to the left under the '>>' icon

Expected Results:  
the bookmarks should be aligned to the right. if they are bookmarks that don't
fit on the toolbar, they should go under the '>>' icon on the left

the bookmarks toolbar is defined as "PersonalToolbar" in xul and css.
Attached image Bookmarks toolbar
do we have existing localizations that are affected by this?  i.e. the available
Hebrew localization at
http://www.mozilla.org/products/firefox/releases/#download, is that affected by
this?  I haven't seen any bugs on that, and 0.8 has been out for a couple months
now.
the current language pack for firefox aligns the bookmarks toolbar to the left,
while the rest if the UI is aligned to the right, in order to avoid this bug.
therefore, users don't see this. but we would like to be able to align the
bookmarks toolbar to the right as well.

Seamonkey had a similar problem, which was only partialy fixed, at bug 218528.
Flags: blocking0.9?
Attached patch Probably the fix... (obsolete) — Splinter Review
Fixes the bug using WIDTHes instead of horizontal x values
Attachment #146499 - Flags: review?(bugs)
when this was partialy fixed in bug 218528, they left a problem in which draging
links to the toolbar droped them one place left of what you expected (left of
where the black vertical line is). did you check that you can drop new links to
the toolbar or drag items around with no problem?
(In reply to comment #5)
> when this was partialy fixed in bug 218528, they left a problem in which draging
> links to the toolbar droped them one place left of what you expected (left of
> where the black vertical line is). did you check that you can drop new links to
> the toolbar or drag items around with no problem?

I checked it with the patch and I don't see any problem with dropping, adding
and reordering the bookmarks toolbar.
(In reply to comment #6)
> (In reply to comment #5)
> > when this was partialy fixed in bug 218528, they left a problem in which draging
> > links to the toolbar droped them one place left of what you expected (left of
> > where the black vertical line is). did you check that you can drop new links to
> > the toolbar or drag items around with no problem?
> 
> I checked it with the patch and I don't see any problem with dropping, adding
> and reordering the bookmarks toolbar.

ok soory, this specific strange behavior still happens. I'll try to find what's
the reason for this. but i'm not promising anything :-)
Comment on attachment 146499 [details] [diff] [review]
Probably the fix...

Remove review request until i work around the dragging problem
Attachment #146499 - Flags: review?(bugs)
not a 0.9 blocker.  A patch that doesn't cause regressions would be accepted,
but this isn't critical for a feature-driven release.  1.0beta is where the
cleanup stuff belongs.
Flags: blocking0.9? → blocking0.9-
(In reply to comment #9)
> not a 0.9 blocker.  A patch that doesn't cause regressions would be accepted,
> but this isn't critical for a feature-driven release.  1.0beta is where the
> cleanup stuff belongs.

If you were using the HebrewL10N version of Firefox, you wodn't denay it so
quickly, it's realy annoying...

and.. just a litle correctioon: there is no regression in the patch. The drag &
drop problem applies only for RTL bookmarks-bar.
if I was using the Hebrew l10n I'd probably have fixed this myself a long time
ago. :)

A partial fix that doesn't hose LTR at all is iffy, since we're basically saying
that RTL has to put up with a buggy interface to have the bookmarks orient RTL.

re-request review, we'll see what the peers say, if its really a net
improvement, okay, but having unexpected behaviour is a bad thing.

its still not a blocker, since its not like its something that's provoked a
massive backlash with users of that l10n, and we're still working towards 1.0. 
If you can figure out a workaround (or even a cheap wallpaper hack) to make this
work properly, the chances improve :)
Status: UNCONFIRMED → NEW
Ever confirmed: true
OK, new patch is fixing the bug and drag & drop issue.

seems to be final...
Attachment #146499 - Attachment is obsolete: true
Attachment #147009 - Flags: review?(bugs)
Attached patch Final Patch (obsolete) — Splinter Review
Much better solution for working around this annoying bug (no change for the
code in LTR cases and no more problems with non-stanard themes or strange
userChrome...).
Attachment #147009 - Attachment is obsolete: true
Attachment #147009 - Flags: review?(bugs)
Attachment #147985 - Flags: review?
Asking again for blocking (now that there is a patch)
Flags: blocking0.9- → blocking0.9?
still not a blocker, but I'll review this later if you request the review from
me (instead of from no one)
Flags: blocking0.9? → blocking0.9-
Attachment #147985 - Flags: review? → review?(mconnor)
Comment on attachment 147985 [details] [diff] [review]
Final Patch

the patch looks OK from the Bidi point of view.
Comment on attachment 147985 [details] [diff] [review]
Final Patch

removing DROP_ON means that  you can't drop onto a folder now
Attachment #147985 - Flags: review?(mconnor) → review-
Attached patch patchSplinter Review
this is essentially Asaf's patch, cleaned up in a few places, with DROP_ON
properly restored
Assignee: bugs → mconnor
Attachment #147985 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Comment on attachment 148793 [details] [diff] [review]
patch

there's a couple minor nites wrt spacing in here, I'll fix that on checkin
Attachment #148793 - Flags: review?(bryner)
(In reply to comment #17)
> (From update of attachment 147985 [details] [diff] [review])
> removing DROP_ON means that  you can't drop onto a folder now
> 

Oh, i missed it up 10x for correction :-)
Comment on attachment 148793 [details] [diff] [review]
patch

checked this in for 0.9, will do the spacing cleanup after.
Attachment #148793 - Flags: review?(bryner)
Whiteboard: fixed-aviary1.0
fixed on trunk, finally.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
QA Contact: bugzilla → toolbars
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
You need to log in before you can comment on or make changes to this bug.