Closed Bug 347514 Opened 13 years ago Closed 11 years ago

Wrong placement of tab drop indicator in RTL locales

Categories

(Firefox :: Tabbed Browser, defect)

2.0 Branch
x86
Linux
defect
Not set

Tracking

()

RESOLVED WORKSFORME
Firefox 2

People

(Reporter: zwnj, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: rtl)

Attachments

(2 files, 1 obsolete file)

Place of marker is wrong for the first place.
Attached image Screenshot (obsolete) —
benham, which build are you using?
Summary: Wrong placement marker in RTL locales → Wrong placement of tab drop indicator in RTL locales
I use a GTK build from CVS (about 4 days ago) with pango enabled.

Configure arguments
--enable-application=browser --prefix=/home/behnam/opt/mozilla/firefox-2.0-fa --enable-pango 
I see this as well in 1.5.0.5, plus when you close a tab Firefox automatically opens the next tab instead of the previous tab, as it does in LTR locales (in other words, it opens the tab to the left, as it does in LTR, although it should open the tab to the right)
I'm sorry, I was wrong. I have not problems with the tab drop indicator. I do have another problem that is visible in your screenshots - the tab isn't flipped horizontally as it should be - i.e. the straight edge is on the left while the curved one in on the right.
The problem do exist yet.
Blocks: NewThemeRTL
If the marker is placed wrong, it makes tab drag and drop much harder to use in RTL locales.  Nominating for driver consideration.
Flags: blocking-firefox2?
Target Milestone: --- → Firefox 2
unable to reproduce, Mano says this looks fine.

unsure if pango might affect it, but unless someone can reproduce in a stock build, there's no way we're blocking on this.
Flags: blocking-firefox2? → blocking-firefox2-
Attachment #232311 - Attachment is obsolete: true
As you can see, the marker for the left end is all right.
These are screenshot of today's nightly build of tbox.
No longer blocks: NewThemeRTL
Blocks: fx35-l10n-fa
No longer blocks: Persian-Fx3.5
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
This has been fixed on trunk.  Resolving as WFM because the 2.0 branch isn't getting any more updates...
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
No longer blocks: fx35-l10n-fa
You need to log in before you can comment on or make changes to this bug.