Skip to content

Conversation

@artembelik
Copy link
Contributor

No description provided.

@artembelik artembelik self-assigned this Aug 21, 2025
@github-actions
Copy link

github-actions bot commented Aug 21, 2025

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

https://koobiq-next--prs-987-pedcy94r.web.app

(expires Mon, 08 Sep 2025 09:05:14 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@artembelik artembelik requested a review from rmnturov August 25, 2025 14:41
@rmnturov
Copy link
Contributor

Сейчас легко ткнуть не туда и фокус ставится на теги. Не туда — это только с точки зрения компонента, а для пользователя он кликал в область в ввода, он опчти попал по крестику. Ожидается, что именно эти вещи сработают, а не то, когда фокус уходит в совершенно другое место

tag-input-click-area.mp4

@rmnturov
Copy link
Contributor

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

Спека
изображение

@rmnturov
Copy link
Contributor

Тосты при редактировании и сохранении не нужны в доке. Они говорят, будто так и надо делать в продуктах. А не надо

@rmnturov
Copy link
Contributor

При вводе длинного текста ожидаю, что тег перенесется на новую строку, общая высота тег инпута увеличится

Сейчас так
изображение

А надо так
изображение

@rmnturov
Copy link
Contributor

При редактировании тега с длинным значением не должно быть уменьшения тега и переноса

Сейчас так
изображение

Надо так
изображение

@rmnturov
Copy link
Contributor

Не работает при редактировании тега

Отмена изменений:
При нажатии клавиши Escape.

@rmnturov
Copy link
Contributor

rmnturov commented Aug 26, 2025

При коротком значении тег не уменьшается до мин. ширины, а должен
изображение

@rmnturov
Copy link
Contributor

Есть возможность сохранить пустой тег, а такого быть не должно
изображение

@rmnturov
Copy link
Contributor

В примере «Редактирование» пусть все созданные юзером теги будут доступны для редактирования

@rmnturov
Copy link
Contributor

Фокуса на кнопку-галочку быть не должно
изображение
изображение

Copy link
Contributor

@NikGurev NikGurev left a comment

Choose a reason for hiding this comment

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

Позиция курсора сохраняется при выходе из режима редактирования. @rmnturov нужно ли его устанавливать в начало при повторном открытии?

Screen.Recording.2025-08-27.at.15.43.24.mov

@rmnturov
Copy link
Contributor

rmnturov commented Aug 28, 2025

минимальная ширина тега, надо больше
слева реализация. справа проект
изображение

@rmnturov
Copy link
Contributor

Позиция курсора сохраняется при выходе из режима редактирования. @rmnturov нужно ли его устанавливать в начало при повторном открытии?
Screen.Recording.2025-08-27.at.15.43.24.mov

@artembelik @NikGurev при входе в редактирование предлагаю выделять текст в теге и ставить курсор в конец

@rmnturov
Copy link
Contributor

Сейчас легко ткнуть не туда и фокус ставится на теги. Не туда — это только с точки зрения компонента, а для пользователя он кликал в область в ввода, он опчти попал по крестику. Ожидается, что именно эти вещи сработают, а не то, когда фокус уходит в совершенно другое место

эта логика не изменялась в рамках этой фичи, кажется что это баг, как это должно работать?

пока оставим по-старому. помечу себе отдлельной задачей

@rmnturov
Copy link
Contributor

Позиция курсора сохраняется при выходе из режима редактирования. @rmnturov нужно ли его устанавливать в начало при повторном открытии?
Screen.Recording.2025-08-27.at.15.43.24.mov

в начало точно не нужно. чаще всего курсор устанавливается в конец

@artembelik
Copy link
Contributor Author

artembelik commented Aug 28, 2025

минимальная ширина тега, надо больше

будет реализовано в рамках DS-4146

@artembelik
Copy link
Contributor Author

при входе в редактирование предлагаю выделять текст в теге и ставить курсор в конец

@rmnturov @NikGurev поправил

@lskramarov lskramarov self-requested a review September 3, 2025 09:07
@rmnturov
Copy link
Contributor

rmnturov commented Sep 4, 2025

Хром макось
Криво сохраняется тег с длинным текстом

tag-long-text.mp4

@artembelik artembelik merged commit 7caac5b into main Sep 4, 2025
4 checks passed
@artembelik artembelik deleted the feat/DS-4091 branch September 4, 2025 09:35
@rmnturov
Copy link
Contributor

Отключить бы в примере с тегами автокомплит барузера
image

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