Closed
Bug 1028701
Opened 10 years ago
Closed 10 years ago
[RTL] Wrong position for tick icon on scroll list
Categories
(Firefox OS Graveyard :: Gaia, defect)
Firefox OS Graveyard
Gaia
Tracking
(b2g-v2.0 affected, b2g-v2.1 verified)
VERIFIED
FIXED
People
(Reporter: mannai.med.mack, Assigned: mannai.med.mack)
References
Details
(Keywords: rtl)
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
vingtetun
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:33.0) Gecko/20100101 Firefox/33.0 (Beta/Release)
Build ID: 20140610030202
Steps to reproduce:
Go to Settings > Language > Click the list and select Arabic (Or any other RTL based language)
Actual results:
The tick icon is on the right of the selected element
Expected results:
The tick icon should be on the left, and the text should be aligned to right
Updated•10 years ago
|
Here is my patch for this bug, added Vivien to review it.
Attachment #8444119 -
Flags: review?(21)
Comment 2•10 years ago
|
||
Comment on attachment 8444119 [details] [diff] [review]
1028701.patch
Review of attachment 8444119 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM. Thanks Mack for the patch!
Attachment #8444119 -
Flags: review?(21) → review+
Comment 3•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 4•10 years ago
|
||
Hey Vivien,
There is still room for improvement in this code, e.g.:
- Using *only* background position instead of using the whole repeated code:
> background: transparent url(value_selector/images/icons/checked.png) left 1.2rem top 50% / 2.1rem no-repeat;
- Removing repeated properties that did not change during the transition from LTR to RTL such as margin-left and color
- Most importantly putting the RTL code at the end of the file with the rest of RTL selectors for consistency with every other CSS file we have. :)
I will open a follow up bug for this one and try to get a PR ready,
Thanks! :)
Flags: needinfo?(21)
Flags: needinfo?(21)
Issue is verified fixed on 2.1 flame
Ticker is in the correct position
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
Issue still occurs on the 2.0 flame
adding affected flag till fix can be verified
Ticker still overlapps words on the right
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
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.0:
--- → affected
status-b2g-v2.1:
--- → verified
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
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.
Description
•