HTML Flitering dla AdGuarda (`$$`)

[krystian3w]

Co mamy ulepszyć?

https://filters.adtidy.org/extension/firefox/filters/237.txt

vs

https://filters.adtidy.org/windows/filters/237.txt https://filters.adtidy.org/android/filters/237.txt https://filters.adtidy.org/mac/filters/237.txt

---

@AdamWr wiesz może dlaczego w pliku dla rozszerzenia nie mamy żadnych reguł filtrowania HTML?

Powinna być część z (przynajmniej te oparte na [tag-content="foo"] i [wildcard="foo*bar"][min-length="x"]):

przekonwertowane $$

```adb !ekliniki.pl$$#popup-message 4kdownload.com$$.thanks-modal alemeksyk.eu$$.pwebbox-plugin-facebook_page_plugin apynews.pl$$#fb-modal arms.com.pl,chrisbenzshop.pl,e-forma.pl,homeoapteka.pl,meblezagrosze.pl,oponyalbert.pl,zegarkiabc.pl$$#newsletter_colorbox bartek.com.pl$$script[tag-content="popupNewsletter"] biznes-host.pl$$#myModalPromo comperia.pl$$#modal_woman cyberfolks.pl$$#myModalDomenowy diamante-wear.com$$.newsletter__popup drzwi-cal.pl$$.pum-theme-promocja-urodzinowa.popmake-overlay e-spar.com.pl$$#modal-welcome homebook.pl$$script[id^="tpl-newsletter-popup"] intermarche.pl$$div[style="display:none"][tag-content="wplightbox"] it.tarnow.pl$$script[tag-content="livingModal"] lento.pl$$script[tag-content="showNotify()"] lookcam.com$$#lookcam-fb-modal makarun.pl$$.makarun-modal mediasam.pl$$#myWorning mprofi.pl$$.tve-leads-lightbox muscat.pl$$#discountPopup ogrodosfera.pl$$#NewsletterExit sportowefakty.wp.pl$$#exitpopup zoo.bydgoszcz.com$$#NewsShow karachan.org$$script[wildcard="style*xD"][min-length="100"] komputerswiat.pl$$#ks_breaking_news abcjanikowo24.pl,bialobrzegi24.net,bielskirynek.pl,bilgoraj.com.pl,brodnica365.pl,chelmza365.pl,chocianow.com.pl,ciechocinek.biz,czasbrodnicy.pl,ddtorun.pl,ddwloclawek.pl,dziendobrypomorze.pl,dziennikhistoryczny.pl,e-chelmno.pl,e-golub.info.pl,e-krajna.pl,e-lubon.pl,e-wabrzezno.info.pl,ejarocin.pl,ekstrasierpc.pl,ekutno.pl,erolnicze.pl,esopot.pl,expresskaszubski.pl,extrawalcz.pl,fakty.nl,faktyct.pl,faktypilskie.pl,gle24.pl,glospowiatusredzkiego.pl,glossokolowa.pl,gniewkowo.eu,gorlice24.pl,gostyn24.pl,grudziadz365.pl,gst24.pl,gwarek.com.pl,gwe24.pl,gwio.pl,haloursynow.pl,hej.mielec.pl,hej.rzeszow.pl,hej.slupsk.pl,ibytow.pl,infomogilno24.pl,infoskierniewice.pl,ipoddebice.pl,isanok.pl,iszczecinek.pl,itvszubin.pl,jastrowie24.pl,jura365.pl,ki24.info,kolniak24.eu,koronowo.net.pl,koscianiak.pl,legio24.pl,leszno24.pl,lowiczanin.info,magazynregionalny.pl,magazynzuzel.pl,miastko24.pl,modnymagazyn.pl,mojakruszwica.pl,motomagazyn.com,myglogow.pl,mypolkowice.pl,myszkow365.pl,naszpruszcz.pl,naszradziejow.pl,nizanskie.info,nowysztum.pl,nrypin.pl,pakosc24.pl,pleszew24.info,plonskwsieci.pl,police24.pl,portalsremski.pl,prawobrzeze24.pl,pulsciechanowa.pl,pulsgminy.pl,pulstygodnia.pl,radzyninfo.pl,roztocze.net,rydzyna24.pl,rzeczkrotoszynska.pl,siecportali.pl,slu24.pl,spotradomsko.pl,superchodziez.pl,swiecie24.pl,szemud24.pl,terazgabin.pl,terazlipno.pl,terazstrzelno.pl,tukutno.pl,tusochaczew.pl,tygodnik.pl,tygodnikketrzynski.pl,tyna.info.pl,uniejow.net.pl,wdrawskupomorskim.pl,wiescimiedzychodzkie.pl,wpu24.pl,wrzesnia.info.pl,wybudujemy.com.pl,zawiercie365.pl,ziemiagorowska.pl,zlomzy.pl,zyciesiedleckie.pl$$.rodo-overlay akamai.com$$.notification--warning alltube.pl,alltube.tv$$#rodo-popup autakrajowe.com.pl,craftserve.pl,infoplanet.pl,motobanda.pl,polski-tenis.pl$$#rodo belsat.eu,buddyzm.info.pl,eurologistics.pl,homedesign.pl,masterleasing.pl,mobilet.pl,petrodom.pl,poznan.ap.gov.pl,sp112.waw.pl,sp18.lublin.eu,whatannawears.com,wsb.net.pl$$.popmake-overlay[tag-content="RODO"] berendsen.pl$$script[data-module="CookieBanner"] bioagris.pl$$#policy-accept-div bmw-motorrad.pl$$.cookielayer brieftip.pl,kaliber.pl$$#cookieModal cenapaliw.pl$$#CBCookieMsg centrumbajki.pl$$.popmake-overlay[tag-content="danych osobowych"] chelmonline.pl,powiatslubicki.pl$$#myModal cooltura24.co.uk,debica24.eu,korsokolbuszowskie.pl,koscierzyna24.info,lokalna24.pl,radiobonton.pl,tarnowska.tv,truso.tv,wejherowo24.info,zgl.pl$$#popModal2 dalmierze.pl$$#Widget3 dlawas.info,halootwock.pl,halowawa.pl,infostrow.pl,meble-4you.pl,ostrzeszowinfo.pl,roztoczanskipn.pl$$#rodo-modal domhouse.pl,gdansk.pl,gokf.gda.pl,opolak.pl,rowerowygdansk.pl,stadionenerga.pl$$.rodo drogeriekoliber.pl$$.popmake-overlay[tag-content="naszej Polityce Prywatności"] dworzec.kielce.pl$$script[tag-content=".popup-aholic97"] e-hotelarz.pl,muzyczneradio.pl,rynek-turystyczny.pl$$#modal-rodo e-legnickie.pl$$script[tag-content="polityka-cookies"] easyprotect.pl$$.popup-aholic227 ekobiuro24.pl$$#dialog-agreement eluban.pl,rybnik.com.pl,slaskibiznes.pl,tujastrzebie.pl,tulodz.pl,tuwodzislaw.pl,tuwroclaw.com,tuzory.pl,zakopane.info$$#pd-modal emas.gdos.gov.pl,projekty.gdos.gov.pl,www.gdos.gov.pl$$[id^="banner-komunikat-rodo-"] ferguson-digital.eu$$#cookie-notice fnglob.pl,polska6.pl$$#modal fryzomania.pl,happydental.pl,pro-sprzet.pl$$#myWorning gryfino.pl$$#templateRODODialog ideafairplay.pl$$#modal-privacy itmarket.pl$$.modal lipinscy.pl$$#popRodoInfoContent listymikolaja.pl$$#popup_rodo_layer lubimyczytac.pl$$script[tag-content="rodoConsent"] mcdonalds.pl$$body > .cookies mdkkolbuszowa.pl,spolem.katowice.pl,wpr.pl$$#popup.modal moja.biedronka.pl$$div.modal.modal-komunikat moja.biedronka.pl$$style[tag-content="body.showpopup"] muzeumazji.pl$$script[tag-content="doRODO"] natura2000.gdos.gov.pl$$#apla-banner neucamed.pl$$.cookies-alert norgips.pl$$#cookiemodal ogloszenia.plock.pl$$#dialog-rodo okazik.pl$$script[tag-content="#cookie-dialog"] pamiatki.pl$$#hw_cookie_law pl.asseco.com$$footer ~ #animatedModal[tag-content="Polityce prywatności"] pl.asseco.com$$footer ~ a.animatedModal[tag-content="Open"] pl.dow.com$$.cookie-notify poszalu.pl$$#gdpr-modal profeto.pl$$#modal-rodo-main pruszczgdanski.pl$$#auto_popup pulawy.eu$$#modalRodo sevenpolska.com$$#flipFlop sklepbiker.pl,tvdzierzgon.pl$$#rodoModal skm.pkp.pl$$#dialog-message-rodo sprzet-fryzjerski.pl$$#myWorningNewsletterPopUp[tag-content="cookies"] styl.fm$$.rodo-info swiat-whisky.sklep.pl$$.POPUP_CONDITION tuzdrowie.pl$$#popupModal tvasta.pl$$#infoModal twojeankiety.pl$$.polPriv ustawlige.com$$#privacy-modal wiemy.to$$#banner[tag-content="RODO"] wtk.pl$$#GDPR-modal www.google.com,www.google.pl$$script[tag-content="consentCookiePayload"] www.google.com,www.google.pl$$script[tag-content="cookieUpdateConsentUrl"] zdajmyrazem.pl$$#gdprModal ztm.poznan.pl$$#PopUP ```

