Closed
Bug 977268
Opened 10 years ago
Closed 10 years ago
[B2G][l10n][Multiple languages] 'Show Password' text is displayed on multiple lines in Join Hidden Network
Categories
(Firefox OS Graveyard :: Gaia::First Time Experience, defect)
Firefox OS Graveyard
Gaia::First Time Experience
ARM
Gonk (Firefox OS)
Tracking
(b2g-v1.4 fixed)
VERIFIED
FIXED
1.4 S3 (14mar)
Tracking | Status | |
---|---|---|
b2g-v1.4 | --- | fixed |
People
(Reporter: pbylenga, Assigned: flod)
References
Details
(Keywords: l12y, Whiteboard: locrun1.3)
Attachments
(3 files)
Description: "Show Password" text string on the Join Hidden Network screen (FTE) has line breaks instead of spaces for multiple locales. Locales unaffected use spaces as expected. Affected: German Greek Bulgarian Catalan Spanish French Hungarian Dutch Russian Serbian Cyrillic Unaffected: Slovak Turkish Bengali Czech English Croatian Italian Polish Portuguese (Brazil) Romanian Serbian Latin Repro Steps: 1) Update a Buri to BuildID: 20140226004002 2) Launch FTE through Developer Options 3) Select an affected language (ie. Greek) on first screen 4) On Wifi setup select Join Hidden Network and observe "Show Password" string. Actual: "Show Password" string uses line breaks instead of spaces. Expected: "Show Password" string uses spaces. v1.3 Environmental Variables: Device: Buri v1.3 MOZ BuildID: 20140226004002 Gaia: da50b2b62ed96993cf4df22e3d69c435b28506bd Gecko: 3c39d1e487d9 Version: 28.0 Firmware Version: v1.2-device.cfg Notes: Repro frequency: 5/5, 100% See attached: screenshot
Reporter | ||
Comment 1•10 years ago
|
||
This issue reproduces on v1.2 v1.2 Environmental Variables: Device: Buri v1.2 MOZ BuildID: 20140220004002 Gaia: 539a25e1887b902b8b25038c547048e691bd97f6 Gecko: 2ea6a65eea23 Version: 26.0 Firmware Version: v1.2-device.cfg
Updated•10 years ago
|
Flags: needinfo?(firefoxos-ux-bugzilla)
Comment 2•10 years ago
|
||
I cannot determine why a line break was used instead of a space, in the string (if in fact that's the source of the problem). Patryk, do you know if this is a visual design issue or something in the string itself?
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(padamczyk)
Comment 3•10 years ago
|
||
From our perspective, this is not intentional, might be a coding issue instead.
Flags: needinfo?(padamczyk)
Assignee | ||
Comment 4•10 years ago
|
||
Translations, no sign of line breaks http://transvision.mozfr.org/string/?entity=apps/communications/ftu/ftu.properties:showPassword&repo=gaia_1_3 My feeling is that the width for that checkbox label is smaller than the available space.
Assignee | ||
Comment 5•10 years ago
|
||
https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/ftu/css/style.css#L531 #hidden-wifi-authentication label p { width: 12rem; bottom: 0.4rem; left: 3rem; } Fixing it to 24rem seems to fit even Greek on a single line, which is the longest one.
Assignee | ||
Updated•10 years ago
|
Summary: [B2G][l10n][Multiple languages] 'Show Password' utilizes line breaks instead of spaces in Join Hidden Network → [B2G][l10n][Multiple languages] 'Show Password' text is displayed on multiple lines in Join Hidden Network
Assignee | ||
Comment 6•10 years ago
|
||
This lets the label use all the horizontal space available. It definitely works, not sure if that's the way this problem is supposed to be fixed.
Assignee: nobody → francesco.lodolo
Attachment #8383814 -
Flags: review?(etienne)
Assignee | ||
Comment 7•10 years ago
|
||
Keon running Italian build with the fix, label for "Show Password" replaced with the Greek one.
Comment 9•10 years ago
|
||
Comment on attachment 8383814 [details] [review] Pull request on Github Redirecting to an FTU peer :)
Attachment #8383814 -
Flags: review?(etienne) → review?(borja.bugzilla)
Comment 10•10 years ago
|
||
Comment on attachment 8383814 [details] [review] Pull request on Github Hi Francesco! I've added a small comment to the patch. Feedback+ by now and let's try a responsive solution! Let me know and I'll take a look to the final patch. Thanks!
Attachment #8383814 -
Flags: feedback+
Assignee | ||
Comment 11•10 years ago
|
||
Thanks, you're absolutely right: using a dynamic width instead of a fixed one is definitely better. I've updated the PR and tested it locally, it works fine. Not sure if you want me to set again the r?, also note that I'll need someone to check-in the patch when ready.
Flags: needinfo?(borja.bugzilla)
Comment 12•10 years ago
|
||
Comment on attachment 8383814 [details] [review] Pull request on Github R+ from my side. This patch is only CSS and is safe to land. We need to ensure that the text is shown properly in every language.
Attachment #8383814 -
Flags: review?(borja.bugzilla) → review+
Flags: needinfo?(borja.bugzilla)
Comment 13•10 years ago
|
||
Asking for 1.4? due to it's a surgical patch which let us to display this text properly in every language.
blocking-b2g: --- → 1.4?
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 14•10 years ago
|
||
Master: 4b9f58e5d79d3ff73e7515970a768cd1e5494885
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v1.4:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S3 (14mar)
Updated•10 years ago
|
blocking-b2g: 1.4? → ---
Assignee | ||
Comment 15•10 years ago
|
||
Verified on Keon with build 20140307, setting phone to el.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•