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.

Sign up for GitHub

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

Jump to bottom

Add filtering in "New tab" windows #881

Closed
vbagirov opened this issue Oct 17, 2017 · 14 comments
Closed

Add filtering in "New tab" windows #881

vbagirov opened this issue Oct 17, 2017 · 14 comments
Assignees
@atropnikov @Mizzick
Labels
Enhancement
Milestone
2.8

Comments

@vbagirov
Copy link
Member

Currently we don't do that, one of the side effects being there's no notification after checking for filter updates.
@vbagirov vbagirov added this to the 2.8 milestone Oct 17, 2017
@ghost
Copy link

ghost commented Oct 18, 2017

А Adguard для Windows может фильтровать "новую вкладку"?

@ameshkov
Copy link
Member

Да для него нет различия новая она или нет:)

@ghost
Copy link

ghost commented Oct 18, 2017

Может меня не правильно поняли, я о вкладке, например, chrome://newtab
В 62 версии Chrome переделали новую вкладку и теперь навязывают свои сервисы прямо в ней.
До 62 версии была возможность обойти это, видоизменив поисковую систему, но теперь нет.
А Adguard кажется полностью игнорирует служебные страницы.

@ameshkov
Copy link
Member

Дефолтная newtab хрома в исключениях в англ фильтре чтоб туда помощник не втыкать:
@@||google.*/_/chrome/newtab$document

Если это правило отключить, все можно прекрасно переделать.

@ghost
Copy link

ghost commented Oct 18, 2017

Хм, отключил правило. Помощника нет, как и инжекта в принципе.
Следов Adguard на view-source не вижу. Может еще какое-то условие есть?

@ameshkov
Copy link
Member

Да не, должно хватить, в журнале фильтрации надо посмотреть

@ghost
Copy link

ghost commented Oct 18, 2017

Установил 6.2 (кстати шикарный журнал теперь сделали)
Несмотря на отключенное правило в фильтре, newtab так и не фильтруется. Ну вот честно не знаю почему.
Но зато с помощью журнала сделал правило, чтобы сторонние сервисы не загружались.
В итоге получилось правило
||google.ru/async/newtab_ogb^
И правило срабатывает всегда, независимо от правила-исключения, которое в английском фильтре.
default

@ameshkov
Copy link
Member

Мне кажется что newtab закэширован по-черному браузером и он его не перезапрашивает просто. Может в профиль перезайти надо?

@ghost
Copy link

ghost commented Oct 18, 2017

Профиль у меня по умолчанию, то есть регистрации в гугле я не имею, профиль всего один.
А у вас инжектится Adguard что ли в новую вкладку?
Если сделать поиском по умолчанию строку
https://www.google.ru/search?q=%s
изменится ли что-нибудь?

@ameshkov
Copy link
Member

Да не, не изменится. Я о чем - думаю хром закэшировал код "новой вкладки" и больше не будет его перезапрашивать какое-то время. Чтобы AG отфильтровал ее и применил правила, надо заставить хром снова попытаться загрузить ее.

@ghost
Copy link

ghost commented Oct 18, 2017

Установил второго пользователя, там естественно поисковая строка какую дает нам гугл.
Новая вкладка инжектится.
Вот с такими параметрами поиска новая вкладка не инжектится
default

@ghost
Copy link

ghost commented Oct 18, 2017

Попробуйте сделать такой поиск и установить по умолчанию

@Mizzick
Copy link
Contributor

Mizzick commented Oct 20, 2017

https://stackoverflow.com/a/45659562
According to this answer it's not fully possible to add filtering for the new tabs.

@Mizzick Mizzick mentioned this issue Oct 23, 2017
Closed
@atropnikov
Copy link
Member

  1. Chrome (Firefox) doesn't inject content scripts into the "New tab"
  2. All requests from the "New tab" are filtering

adguard pushed a commit that referenced this issue Jun 7, 2021
@maximtop
Pull request #881: AG-8201 open editor in the fullscreen window
f43de63 
Merge in EXTENSIONS/browser-extension from feature/AG-8201 to feature/AG-2737

Squashed commit of the following:

commit 5352964
Merge: e5b7165 151d89f
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Mon Jun 7 11:37:44 2021 +0300

    Merge branch 'feature/AG-8201' of ssh://bit.adguard.com:7999/extensions/browser-extension into feature/AG-8201

commit e5b7165
Merge: b4c496f 71f6937
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Mon Jun 7 11:31:05 2021 +0300

    Merge branch 'feature/AG-2737' into feature/AG-8201

commit 151d89f
Merge: fdd628a b4c496f
Author: anton <a.kaprielov@adguard.com>
Date:   Thu Jun 3 15:16:24 2021 +0300

    Merge branch 'feature/AG-8201' of ssh://bit.adguard.com:7999/extensions/browser-extension into feature/AG-8201

commit fdd628a
Author: anton <a.kaprielov@adguard.com>
Date:   Thu Jun 3 15:16:18 2021 +0300

    feature/AG-8201 fix white border

commit b4c496f
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Thu Jun 3 15:02:10 2021 +0300

    AG-8201 add comment

commit 41205cc
Merge: 8ae6066 d4a6d9f
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Thu Jun 3 14:46:40 2021 +0300

    Merge branch 'feature/AG-8201' of ssh://bit.adguard.com:7999/extensions/browser-extension into feature/AG-8201

commit 8ae6066
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Thu Jun 3 14:46:31 2021 +0300

    AG-8201 add tooltips to icon buttons

commit d4a6d9f
Author: anton <a.kaprielov@adguard.com>
Date:   Thu Jun 3 12:16:59 2021 +0300

    feature/AG-8201 fix actions outline

commit aed4c86
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Thu Jun 3 11:42:51 2021 +0300

    AG-8201 always get wrap button state from localstorage

commit b85ad62
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Thu Jun 3 11:22:29 2021 +0300

    AG-8201 rules considered saved when, they saved to storage

commit 2601c09
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Wed Jun 2 15:36:49 2021 +0300

    AG-8201 reduce editor renderings on user rules enabled state toggle

commit 979ac08
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Wed Jun 2 11:53:21 2021 +0300

    AG-8201 remove notify on unload

commit a139722
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Wed Jun 2 11:52:15 2021 +0300

    AG-8201 use background memory to keep editor state

commit 40c520e
Merge: 01b10fc 50e0bbd
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Fri May 28 13:42:39 2021 +0300

    Merge branch 'feature/AG-2737' into feature/AG-8201

commit 01b10fc
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Fri May 28 11:57:52 2021 +0300

    AG-8201 do not remove unsaved content on rules wrap mode toggle

commit b341c9a
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Fri May 28 11:23:46 2021 +0300

    AG-8201 do not close window if there are unsaved changes

commit 5c3e907
Merge: 67f067a 0ec0b33
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Fri May 28 10:45:52 2021 +0300

    Merge branch 'feature/AG-2737' into feature/AG-8201

commit 67f067a
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Wed May 26 07:58:35 2021 +0300

    AG-8201 close window

commit cdc292e
Author: anton <a.kaprielov@adguard.com>
Date:   Tue May 25 18:26:13 2021 +0300

    feature/AG-8201 fix styles

commit 37a69c9
Merge: 92c2267 47f4c29
Author: Maxim Topciu <maximtop@gmail.com>
Date:   Tue May 25 17:58:47 2021 +0300

    Merge branch 'feature/AG-2737' into feature/AG-8201

... and 12 more commits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atropnikov atropnikov

@Mizzick Mizzick

Labels
Enhancement
Projects
None yet
Milestone
2.8
Development

No branches or pull requests
4 participants
@atropnikov @ameshkov @vbagirov @Mizzick