[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

Cleanup duplicate rules in MobileFilter (specific_web.txt) #177287

Open
3 tasks done
SKEIDs opened this issue Apr 17, 2024 · 0 comments
Open
3 tasks done

Cleanup duplicate rules in MobileFilter (specific_web.txt) #177287

SKEIDs opened this issue Apr 17, 2024 · 0 comments
Labels
Feature Request

Comments

@SKEIDs
Copy link
Contributor
SKEIDs commented Apr 17, 2024 •
edited
Loading

Prerequisites

  • I checked the documentation and found no answer;
  • I checked to make sure that this issue has not already been filed;
  • This is not an ad/bug report.

Problem description

There are multiple duplicate rules in the filters.

###ad-top-2
###article-1_5
##.article-mobile-ad
##.ccmcss_oas_top
##.center-spot
##.displayAd

##.div_sticky2
##.header-fixed-area
##.infeed
##.metro-amp-ad
##.single_ads

###.top
##div[class^="TwoColumnsWithHorizontalAdMosaicWrapper"]
##iframe[width="300"][height="100"]

gptoday.net###gptoday_ros_bravo_mobile
4gamers.com.tw##.mobile-bottom-ads
4gamers.com.tw##.interstitial-ads
m.winfuture.de#?##articleBody > div:has(> div[id^="div-gpt-ad"])

Proposed solution

Consolidate or remove rules if possible.

Additional information

These rules are grouped by domain, so I was unable to determine whether the rules should be merged.
If necessary, please merge the rules on the maintainer side or send a comment on the pull request.
Here are some examples of such rules:

##.ad-inread
##.adsbygoogle
##.amp-flying-carpet-text-border
##.pop-fade
##amp-fx-flying-carpet
##article[data-teaser-type="unspecified_group"]
##style[type="text/css"] + div[class]:not([id]):not([style]):not([class*=" "])

m.formel1.de##.b-top-container
formel1.de##.b-top-container

wetter.com##div[id^="somtag-container"]
wetter.com#?#div[id^="somtag-container"]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SKEIDs