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

Change "Brave Web Store" strings to "Web Store" #2983

Closed
mkarolin opened this issue Jan 16, 2019 · 1 comment · Fixed by brave/brave-core#1358
Closed

Change "Brave Web Store" strings to "Web Store" #2983

mkarolin opened this issue Jan 16, 2019 · 1 comment · Fixed by brave/brave-core#1358

Comments

@mkarolin
Copy link
Contributor

Because of localization issues with string containing tags and consequently brave/brave-core#1080 (which fixed #1013 and #2497) being reverted (by #2982), strings showing "Brave Web Store" would show up on Extensions page and elsewhere. We need to apply the same fix as was done in #2625 to 0.59.x and above - replace Chrome Web Store to just Web Store, so that it doesn't get affected by global replacement of Chrome with Brave.

@mkarolin mkarolin self-assigned this Jan 16, 2019
mkarolin added a commit that referenced this issue Jan 16, 2019
'Web Store' in order to avoid ending up with 'Brave Web Store' when
'Chrome' is globally replaced with 'Brave'.

Fixes #2983
mkarolin added a commit to brave/brave-core that referenced this issue Jan 16, 2019
IDS_MD_EXTENSIONS_BRAVE_MORE_EXTENSIONS due to localization issues.
- Updates extensions web UI page to use the default Chromium string
instead of the removed IDS_MD_EXTENSIONS_BRAVE_MORE_EXTENSIONS.
- Reverts patch that made those strings accessible to the web UI.
- Includes updated .grd[p] files resulting from running
chromium_rebase_l10n as a part of fix for brave/brave-browser#2983.

Fixes brave/brave-browser#2982
Fixes brave/brave-browser#2983
mkarolin added a commit that referenced this issue Jan 16, 2019
'Web Store' in order to avoid ending up with 'Brave Web Store' when
'Chrome' is globally replaced with 'Brave'.

Fixes #2983
mkarolin added a commit to brave/brave-core that referenced this issue Jan 16, 2019
IDS_MD_EXTENSIONS_BRAVE_MORE_EXTENSIONS due to localization issues.
- Updates extensions web UI page to use the default Chromium string
instead of the removed IDS_MD_EXTENSIONS_BRAVE_MORE_EXTENSIONS.
- Reverts patch that made those strings accessible to the web UI.
- Includes updated .grd[p] files resulting from running
chromium_rebase_l10n as a part of fix for brave/brave-browser#2983.

Fixes brave/brave-browser#2982
Fixes brave/brave-browser#2983
mkarolin added a commit to brave/brave-core that referenced this issue Jan 16, 2019
IDS_MD_EXTENSIONS_BRAVE_MORE_EXTENSIONS due to localization issues.
- Updates extensions web UI page to use the default Chromium string
instead of the removed IDS_MD_EXTENSIONS_BRAVE_MORE_EXTENSIONS.
- Reverts patch that made those strings accessible to the web UI.
- Includes updated .grd[p] files resulting from running
chromium_rebase_l10n as a part of fix for brave/brave-browser#2983.

Fixes brave/brave-browser#2982
Fixes brave/brave-browser#2983
mkarolin added a commit that referenced this issue Jan 16, 2019
'Web Store' in order to avoid ending up with 'Brave Web Store' when
'Chrome' is globally replaced with 'Brave'.

Fixes #2983
@mkarolin mkarolin added this to the 0.59.x - Beta milestone Jan 17, 2019
@btlechowski
Copy link

btlechowski commented Jan 21, 2019

Verification passed on

Brave 0.59.26 Chromium: 72.0.3626.64 (Official Build) beta (64-bit)
Revision eaa9668e80ce5405e7f1902579558ea725c06ca1-refs/branch-heads/3626@{#708}
OS Windows 7

Used test plan from brave/brave-core#1358

image
image
image

Verification passed on

Brave 0.59.26 Chromium: 72.0.3626.64 (Official Build) beta(64-bit)
Revision eaa9668e80ce5405e7f1902579558ea725c06ca1-refs/branch-heads/3626@{#708}
OS Linux

image
image

Verification PASSED on macOS 10.14.2 x64 using the following build:

Brave 0.59.28 Chromium: 72.0.3626.64 (Official Build) beta(64-bit)
Revision eaa9668e80ce5405e7f1902579558ea725c06ca1-refs/branch-heads/3626@{#708}
OS Mac OS X

screen shot 2019-01-23 at 4 24 28 pm

screen shot 2019-01-23 at 4 24 35 pm

@brave brave deleted a comment from yodaross Jan 21, 2019
petemill pushed a commit to brave/brave-core that referenced this issue Jul 27, 2020
'Web Store' in order to avoid ending up with 'Brave Web Store' when
'Chrome' is globally replaced with 'Brave'.

Fixes brave/brave-browser#2983
petemill pushed a commit to brave/brave-core that referenced this issue Jul 28, 2020
'Web Store' in order to avoid ending up with 'Brave Web Store' when
'Chrome' is globally replaced with 'Brave'.

Fixes brave/brave-browser#2983
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment