Closed
Bug 751841
Opened 12 years ago
Closed 12 years ago
[RTL] Line numbering broken if line starts with RTL character
Categories
(Toolkit :: View Source, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: tomer, Assigned: smontagu)
References
Details
(Keywords: rtl)
Attachments
(3 files)
screenshot (Mozilla/5.0 (X11; Linux i686; rv:14.0) Gecko/20120430 Firefox/14.0a2)
Firefox doesn't display line numbering right when the content has lines that start with RTL characters such as Arabic and Hebrew. Instead, it appears behind the source text making the source less readable and the line numbering to appear to fewer lines. The issue does reproduce only to lines that starts with RTL character, so it won't reproduce on pages that the author have indented the source lines in the <body> block…
|
Reporter | |
Comment 1•12 years ago
|
||
See lines {9,11,16,17}.
|
||
Updated•12 years ago
|
Component: Developer Tools → View Source
Product: Firefox → Toolkit
QA Contact: developer.tools → view.source
|
Assignee | |
Comment 2•12 years ago
|
||
This is caused by the line numbers participating in bidi reordering. Changing from unicode-bidi: embed to unicode-bidi: -moz-isolate would be enough to fix it, except for bug 717811, so I have added a workaround for that as well.
|
|
Assignee | |
Updated•12 years ago
|
Attachment #627862 -
Flags: review?(ehsan)
|
||
Comment 3•12 years ago
|
||
Comment on attachment 627862 [details] [diff] [review] Patch Hmm, does "\200e" work, or did you mean "\u200e"? r=me with whichever works correctly.
Attachment #627862 -
Flags: review?(ehsan) → review+
|
|
Assignee | |
Comment 4•12 years ago
|
||
No, I meant what I wrote: http://www.w3.org/TR/CSS2/syndata.html#escaped-characters
|
|
Assignee | |
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e98f0fc3d02e
Assignee: nobody → smontagu
Target Milestone: --- → mozilla15
|
||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e98f0fc3d02e
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•