Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow up - Shields Panel and Rewards Panel are not translated to Norwegian on Windows #5546

Closed
btlechowski opened this issue Aug 5, 2019 · 6 comments
Labels
closed/no-milestone l10n OS/Windows priority/P2 A bad problem. We might uplift this to the next planned release. QA/Test-Plan-Specified QA/Yes

Comments

@btlechowski
Copy link

Follow up to #3425
Still reproducible on Windows. Works on MacOS.

Steps to Reproduce

  1. Change Brave language to Norwegian
  2. Open Brave Shields
  3. Open Rewards Panel

Actual result:

Shields panel is not translated
image

Rewards Panel is not translated:
image

Expected result:

Shields panel is translated
Rewards Panel is translated:

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.68.118 Chromium: 76.0.3809.87 (Offisiell delversjon) beta (64-bit)
Versjon 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Windows 7 Service Pack 1 (Build 7601.24494)
Brave 0.70.36 Chromium: 76.0.3809.87 (Offisiell delversjon) nightly (64-bit)
Versjon 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Windows 7 Service Pack 1 (Build 7601.24494)

cc @brave/legacy_qa @mkarolin @bsclifton

@rebron
Copy link
Collaborator

rebron commented Oct 1, 2019

@btlechowski Can you take a look? Is this already fixed?

@btlechowski
Copy link
Author

Still reproducible on all channels

Brave 0.69.130 Chromium: 77.0.3865.90 (Offisiell delversjon) (64-bit)
Versjon 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 7 Service Pack 1 (Build 7601.24519)
Brave 0.70.104 Chromium: 77.0.3865.90 (Offisiell delversjon) beta (64-bit)
Versjon 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 7 Service Pack 1 (Build 7601.24519)
Brave 0.71.74 Chromium: 77.0.3865.75 (Official Build) dev (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Windows 7 Service Pack 1 (Build 7601.24519)
Brave 0.72.62 Chromium: 77.0.3865.90 (Official Build) nightly (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 7 Service Pack 1 (Build 7601.24519)

image
image
image
image

@bsclifton
Copy link
Member

@mkarolin @atsyed87 is there a way to verify that we DO have translations for Norwegian on desktop (in Transifex)? Maybe there's a bug with pulling them? I'd have to think we've gotten some by now

@atsyed87
Copy link

atsyed87 commented Oct 2, 2019 via email

@mkarolin
Copy link
Contributor

mkarolin commented Oct 3, 2019

As far as I recall, we have proper localization for the browser chrome and webUI, but not for the Shields/Rewards extensions, so this is likely to do with how the JS side requests strings for the given locale.
The strings come down from Transifext as locale 'no' and we rename it to 'nb' when we package resources (see components/brave_extension/extension/brave_extension/BUILD.gn and components/brave_rewards/resources/extension/BUILD.gn).
It would be useful to check what locale is being specified when strings are requested from the extensions.

@btlechowski
Copy link
Author

No longer reproducible

Brave 1.4.95 Chromium: 80.0.3987.122 (Official Build) (64-bit)
Revision cf72c4c4f7db75bc3da689cd76513962d31c7b52-refs/branch-heads/3987@{#943}
OS Windows 7 Service Pack 1 (Build 7601.24544)

image
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/no-milestone l10n OS/Windows priority/P2 A bad problem. We might uplift this to the next planned release. QA/Test-Plan-Specified QA/Yes
Projects
None yet
Development

No branches or pull requests

5 participants