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

Incognito still being generated from translated strings #2147

Closed
bbondy opened this issue Nov 15, 2018 · 7 comments
Closed

Incognito still being generated from translated strings #2147

bbondy opened this issue Nov 15, 2018 · 7 comments
Assignees

Comments

@bbondy
Copy link
Member

bbondy commented Nov 15, 2018

This issue #738 changed Incognito -> Private for all source strings, but the translations are still being preserved. I think we need to add some 1 time code to clear out the translations, and maybe also to make sure we don't re-upload them when we do npm run push_l10n

@tildelowengrimm tildelowengrimm added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Nov 27, 2018
@tildelowengrimm tildelowengrimm added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P5 Not scheduled. Don't anticipate work on this any time soon. labels Nov 28, 2018
@bbondy bbondy added this to the 1.x Backlog milestone Dec 2, 2018
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Oct 14, 2019

Still reproduces with

Brave 0.69.135 Chromium: 77.0.3865.120 (Official Build) (64-bit)
Revision 416d6d8013e9adb6dd33b0c12e7614ff403d1a94-refs/branch-heads/3865@{#884}
OS macOS Version 10.13.6 (Build 17G5019)

Noticed English (UK) uses Incognito instead of Private terminology.

Screen Shot 2019-10-14 at 15 19 51

Screen Shot 2019-10-14 at 15 20 06

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 15, 2019

Reproduced in Windows 10 x64 - 0.70.116 - Encountered the issue for UK, INDIA and Germany locales

image

Germany:

image

@btlechowski
Copy link

I closed the wrong issue. Sorry.

@btlechowski
Copy link

btlechowski commented Oct 21, 2019

Still reproducible on Linux #6564

Affected locales: en_GB/en_AU/en_NZ
Not affected locales: en_US

image

Brave 0.70.119 Chromium: 78.0.3904.50 (Official Build) (64-bit)
Revision 2accdc52c79976e264cd2694df6db31d1fccd8e8-refs/branch-heads/3904@{#658}
OS Ubuntu 18.04 LTS

@petemill
Copy link
Member

petemill commented Nov 1, 2019

Hmm, I'm investigating this but not sure I understand what happened yet.

About a year ago, we would have submitted changes in the source US strings from e.g. "New Incognito Window" to "New Private Window".

However, in transifex looks like we (or it automatically) accepted the original translations for certain languages. So British English has the string translated (and approved) as "New Incognito Window"

Similarly, spanish

But German does have the correct translation

@bsclifton
Copy link
Member

Fixed with brave/brave-core#3922

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 11, 2019

Verification passed on

Brave 0.71.114 Chromium: 78.0.3904.97 (Official Build) (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Locale UK
image

Locale India
image

Locale Germany
image

Locale Australia
image

Locale New Zealand
image

Verification passed on

Brave 0.71.114 Chromium: 78.0.3904.97 (Official Build) (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Ubuntu 18.04 LTS

en_GB
image

en_AU
image

en_NZ
image

de_DE
image

Verified passed with

Brave 0.71.114 Chromium: 78.0.3904.97 (Official Build) (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS macOS Version 10.13.6 (Build 17G5019)

EN UK
EN-UK1
EN-UK2
EN-UK3
EN-UK4

EN AU
EN-AU1
EN-AU2
EN-AU3
EN-AU4

DE
DE1
DE2
DE3
DE4

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

No branches or pull requests

9 participants