929733 - [l10n][el] User Settings page, "Account" tab Greek translation doesn't fit inside tab

Status: RESOLVED WONTFIX

(Marketplace Graveyard :: Consumer Pages, defect, P4)

Description

[Thanos Lefteris [:alefteris]]

Steps to reproduce:

Open the Marketplace app from Firefox OS and click the gear icon to open the user settings, using the Greek translation. Can also be tested by visiting https://marketplace-dev.allizom.org/settings?lang=el with desktop Firefox and using the Responsive Design View tool.


Actual results:

The "Account" translation "Λογαριασμός" (1st tab) can't fit inside the tab and the last character of the word wraps bellow.


Expected results:

Word fits in tab.

Comment 1

[Alfredos-Panagiotis Damkalis [:fredy]]

Attachment: 2013-10-22-17-30-23.png

Confirm and adding screenshot

Comment 2

[Matt Basta [:basta]]

Wil, can you make a call here? We're already shrinking the text. Should this go back to UX or should we send this to the localizers?

Same with bug 929722

Comment 3

[Wil Clouser [:clouserw]]

Alexandros has been doing the L10n here.  Alexandros - is there anything we can do to shorten those strings?

Comment 4

[Christopher Van Wiemeersch [:cvan]]

We linefit at two lines; I agree it looks a bit awkward but it's not wrapping incorrectly.

Comment 5

[alco]

Disappointingly enough, I cannot change the string to something shorter since I would change the meaning. That's a common word used in greek. Could we find another workaround?

Comment 6

[Wil Clouser [:clouserw]]

I can pass it over to UX, but nothing is jumping out at me.  Since it's technically not broken (it's wrapping) it's still going to be a p4.

Comment 8

[Alfredos-Panagiotis Damkalis [:fredy]]

Attachment: example from 15px to 14px

Decreasing by 1px the body font-size (from 15px to 14px) in https://marketplace.firefox.com/media/fireplace/css/include.css solves the problem, as it is shown in attached screenshot.

Comment 9

[Tony Santos [:tsmuse]]

(In reply to Alfredos-Panagiotis Damkalis [:fredy] from comment #8)
> Created attachment 829865 [details]
> example from 15px to 14px
> 
> Decreasing by 1px the body font-size (from 15px to 14px) in
> https://marketplace.firefox.com/media/fireplace/css/include.css solves the
> problem, as it is shown in attached screenshot.

Ship it :)

Comment 10

[Matt Basta [:basta]]

That's not a satisfactory fix. The way the font is rendered on the device is not guaranteed across platforms. Just because it works in this browser/OS combo doesn't mean it'll work universally. We should work to have a sounder fix.

Comment 11

[Tony Santos [:tsmuse]]

this menu is changing with the feed, so if there isn't a patch we can put on this one, then it will be gone soon.

Comment 12

[Wil Clouser [:clouserw]]

(In reply to Tony Santos [:tsmuse] from comment #11)
> this menu is changing with the feed, so if there isn't a patch we can put on
> this one, then it will be gone soon.

If you're saying wontfix the bug, please do.  Otherwise give us direction, thanks.

Comment 13

[Tony Santos [:tsmuse]]

If the solution in comment 8, that I endorsed,  isn't sufficient than I'll close it as wontfix.