Skip to content

Conversation

@artembelik
Copy link
Contributor

@artembelik artembelik commented Sep 19, 2025

@artembelik artembelik self-assigned this Sep 19, 2025
@artembelik artembelik changed the base branch from feat/DS-4092 to main September 19, 2025 13:22
@github-actions
Copy link

github-actions bot commented Sep 19, 2025

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

https://koobiq-next--prs-1025-0bvge937.web.app

(expires Tue, 07 Oct 2025 14:11:44 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@artembelik artembelik force-pushed the feat/DS-4093 branch 2 times, most recently from 807f062 to bb5390c Compare September 22, 2025 09:47
@artembelik artembelik marked this pull request as ready for review September 22, 2025 10:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds drag-and-drop functionality to the tag list component, allowing users to reorder tags by dragging them. The implementation leverages Angular CDK's drag-and-drop module and adds new properties to control when dragging is enabled.

  • Added draggable property to KbqTagList and corresponding CSS classes for styling
  • Integrated CDK drag-drop directives with both KbqTag and KbqTagList components
  • Created example components demonstrating the new drag-and-drop functionality

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tools/public_api_guard/components/tags.api.md Updated public API to include new drag-drop imports, draggable properties, and event types
packages/components/tags/tag-list.component.ts Added draggable property, drop list integration, and event handling for tag reordering
packages/components/tags/tag.component.ts Added drag directive integration and draggable state management
packages/components/tags/*.scss Added CSS styling for drag states and cursor behavior
packages/docs-examples/ Added example components and documentation demonstrating drag-and-drop functionality
packages/components-dev/tag/module.ts Updated development module to showcase new drag-and-drop examples

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@rmnturov
Copy link
Contributor

Сейчас непонятно теги перставляются, нужно или сделать по спеке (показывать индикатор-полоску между тегами) или преставлять теги на лету во время перетаскивания

@rmnturov
Copy link
Contributor

после перетаскивания фокус должен оказать на теге, которые перетаскивали. тянули мышкой, поэтому рамки фокуса не нужно

@rmnturov
Copy link
Contributor

порой вот такой внешний вид ловлю → две рамки фокуса. так быть не должно
image

@rmnturov
Copy link
Contributor

rmnturov commented Sep 29, 2025

Давай не будет добавлять рамку фокуса в момент перетаскивания, ни на превью, ни на плейсхолдер бледный. Рамка фокуса у нас видна только при управляения с клавы

@rmnturov
Copy link
Contributor

rmnturov commented Sep 29, 2025

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

@artembelik
Copy link
Contributor Author

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

добавил

@artembelik artembelik marked this pull request as draft September 30, 2025 13:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 18 out of 18 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@artembelik artembelik marked this pull request as ready for review October 1, 2025 13:01
@artembelik artembelik requested a review from NikGurev October 1, 2025 13:01
@rmnturov
Copy link
Contributor

rmnturov commented Oct 2, 2025

Вот я перетаскиваю тег 0. Для меня странно, что выбрался тег 1 после перетатскивания. Рамка фокуса на контроле видна. Контрол в фокусе? Пытаюсь ввести текст, не получается.

Что я ожидаю: тянешь тег, никаких синих тегов не видишь. И уж точно синим не должен становиться тег, который ты не тянул. После перетаскивания мышкой в принципе норм, если фокус будет вставать в область создания нового тега и контрол будет с синей рамкой

taglist-dnd.mp4

@artembelik
Copy link
Contributor Author

Вот я перетаскиваю тег 0. Для меня странно, что выбрался тег 1 после перетатскивания. Рамка фокуса на контроле видна. Контрол в фокусе? Пытаюсь ввести текст, не получается.

Что я ожидаю: тянешь тег, никаких синих тегов не видишь. И уж точно синим не должен становиться тег, который ты не тянул. После перетаскивания мышкой в принципе норм, если фокус будет вставать в область создания нового тега и контрол будет с синей рамкой

выбор тегов будет исправлен в рамках DS-4161, фокус поправлю

@artembelik artembelik requested a review from NikGurev October 2, 2025 14:06
@artembelik artembelik merged commit a422338 into main Oct 3, 2025
4 checks passed
@artembelik artembelik deleted the feat/DS-4093 branch October 3, 2025 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants