Grupa wp.pl - Githubissues

seba2282 commented 3 years ago

Lipa będzie z adocean i na domenach TVN głównie.

Widzę, że jest sporo roboty przy tych adserverach. I muszę to jakoś mądrze zrobić. Żeby mi strony nie padały.

seba2282 commented 3 years ago

Testuję te reguły. Widzę, że na razie są w porządku te na blokowanie homepages. Ale zrobiłem mały błąd. Miało być smartadserver. Więc skorygowałem z ręcznym dodaniem do zakładki moje filtry w ublock origin i testuję przed ich dodaniem do filtra.

I na tweet na dp też pracują reguły.

Są puste pola po reklamach na dobreprogramy i w środku ich symbol "WP".

seba2282 commented 3 years ago

@AdamWr, Adguard blokuje wideo na wp tak sądzę na regule z wpimg. Możesz to sprawdzić np. na stronie: https://wideo.wp.pl/dlaczego-mlodzi-kochaja-ekipe-friz-to-lepsza-wersja-nas-6633603995887745v

AdamWr commented 3 years ago

U mnie wideo działa prawidłowo.

Screenshot

![image](https://user-images.githubusercontent.com/29142494/116259877-7ff9f800-a776-11eb-9ff9-64eced7f6ee5.png)

Która konkretnie lista powoduje ten problem?

seba2282 commented 3 years ago

bug

seba2282 commented 3 years ago

@AdamWr, a jakie jeszcze listy masz włączone bo i RU Adguard coś też blokuje. I musiałbym chyba na spokojnie posprawdzać wyłączając filtry. I nie wiem do końca który. Ale jak mówisz, że ci działa, to wtedy co? Jakieś filtry trzecie?

AdamWr commented 3 years ago

Myślę, że najprościej będzie jak wyłączysz po kolei wszystkie listy filtrów i sprawdzisz która lista (albo listy) powodują ten problem.

seba2282 commented 3 years ago

@AdamWr w takim razie sprawdzę to za jakiś moment. Dzięki za sugestie.

seba2282 commented 3 years ago

RUS: RU AdList jest winowajcą. Czyli third part filter, @AdamWr :)

krystian3w commented 3 years ago

To musisz pisać do "Dimisy" jak Adam nie zrobi przeklejki.

https://github.com/easylist/ruadlist

seba2282 commented 3 years ago

@krystian3w, dodałem regułę do odblokowania, co zablokował rus ru adlist. Jak będę jutro na PC, to spróbuję do nich napisać + uwzględnię regułę, którą zastosowałem. I albo oni wywalą tą z "||" na wideo z wp, albo dodadzą "@@||"

krystian3w commented 3 years ago

wystarczy pewnie, że dopiszą: domain=~wp.pl ale "OK"...

seba2282 commented 3 years ago

wystarczy pewnie, że dopiszą: domain=~wp.pl ale "OK"...

Mogą i tak, @krystian3w

seba2282 commented 3 years ago

To musisz pisać do "Dimisy" jak Adam nie zrobi przeklejki.

https://github.com/easylist/ruadlist

Zaraportowałem do nich i dałem dwa możliwe fixy, aby dodali. :) @krystian3w i @AdamWr

https://github.com/easylist/ruadlist/issues/17

Edit: Bardzo szybko zafiksowali problem. :)

seba2282 commented 3 years ago

Cytaty tweetów się nie zaliczają jak są z kont innych niż twitter.com/dobreprogramy, wytnij sobie ręcznie cytowany zewnętrzny tweet:
dobreprogramy.pl#?#div[class][style*="min-height:"][style*="200px"]:-abp-has(.twitter-tweet)
dobreprogramy.pl##.twitter-tweet
Na moje nie da się łatwo wykrywać ID profilu twitter, w przypadku FB niby im przeszkadza ukrywanie postów self-promo i wymyślają już różne metody, że post nie jest z konta fb.com/dobreprogramy.

Będzie to dodane w reguły także, @krystian3w ?

seba2282 commented 3 years ago

www.dobreprogramy.pl##a.Ra7.ant-btn:nth-of-type(2) Do ukrycia DP logowanie FB. Zostawiam na razie wp login i google.

krystian3w commented 3 years ago