[!Note] Zakładam że nie jest to generyczne odrzucanie obsługi ID pisanego: z $$#ID zamiast $$[id="ID"] lub klas z $$.class zamiast $$[class="class"]

[!Note] Bazując na liście "dawne RSS" złe może być użycie if-a:

!#if (!adguard_ext_android_cb && !adguard_app_ios)

bo nam nie jest wygodnie używać { display: none !important } na kilkuset liniach, by filtr nie ruszył z bezpłatną apką dla iOS (w CB dla Androida raczej nie ma wielu naszych list). I chyba powrót do https://github.com/FiltersHeroes/PolishAntiAnnoyingSpecialSupplement/commit/f82a817001cc669210d839c40b38ed50ebb9d1c1 nie pomoże.

Zasady pisania

• [X] Zapoznałem się z netykietą

[AdamWr]

Tego typu reguły zostały wykluczone (jeśli się nie mylę, to prawie 2 lata temu - https://github.com/AdguardTeam/FiltersRegistry/pull/596) z filtrów dla rozszerzenia do Firefoxa ze względu na to, że powodowały problem na niektórych stronach - https://github.com/AdguardTeam/AdguardBrowserExtension/issues/2249 Oprócz tego jeszcze to - https://github.com/AdguardTeam/AdguardBrowserExtension/issues/2646 Zgaduję, że jak ten drugi problem zostanie naprawiony, to wtedy ponownie zostaną włączone te reguły.

[github-actions[bot]]

Ten wątek został automatycznie zablokowany, ponieważ po jego zamknięciu nie było żadnej aktywności. Proszę otworzyć nowe zgłoszenie dla powiązanych problemów.