Closed
Bug 421690
Opened 16 years ago
Closed 16 years ago
Firefox with he/ar UI displays print headers reversed for non-rtl pages
Categories
(Core :: Layout: Text and Fonts, defect, P1)
Core
Layout: Text and Fonts
Tracking
()
RESOLVED
FIXED
People
(Reporter: tomer, Assigned: smontagu)
References
(Blocks 1 open bug, )
Details
(Keywords: rtl)
Attachments
(3 files)
|
17.06 KB,
image/png
|
Details | |
|
43.99 KB,
image/png
|
Details | |
|
6.80 KB,
patch
|
roc
:
review+
roc
:
superreview+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
mozilla.org print preview
When I try to print English page from Firefox with Hebrew layout, I gets the print headers reversed. I.e., "Page 1 of 2" displayed as "עמוד 1 מתוך 2" instead of "עמוד 1 מתוך 2". This issue affect both the Print Preview and the actual print. I can't reproduce it on http://www.wikipedia.org, but on http://www.mozilla.org it is visible (both do not specify dir=rtl, and are UTF-8 encoded). I'm attaching a screenshot.
|
Reporter | |
Updated•16 years ago
|
Summary: Firefox with he/ar UI show print headers reversed for non-rtl pages → Firefox with he/ar UI displays print headers reversed for non-rtl pages
|
|
Reporter | |
Comment 1•16 years ago
|
||
Note that it is displayed correctly in the printer settings dialog.
|
Assignee | |
Comment 2•16 years ago
|
||
This is a regression since Firefox 2.
Component: Printing → Layout: BiDi Hebrew & Arabic
Flags: blocking1.9?
OS: Linux → All
Product: Toolkit → Core
QA Contact: printing → layout.bidi
Hardware: PC → All
|
|
Assignee | |
Comment 3•16 years ago
|
||
This is happening because bidi is disabled when there are no RTL characters on the page apart from the headers and footers (which is why the bug doesn't appear on wikipedia).
Assignee: nobody → smontagu
Status: NEW → ASSIGNED
Attachment #308276 -
Flags: superreview?(roc)
Attachment #308276 -
Flags: review?(roc)
Attachment #308276 -
Flags: superreview?(roc)
Attachment #308276 -
Flags: superreview+
Attachment #308276 -
Flags: review?(roc)
Attachment #308276 -
Flags: review+
|
|
Assignee | |
Updated•16 years ago
|
Attachment #308276 -
Flags: approval1.9?
|
||
Comment 4•16 years ago
|
||
Marking as blocking1.9+ w/P2. Also, approving patch.
Flags: blocking1.9? → blocking1.9+
Priority: -- → P2
|
|
||
Updated•16 years ago
|
Attachment #308276 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Comment 5•16 years ago
|
||
Checked in. Marking in-testsuite- since as far as I know we have no way to reftest print preview, but feel free to change that if I am wrong.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Priority: P2 → P1
|
||
Updated•16 years ago
|
Blocks: Persian-Fx3.5
|
|
||
Updated•16 years ago
|
Blocks: fx35-l10n-fa
|
|
||
Updated•16 years ago
|
No longer blocks: Persian-Fx3.5
|
|
||
Comment 6•16 years ago
|
||
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
|
||
Updated•16 years ago
|
Blocks: Persian-Fx3.5
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: layout.bidi → layout.fonts-and-text
|
|
||
Updated•15 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
•