Closed Bug 859093 Opened 12 years ago Closed 12 years ago

broken handling of parentheses near inlines with unicode-bidi: isolate

Categories

(Core :: Layout: Text and Fonts, defect)

10 Branch
x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
firefox22 --- wontfix
firefox23 --- fixed

People

(Reporter: amir.aharoni, Assigned: smontagu)

References

Details

(Keywords: regression, rtl)

Attachments

(2 files)

Attached file demonstration of the problem (deleted) —
If there are closing parentheses after the closing tag of <span dir="auto">, they are shown on the wrong side of the word. See the attached HMTL file.
Regression range: m-c good=2011-10-19 bad=2011-10-20 http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9fa62f76f1cf&tochange=311fdb9b38b7 Suspected bug: Jonathan Kew — bug 694986 - trailing bracket misplaced in LTR language names when language-picker UI is displayed RTL. r=mfinkle
Blocks: 694986
Keywords: regression, rtl
Version: 20 Branch → 10 Branch
Bug 694986 touched -only- mobile JS code in mobile/chrome/content/localePicker.js, which was specific to the XUL fennec UI, so it's never been part of desktop Firefox, and isn't even present on mobile any longer. (See https://hg.mozilla.org/mozilla-central/rev/387f388bca1f.) So I don't think that can have caused the issue here.
No, this is certainly caused by an error in bug 613149
You're right.
Blocks: 613149
No longer blocks: 694986
Attached patch Patch (deleted) — Splinter Review
This was a very silly mistake :(
Assignee: nobody → smontagu
Attachment #734631 - Flags: review?(roc)
The issue isn't with dir=auto as such, but with any inline element with unicode-bidi: -moz-isolate.
Summary: broken handling of parentheses near dir=auto → broken handling of parentheses near inlines with unicode-bidi: isolate
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Comment on attachment 734631 [details] [diff] [review] Patch [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 613149, and exposed in more contexts by bug 548206 User impact if declined: regressions in reordering of bidirectional text -- in particular it will appear as if the new bidi features of isolation and autodirection make reordering less accurate instead of more accurate as intended Testing completed (on m-c, etc.): baked on m-c since 2013-04-08 Risk to taking this patch (and alternatives if risky): Minimal. String or IDL/UUID changes made by this patch: None
Attachment #734631 - Flags: approval-mozilla-aurora?
Attachment #734631 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Depends on: 870794
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: