All users were logged out of Bugzilla on October 13th, 2018

[BB][RTL] Drawer is messed up when used in RTL languages

VERIFIED FIXED in Firefox OS v2.0

Status

VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: nefzaoui, Assigned: nefzaoui)

Tracking

unspecified
2.0 S5 (4july)

Firefox Tracking Flags

(b2g-v2.0 verified, b2g-v2.1 verified)

Details

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
Created attachment 778456 [details]
Drawer_status.jpg

User Agent: Mozilla/5.0 (Windows NT 6.2; rv:25.0) Gecko/20130716 Firefox/25.0 (Nightly/Aurora)
Build ID: 20130716030202



Actual results:

Using Firefox OS in Arabic causes a little (too much) of a chaos in building blocks. Most elements that live in Shared/style and shared/style_unstable need to get support for dir="rtl" cases.
Screenshots show the current status without RTL support.
Note that what you see is after I finished working on bug 883673, I was even worse.
(Assignee)

Updated

5 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Updated

5 years ago
Blocks: 906270

Comment 1

4 years ago
Ahmed, at risk of abusing your time, can you confirm if these drawer issues happen in 2.0? I ask because there are a lot of changes to these areas in 2.0. If they still exist, we can add these to the RTL backlog for 2.1. Thanks!
Flags: needinfo?(nefzaoui.ahmed)
(Assignee)

Comment 2

4 years ago
No it's fine :)
Just checked, they still exists, I'll submit a patch.
BTW, do we have a deadline ?
Assignee: nobody → nefzaoui.ahmed
Flags: needinfo?(nefzaoui.ahmed)

Comment 3

4 years ago
Thanks, Ahmed! There is no deadline. I am just trying to find Arabic language and RTL improvements that (if low risk) might make gaia approval for 2.0, and if not it can land in 2.1. Thanks!
(Assignee)

Comment 4

4 years ago
Created attachment 8441938 [details] [review]
PR to Github

However I'm not sure about who to flag for r?
Stephany, is that you who should I be flagging?

Comment 5

4 years ago
Hi Ahmed,

Very sorry for the delay. If you click on Details, you will see a new set of flags. If you go to set the "review" ? you will "suggested reviewers" appear and you should choose one of those suggested people. Please let me know if you have any other questions. Thanks!

Updated

4 years ago
Attachment #8441938 - Flags: review?(fabrice)
Component: General → Gaia
Comment on attachment 8441938 [details] [review]
PR to Github

Redirecting the review to Tim.
Attachment #8441938 - Flags: review?(fabrice) → review?(timdream)
Attachment #8441938 - Flags: review?(timdream) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
master: https://github.com/mozilla-b2g/gaia/commit/5a804cb3b3dc348927c1912d1e6f1b21d91b3826
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S5 (4july)

Comment 8

4 years ago
Issue is verified fixed on 2.1 flame and 2.0 flame 

Drawers are aligned correctly and look like they are no longer broken.

Flame 2.1

Device: Flame 2.1 (319mb)(Kitkat Base)(Shallow Flash)
BuildID: 20141126001202
Gaia: db2e84860f5a7cc334464618c6ea9e92ff82e9dd
Gecko: 211eae88f119
Version: 34.0 (2.1)
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Flame 2.0

Device: Flame 2.0 (319mb)(Kitkat Base)(Shallow Flash)
Build ID: 20141126000203
Gaia: f9d6e3d83c3922e9399a6c27f5ce4cdd27bdfd05
Gecko: 45112935086f
Version: 32.0 (2.0)
Firmware Version: v188-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
Flags: needinfo?(ktucker)

Updated

4 years ago
Status: RESOLVED → VERIFIED
status-b2g-v2.0: --- → verified
status-b2g-v2.1: --- → verified
QA Whiteboard: [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Mass Edit: adding the [rtl-meta]
Whiteboard: [rtl-meta]
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][rtl-impact]
Whiteboard: [rtl-meta]
You need to log in before you can comment on or make changes to this bug.