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

Some UI changes for editor #2125

Closed
XX-J opened this issue Jun 24, 2022 · 5 comments
Closed

Some UI changes for editor #2125

XX-J opened this issue Jun 24, 2022 · 5 comments

Comments

@XX-J
Copy link

XX-J commented Jun 24, 2022

1. Добавьте, пожалуйста, выключатель пользовательского фильтра на отдельную страницу редактирования этого самого пользовательского фильтра.

2022 06 24-2

А то неудобно - редактируешь на одной странице, а лезть за полным отключением надо в настройки - на другую страницу. Я всегда держу страницу редактирования пользовательских правил открытой, а держать открытой ещё и страницу настроек из-за одной из них - это перебор. (Отсюда: #1352 (comment))


2. Избыточная высота футера - можно сделать в 2 раза меньше.

3. Отступ перенесённой строки 4 пробела - достаточно 2-х.

2022 06 24-3


AdGuard v4.0.181   |   FireFox v78.13 ESR   |   Win7SP1

@maximtop maximtop changed the title Some UI changes in Filtering log Some UI changes for editor Jul 11, 2022
@XX-J
Copy link
Author

XX-J commented Jul 12, 2022

При изменении высоты футера сообщение "Сохранено" можно перенести внутрь кнопки "Сохранить", временно меняя в ней текст.

2022 07 12-1

@XX-J
Copy link
Author

XX-J commented Jul 20, 2022

4. Перенести кнопку "Сохранить" в центр или поставить её после кнопок "Импорт" "Экспорт". Кнопка "Сохранить" самая используемая (по крайней мере для меня) и по этому к ней можно было бы быстрее дотянуться мышью, если бы она была расположена ближе к центру.

2022 07 20-1

adguard pushed a commit that referenced this issue Sep 8, 2022
Merge in EXTENSIONS/browser-extension from feature/AG-14968 to master

Squashed commit of the following:

commit b2a6c24
Merge: bae2693 6e12572
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Tue Sep 6 19:53:49 2022 +0300

    Merge branch 'master' into feature/AG-14968

commit bae2693
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Fri Aug 26 19:53:33 2022 +0300

    clear subscription forr useEffect

commit 5856d82
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Fri Aug 26 19:31:51 2022 +0300

    subscribe user rules editor store to settings updates

commit da13602
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Mon Aug 22 17:27:54 2022 +0300

    fix checkbox label background

commit d49d08f
Merge: d254f04 a4a106a
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Mon Aug 22 17:00:04 2022 +0300

    resolve conflict

commit d254f04
Author: Anton <a.kaprielov@adguard.com>
Date:   Wed Aug 3 11:37:04 2022 +0300

    AG-14968 fix switcher in fullscreen mode

commit 807566a
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Wed Aug 3 01:20:05 2022 +0300

    improve layout (not really)

commit 917fd95
Author: Stanislav A <s.atroschenko@adguard.com>
Date:   Tue Aug 2 00:50:29 2022 +0300

    add user-filter-enabled button to fullscreen user rules
@XX-J
Copy link
Author

XX-J commented Oct 7, 2022

Спасибо Большое за 1-е и 4-е!

Но что насчёт 2-го и 3-го пункта?

Суммирую:

2. Избыточная высота футера - можно сделать в 2 раза меньше.

3. Отступ перенесённой строки 4 пробела - достаточно 2-х.

2022 06 24-3

При изменении высоты футера сообщение "Сохранено" можно перенести внутрь кнопки "Сохранить", временно меняя в ней текст.

2022 07 12-1

@maximtop
Copy link
Contributor

Второй и третий пункты мы менять не будем.

@XX-J
Copy link
Author

XX-J commented Oct 11, 2022

Второй и третий пункты мы менять не будем.

Почему?

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

No branches or pull requests

4 participants