FiltersHeroes / PolishAnnoyanceFilters

Polskie Filtry Elementów Irytujących ukrywają i blokują wyskakujące okienka, widgety, newslettery, powiadomienia push, strzałki, niezgodne z tematem artykułów otagowane linki wewnętrzne i inne drażniące elementy (Polskie Filtry RODO-Ciasteczkowe są już w nich zawarte).
https://polishannoyancefilters.netlify.app
Other
50 stars 4 forks source link

Poszatkowane filtry #1463

Closed krystian3w closed 5 years ago

krystian3w commented 5 years ago

Jak zrobiłem porównanie z filtrami dla np. Firefox:

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

to serwer sobie wyrwał połowicznie filtry na np. pop-upy i usunął naprawy przewijania / usunięcia rozmazania / przezroczystości.

Tak na wypadek jakby gdzieś się coś przyblokowało z ograniczeń technologicznych.


Nie badałem na ile pop-upy wyskakują na Androidzie ani tym bardziej na iOS :iphone: z ograniczeniami technologicznymi.

hawkeye116477 commented 5 years ago

No to trzeba będzie dołożyć warunek !#if (!adguard_ext_android_cb && !adguard_ext_safari && !adguard_app_ios) do całego supplementu. Ale naszej listy tak w ogóle to nie ma w AdGuard CB, przynajmniej na Androidzie.

krystian3w commented 5 years ago

Tylko uważaj na uBO, bo nagle przy dwóch zagnieżdżonych warunkach będzie ten drugi ignorować - nagle zniknie sens np. popupy-userCSS.txt.

hawkeye116477 commented 5 years ago

A no to faktycznie może być problem. Niby można jeszcze dodać !+ NOT_PLATFORM(ext_safari, ext_android_cb, ios), ale pewnie to w tym przypadku nie przejdzie. A inaczej też raczej to nie rozwiążemy, no chyba, że gorhill naprawi zagnieżdżone warunki albo ameshkov zrobi tak, by ten ich kompilator sam dodawał dyrektywy, żeby includowane listy nie działały w ACB.

krystian3w commented 5 years ago

Coś przeczuwam że by wymusili na nas ukrywanie CSS-em pop-upa zamiast filtrem ABP.

domain.xyz#$#.popup { display: none!important; }
hawkeye116477 commented 5 years ago

@krystian3w No tak też może można, ale ja tam myślę raczej, że może trzeba napisać do gorhilla, by ogarnął zagnieżdżone warunki, bo AG to naprawił, to teraz chyba kolej na uBO :smile:. No jeszcze ewentualnie moglibyśmy zrezygnować z oddzielnych reguł, ale raczej to nie wchodzi w grę, bo o ile specjalne dla FF można byłoby usunąć, to znowu dla Nano muszą zostać.

krystian3w commented 5 years ago

Ale to też nic w sumie nie da, bo nagle się walnie w przestarzałych wersjach uBO, może jednak trzeba by to pchać w AdGuard, może powinni dodać wykrywanie że CB / iOS nie mają html filteringu czy np. userCSS?

hawkeye116477 commented 5 years ago

Stare niby nie wspieramy :smile:.


może powinni dodać wykrywanie że CB / iOS nie mają html filteringu czy np. userCSS?

Raczej to nic nie da, bo są też normalne reguły przecież. Musieliby raczej zrobić tak, by dokładać warunek, by includowana lista nie była dla AG CB/iOS/Safari.

krystian3w commented 5 years ago

Safari 1.X "extension" używa listy iOS, a dopiero 2.x/3.x to:

https://filters.adtidy.org/extension/safari/filters/237.txt :question:

hawkeye116477 commented 5 years ago

@krystian3w Nie wiem, nie posiadam Maca ani iPhona. Jestem zwolennikiem Androida i mam Xiaomi :smile:.

krystian3w commented 5 years ago
!
! Otagowane linki wewnętrzne w artykułach
!
!
! Popupy
+!
+!
! Popupy - specjalne instrukcje

Chyba ten pierwszy if działa. Ale raczej dalej będzie problem np. popupy_bez_html.txt tu niby twierdzi, że może sobie wziąć reguły i wywalić CSS, albo niby zasysa CSS, nakładany liniowo (tu raczej niedoprogramowali do jakiej grupy liczyć taki filtr).

hawkeye116477 commented 5 years ago

@krystian3w Tego nie możemy dodać !#if (!adguard_ext_android_cb && !adguard_ext_safari && !adguard_app_ios), bo wtedy problem jest w uBO, że resztę warunków olewa.

krystian3w commented 5 years ago

przecież usunąłem jakieś 10 minut później.

hawkeye116477 commented 5 years ago

@krystian3w A skąd, zostało, nie trafiło do GH, więc ja usunąłem 20 minut temu :smile:

krystian3w commented 5 years ago

A dziwne, sprawdzałem na vivrehome i potem juz nie wycinało popupu...

krystian3w commented 5 years ago

Problem niby naprawiony: https://github.com/gorhill/uBlock/commit/de41c1bf533d7a58400ea91f8361620912375bb1

krystian3w commented 5 years ago

na razie łagodne poprawki w plikach pod AdGuard

krystian3w commented 5 years ago

Jakby to naprawiać tylko w template, to raczej tylko tak na razie (najwcześniej to chyba na sylwestra 2019 / zima 2020):

!#if !adguard_ext_android_cb 
!#if !adguard_ext_safari 
!#if !adguard_app_ios

! filter for modern uBlock Origin and AdGuard Adblocker / AdGuard APP (macOS, Windows, Android).
! "modern" - support `:style()` filters.

!#endif
!#endif
!#endif

Nie wiem czy Gorhill doda wersje skróconą do nawiasu.

github-actions[bot] commented 3 years ago

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.