Skip to content

Conversation

@NikGurev
Copy link
Contributor

@NikGurev NikGurev commented Oct 3, 2025

No description provided.

@NikGurev NikGurev self-assigned this Oct 3, 2025
@NikGurev NikGurev added the enhancement New feature or request label Oct 3, 2025
@github-actions
Copy link

github-actions bot commented Oct 3, 2025

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

https://koobiq-next--prs-1048-mfd3sf4y.web.app

(expires Sat, 18 Oct 2025 09:59:54 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@NikGurev NikGurev requested a review from rmnturov October 7, 2025 15:46
@NikGurev NikGurev marked this pull request as ready for review October 7, 2025 15:46
@NikGurev NikGurev requested a review from lskramarov as a code owner October 7, 2025 15:46
@rmnturov
Copy link
Contributor

rmnturov commented Oct 8, 2025

Первый пример. Редактирую тег, жму Enter, жму Tab — фокус будто сбросился на боди и начал я навигацию с начала страницы. Ожидаю, что фокус после Enter вернется на элемент списка параметров, просто рамки не будет видно.

@rmnturov
Copy link
Contributor

rmnturov commented Oct 8, 2025

Фокусируюсь на кнопке Отмена в Textarea. Активрую ее по Enter. Фокус оказывается в Textarea. Ожидаю выхода из редактирования, отмену изменений, фокус на элемент списка параметров в режиме просмотра
Clipboard-20251008-064709-234

@rmnturov
Copy link
Contributor

rmnturov commented Oct 8, 2025

Выбрал много опций. Выпадашка стала перекрывать селект с тегами, который увеличился по высоте
image

@NikGurev
Copy link
Contributor Author

NikGurev commented Oct 8, 2025

Выбрал много опций. Выпадашка стала перекрывать селект с тегами, который увеличился по высоте

Это баг select'а , сделаю отдельной задачей

@NikGurev
Copy link
Contributor Author

NikGurev commented Oct 9, 2025

@rmnturov поправил поведение по кнопке "Отмена" и нажатие на Tab.

@lskramarov перепроверь пожалуйста, добавил вспомогательную сущность KbqFocusRegionItem , которая трекает уход из элемента по табу.
cdkFocusMonitor не подошел, потому что нужно трекать именно keydown события в данном случае

@rmnturov
Copy link
Contributor

rmnturov commented Oct 13, 2025

@NikGurev
чет скролл блокируется при редактировании, мы в какой-то момент, мне кажется, это убирали. в момент редактирования сохранялсь прокрутка, теперь это снова появилось. или я запамятовал?

мне кажется, мы убирали модальность также (можно было ховерить на другие элементы)

я даже доку переделывал
image

@rmnturov
Copy link
Contributor

из-за блокировки скролла возможны проблемы с textarea, когда она увеличится и выйдет за пределы экрана
image

@NikGurev
Copy link
Contributor Author

NikGurev commented Oct 13, 2025

@rmnturov , @lskramarov
приложу для справки

Cкролл не работает в Google Chrome, потому что backdrop может отлавливать события мыши

@NikGurev NikGurev merged commit 2892067 into main Oct 13, 2025
4 checks passed
@NikGurev NikGurev deleted the feat/DS-4160 branch October 13, 2025 12:22
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.

4 participants