Nie będzie, potem ktoś zgłosi, że zewnętrzny cytat twitter jest ukrywany a news tak zbudowany, że nawet ktoś nieznający angielskiego powinien go widzieć.

Na razie ukrywam tylko self-promo dodawane z pomocą wtyczki Facebook np. w newsie o CERT i przekrętach na "kuriera" reklamują swój wpis o BitCoinach. Ale jak zaciemnią budowanie ramki Facebook to i to pewnie się skończy, bo już nawet sięgnęli po ID profilu, byle by filtr prosty CSS nie zadziałał.

Filtr na ukrycie logowania Facebookiem można zrobić lepiej:

##a[href*="facebook.com/"][href*="/dialog/oauth?"]

##a[href*="1login.wp.pl/"][href*="/ol-authprovider/auth?"]

##a[href*="accounts.google.com/"][href*="/oauth2/"][href*="/auth?"]

Na tę chwilę nie ma wciąż w CMS opcji "odepnij/przypnij konto społecznościowe" (match chyba zachodzi jak na FB/Google/1Login używasz ten sam e-mail) czy nawet zmiany e-mail na inny...

seba2282 commented 3 years ago

Więc WP próbuje uprzykrzać życie.

seba2282 commented 3 years ago

Dla pilot: pilot.wp.pl##.bKXAMv.sc-16zk3if-2.isrxuh.jv62pm-1.coMNkW.sc-AxjAm tylko facebook logowanie. Apple i wp zostawiłem. Dziwne mają te identyfikatory.

krystian3w commented 3 years ago

Losowanko by było trudniej blokować lub cuda korzystania z frameworków jak React (kod z czytelnymi klasami mają na wersjach developerskich tylko).

seba2282 commented 3 years ago

www.wp.pl##.jVHKjF.wc57lf-3.ineDsg.BFLvE www.wp.pl###wp-weather-widget albo to, aby pozbyć się jednej reklamy xD :D

seba2282 commented 3 years ago

Testuję twoją regułę na FB, @krystian3w

krystian3w commented 3 years ago

##a[href="/api/v1/user_auth/sso"] + span + button

Niby czytelniejsza dla Pilota, Apple wytnie np.:

##a[href="/api/v1/user_auth/sso"] + span + button + button

lub oba:

##a[href="/api/v1/user_auth/sso"] + span ~ button:nth-of-type(-n+2)

seba2282 commented 3 years ago

Nie będzie, potem ktoś zgłosi, że zewnętrzny cytat twitter jest ukrywany a news tak zbudowany, że nawet ktoś nieznający angielskiego powinien go widzieć.

Na razie ukrywam tylko self-promo dodawane z pomocą wtyczki Facebook np. w newsie o CERT i przekrętach na "kuriera" reklamują swój wpis o BitCoinach. Ale jak zaciemnią budowanie ramki Facebook to i to pewnie się skończy, bo już nawet sięgnęli po ID profilu, byle by filtr prosty CSS nie zadziałał.

Filtr na ukrycie logowania Facebookiem można zrobić lepiej:

##a[href*="facebook.com/"][href*="/dialog/oauth?"]

##a[href*="1login.wp.pl/"][href*="/ol-authprovider/auth?"]

##a[href*="accounts.google.com/"][href*="/oauth2/"][href*="/auth?"]

Na tę chwilę nie ma wciąż w CMS opcji "odepnij/przypnij konto społecznościowe" (match chyba zachodzi jak na FB/Google/1Login używasz ten sam e-mail) czy nawet zmiany e-mail na inny...

Na DP nie działa mi reguła dla FB przez te id, co napisałem u góry. Co otrzymałem przy próbie ukrycia FB login.

krystian3w commented 3 years ago

strona ma generichide, doklej domenę lub pl przed haszami jak wklejasz w zakładkę "Moje Filtry" lub w AG "Reguły użytkownika".

https://github.com/MajkiIT/polish-ads-filter/blob/032a6b8fa0c232c0e7e5439b8e86b212467242ff/polish-adblock-filters/adblock.txt#L5162

Jak to nie ghide, to nie wiem w czym problem. A ghide było dodane z racji mrożenia się starego layoutu z reklamami (padnięte powiadomienia/sidebar z niewidzialnymi pozycjami lub bardzo nisko jakby z wielkim margin-top).

