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

UK English sometimes Private/Private Window is not capitalized #6872

Closed
LaurenWags opened this issue Nov 11, 2019 · 8 comments
Closed

UK English sometimes Private/Private Window is not capitalized #6872

LaurenWags opened this issue Nov 11, 2019 · 8 comments

Comments

@LaurenWags
Copy link
Member

Description

While testing #2147 I noticed that the capitalization of Private Window isn't consistent across translations.

For example, here is English (US). Private/Private Window is capitalized as I would expect:
hamburger menu:
EN-US1

on the window:
EN-US2

from Brave menu (macOS):
EN-US3

right click:
EN-US4

Compared to English (UK), only 2 of the 4 items above are capitalized correctly - Hamburger menu and Brave menu (macOS):
EN-UK1
EN-UK4

On the window and right click, Private/Private Window is not capitalized as I would expect:
EN-UK2
EN-UK3

Steps to Reproduce

Use a language other than English (US).

  1. Launch Brave with a clean profile.
  2. View the following places:
  • hamburger menu
  • Private Window
  • right click on a link to view context menu
  • Brave menu (macOS/Linux)
  1. See inconsistencies across capitalization.

Actual result:

English (AU) - is similar to English UK:
EN-AU1
EN-AU2
EN-AU3
EN-AU4

DE - a little different, sometimes Private isn't capitalized, but Window is
DE1
DE2
DE3
DE4

Expected result:

Consistency in capitalization for Private/Private Window

Reproduces how often:

easily

Brave version (brave://version info)

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)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the dev channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@bsclifton
Copy link
Member

cc: @petemill (I know you fixed the hamburger menu usage of this via brave/brave-core#3922)

@petemill
Copy link
Member

🤦‍♂

@bsclifton
Copy link
Member

Fixed in 0.71.x with brave/brave-core#3970 (other versions will get fixed when pulling translations for C79)

@LaurenWags
Copy link
Member Author

To confirm, brave/brave-core#3970 will only fix English (UK) correct, no other languages?

@bsclifton
Copy link
Member

@LaurenWags that is correct! 😄

@petemill
Copy link
Member

petemill commented Nov 11, 2019

@LaurenWags with regards to other languages, I learnt (when dealing with the translations for these strings, although it would apply to many other strings we have) that different languages do not do 'title case'. So, we'd need to take it on a case-by-case basis as to whether the translation for a specific language is was correctly translated to title-case when appropriate for that language.

I'll amend the title of this issue to make it clear that it only applies to UK English.

@petemill petemill changed the title sometimes Private/Private Window is not capitalized UK English sometimes Private/Private Window is not capitalized Nov 11, 2019
@LaurenWags
Copy link
Member Author

That's fair. Mostly wanted to clarify so when @brave/legacy_qa goes to test this one, they don't bother with the other languages I had listed. We can do as you suggest and deal with them on a case by case basis and log issues appropriately as they come up. cc @kjozwiak @rebron

@LaurenWags
Copy link
Member Author

LaurenWags commented Nov 12, 2019

Verified passed with

Brave 0.71.115 Chromium: 78.0.3904.97 (Official Build) (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified STR from description
    hamburger menu

Screen Shot 2019-11-12 at 10 14 24 AM

Private Window - logged #6890
Screen Shot 2019-11-12 at 10 15 05 AM

right click on a link to view context menu - logged #6891
Screen Shot 2019-11-12 at 10 15 33 AM

Brave menu (macOS/Linux)
Screen Shot 2019-11-12 at 10 15 48 AM

Verification passed on

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

en_GB is not capitalized at all on Linux

  • Verified STR from description
    image
    image
    image

Verification passed on

Brave 0.71.115 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)
  • Verified STR from the description

image

image

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

5 participants