-
Notifications
You must be signed in to change notification settings - Fork 792
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
search.brave.com: ads #17949
Comments
Compatible with EasyList, and what was done on sending report, so that broke harmony in sections under "Fix:" (...?). |
Hi, as this is fixable by Easylist, please file your report here: https://github.com/easylist/easylist/issues/ (Just copy your message there). |
Isn't that self-promo, which is out of scope of EasyList and uAssets? Or am I an idiot and misread this? |
Can be addresed in EasyList Annoyances or legacy "Fanboy Annoyances". For the time being, I do not track whether gorhill has activated the merging of the cookie list and social media in its mirror (So that people would not change old habits). Theoretically, it minimally resembles a widget on Reddit, which is only suitable for addressing as an annoyance (aside from the fact that an interface based on something similar to react has no readable class/id). |
Outlook "ads" or "self-promo" is also handled in uAssets. Personally I think that if self-promo is annoying, it should be handled. |
So made PR. |
Any ETA, if I may ask? |
Which browser are you using? |
I can't repo, but if you want to block Brave search ads with Brave shields, use Aggresive mode in shields on search.brave.com |
@luftra can't repro on Firefox or Brave |
Thanks |
Prerequisites
URL address of the web page
https://search.brave.com/search?spellcheck=0&q=js+change+element+colors
Category
ads
Description
it shows the premium subscription ad to remove ads.
Fix:
Screenshot(s)
Screenshot(s)
Configuration
Settings
The text was updated successfully, but these errors were encountered: