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

Support procedural cosmetic filtering #16935

Closed
antonok-edm opened this issue Jul 13, 2021 · 5 comments · Fixed by brave/brave-core#24688
Closed

Support procedural cosmetic filtering #16935

antonok-edm opened this issue Jul 13, 2021 · 5 comments · Fixed by brave/brave-core#24688
Assignees
Labels
feature/shields/adblock Blocking ads & trackers with Shields features/shields/cosmetic-filtering OS/Android Fixes related to Android browser functionality OS/Desktop OS/iOS Fixes related to iOS browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-macOS-arm64 QA/Test-All-Platforms QA/Yes release-notes/include uBO-parity
Milestone

Comments

@antonok-edm
Copy link
Collaborator

antonok-edm commented Jul 13, 2021

Procedural cosmetic filtering is useful for the most evasive kinds of in-content sponsored/promoted material. It'd be great to implement this and tie it to aggressive shields settings, to handle the last few examples of cosmetic items slipping through on Reddit, YouTube, Facebook, etc.

See also:

@antonok-edm
Copy link
Collaborator Author

Another use/test case: sponsored posts on stuff.co.nz

image

Should be handled by stuff.co.nz##[itemprop]:has(.sponsored-flag)

@intrnl
Copy link

intrnl commented Apr 12, 2022

How many filters makes use of other procedural selector? (e.g. :has-text)
I'd imagine Chrome's upcoming support for :has is going to make this slightly less necessary.

@antonok-edm
Copy link
Collaborator Author

antonok-edm commented Apr 12, 2022

How many filters makes use of other procedural selector? (e.g. :has-text)

I did a survey of procedural filter operator counts in Brave's default lists a few days ago:

:has
    164
:has-text
    336
:matches-css
    15
:matches-css-before
    19
:matches-css-after
    2
:matches-path
    12
:not
    165
:upward
    239
:xpath
    49

It's not an exact metric since some filters can contain multiple chained operators, and :not is a special case since it's also a valid CSS pseudo-class, but this list still gives a good idea of how important each is. has-text in particular would be great to support.

I'd imagine Chrome's upcoming support for :has is going to make this slightly less necessary.

The recent efforts on :has are commendable, but it actually has a really long history and who knows what else may delay the implementation. Plus, support procedural cosmetic filters in general should make :has fairly simple anyways.

edit: 172 occurrences of has-text, or roughly half, are actually from HTML filters (with a ##^ separator in the rule). That uses a totally separate system from procedural filtering, even though the rules look similar. I haven't seen any other procedural filters used for HTML filtering.

edit again: :has is actually implemented now! So just the other procedural filters are left now.

@dharnil
Copy link

dharnil commented Mar 6, 2024

Hi Braver, Is there any update on this open ticket. I happy to contribute.

@MadhaviSeelam
Copy link

MadhaviSeelam commented Oct 18, 2024

Verification PASSED using

Brave | 1.72.83 Chromium: 130.0.6723.58 (Official Build) beta (arm64)
-- | --
Revision | b49615851cd2dc4b1c3f5f13d3207d3c032bfd8d
OS | macOS Version 14.6.1 (Build 23G93)
  1. Installed 1.72.83
  2. launched Brave
  3. opened brave://settings/shields/filters in a new tab
  4. added https://antonok.com/tmp/procedural-filter-tests/test-extended-css-rules.txt to Custom lists
  5. visited https://antonok.com/tmp/procedural-filter-tests/index.html in a new tab

Confirmed following numbered test cases are passed

  - 1,2.3
  - 6,7,8,9
  - 20,21
  - 24
  - 26
  - 28,29
step 4 step 5a step 5b
Image Image Image

@MadhaviSeelam MadhaviSeelam added OS/macOS-arm64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/shields/adblock Blocking ads & trackers with Shields features/shields/cosmetic-filtering OS/Android Fixes related to Android browser functionality OS/Desktop OS/iOS Fixes related to iOS browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-macOS-arm64 QA/Test-All-Platforms QA/Yes release-notes/include uBO-parity
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants