Closed
Bug 1128981
Opened 10 years ago
Closed 10 years ago
[RTL] "No SIM inserted" message etc is misplaced for RTL languages
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(feature-b2g:2.2+, b2g-v2.2 verified, b2g-master verified)
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
(Whiteboard: [systemsfe])
Attachments
(1 file)
46 bytes,
text/x-github-pull-request
|
kgrandon
:
review+
bajaj
:
approval-gaia-v2.2+
|
Details | Review |
STR: With no SIM installed, change language to Arabic, then pull down the Notification panel.
The Arabic version of the message
No SIM inserted
Insert a SIM to enable Usage
should be flush-right, but appears on the left-hand side of the screen, somewhat truncated at the edge.
I would post a screenshot, but am having trouble getting my current build to produce one. :(
Assignee | ||
Comment 1•10 years ago
|
||
Also, the second line of this text (the Arabic rendering of "Insert a SIM to enable Usage") is clipped at the bottom, making some of the characters illegible. This happens because of the use of an overly-constrained line height in the CSS for the p.meta element.
Assignee | ||
Comment 2•10 years ago
|
||
Comment 3•10 years ago
|
||
Thanks for the patch Jonathan! Please request review from a system peer like :alive or :kgrandon.
Component: Gaia::Cost Control → Gaia::System
Flags: needinfo?(jfkthame)
Whiteboard: [systemsfe]
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8558590 [details] [review]
[PR] fix the Cost Control widget rendering for RTL languages
Oops, I thought I'd flagged this for review already, but apparently not. Sorry for the delay.
Flags: needinfo?(jfkthame)
Attachment #8558590 -
Flags: review?(kgrandon)
Updated•10 years ago
|
feature-b2g: --- → 2.2+
Updated•10 years ago
|
Assignee: nobody → jfkthame
Comment 6•10 years ago
|
||
Comment on attachment 8558590 [details] [review]
[PR] fix the Cost Control widget rendering for RTL languages
This seems to do the job for me, thanks!
Attachment #8558590 -
Flags: review?(kgrandon) → review+
Comment 7•10 years ago
|
||
Comment 8•10 years ago
|
||
Comment on attachment 8558590 [details] [review]
[PR] fix the Cost Control widget rendering for RTL languages
Requesting uplift for this RTL feature.
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Implementation.
[User impact] if declined: Poor RTL experience when viewing the usage widget.
[Testing completed]: Manual.
[Risk to taking this patch] (and alternatives if risky): Simple styling change, so low risk.
[String changes made]: None.
Attachment #8558590 -
Flags: approval-gaia-v2.2?(bbajaj)
Updated•10 years ago
|
Attachment #8558590 -
Flags: approval-gaia-v2.2?(bbajaj) → approval-gaia-v2.2+
Comment 9•10 years ago
|
||
status-b2g-v2.2:
--- → fixed
Target Milestone: --- → 2.2 S5 (6feb)
Comment 10•10 years ago
|
||
This issue is verified fixed on the latest Nightly 3.0 build but is not yet on the latest night 2.2 Build. Will verify that one on another day.
Results: The "No Sim inserted" message is correctly on the right side and does not get truncated.
Environmental Variables:
Device: Flame 3.0
BuildID: 20150206010204
Gaia: 94af4b42d2ace6c9f38f31de77240604fac68af1
Gecko: 7c5f187b65bf
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 38.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:38.0) Gecko/38.0 Firefox/38.0
Environmental Variables:
Device: Flame 2.2
BuildID: 20150206002505
Gaia: a52999ce7f783177deb17e267bf003a53e6fde06
Gecko: 01446d5231ef
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0a2 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Comment 11•10 years ago
|
||
Jayme, assigning this to you so you can verify 2.2 on Monday.
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker) → needinfo?(jmercado)
Comment 12•10 years ago
|
||
This issue is verified fixed on Fame 2.2.
Results: The "No SIM inserted message is correctly on the right and is not cut off.
Environmental Variables:
Device: Flame 2.2
BuildID: 20150209002504
Gaia: e827781324cbde91d2434b388f5dead3303a85ee
Gecko: 0552759956d3
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0a2 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(jmercado) → needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Comment 13•10 years ago
|
||
Test case has been added in moztrap:
https://moztrap.mozilla.org/manage/case/15475/
Flags: in-moztrap+
You need to log in
before you can comment on or make changes to this bug.
Description
•