seba2282 commented 3 years ago

Teraz rozumiem. Jeżeli chcę dokleić domenę wp, ze wszystkimi subdomenami (dp, pilot, główna wp) to wtedy jak? I czy dałbyś też jak usunąć przycisk od Apple, jeżeli bym się zdecydował go usunąć też. Na pilot jest zaloguj się za pomocą Apple.

seba2282 commented 3 years ago

##a[href="/api/v1/user_auth/sso"] + span + button

Niby czytelniejsza dla Pilota, Apple wytnie np.:

##a[href="/api/v1/user_auth/sso"] + span + button + button

lub oba:

##a[href="/api/v1/user_auth/sso"] + span ~ button:nth-of-type(-n+2)

To spróbuję w takim razie za moment. 😉

hawkeye116477 commented 3 years ago

@seba2282 Mam dla ciebie lekturę do poczytania w wolnej chwili/przed snem:

https://github.com/gorhill/uBlock/wiki/Static-filter-syntax https://adblockplus.org/filter-cheatsheet

seba2282 commented 3 years ago

@seba2282 Mam dla ciebie lekturę do poczytania w wolnej chwili/przed snem:

https://github.com/gorhill/uBlock/wiki/Static-filter-syntax https://adblockplus.org/filter-cheatsheet

@hawkeye116477, dzięki za lekturę xD 😉

krystian3w commented 3 years ago

z racji, że na innych WP pewnie też jest ghide to powinno ruszać jak przed haszami wstawisz pl. Oczywiście jak na jakimś kuchnia wp pl zbudowali inaczej logowanie, to filtr nie chwyci i pod tamten layout trzeba będzie kolejny prosty filtr "CSS".

seba2282 commented 3 years ago

Teraz działa jak należy. na dp i wp pilot.

seba2282 commented 3 years ago

@krystian3w takie małe pytanie. Jak ty wyciągasz prawdziwe elementy z tych dziwnych id? Jak ja próbuję zablokować coś, to ta konsola UBO widzi ID. Ty skąd bierzesz normalne reguły i one też działają, nawet lepiej.

krystian3w commented 3 years ago

To nie są ID, po prostu używam DevTools i kursu "CSS".

seba2282 commented 3 years ago

@krystian3w więc rozumiem.

seba2282 commented 3 years ago

Muszę teraz spojrzeć, bo na Firefox android nie działają reguły na chowanie przycisków logowania za pomocą na DP. A działały, tak jak mówiłeś z dopiskiem PL przed haszami, @krystian3w, więc sądzę WP coś modyfikuje. P.S. Widzę, że na chromium Edge jest wszystko normalnie

hawkeye116477 commented 3 years ago

więc sądzę WP coś modyfikuje

To nie WP, tylko tak jak Krystian pisał => https://github.com/MajkiIT/polish-ads-filter/issues/18472#issuecomment-828745406, jeżeli ktoś doda tego typu regułę na listę, to znaczy, że wyłączone są wtedy reguły ogólne, odsyłam do https://github.com/gorhill/uBlock/wiki/Static-filter-syntax#generichide. A jest to często robione, by zapobiec komunikat anty-adblockowym, bo często tak webmasterzy robią, że sprawdzają jakąś klasę, id, która jest na głównej liście adblockera czy jest widoczna. Dlatego też wtedy jeżeli chcesz coś schować, to musisz dopisać jakąś domenę albo końcówkę, np. pl.

krystian3w commented 3 years ago

Ja tam nie widzę z element pickerem AG by nie działało:

pl##a[href*="facebook.com/"][href*="/dialog/oauth?"], a[href*="1login.wp.pl/"][href*="/ol-authprovider/auth?"], a[href*="accounts.google.com/"][href*="/oauth2/"][href*="/auth?"]

seba2282 commented 3 years ago

Na firefox ci działa? Piszę, że na chromium edge PC działa, a na mobilnym firefox z ublock origin nie działa. Przynajmniej u mnie. I @hawkeye116477 odsyła mnie do komentarzy. Napisałem wyraźnie, gdzie u mnie tkwi problem.

krystian3w commented 3 years ago

Nie widzę by wersja mobilna miała swój szablon, to RWD wersji PC.

Jak chcesz wyciąć resztki po reklamach to sprawdź:

