lightning-browser-extension icon indicating copy to clipboard operation
lightning-browser-extension copied to clipboard

Changing the language does not work

Open AnastasiaVolk opened this issue 3 years ago • 7 comments

Bug description

In the settings there is an option to change language, but it doesn't work

Screenshots

Image

Steps to reproduce the behaviour:

  1. Click on Alby Extension Icon
  2. Pop up window appears
  3. Click on the "menu" button in the right
  4. The menu appears
  5. Click on "settings"
  6. "Settings" page appears
  7. In the line "Language" try to change it

Expected behavior

The language changes to the chosen one

Suggestion

Either remove the option for changing or make it work

Alby information

Alby Version: [v1.12.0] Alby installed through Google Chrome

Device information:

OS: [macOS Monterey 12.0.1 ]
 Browser [chrome]
 Browser Version [Chrome Version 103.0.5060.114]

Are you working on this issue?

No

AnastasiaVolk avatar Jul 25 '22 16:07 AnastasiaVolk

Not all languages have been translated completely yet. This will happen over time via Weblate. I think this can be closed.

escapedcat avatar Jul 26 '22 02:07 escapedcat

Maybe it makes sense to remove this option before it's done? It gives the impression of a very raw product

AnastasiaVolk avatar Jul 26 '22 07:07 AnastasiaVolk

True, I understand that. Doubt translations will be 100% finished soon. We kinda wanted this in so people are aware of it.

Maybe we could improve the info here: image

  1. Change to "Translations are not 100% finished yet. Help us translate Alby into your language!
  2. "translate Alby into your language" link this to the Weblate service we use for translations

escapedcat avatar Jul 26 '22 07:07 escapedcat

Yes, then it turns out to be even useful! love it!

P.S. I think translations are very important, actually. For example, most Russians don't speak English at all. Neither do Spaniards. It might even stop them from using the product.

AnastasiaVolk avatar Jul 26 '22 08:07 AnastasiaVolk

I agree that the current setup is not correct, we should only show a language in the selector when the entire app has been translated. It doesn't make sense to show it otherwise, so we could only show English for now and then when we have full translations add the other languages.

secondl1ght avatar Jul 26 '22 14:07 secondl1ght

For the entire app to be translatable we need to switch all strings to i18n placeholders first. You up for it?

escapedcat avatar Jul 27 '22 02:07 escapedcat

PR #1244 is in for this issue, will now look into updating remaining strings into i18n placeholders.

secondl1ght avatar Aug 05 '22 22:08 secondl1ght

Will be done with the next release. Done in master already.

escapedcat avatar Sep 09 '22 08:09 escapedcat