Closed Bug 1136996 Opened 9 years ago Closed 9 years ago

[RTL][2.2 Only][Notifications] Not enough padding on top edge when first line of notification is in Arabic

Categories

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

ARM
Gonk (Firefox OS)
defect

Tracking

(b2g-v2.2 wontfix, b2g-v2.5 unaffected, b2g-master unaffected)

RESOLVED WONTFIX
Tracking Status
b2g-v2.2 --- wontfix
b2g-v2.5 --- unaffected
b2g-master --- unaffected

People

(Reporter: pcheng, Unassigned)

References

Details

(Whiteboard: [3.0-Daily-Testing][systemsfe])

Attachments

(4 files)

Description:
Text looks like it's touching the top edge of screen when first line of the notification is in Arabic.

Prerequisites:
In RTL language

STR:
1) Press Power button + Volume down button to take a screenshot

- observe the notification

2) Go to Calendar, set up an event, and receive the event's notification

- observe the notification

Expected: The blue notification banner shows text within a certain padding surrounding the text

Actual: Text is touching the top edge of the notification. See screenshots.

Device: Flame 3.0 Master (full flash 319MB)
BuildID: 20150225010244
Gaia: f6bfd854fe4746f21bc006eac145365e85f98808
Gecko: 0a8b3b67715a
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 39.0a1 (3.0 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
This issue occurs on Flame 2.2.

Device: Flame 2.2
BuildID: 20150225002505
Gaia: ca64f2fe145909f31af266b1730874051ba76c78
Gecko: 16804008c29f
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [rtl-impact][QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Whiteboard: [3.0-Daily-Testing]
Priority: -- → P2
Whiteboard: [3.0-Daily-Testing] → [3.0-Daily-Testing][systemsfe]
QA Whiteboard: [rtl-impact][QAnalyst-Triage?] → [rtl-impact][QAnalyst-Triage+]
Flags: needinfo?(ktucker)
I think this one is also a layout problem just like https://bugzilla.mozilla.org/show_bug.cgi?id=1136860#c2. I tried another language which top edge also has not enough padding.
QA Whiteboard: [rtl-impact][QAnalyst-Triage+] → [rtl-impact][QAnalyst-Triage+][MGSEI-Triage+]
Attached image AreisKK_v2.5.png
This issue cannot be reproduced on latest FlameKK v2.2&v2.5 build, AriesKK v2.5 build in Arabic language by the same STR in comment 0.
Actual Result:
The blue notification banner shows text within a certain padding surrounding the text.

See attachment:AreisKK_v2.5.png.
Reproducing rate:0/10

Device:FlameKK v2.2 build
Build ID               20150811032503
Gaia Revision          102f1299e9eafe3760e1deb44d556b5c4f36b5af
Gaia Date              2015-08-06 20:46:56
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/13c447ed929e
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150811.065030
Firmware Date          Tue Aug 11 06:50:39 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device:FlameKK v2.5 build
Build ID               20150811150203
Gaia Revision          2d7f369fd923b6df3b00d76844c413c1202c04ba
Gaia Date              2015-08-11 09:58:44
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/38c1ea9ccae31700630f1fe0d651e94c0c5b9e1d
Gecko Version          43.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150811.182056
Firmware Date          Tue Aug 11 18:21:06 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device:AriesKK v2.5 build
Build ID               20150810201912
Gaia Revision          fa89e03dc489e79baa0e74cb1d205260c7924caa
Gaia Date              2015-08-10 10:07:57
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/3f135a8ead22
Gecko Version          43.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150810.194258
Firmware Date          Mon Aug 10 19:43:05 UTC 2015
Bootloader             s1

Note: Per Comment 2, keep the original status for other languages.
No longer blocks: notifications-rtl
Hi Lancy,
Please just mark status-b2g:2.5 unaffected instead of removing blocking from meta bug 1072027.
Could you also verify whether the issue still happen on master?
Thanks!
Flags: needinfo?(yulan.zhu)
Attached image Notification_master.png
Hi Josh,
This issue cannot be reproduced on latest AriesKK&FlameKK master in Arabic language by the same STR in comment 0.
Actual Result:
The blue notification banner shows text within a certain padding surrounding the text.
See attachment:Notification_master.png
Reproducing rate:0/10

Device:AriesKK master build
Build ID               20151117222127
Gaia Revision          28d63cf3bdc4417f7ad8cab2230f096bf9f6d3b5
Gaia Date              2015-11-17 07:35:12
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/8c3a0b58fd51c02c8aca138b0da58445becb037f
Gecko Version          45.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151117.214032
Firmware Date          Tue Nov 17 21:40:40 UTC 2015
Bootloader             s1

Device:FlameKK master 512mb  build
Build ID               20151117150203
Gaia Revision          28d63cf3bdc4417f7ad8cab2230f096bf9f6d3b5
Gaia Date              2015-11-17 07:35:12
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/8c3a0b58fd51c02c8aca138b0da58445becb037f
Gecko Version          45.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151117.183602
Firmware Date          Tue Nov 17 18:36:15 EST 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0
Flags: needinfo?(yulan.zhu) → needinfo?(jocheng)
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(jocheng)
Resolution: --- → WONTFIX
Summary: [RTL][Notifications] Not enough padding on top edge when first line of notification is in Arabic → [RTL][2.2 Only][Notifications] Not enough padding on top edge when first line of notification is in Arabic
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: