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

Add to Chrome on CWS main page #5730

Closed
btlechowski opened this issue Aug 20, 2019 · 6 comments · Fixed by brave/brave-core#4013
Closed

Add to Chrome on CWS main page #5730

btlechowski opened this issue Aug 20, 2019 · 6 comments · Fixed by brave/brave-core#4013

Comments

@btlechowski
Copy link

Follow up to #790

Steps to Reproduce

  1. Open https://chrome.google.com/webstore/category/extensions

Actual result:

Add to Chrome on CWS main page
image

Expected result:

Add to Brave on CWS main page

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 0.69.110 Chromium: 76.0.3809.100 (Official Build) beta (64-bit)
Revision ed9d447d30203dc5069e540f05079e493fc1c132-refs/branch-heads/3809@{#990}
OS Ubuntu 18.04 LTS

cc @brave/legacy_qa @rebron

@bsclifton
Copy link
Member

bsclifton commented Aug 20, 2019

Functionality originally fixed with brave/brave-core#2845

Looks like it regressed completely (at least on Nightly)... content script has correct selector still

@bsclifton
Copy link
Member

bsclifton commented Aug 20, 2019

Ah - if you refresh after going to a detail page, it then executes the content script properly (weird that a refresh is needed though). Also, directly navigating to a detail page works fine, as expected

This issue (hover showing Add to Chrome) can be fixed by adding https://chrome.google.com/webstore/category/extensions (or a better selector) to the manifest.json here:
https://github.com/brave/brave-core/blob/05c7b6199273870ecc536798a0f223c5f2d5794d/components/brave_extension/extension/brave_extension/manifest.json#L36-L38

@LaurenWags
Copy link
Member

LaurenWags commented Aug 29, 2019

Reproduced originally described issue as well as issue described with #5730 (comment)

Brave 0.69.119 Chromium: 76.0.3809.132 (Official Build) beta (64-bit)
Revision fd1acc410994a7a68ac25bc77513d443f3130860-refs/branch-heads/3809@{#1035}
OS Mac OS X

Screen Shot 2019-08-29 at 3 52 32 PM

Screen Shot 2019-08-29 at 3 46 01 PM
And after page refresh:
Screen Shot 2019-08-29 at 3 46 12 PM

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Aug 30, 2019
@eljuno
Copy link

eljuno commented Oct 4, 2019

@kjozwiak
Copy link
Member

kjozwiak commented Oct 7, 2019

@rebron @bsclifton looks like this regressed again. Originally it was fixed via #790 and checked by QA via the following verifications #790 (comment).

I also checked and reproduced using 0.69.119 Chromium: 76.0.3809.132 which was originally used for verification as per #790 (comment). Definitely seems like something was changed on Google's side.

CCing @brave/legacy_qa so they're aware of the issue.

@btlechowski
Copy link
Author

btlechowski commented Jan 4, 2020

Verification passed on

Brave 1.3.81 Chromium: 79.0.3945.88 (Official Build) beta (64-bit)
Revision c2a58a36b9411c80829b4b154bfcab97e581f1f3-refs/branch-heads/3945@{#954}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#4013 and from the description
Verified extensions main page and individual extension pages.

image
image
image

Verification passed on

Brave 1.3.87 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verified passed with

Brave 1.3.88 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS macOS Version 10.14.6 (Build 18G103)

Extensions Main page (hover over image at top)
Screen Shot 2020-01-10 at 9 25 35 AM

Individual extension page before adding:
Screen Shot 2020-01-10 at 9 28 48 AM
Screen Shot 2020-01-10 at 9 29 05 AM
Screen Shot 2020-01-10 at 9 29 26 AM

Individual extension page after adding:
Screen Shot 2020-01-10 at 9 30 16 AM

Searching for extensions:
Screen Shot 2020-01-10 at 9 30 40 AM

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.

7 participants