1157429 - [l10n] [Greek / Ελληνικά] The battery is low message runs outside the bounding box / notification area

Status: RESOLVED FIXED

(Mozilla Localizations :: el / Greek, defect)

Description

[Joshua Mitchell (Inactive)]

Attachment: GreekLowBattery.png

Description:
When the battery drops below 10% a message 'Your battery is low. Please plug in your charger.' is given. The text of this message in Greek runs onto a 3rd line, which is too many lines for the notification area, causing it to partially overflow onto the rest of the screen. If the current app is white (like settings) it makes that part of the notification difficult to see. (shown in screenshot)


Repro Steps:
1) Update a Flame to 20150422010202
2) Set the language to Greek
3) Let the power run down to 10% (or have one at 10% or less and plug and unplug the charge cable)


Actual:
Warning message overruns message area


Expected:
message will fit on two lines to fit within the notification area



Environmental Variables:
Device: Flame 3.0 (KK - Nightly - Full Flash - 319mem)
Build ID: 20150422010202
Gaia: 15134b080b5f406e5aa36f5136c17dafb4e31f64
Gecko: 946ac85af8f4
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 40.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0



Repro frequency: 5/5
See attached: screenshot

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

Joshua, the description of Gaia:L10n component is: "Gaia's localization library. (Bugs about specific localizations and translation requests should go into the Mozilla Localization product.)"

Please, when you report bugs against specific localizations, use Mozilla Localization:ab-CD component/product.

Comment 2

[Delphine Lebédel]

Joshua: can you please look on 2.2 and 2.1 and tell us what it's like there? We'll need to see if it's a regression or not. thanks!

Comment 3

[Joshua Mitchell (Inactive)]

This issue also occurs on 2.2 and 2.1

HOWEVER - on 2.1 there is a different, (but longer) message that actually extends to 4 lines and over-runs the notification area even more - see additional screenshot

Device: Flame 2.2 (KK - Nightly - Full Flash - 319mem)
Build ID: 20150423002502
Gaia: b838d0e7c163e66660dcb6e387d8339944a7a30e
Gecko: 8dce56574f28
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0


Device: Flame 2.1 (KK - Nightly - Full Flash - 319mem)
Build ID: 20150422001201
Gaia: bbe983b4e8bebfec26b3726b79568a22d667223c
Gecko: 685fa69b59dc
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 34.0 (2.1)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Comment 4

[Joshua Mitchell (Inactive)]

Attachment: Greek2.1.png

Comment 5

[Delphine Lebédel]

Asking Greek localizers if they can shorten his sentence! If not, we might need a UX fix to this (Greek is a shipping locale). Thanks!

Comment 6

[Iosif Psychas [:TheGallery]]

The other active FXOS localizer is Fredy, I am CCing him and requesting info. I think we might be able to shorten the second sentence. 

Fredy? Do you think we could remove "Please" from the 2nd sentence? For the first one we can replace the 2.1 string with 2.2's.

Comment 7

[Alfredos-Panagiotis Damkalis [:fredy]]

I want to try by removing the word "σας" from the string, if this doesn't work then will go to remove the "Please" word.

Comment 8

[Iosif Psychas [:TheGallery]]

Sure let's try that.

Comment 9

[Alfredos-Panagiotis Damkalis [:fredy]]

Attachment: patch for bug 1157429

Fix has been tested and works fine in v2.2 in flame and keon devices.

Needinfo flod for landing the change. Thanks!

Comment 10

[Francesco Lodolo [:flod]]

Commit
https://hg.mozilla.org/gaia-l10n/el/rev/cb428d10f115