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

Status: RESOLVED DUPLICATE of bug 1064148

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

Description

[Oliver Nelson [:oliverthor]]

Attachment: english_vs_arabic_lockscreen_notifications.png

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

Comment 1

[bhavana bajaj [:bajaj]]

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

Comment 2

[Dylan Oliver [:doliver]]

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.

Comment 3

[Shine(Leave from Mozilla)]

> 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

Comment 4

[Shine(Leave from Mozilla)]

Attachment: Flame2.2Arabic_screenshot

Comment 5

[Shine(Leave from Mozilla)]

Attachment: logcat

Comment 6

[Dylan Oliver [:doliver]]

RTL update: marking required bugs as feature-b2g:2.2+ (and removing blocking flags)

Comment 7

[GitHub Autolander Bot]

Attachment: [PullReq] mikehenrty:bug-1116630-rtl-lockscreen to mozilla-b2g:master

Comment 8

[Ahmed Nefzaoui [:Nefzaoui]]

This has been already been addressed in bug 1064148 which in itself depends on bug 1058799. :)

Comment 9

[Alissa（leave from Mozilla）]

Test case has been added in moztrap:15381

Comment 10

[Michael Henretty [:mikehenrty][:mhenretty]]

(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?

Comment 11

[Ahmed Nefzaoui [: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 :)

Comment 12

[Michael Henretty [:mikehenrty][:mhenretty]]

(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!