Better RTL support for Statistics Dashboard

VERIFIED FIXED in 5.0.6

Status

addons.mozilla.org Graveyard
Developer Pages
VERIFIED FIXED
9 years ago
3 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

({rtl})

unspecified
5.0.6

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
+++ This bug was initially created as a clone of Bug #450373 +++

The statistics dashboard also needs some RTL love...
(Assignee)

Updated

9 years ago
Blocks: 494583
Target Milestone: --- → 5.0.6
(Assignee)

Comment 1

9 years ago
Created attachment 381516 [details] [diff] [review]
Patch (v1)
Attachment #381516 - Flags: review?(clouserw)
(Assignee)

Updated

9 years ago
Attachment #381516 - Flags: review?(clouserw) → review?(fwenzel)
Comment on attachment 381516 [details] [diff] [review]
Patch (v1)

I can't read farsi, but your changes make sense, visually, and ltr is unchanged.

On a side note, I also appreciate that you removed the in-line style attributes!
Attachment #381516 - Flags: review?(fwenzel) → review+
(Assignee)

Comment 3

9 years ago
Committed in r27101.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

9 years ago
Verified on <https://preview.addons.mozilla.org/fa/statistics/addon/4976/>
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.