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.

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

uBlock filtry #115

Closed
vobruba-martin opened this issue Nov 29, 2017 · 14 comments
Closed

uBlock filtry #115

vobruba-martin opened this issue Nov 29, 2017 · 14 comments
Assignees

Comments

@vobruba-martin
Copy link
Contributor

Vzhledem k tomu, ze filtry pro uBlock rozbijeji fungovani ABP pro Chrome/Operu, neni vhodne je pridavat do filters.txt. Napada me tedy, jestli by bylo reseni, kdyby doslo k vytvoreni filtru primo pro uBlock.

Je otazka, jakym zpusobem by to pripadne mohlo fungovat:

  1. filters_ublock.txt by obsahoval stejny obsah jako filters.txt + pravidla pro uBlock.
  • Bylo by to horsi na spravu, protoze by bylo pro kazde pravidlo nutne upravovat 2 soubory.
  • Asi by bylo snazsi do uBlocku prosadit jen zmenu URL.
  1. filters_ublock.txt by obsahoval jen pravidla pro uBlock.
  • Snazsi na spravu.
  • Nevim, jakym zpusobem by se to dalo nejak rozumne prosadit do uBlocku.

/cc @tomasko126 @Aslanex

@vobruba-martin
Copy link
Contributor Author

Zrovna koukam, ze diky commitu b41e211, mam Chrome s ABP na zive.cz i mobilmania.cz opet plne reklam.

@MilanKK
Copy link

MilanKK commented Nov 30, 2017

Tento filtr:
mobilmania.cz,zive.cz##.ar-detail + div
blokne psaní příspěvků na zive.cz
= uživatelé s cz easylistem teď nemůžou na zive.cz vkládat příspěvky.
Musel jsem si v mých filtrech nastavit:
zive.cz#@#.ar-detail + div

@vobruba-martin
Copy link
Contributor Author

Koukal jsem na zdroj uBlocku a PR pro zmenu URL by slo vytvorit snadno. Sel bych tedy 1. cestou:

  • Vytvorit kopii stavajich filtru.
  • Poslat do uBlocku PR pro zmenu URL.
  • Vycistit stavajici filtry od uBlock pravidel.

@Aslanex
Copy link
Collaborator

Aslanex commented Dec 1, 2017

To by neměl být problém, co na to @tomasko126 ?

@tomasko126
Copy link
Owner

@Aslanex @vobruba-martin Súhlasím s 1. možnosťou.

@tomasko126 tomasko126 self-assigned this Dec 3, 2017
@tomasko126
Copy link
Owner

tomasko126 commented Dec 3, 2017

@Aslanex @vobruba-martin
V commite e03eea7 bol vytvorený filters_ublock.txt, ktorého obsah bol vytvorený na základe filters.txt mínus ABP špecifické filtre.
Taktiež bol vyčistený filters.txt od uBlock špecifických filtrov.

@tomasko126
Copy link
Owner

@MilanKK Daný filter bol odstránený v 0a1f9f0.

@vobruba-martin
Copy link
Contributor Author

@tomasko126
To asi uplne neni stastne reseni, protoze ABP specificke filtry umi uBlock interpretovat spravne. Jsou tam nejake vyjimky, ale to bude chtit resit pro kazde pravidlo zvlast. Momentalne by doslo k tomu, ze veci, ktere do ted v uBlocku byly blokovany spravne, prestanou byt v uBlocku blokovany kvuli absenci tech ABP specifickych filtru.

Zkusim to kdyztak projit, jakmile si najdu chvili a take vytvorim PR v uBlock repozitari.

@vobruba-martin
Copy link
Contributor Author

PR v uBlock repozitari je zde: gorhill/uBlock#3312

@tomasko126
Copy link
Owner

@vobruba-martin Všetky ABP špecifické filtre v zozname filtrov som pridal na koniec súboru filters.txt. Je niekde (pravdepodobne v repo uBlocku) zoznam ABP špecifických filtrov, ktoré sa automaticky konvertujú v uBlocku na uBlock pravidlá?

@vobruba-martin
Copy link
Contributor Author

Ja to chapu tak, ze uBlock by mel podporovat vsechny ABP syntaxe. Viz https://github.com/gorhill/uBlock/wiki/Static-filter-syntax

Kazdopadne vim minimalne o jedne vyjimce:
:-abp-has() se sice automaticky konvertuje na :has(), ale v uBlocku muze :has() obsahovat pouze validni CSS selector kvuli pripadne budouci kompatibilite v browserech. V ABP muze :-abp-has() obsahovat napr. regularni vyraz nebo dalsi ABP specificky selector. Zalezi tedy pripad od pripadu, kdy muze jedno pravidlo fungovat v obou blokatorech a kdy je potreba to vyresit pro kazdy z nich zvlast.

@vobruba-martin
Copy link
Contributor Author

V 51f2c3c jsem synchronizoval uBlock filtry. Vsechny ABP specificke filtry, ktere tam mame, by mely byt kompatibilni s uBlockem, protoze :-abp:has() pravidla vzdy obsahuji validni CSS selector.

@vobruba-martin
Copy link
Contributor Author

Tak myslim, ze mame vyreseno :-)

@krystian3w
Copy link

#136

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants