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

Add $important modifier to the rule constructor #626

Closed
Alex-302 opened this Issue Apr 3, 2017 · 11 comments

Comments

Projects
None yet
5 participants
@Alex-302
Member

Alex-302 commented Apr 3, 2017

... like in the app:
image

@ameshkov ameshkov added this to the 3.0 milestone Apr 3, 2017

@atropnikov atropnikov modified the milestones: 2.6, 3.0 May 22, 2017

@Mizzick Mizzick self-assigned this May 22, 2017

atropnikov added a commit that referenced this issue May 23, 2017

@atropnikov atropnikov closed this May 23, 2017

@XX-J

This comment has been minimized.

Show comment
Hide comment
@XX-J

XX-J Aug 21, 2017

Почему не срабатывает следующее правило блокировки:
cdn.mozilla.net/*/jsi18n.js$important ?
Ведь оно должно превалировать перед исключающем правилом из "Английского фильтра":
@@||cdn.mozilla.net$document

После отключения "Английского фильтра" моё правило начинает работать.


Pale Moon v26.5 (WinXP) - Adguard legacy v2.6.7
Английский фильтр | Русский фильтр | Фильтр раздражителей
Фильтр виджетов социальных сетей | Фильтр счетчиков и системы аналитики

XX-J commented Aug 21, 2017

Почему не срабатывает следующее правило блокировки:
cdn.mozilla.net/*/jsi18n.js$important ?
Ведь оно должно превалировать перед исключающем правилом из "Английского фильтра":
@@||cdn.mozilla.net$document

После отключения "Английского фильтра" моё правило начинает работать.


Pale Moon v26.5 (WinXP) - Adguard legacy v2.6.7
Английский фильтр | Русский фильтр | Фильтр раздражителей
Фильтр виджетов социальных сетей | Фильтр счетчиков и системы аналитики

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Aug 22, 2017

Member

Не, только не с $document. Другое дело, что это правило (сдн.мозилла) вообще не очень нужно.

@Alex-302 может удалим его?

Member

ameshkov commented Aug 22, 2017

Не, только не с $document. Другое дело, что это правило (сдн.мозилла) вообще не очень нужно.

@Alex-302 может удалим его?

@XX-J

This comment has been minimized.

Show comment
Hide comment
@XX-J

XX-J Aug 22, 2017

Не, только не с $document.

Оказывается Ваш $important Не настолько и Important... Почему??

может удалим его?

Я - за. По моему - это лишнее предостережение.

XX-J commented Aug 22, 2017

Не, только не с $document.

Оказывается Ваш $important Не настолько и Important... Почему??

может удалим его?

Я - за. По моему - это лишнее предостережение.

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Aug 22, 2017

Member

Тут скорее вопрос другой - почему $document применяется к запросам с типом, отличным от DOCUMENT, это не его задача

Member

ameshkov commented Aug 22, 2017

Тут скорее вопрос другой - почему $document применяется к запросам с типом, отличным от DOCUMENT, это не его задача

@Alex-302

This comment has been minimized.

Show comment
Hide comment
@Alex-302

Alex-302 Aug 22, 2017

Member

@ameshkov ок, удаляю. Правило очень староею

Member

Alex-302 commented Aug 22, 2017

@ameshkov ок, удаляю. Правило очень староею

Alex-302 added a commit to AdguardTeam/AdguardFilters that referenced this issue Aug 22, 2017

@XX-J

This comment has been minimized.

Show comment
Hide comment
@XX-J

XX-J Aug 22, 2017

DOCUMENT - это не его задача

Согласно Справке document включает в себя jsinject, а уж тот, в свою очередь, блокирует *.js . Поэтому, можно сказать, что - Его.

XX-J commented Aug 22, 2017

DOCUMENT - это не его задача

Согласно Справке document включает в себя jsinject, а уж тот, в свою очередь, блокирует *.js . Поэтому, можно сказать, что - Его.

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov Aug 22, 2017

Member

jsinject - отключает JS-правила на странице. Так что цель все же именно страница.

Member

ameshkov commented Aug 22, 2017

jsinject - отключает JS-правила на странице. Так что цель все же именно страница.

@XX-J

This comment has been minimized.

Show comment
Hide comment
@XX-J

XX-J Aug 22, 2017

jsinject - Запрещает добавление javascript-кода на страницу.
Я так понял что из Всех мест, и со страницы, и с внешних файлов.
Но если задумка была ТОЛЬКО со страницы, то на лицо явно не правильная работа.

XX-J commented Aug 22, 2017

jsinject - Запрещает добавление javascript-кода на страницу.
Я так понял что из Всех мест, и со страницы, и с внешних файлов.
Но если задумка была ТОЛЬКО со страницы, то на лицо явно не правильная работа.

@Alex-302

This comment has been minimized.

Show comment
Hide comment
@Alex-302

Alex-302 Aug 22, 2017

Member

Я так понял что из Всех мест, и со страницы, и с внешних файлов.

не из Всех мест. При добавлении правила с этим модификатором Адгард не встраивает свой JS код в страницы. Чужой JS(в коде страницы и прочие скрипты) не затрагивается.

Member

Alex-302 commented Aug 22, 2017

Я так понял что из Всех мест, и со страницы, и с внешних файлов.

не из Всех мест. При добавлении правила с этим модификатором Адгард не встраивает свой JS код в страницы. Чужой JS(в коде страницы и прочие скрипты) не затрагивается.

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov
Member

ameshkov commented Aug 22, 2017

@XX-J

This comment has been minimized.

Show comment
Hide comment
@XX-J

XX-J Aug 22, 2017

Ааа, я понял - попутал свой-чужой js.

Прикинул что похоже на urlblock. Проверил - и вправду Он. Загадка с document - решена.

XX-J commented Aug 22, 2017

Ааа, я понял - попутал свой-чужой js.

Прикинул что похоже на urlblock. Проверил - и вправду Он. Загадка с document - решена.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment