Skip to content

Conversation

@lskramarov
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jul 21, 2025

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

https://koobiq-next--prs-927-cose1efl.web.app

(expires Mon, 28 Jul 2025 13:27:49 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@lskramarov lskramarov self-assigned this Jul 22, 2025
@lskramarov lskramarov marked this pull request as ready for review July 22, 2025 13:19
@rmnturov
Copy link
Contributor

Когда фокус падает на очистку, то не надо выделять рамкой само поле. Две синие рамки одновременно получается — так не хорошо.

Когда фокус на крестик, то рамка только на крестике. На поле нет синей рамки

@rmnturov
Copy link
Contributor

На обрезанный тег давай навесим тайтл (тултип с полным названием) по ховеру
изображение

@rmnturov
Copy link
Contributor

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

@rmnturov
Copy link
Contributor

убери плиз эту подпись в примере
изображение

@lskramarov
Copy link
Contributor Author

На обрезанный тег давай навесим тайтл (тултип с полным названием) по ховеру

убрал длинный текст из примера

@lskramarov
Copy link
Contributor Author

Когда фокус падает на очистку, то не надо выделять рамкой само поле. Две синие рамки одновременно получается — так не хорошо.

Когда фокус на крестик, то рамка только на крестике. На поле нет синей рамки

сейчас везде такое поведение, предлагаю изменить его в DS-3983 или вообще отдельной задачей

@rmnturov
Copy link
Contributor

Когда фокус падает на очистку, то не надо выделять рамкой само поле. Две синие рамки одновременно получается — так не хорошо.
Когда фокус на крестик, то рамка только на крестике. На поле нет синей рамки

сейчас везде такое поведение, предлагаю изменить его в DS-3983 или вообще отдельной задачей

Давай не в рамках этой задачи, хорошо. Сами смотрите, в следующей или в новой, подалуйста

@lskramarov lskramarov merged commit c85119f into main Jul 23, 2025
4 checks passed
@lskramarov lskramarov deleted the feat/DS-3120 branch July 23, 2025 16:57
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