Closed Bug 1116630 Opened 10 years ago Closed 10 years ago

[RTL][Lockscreen] Notifications viewed on lockscreen are not right-aligned

Categories

(Firefox OS Graveyard :: Gaia::System, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

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

RESOLVED DUPLICATE of bug 1064148
2.2 S5 (6feb)
feature-b2g 2.2+
Tracking Status
b2g-v2.2 --- affected

People

(Reporter: onelson, Assigned: mikehenrty)

References

Details

(Whiteboard: [systemsfe])

Attachments

(4 files)

Summary (title) Field: [RTL][Lockscreen] Notifications viewed on lockscreen are not right-aligned Description: When a user receives a notification (missed call, text message, screenshot-taken) and it is viewed on the lockscreen, they will notice that in a RTL-defined language that the notification is not translated properly from right-to-left. Pre-Req: * Change Language to a RTL [Arabic] Repro Steps: 1) Update a Flame device to BuildID: 20141230010205 2) Lock phone. 3) Call test phone from a seperate device. 4) Text test phone from a seperate device. 5) From test phone, tap power button to view lockscreen display. 6) Take a screenshot of the lockscreen. 7) Observe orientation of the lockscreen notifications. Actual: Notifications observed on the lockscreen are not translated properly for a RTL-specific language. Expected: Notifications observed on the lockscreen respect the RTL linguistics. Environmental Variables: Device: Flame 2.2 Master (319mb)(Kitkat Base)(Full Flash) BuildID: 20141230010205 Gaia: 322ef5116a5827a30c9a3cd9b842449a9c66a5b3 Gecko: 67872ce17918 Gonk: a814b2e2dfdda7140cb3a357617dc4fbb1435e76 Version: 37.0a1 (2.2 Master) Firmware: V18D User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0 Repro frequency: 5/5 See attached: screenshot
QA Whiteboard: [QAnalyst-Triage?][rtl-impact]
Flags: needinfo?(pbylenga)
QA Whiteboard: [QAnalyst-Triage?][rtl-impact] → [QAnalyst-Triage+][rtl-impact]
Flags: needinfo?(pbylenga)
Looks like SMS is the only that is not respecting RTL. We should fix that to be consistent. Also adding qawanted to test notification across apps to confirm if the bug exists elsewhere
blocking-b2g: --- → 2.2+
Priority: -- → P1
Pending the results from the qawanted, I think we should downgrade this to P2 unless more severe breakage is revealed. The SMS notification formatting is incorrect but it appears to be usable.
Flags: needinfo?(bbajaj)
Keywords: qawanted
QA Contact: ychung
> Also adding qawanted to test notification across apps to confirm if the bug > exists elsewhere Hi Bhavana, This problem can be repro on Calendar, Bluetooth apps. See attachment: logcat_1811.txt & Flame2.2Arabic_screenshot.png Repro time: 18:11 Rate: 5/5 Flame 2.2 build: Gaia-Rev e4f9b5da3751798f9cc5d95f302c30722cc11fca Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/75a462a58d7a Build-ID 20150121002607 Version 37.0a2 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20150121.040751 FW-Date Wed Jan 21 04:08:02 EST 2015 Bootloader L1TC000118D0
Keywords: qawanted
Attached image Flame2.2Arabic_screenshot (deleted) —
Attached file logcat (deleted) —
QA Contact: ychung
RTL update: marking required bugs as feature-b2g:2.2+ (and removing blocking flags)
blocking-b2g: 2.2+ → ---
feature-b2g: --- → 2.2+
Whiteboard: [systemsfe]
Assignee: nobody → mhenretty
Flags: needinfo?(bbajaj)
This has been already been addressed in bug 1064148 which in itself depends on bug 1058799. :)
Test case has been added in moztrap:15381
Flags: in-moztrap+
(In reply to Ahmed Nefzaoui [:Nefzaoui] from comment #8) > This has been already been addressed in bug 1064148 which in itself depends > on bug 1058799. :) Yeah, I see our patches are very similar. So is this bug a dupe of bug 1064148 then?
Flags: needinfo?(nefzaoui)
(In reply to Michael Henretty [:mhenretty] from comment #10) > (In reply to Ahmed Nefzaoui [:Nefzaoui] from comment #8) > > This has been already been addressed in bug 1064148 which in itself depends > > on bug 1058799. :) > > Yeah, I see our patches are very similar. So is this bug a dupe of bug > 1064148 then? Yes :)
Flags: needinfo?(nefzaoui)
(In reply to Ahmed Nefzaoui [:Nefzaoui] from comment #11) > (In reply to Michael Henretty [:mhenretty] from comment #10) > > (In reply to Ahmed Nefzaoui [:Nefzaoui] from comment #8) > > > This has been already been addressed in bug 1064148 which in itself depends > > > on bug 1058799. :) > > > > Yeah, I see our patches are very similar. So is this bug a dupe of bug > > 1064148 then? > > Yes :) Excellent, thank you!
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Target Milestone: --- → 2.2 S5 (6feb)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: