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

Block script doesn't show all blocked items at once and lists irrelevant items blocked #230

Closed
srirambv opened this issue May 27, 2018 · 4 comments · Fixed by brave/brave-core#494

Comments

@srirambv
Copy link
Contributor

Description

Block script doesn't show all blocked items at once and lists irrelevant items blocked

Steps to Reproduce

  1. Build from source (2e9c470)
  2. Visit brianbondy.com in a new tab
  3. Block scripts from shields, shows one entry for blocked item
  4. Disable the blocked item and click apply
  5. Two more blocked items show up which are not relevant

Actual result:

block script

Blocked scripts on current muon browser for comparison
image

Expected result:

Should show all blocked items when block script is enabled and only block relevant items

Reproduces how often:

Easy

Brave version (about:brave info)

2e9c470

cc: @bbondy

@srirambv srirambv added bug feature/shields The overall Shields feature in Brave. release/blocking labels May 27, 2018
@bbondy bbondy added this to the Milestone 3: June-July milestone Jun 6, 2018
@yrliou
Copy link
Member

yrliou commented Sep 25, 2018

screen shot 2018-09-24 at 9 15 25 pm

Above is the screenshot for the current result. The youtube.com entry won't be shown before we unblock script on brianbondy.com because it won't be loaded at all if script on brianbondy.com is blocked. So not showing brianbondy.com & youtube.com at once seems to be the expected result to me.

screen shot 2018-09-24 at 9 27 22 pm

The two youtube scripts blocked also seem to be valid for me judging from the above screenshot of devtool. These two scripts are from the youtube iframe in this page.

@bbondy
Copy link
Member

bbondy commented Sep 25, 2018

I think maybe just the UI is confusing because if I re-block on brianbondy.com scripts, then I expect for youtube to not show up in that case again, but the radio button is still there.

@yrliou
Copy link
Member

yrliou commented Sep 27, 2018

QA note: Please see the test plan specified in brave/brave-core#494, thanks!

@LaurenWags
Copy link
Member

LaurenWags commented Oct 4, 2018

Verified passed with

Brave 0.55.10 Chromium: 70.0.3538.22 (Official Build) beta(64-bit)
Revision ac9418ba9c3bd7f6baaffa0b055dfe147e0f8364-refs/branch-heads/3538@{#468}
OS Mac OS X

Verification Passed on

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta (64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Linux

Verification passes on

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta (64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Windows 7

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.

5 participants