! ## element typowo reklamowy:
polygamia.pl,www.dobreprogramy.pl##div[data-index][data-slot][style]

! ##img[src*="wpimg.pl/Zi9i"]:upward(1) - dawna grafika:
polygamia.pl,www.dobreprogramy.pl##div[class*=" "] > div:first-child + img[src*="wpimg.pl/"][alt]:upward(1)

! ##img[src*="wpimg.pl/"][src*="/bg.png"]:upward(2) - dawna grafika:
polygamia.pl,www.dobreprogramy.pl##img[src*="wpimg.pl/"][alt]:first-child + div[class] > style:first-child + div:last-child:upward(2)

seba2282 commented 3 years ago

Nie widzę by wersja mobilna miała swój szablon, to RWD wersji PC.

Jak chcesz wyciąć resztki po reklamach to sprawdź:

! ## element typowo reklamowy:
polygamia.pl,www.dobreprogramy.pl##div[data-index][data-slot][style]

! ##img[src*="wpimg.pl/Zi9i"]:upward(1) - dawna grafika:
polygamia.pl,www.dobreprogramy.pl##div[class*=" "] > div:first-child + img[src*="wpimg.pl/"][alt]:upward(1)

! ##img[src*="wpimg.pl/"][src*="/bg.png"]:upward(2) - dawna grafika:
polygamia.pl,www.dobreprogramy.pl##img[src*="wpimg.pl/"][alt]:first-child + div[class] > style:first-child + div:last-child:upward(2)

Tylko ja mówię o zaloguj się za pomocą. Nie o reklamach. @krystian3w. I próbuję trochę przeanalizować kod strony. Wiesz, w kodzie są te dziwne numerki, co podałem na początku do ukrycia zaloguj za pomocą facebook. Jak ty wyciągnąłeś prawdziwe adresy z pod flex (bo tak mi oznacza w kodzie podczas wykonaj inspekcję w microsoft edge), to jest dla mnie zagadką. :)

krystian3w commented 3 years ago

Sprawdź więc:

pl#?#a > span:contains(/^(1Login od WP|Facebook|Google)$/):upward(a)

Niby telefony nie mają wypełnionego href="...".

seba2282 commented 3 years ago

Sprawdzam. Domyślam, się, że na pilot będzie podobnie, tylko z dodanym apple.

krystian3w commented 3 years ago

Tam mniej machlojek lub realnie trzeba sięgnąć po telefon.

seba2282 commented 3 years ago

I zostaje tylko napis zaloguj się za pomocą. Nie da się tego dokleić w jedno?

krystian3w commented 3 years ago

Raczej trzeba na 2 filtry proceduralne pod telefon:

pl#?##loginForm div:contains(lub zaloguj się)

chyba, że ufasz działaniu nth-last-child:

pl###loginForm > div:nth-last-child(-n+2)

seba2282 commented 3 years ago

Mam dodane i działają.

krystian3w commented 3 years ago

Na pilot takie coś bez proceduralności:

pl##div[data-gtm-element="close_modal_overlay"] ~ div > div > form ~ div:not(:last-child)

PS. pop-up kradnie event pisania w polu na tekst element pickera i niby tylko ręczna zabawa z nadaniem klika razy display: none pomaga.

seba2282 commented 3 years ago

Czyli finalnie działają. te reguły: pl###loginForm > div:nth-last-child(-n+2) pl##div[data-gtm-element="close_modal_overlay"] ~ div > div > form ~ div:not(:last-child)

Nie rozumiem do końca teraz końcówki twojej wypowiedzi. Kiedy dokładnie trzeba kilka razy to dodać? @krystian3w

krystian3w commented 3 years ago

Kilka filtrów jak wolisz użyć contains / has / upward.

AroKol78 commented 3 years ago

pytania 2 dotyczące dobreprogramy.pl

czy nadal jest potrzebne łączenie z googletagservices (na polygamia.pl jest odłączone) ? (@@||googletagservices.com/tag/js/gpt.js$domain=wp.pl|dobreprogramy.pl|o2.pl|pudelek.pl|so-magazyn.pl)
konieczne jest łączenie z pushpushgo.com ?

MajkiIT / polish-ads-filter

Grupa wp.pl #18472