Browser-laptop: Pocket was translated when it should not.

Created on 2 Mar 2017  路  10Comments  路  Source: brave/browser-laptop

Test plan

See https://github.com/brave/browser-laptop/pull/8110

Original issue description

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

  • Describe the issue you encountered:
    On the about:preferences#advanced page, it was discovered that 'Pocket' was translated.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Tested on OS X

  • Brave Version (revision SHA):
    0.13.5 rc 1

  • Steps to reproduce:

    1. Change your language to something other than English (I chose Chinese)
    2. Visit about:preferences#advanced
    3. Observe the final toggle
  • Actual result:
    Pocket is translated.

  • Expected result:
    As a brand, it should not, just as we do not translate Widevine or Flash.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    screen shot 2017-03-02 at 9 08 52 am

  • Any related issues:

Qchecked-Linux Qchecked-Win32 Qchecked-Win64 Qchecked-macOS Qtest-plan-specified bug l10n release-noteexclude

All 10 comments

It seems that this is an issue specific to Chinese.

https://github.com/brave/browser-laptop/search?l=INI&q=Pocket&utf8=%E2%9C%93

Is there a way to leave a note in Transifex to not to translate Pocket?

In Chinese a foreign product name is often translated in Chinese characters (for example Microsoft is called "寰蒋"). Still in this case, according to this official page the name should not be translated.

Leaving the note would help for a while but not the perfect solution as the note can be overlooked or ignored.

Moved to 0.13.7

Its an issue with many other languages not just Chinese. I see pocket is translated in Bengali and other languages as well

Removing milestone. If we'd like to see this done soon (and assigned a milestone), let's find an owner please 馃槃

What can be done in browser-laptop to fix this problem?

@Liunkae can we use a token like {{NAME}} in the middle of the string? For example:
Enable {{NAME}}

In browser-laptop, we'd have to do a manual string replace for this. But it would work nicely, IMO

Enable {{ExtentionName}} would be a global solution.

PR: #8110

I specified release-notes/exclude, but please feel free to change back if this is something we'd like to include 馃槃

Was this page helpful?
0 / 5 - 0 ratings

Related issues

briannyeko picture briannyeko  路  3Comments

octohedron picture octohedron  路  3Comments

luixxiul picture luixxiul  路  3Comments

luixxiul picture luixxiul  路  3Comments

bsclifton picture bsclifton  路  3Comments