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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(feature-b2g:2.2+, b2g-v2.2 verified, b2g-master verified)

VERIFIED FIXED
2.2 S5 (6feb)
feature-b2g 2.2+
Tracking Status
b2g-v2.2 --- verified
b2g-master --- verified

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

(Whiteboard: [systemsfe])

Attachments

(1 file)

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. :(
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.
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]
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)
feature-b2g: --- → 2.2+
Assignee: nobody → jfkthame
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+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
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)
Attachment #8558590 - Flags: approval-gaia-v2.2?(bbajaj) → approval-gaia-v2.2+
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)
Jayme, assigning this to you so you can verify 2.2 on Monday.
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker) → needinfo?(jmercado)
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)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
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.

Attachment

General

Created:
Updated:
Size: