uBlockOrigin / uBlock-issues

This is the community-maintained issue tracker for uBlock Origin
https://github.com/gorhill/uBlock
945 stars 81 forks source link

Support ABP/EL `{remove: true;}` as alias for `:remove()` #3451

Closed stephenhawk8054 closed 1 day ago

stephenhawk8054 commented 2 days ago

Prerequisites

I tried to reproduce the issue when...

Description

Currently ABP is supporting removing elements by {remove: true;}: https://gitlab.com/eyeo/adblockplus/abc/webext-ad-filtering-solution/-/releases/1.3.0

And looks like EL is starting to support that syntax as well: https://github.com/easylist/easylist/commit/99c717fac

So I think we could add alias support for {remove: true;} then we can delegate some filters in uAssets to EL?

A specific URL where the issue occurs.

https://www.euronews.com/next/2024/11/12/europes-answer-to-google-ecosia-and-qwant-partner-to-build-new-search-index

Steps to Reproduce

  1. Check for the filter euronews.com###o-site-hr__leaderboard-wallpaper.u-show-for-xlarge {remove:true;} in EL
  2. The filter is marked as invalid

image

Expected behavior

{remove: true;} filters are supported

Actual behavior

{remove: true;} filters are not supported

uBO version

1.61.0

Browser name and version

Firefox 132.0.2

Operating System and version

Windows 10

uBlock-user commented 1 day ago

{remove: true;} filters are supported

AdGuard's {remove: true;} filter is supported, it's just the starting syntax of ABP that is different.

stephenhawk8054 commented 1 day ago

Yeah, if I'm not wrong, AG's syntax is

euronews.com#$##o-site-hr__leaderboard-wallpaper.u-show-for-xlarge {remove:true;}

which is supported in uBO.