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

Extension pages are half translated when browser and extension are on different language #6308

Closed
srirambv opened this issue Oct 3, 2019 · 7 comments · Fixed by brave/ethereum-remote-client#78

Comments

@srirambv
Copy link
Contributor

srirambv commented Oct 3, 2019

Description

Reported by @SergeyZhukovsky.
Extension pages are half translated when browser and extension are on different language

Steps to Reproduce

Scenario 1:

  1. Set browser language to English and add Russian to the list of browser languages
  2. Enable crypto wallet
  3. Extension page has mixed translations

Scenario 2:

  1. Set browser language to English and add Russian to the list of browser languages
  2. Add an extension like MM and set extension language to Russian
  3. Extension page has mixed translations

Actual result:

image
image

Expected result:

All text should be translated

Reproduces how often:

Easy

Brave version (brave://version info)

Version/Channel Information:

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

Other Additional Information:

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

Miscellaneous Information:

@kjozwiak
Copy link
Member

kjozwiak commented Oct 8, 2019

@bbondy should we move this into https://github.com/brave/brave-browser/milestone/55 as we're going to need a new build for #5698 and possibly #5106?

@kjozwiak kjozwiak added this to the Ethereum Remote Client 1.0.10 milestone Oct 8, 2019
@kjozwiak
Copy link
Member

kjozwiak commented Oct 9, 2019

@ryanml @bbondy looks like I can still reproduce this using https://github.com/brave/ethereum-remote-client/releases/tag/0.1.15 on both Windows 10 x64 & macOS 10.14.6 x64. Examples:

macOS 10.14.6 x64 example:

Screen Shot 2019-10-09 at 1 01 21 AM

Windows 10 x64 example:

Annotation 2019-10-09 010448

@srirambv can you check Linux and see if you can reproduce on your end with https://github.com/brave/ethereum-remote-client/releases/tag/0.1.15?

@ryanml ryanml reopened this Oct 9, 2019
@srirambv
Copy link
Contributor Author

srirambv commented Oct 9, 2019

Reproduced on Linux with 0.1.15
image

@bbondy
Copy link
Member

bbondy commented Oct 9, 2019

With regards to newLocalWalletSubText I just cleared out the english strings on each language and did an order to get the string picked up for translation. Once that's done in a few days I can pull the translations and it'll be included in the next extension update.

@kjozwiak kjozwiak removed this from the Ethereum Remote Client 1.1.15 milestone Oct 9, 2019
bbondy added a commit to brave/ethereum-remote-client that referenced this issue Oct 9, 2019
ryanml pushed a commit to brave/ethereum-remote-client that referenced this issue Dec 5, 2019
@bbondy bbondy added the priority/P3 The next thing for us to work on. It'll ride the trains. label Dec 20, 2019
@bbondy
Copy link
Member

bbondy commented Dec 20, 2019

@srirambv can you check again?

@srirambv
Copy link
Contributor Author

This still happens on current component and the rebased component
Screen Shot 2020-01-10 at 12 07 45 PM

@bbondy
Copy link
Member

bbondy commented Jan 17, 2020

Closing as dupe of #7791

@srirambv srirambv removed the feature/web3/wallet Integrating Ethereum+ wallet support label Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants