Closed Bug 350597 Opened 18 years ago Closed 15 years ago

[RTL] XML preview header should be RTL in RTL locales

Categories

(Core :: XML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: zwnj, Assigned: ehsan.akhgari)

References

(Blocks 1 open bug)

Details

(Keywords: fixed1.9.1, intl, rtl)

Attachments

(2 files)

XML preview's header should be right-aligned and have rtl direction in rtl locales.

I'm going to attach an screenshot.
No longer blocks: Persian-Fx2.0
Component: Layout: BiDi Hebrew & Arabic → XML
OS: AIX → All
Summary: [RTL] XML preview header → [RTL] XML preview header should be RTL in RTL locales
Version: 1.8 Branch → Trunk
Assignee: nobody → xml
QA Contact: layout.bidi → ashshbhatt
A related bug: bug 417489
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
Assignee: xml → ehsan.akhgari
QA Contact: ashshbhatt → xml
Target Milestone: Future → ---
Attached patch PatchSplinter Review
Trivial patch.
Attachment #359974 - Flags: superreview?(jonas)
Attachment #359974 - Flags: review?(jonas)
Status: NEW → ASSIGNED
Attachment #359974 - Flags: superreview?(jonas)
Attachment #359974 - Flags: superreview+
Attachment #359974 - Flags: review?(jonas)
Attachment #359974 - Flags: review+
Thanks for the super-quick review, Jonas!

<http://hg.mozilla.org/mozilla-central/rev/bdde600b135e>
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-litmus?
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Attachment #359974 - Flags: approval1.9.1?
Comment on attachment 359974 [details] [diff] [review]
Patch

This is a very low-risk patch which would bring a little more RTL goodness to users of RTL locales.
Comment on attachment 359974 [details] [diff] [review]
Patch

a191=beltzner
Attachment #359974 - Flags: approval1.9.1? → approval1.9.1+
Flags: in-litmus?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: