1072029 - Notifications layout should not be empty or misaligned in Arabic.

Status: VERIFIED FIXED

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

Description

[Stephany Wilkes]

Attachment: 2014-09-23-14-56-16.png

See attached screenshot for illustration of how notifications display is broken when Arabic is selected. 

UX considers this a blocker as Notifications are neither legible nor usable.

Comment 2

[Ahmed Nefzaoui [:Nefzaoui]]

Attachment: Link to Github pull-request

Comment 3

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8494538 [details]
Link to Github pull-request

Thank you for the fix, please get ui-review+ before landing since I didn't check the visual myself.

Comment 4

[Ahmed Nefzaoui [:Nefzaoui]]

Attachment: After Patch - 2014-09-24-22-58-10

Thanks Tim :)
Stephany? :)

Comment 5

[Stephany Wilkes]

Comment on attachment 8494568 [details]
After Patch - 2014-09-24-22-58-10

Beautiful. Thanks, Ahmed! ui-review+

Comment 6

[Ahmed Nefzaoui [:Nefzaoui]]

Thanks :)

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/gaia/commit/65f9fd643bd1aaee74c217ec8c2318035152e23f

Comment 8

[Sue]

Attachment: 2015-01-30-11-44-50.png

This issue has been verified pass on Flame 2.2, notifications layout displayed right-aligned correctly in Arabic.
See attachment:2015-01-30-11-44-50.png
Rate:0/5

Flame 2.2 build:
Gaia-Rev        6e494f1d2676d231abba7dcc2e2822d1170d2d02
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/5e6fac01a72f
Build-ID        20150129003432
Version         37.0a2
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150129.042943
FW-Date         Thu Jan 29 04:29:53 EST 2015
Bootloader      L1TC000118D0

Comment 9

[Alissa（leave from Mozilla）]

Test case has been added in moztrap:
https://moztrap.mozilla.org/manage/case/15553/
https://moztrap.mozilla.org/manage/case/15676/