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

Изменение цветовой индикации фильтров на странице open issues #294

Closed
Shvidunov opened this issue Jul 27, 2023 · 12 comments
Labels

Comments

@Shvidunov
Copy link

Предлагаемая задача

на странице open issues все фильтры при первом переходе выглядят активными (синего цвета), хотя они не задействованы.
Если пошагово нажать на каждый - происходит фильтрация по лейблам, но при этом исчезают все нерешенные проблемы без лейблов. в итоге все активные фильтры выглядят также как и все неактивные.

image

image

Решения задачи

отключить индикацию фильтров до их непосредственного использования (серые фильтры при открытии страницы), возможно добавить фильтр no labels

@NikGor
Copy link
Contributor

NikGor commented Aug 15, 2023

кроме цветовой индикации не работает пагинация, переход на 2 страницу приводит к Not Found: The requested resource was not found on this server. Создадите отдельное issue на проблему?

@NikGor
Copy link
Contributor

NikGor commented Aug 22, 2023

реализовал аналогичный фильтр по лейблам у себя в локальном проекте Task Manager.
https://github.com/NikGor/TaskManager

беру issue в работу

@fey
Copy link
Collaborator

fey commented Aug 23, 2023

@NikGor можете показать, как это выглядит? Потыкал проект, не увидел https://python-project-52-production.up.railway.app/tasks/

@fey
Copy link
Collaborator

fey commented Aug 23, 2023

Вообще тут надо скорректировать логику, по которой будут подсвечиваться лейблы.
Если не выбран ни один лейбл - то все серого цвета.
Если выбран лейбл по которому фильтрация - то только он подсвечивается.
Если нажимаем на следующий лейбл, то он добавляется и посвечивается тоже.

Также есть баг щас с тем, что допустим выбрали один из лейблов https://friends.hexlet.io/issues/?contribution_labels=enhancement
То будут подсвечены все лейблы, которые есть у ищщусов (даже тот лейбл, который мы не выбирали для фильтрации).

Короче говоря лейбл горит синим, если по нему фильтруем

image

@NikGor
Copy link
Contributor

NikGor commented Aug 23, 2023

@NikGor можете показать, как это выглядит? Потыкал проект, не увидел https://python-project-52-production.up.railway.app/tasks/

да, я оставил 4-й проект как есть, и таск-менеджер у меня как сайд-проект: https://github.com/NikGor/TaskManager
Но у меня это поведение все-таки сделано через JS

@NikGor
Copy link
Contributor

NikGor commented Aug 23, 2023

Вообще тут надо скорректировать логику, по которой будут подсвечиваться лейблы. Если не выбран ни один лейбл - то все серого цвета. Если выбран лейбл по которому фильтрация - то только он подсвечивается. Если нажимаем на следующий лейбл, то он добавляется и посвечивается тоже.

Также есть баг щас с тем, что допустим выбрали один из лейблов https://friends.hexlet.io/issues/?contribution_labels=enhancement То будут подсвечены все лейблы, которые есть у ищщусов (даже тот лейбл, который мы не выбирали для фильтрации).

Короче говоря лейбл горит синим, если по нему фильтруем

image

понял. Да, так и будет логично.
Еще вроде как не хватает кнопки ресета всех фильтров

@DREU007
Copy link
Contributor

DREU007 commented Nov 23, 2023

Беру в работу.

@DREU007
Copy link
Contributor

DREU007 commented Nov 25, 2023

@fey, Пофиксил отображение цветов. Вопрос по кнопке ресета. Сделать отдельно для всех фильтров или только для label?

@Shvidunov
Copy link
Author

получилось наоборот - активные серым цветом выделяются. Может есть смысл наоборот сделать? Неактивные серые (как то логичнее), активные синие. Но тогда при переходе на страницу все фильтры должны быть отключены (серый цвет).

@fey
Copy link
Collaborator

fey commented Nov 25, 2023

давайте активные зеленым сделаем, а неактивные - серым.
Посмотрите какие есть классы в бутстрапе. По-моему для зеленого можно success класс использовать, а для серого - secondary или outline если это возможно.

@fey
Copy link
Collaborator

fey commented Nov 25, 2023

@DREU007 обычно на форме есть кнопка-ресета для всей формы.

@fey fey closed this as completed Nov 25, 2023
@DREU007
Copy link
Contributor

DREU007 commented Nov 27, 2023

Задеплоил
GitHub

sgmdlt added a commit that referenced this issue Dec 6, 2023
[#294] Fix: label filter color & Feat: filter reset button
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

4 participants