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

Cosmetic filtering should be enabled by default #8068

Closed
antonok-edm opened this issue Feb 5, 2020 · 6 comments · Fixed by brave/brave-core#4511
Closed

Cosmetic filtering should be enabled by default #8068

antonok-edm opened this issue Feb 5, 2020 · 6 comments · Fixed by brave/brave-core#4511

Comments

@antonok-edm
Copy link
Collaborator

antonok-edm commented Feb 5, 2020

Description

Currently merged implementation of cosmetic filtering is off by default, new 1p/3p heuristics should allow it to be enabled by default.

Test plan

See #5381

@bsclifton
Copy link
Member

@antonok-edm what is a quick way to tell if cosmetic filtering is enabled? Is there a specific page you can mention (maybe edit the original post above to provide the example as a test plan)

@szaimen
Copy link

szaimen commented Feb 16, 2020

I am wondering, too...
Also it seem to doesn't work on some sites. Or is there a learning curve?

@bsclifton
Copy link
Member

bsclifton commented Feb 17, 2020

cc: @antonok-edm @ryanbr @snyderp

@szaimen I believe the rules are downloaded as part of the regular adblock definition download and then they should be applied. As this was just merged, it's only enabled on Nightly

Which version of Brave are you using? If you're on Developer, it may also be available, but you'd need to go to brave://flags/#brave-adblock-cosmetic-filtering to enable it

It's not merged to Beta (1.4.x) or Release (1.3.x) yet

@bbondy bbondy modified the milestones: 1.6.x - Beta, 1.7.x - Dev Mar 10, 2020
@kjozwiak
Copy link
Member

@bsclifton @antonok-edm is there a test plan that QA can run through to make sure this is working as expected?

@bsclifton
Copy link
Member

@kjozwiak yup! Added 😄

@LaurenWags
Copy link
Member

LaurenWags commented Apr 6, 2020

Verified passed with

Brave 1.7.86 Chromium: 80.0.3987.163 (Official Build) (64-bit)
Revision e7fbe071abe9328cdce4ffedac9822435fbd3656-refs/branch-heads/3987@{#1037}
OS macOS Version 10.14.6 (Build 18G3020)

Verification passed on

Brave 1.7.86 Chromium: 80.0.3987.163 (Official Build) (64-bit)
Revision e7fbe071abe9328cdce4ffedac9822435fbd3656-refs/branch-heads/3987@{#1037}
OS Ubuntu 18.04 LTS

Verification passed on

Brave 1.7.86 Chromium: 80.0.3987.162 (Official Build) dev (64-bit)
Revision e7fbe071abe9328cdce4ffedac9822435fbd3656-refs/branch-heads/3987@{#1037}
OS Windows 10 OS Version 1803 (Build 17134.523)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment