feat(tags): added tag selection on focus for non multiple tag-list (#DS-4212) #1029
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR implements tag selection on focus for non-multiple tag lists, enhancing the user interaction pattern by allowing tags to be selected when focused rather than only through explicit click interactions with modifier keys.
- Added automatic tag selection on focus and deselection on blur for standalone tags and non-multiple tag lists
- Updated documentation to reflect the new selection behavior
- Consolidated test components to simplify test structure
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/public_api_guard/components/tags.api.md | Removed undocumented comments from public API |
| packages/components/tags/tag.ru.md | Updated Russian documentation for new selection behavior |
| packages/components/tags/tag.en.md | Updated English documentation for new selection behavior |
| packages/components/tags/tag.component.ts | Added focus/blur selection logic and improved documentation |
| packages/components/tags/tag.component.spec.ts | Consolidated test components and added focus/blur selection tests |
| packages/components/tags/tag-list.ru.md | Updated Russian documentation for tag-list selection behavior |
| packages/components/tags/tag-list.en.md | Updated English documentation for tag-list selection behavior |
| packages/components/tags/tag-list.component.ts | Added focus/blur event handling for non-multiple mode |
| packages/components/tags/tag-list.component.spec.ts | Added comprehensive tests for new selection behavior |
| packages/components/tags/tag-input.ru.md | Updated Russian documentation for tag-input selection behavior |
| packages/components/tags/tag-input.en.md | Updated English documentation for tag-input selection behavior |
| packages/components/tags/tag-autocomplete.ru.md | Updated Russian documentation for tag-autocomplete selection behavior |
| packages/components/tags/tag-autocomplete.en.md | Updated English documentation for tag-autocomplete selection behavior |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
Visit the preview URL for this PR (updated for commit e23d927): https://koobiq-next--prs-1029-q2nt4ji5.web.app (expires Sun, 05 Oct 2025 10:25:58 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c |
artembelik
changed the title
|
в примере с выбором тегов нудно добавить фичу удаления тегов. без удаления нет смысла выбирать несколько тегов. Поэтому страннно получается, мы дает выбирать, но зачем? ) |
|
когда нет мультивыбора, но нет смысла выделять ярким фоном тег (в режиме одиночного выбора достаточно только рамки фокуса) |
исправим в рамках DS-4224 |
как договорились удалил примеры с selectable, оставил только описание |
artembelik
force-pushed
the
feat/DS-4212
branch
from
c236922 to
df7057d
Compare
|
Кликаем по крестику → фокус перемещается. Рамки фокуса не надо показывать видимой, тк было управление мышкой. Рамка фокуса показывается только при управлении с клавы. Если тег удалили с клавы, то уже надо показывать фокус на лементе, который зафокусился следом. Если фокус перешел в область ввода, то рамка нужна даже если до этого было взаимодейтсвие мышкой |
artembelik
force-pushed
the
feat/DS-4212
branch
from
df7057d to
c737e0a
Compare
поправил @rmnturov переместил "выбор тегов" под "удаление" в документации, как обсуждали |
|
Каретка инпута коряво обрезается. Отдельной задачей поправь плиз. Приоритет P3 |
посмотрю в рамках DS-4204 |
No description provided.