Skip to content

Conversation

@NikGurev
Copy link
Contributor

@NikGurev NikGurev commented Oct 10, 2025

Redesigned the focus logic, following the UX principle: 'content first, everything else after.'
Initially wanted to implement it using CdkFocusTrap and focus delegation, but realized that this approach works differently across browsers.
In the end, implemented it natively by adding an overlay that renders the focus border.
This way, the focus order became correct:

Content -> Overlay (with focus border) -> Menu


Press for description in ru-RU

Переделал логику фокусировки, следуя UX-правилу: "сначала контент, потом все остальное".

Сначала хотел реализовать с помощью CdkFocusTrap и прокидыванием фокуса, но понял, что такой подход в браузерах работает по-разному.

По итогу, реализовал нативно, добавив оверлей, который отрисовывает рамку фокуса.

Таким образом, порядок фокусировки стал верный:

Контент -> Оверлей (с рамкой фокуса) -> Меню

@NikGurev NikGurev self-assigned this Oct 10, 2025
@NikGurev NikGurev added the enhancement New feature or request label Oct 10, 2025
Base automatically changed from feat/DS-4160 to main October 13, 2025 12:22
@github-actions
Copy link

github-actions bot commented Oct 14, 2025

Visit the preview URL for this PR (updated for commit ca0d162):

https://koobiq-next--prs-1052-2zmlglxv.web.app

(expires Wed, 22 Oct 2025 09:36:11 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@NikGurev NikGurev marked this pull request as ready for review October 15, 2025 12:56
@NikGurev NikGurev requested a review from lskramarov as a code owner October 15, 2025 12:57
@NikGurev NikGurev requested a review from rmnturov October 15, 2025 12:59
@NikGurev NikGurev requested a review from artembelik October 16, 2025 10:38
@rmnturov
Copy link
Contributor

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

@NikGurev
Copy link
Contributor Author

NikGurev commented Oct 17, 2025

@rmnturov убрал пример с тегами, заменил на бейджи

@NikGurev NikGurev merged commit f8af5f3 into main Oct 17, 2025
4 checks passed
@NikGurev NikGurev deleted the feat/DS-4210 branch October 17, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants