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

blocked Trackers and ads count is shown when the shield is down #22422

Closed
Tracked by #22546
GeetaSarvadnya opened this issue Apr 20, 2022 · 4 comments
Closed
Tracked by #22546

blocked Trackers and ads count is shown when the shield is down #22422

GeetaSarvadnya opened this issue Apr 20, 2022 · 4 comments
Assignees

Comments

@GeetaSarvadnya
Copy link

Description

blocked Trackers and ads count is shown when the shield is down

Steps to Reproduce

  1. Clean profile 1.38.103
  2. Open brave://settings/shields
  3. Enable global Block scripts settings
  4. Open cnn.com or buzzfeed.com
  5. Open shield and turn off and on the shield a couple of times, observe trackers count is shown when shield is down

Actual result:

blocked Trackers and ads count is shown when the shield is down

Recorded file path:
https://drive.google.com/file/d/1CVn-FiEKQkCrWIG8WtigvFsIVJSqVfUt/view?usp=sharing

image

Expected result:

blocked Trackers and ads count should not be shown when the shield is down

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.38.103 Chromium: 100.0.4896.127 (Official Build) beta (64-bit)
Revision ff0d0695743e65305d7194f9bd309e5e1c824aa0-refs/branch-heads/4896_88@{#4}
OS Windows 10 Version 21H2 (Build 19044.1645)

Version/Channel Information:

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

Other Additional Information:

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

Miscellaneous Information:

cc: @brave/qa-team @nullhook @rebron

This was referenced Apr 20, 2022
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Apr 22, 2022
@rebron rebron mentioned this issue Apr 25, 2022
7 tasks
@aguscruiz
Copy link

Design for Shields down here: https://www.figma.com/file/3A6F6VrxVahiZFxLr7j7FO/Desktop-Shields?node-id=1804%3A30480

  • Changes number for a dash
  • Text below URL on header changes from "Trackers, ads, and more blocked" to "Not protected by Brave Shields" when shields down

image

@rebron rebron added this to the 1.41.x - Nightly milestone Jun 1, 2022
@rebron
Copy link
Collaborator

rebron commented Jun 1, 2022

Closed with brave/brave-core#13209

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Jun 9, 2022

Verification PASSED on

Brave | 1.41.46 Chromium: 102.0.5005.78 (Official Build) nightly (64-bit)
-- | --
Revision | df6dbb5a9fd82af3f567198af2eb5fb4876ef99c-refs/branch-heads/5005_59@{#3}
OS | Windows 10 Version 21H2 (Build 19044.1706)

Dark theme Light theme
image image

@stephendonner
Copy link

Verified PASSED using

Brave 1.41.47 Chromium: 102.0.5005.78 (Official Build) nightly (x86_64)
Revision df6dbb5a9fd82af3f567198af2eb5fb4876ef99c-refs/branch-heads/5005_59@{#3}
OS macOS Version 12.5 (Build 21G5037d)

NOTE: encountered #23245

Steps

  1. installed 1.41.47
  2. launched Brave
  3. loaded thenextweb.com
  4. toggled Shields to Off/Disabled

Verified the new design for shield down #22422 (comment) and ensured the following:

  • blocked trackers and ads count is NOT shown when the shield is down
  • dash - is shown in place of the number when the shield is down
  • changed text Not protected by Brave Shields is shown when Shields are down
Dark mode Light mode
Screen Shot 2022-06-09 at 3 05 08 AM Screen Shot 2022-06-09 at 3 05 46 AM

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