822779 - [FTE] [Portuguese] Text Overlap in Tutorial screen

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::First Time Experience, defect, P1)

Description

[Marcia Knous [:marcia]]

Attachment: screenshot of issue

unagi, seen while running:

Gecko: 57b016ab196e
Gaia: b8524e43d4eb

STR
1. Set language to Portuguese in First Run
2. Skip through the screens to get to the tutorial
3. Observe the attached screenshot

Nominating since this is bad first time experience for this language.

Comment 1

[Maria Angeles Oteo (:oteo)]

I hope it will be fixed with 818588 lands as it includes shorter strings

Comment 2

[Fernando Campo (:fcampo)]

To avoid this kind of things we should include notes on maximum of characters for each label, or css rules to lower the size of the text when gets too big.

Asking for some UX feedback to check better options

Comment 3

[Victoria Gerchinhoren [:vicky]]

We defined 3 lines of text as maximum space, I count a maximum of 110 carachters including blank spaces. But, the recommendation for this kind of process is "less is more", avoid long explanations and keep it simple. Take into account the both image and text support each other.

Thanks!

Comment 4

[Fernando Campo (:fcampo)]

So the fix for this (and other languages) would be using smaller texts with a 110 char limit?
Is this acceptable for the l10n-drivers?

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Fernando Campo (:fcampo) from comment #2)
> To avoid this kind of things we should include notes on maximum of
> characters for each label, or css rules to lower the size of the text when
> gets too big.
>

This is hard since the size depends on the fonts used for each character :(

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Fernando Campo (:fcampo) from comment #4)
> So the fix for this (and other languages) would be using smaller texts with
> a 110 char limit?
> Is this acceptable for the l10n-drivers?

Smaller text sounds the best option available for now.

Comment 7

[Axel Hecht [:Pike]]

I don't think that bug 818588 had an impact here.

Jeferson, could you shorten the strings reasonably?

The other thing I see on that screenshot that we could make smaller is the "tap" image, how about we use some CSS to scale that down to the available space if need be?

Comment 8

[Jeferson Hultmann]

(In reply to Axel Hecht [:Pike] from comment #7)
> Jeferson, could you shorten the strings reasonably?

I just pinged B2G localizers.

Comment 9

[Fernando Campo (:fcampo)]

As this is decided to be solved shortening the strings, de-assigning myself

Comment 10

[Axel Hecht [:Pike]]

fcampo, I don't see that decision documented here?

All I see is that we asked if that's feasible for the pt-BR locale, but I don't see a response on that here.

Comment 11

[Ben Francis [:benfrancis]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/7339

Discussed with Patryk from UX and agreed on these font sizes.

Comment 12

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 698749 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/7339

https://github.com/mozilla-b2g/gaia/commit/e95e507b58d74cce2688068a5ee47b22899c003d

Comment 13

[Rebecca Billings [:rbillings]]

The 1/12/13 still repros this problem when selecting Portuguese as a language.

Comment 14

[Axel Hecht [:Pike]]

Rebecca, can you give us an updated screenshot?

Comment 15

[Marcia Knous [:marcia]]

Attachment: Screenshot running FTU from Developer Mode

Here is what I see using the latest nightly that I just smoketested today. I can test a fresh flash as well.

Comment 16

[Jason Smith [:jsmith]]

(In reply to Rebecca Billings from comment #13)
> The 1/12/13 still repros this problem when selecting Portuguese as a
> language.

Please file a new bug for the issue you encountered and link it as a dependency on this bug. Reopening makes tracking blockers a royal pain, so I'd suggest filing followups and linking appropriately.

Comment 17

[Marcia Knous [:marcia]]

Fresh flash works for me as well, I get the same screenshot posted above in Comment 15.

(In reply to Marcia Knous [:marcia] from comment #15)
> Created attachment 701529 [details]
> Screenshot running FTU from Developer Mode
> 
> Here is what I see using the latest nightly that I just smoketested today. I
> can test a fresh flash as well.