Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Preferences : Payments : Text clipping for localized strings. #3539

Closed
alexwykoff opened this issue Aug 29, 2016 · 4 comments
Closed

Preferences : Payments : Text clipping for localized strings. #3539

alexwykoff opened this issue Aug 29, 2016 · 4 comments

Comments

@alexwykoff
Copy link
Contributor

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
While testing #3414 it was discovered that 'Brave Payments Beta' was being clipped for non-English languages.

Expected behavior:
The translated string for 'Brave Payments Beta' should be visible for Bengali (Bangladesh), German, Spanish, French, Hindi, Indonesian, Japanese, Dutch, Portuguese, Tamil, Telegu, Turkish, and Ukranian.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Tested on OS X
  • Brave Version:
    0.11.6 pre-beta6
  • Steps to reproduce:
    1. With the latest build, change your language to one of the affected languages above.
    2. Navigate to Preferences -> Payments, observe the top of the page.
  • Screenshot if needed:
    screen shot 2016-08-29 at 3 52 21 pm
  • Any related issues:
@jkup
Copy link
Contributor

jkup commented Aug 30, 2016

This is fixed (for now) as the strings have been updated to just say Brave Software. Although as translations come in they might break the design.

@bradleyrichter the problem here is just that in order to get the text "Brave Payments" to stick to the bottom I (think I) need to use position absolute which means that it can't have a flexible width.

Tonight I don't think it's a blocker for 11.6 because there are currently no translations for that string so we're back to them all saying "Brave Payments" which fits just fine.

Happy to work on a more long-term solution ASAP.

@luixxiul
Copy link
Contributor

Please be careful when you use pixels, from a perspective of localization. Text overflow happens easily. cf: #2096

@luixxiul luixxiul added the l10n label Aug 30, 2016
@bradleyrichter
Copy link
Contributor

oops. This shouldn't be localized. It's a brand name. The fix is to remove it from transflex or make a logo file.

@bradleyrichter
Copy link
Contributor

@jkup Please remove the translflex tag from the Top Logo-title so this instance skips the translation. Leave the rest as is so they are translated.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants