Closed
Bug 421690
Opened 17 years ago
Closed 17 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 |
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•17 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•17 years ago
|
||
Note that it is displayed correctly in the printer settings dialog.
Assignee | ||
Comment 2•17 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•17 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•17 years ago
|
Attachment #308276 -
Flags: approval1.9?
Comment 4•17 years ago
|
||
Marking as blocking1.9+ w/P2. Also, approving patch.
Flags: blocking1.9? → blocking1.9+
Priority: -- → P2
Updated•17 years ago
|
Attachment #308276 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 5•17 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: 17 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Priority: P2 → P1
Updated•17 years ago
|
Blocks: Persian-Fx3.5
Updated•17 years ago
|
Blocks: fx35-l10n-fa
Updated•17 years ago
|
No longer blocks: Persian-Fx3.5
Comment 6•17 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•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
•