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

Remove Block all text when there are no entries - follow up to 1906 #2371

Closed
LaurenWags opened this issue Dec 5, 2018 · 4 comments
Closed
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. closed/wontfix feature/shields The overall Shields feature in Brave.

Comments

@LaurenWags
Copy link
Member

Description

Issue #1906 was not resolved, logging this follow up issue.

Remove Block all text when there are no entries

Steps to Reproduce

  1. Build from master or run 0.57.16.
  2. Visit a site. Enable Scripts Blocked (text is also a little confusing) @karenkliu relating to the conversation on Shields v2 GA version brave-extension#70 (comment)
  3. Click on the expand button to see the items blocked
  4. Block all text is shown even though there are no items under allowed list

Actual result:

image

Expected result:

Should not have unwanted text which is confusing

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.57.16 Chromium: 71.0.3578.80 (Official Build) (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    yes

Website problems only:

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

Additional Information

cc: @rebron since this was originally slated for 0.57.x but was not resolved.
cc @brave/legacy_qa

@LaurenWags LaurenWags added feature/shields The overall Shields feature in Brave. design A design change, especially one which needs input from the design team QA/Yes QA/Test-Plan-Specified feature/shields/panel Front-end design and functionality of the Shields panel. labels Dec 5, 2018
@LaurenWags LaurenWags added this to the 1.x Backlog milestone Dec 5, 2018
@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@cezaraugusto
Copy link
Contributor

cc @karenkliu agree on this?

@karenkliu
Copy link

@cezaraugusto Yes, correct.

@karenkliu
Copy link

Hi @cezaraugusto, just noting that this issue is still open:
Screen Shot 2019-06-13 at 3 49 02 PM

@rebron rebron added closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. closed/wontfix and removed design A design change, especially one which needs input from the design team QA/Yes QA/Test-Plan-Specified feature/shields/panel Front-end design and functionality of the Shields panel. labels May 27, 2022
@rebron
Copy link
Collaborator

rebron commented May 27, 2022

Closing as stale/wontfix.

@rebron rebron closed this as completed May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. closed/wontfix feature/shields The overall Shields feature in Brave.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants