Skip to content

Conversation

@tungulin
Copy link
Contributor

@tungulin tungulin commented Oct 21, 2025


Описание

Данный MR исправляет ширину placeholder учитывая контролы(кнопка очистки и кастомная иконка). Основывался на закрытом MR

Изменения

  • Доработал логику отображения контролов: сделал так, чтобы при скрытии они удалялись из dom дерева. Сделал это с помощью хука useCSSKeyframesAnimationController для отслеживания состояния анимаций
  • Добавил элипсис для плейсхолдера и отступ справа
  • Добавил скришоты кейса с длинным плейсхолдером

@tungulin tungulin marked this pull request as ready for review October 21, 2025 09:30
@tungulin tungulin requested a review from a team as a code owner October 21, 2025 09:30
@EldarMuhamethanov
Copy link
Contributor

@tungulin Привет! спасибо за правку. Можешь подмержить актуальный master? А то судя по diff сделано от очень старой ветки

@tungulin tungulin force-pushed the tungulin/8559/search-fix-placeholder branch from a400695 to 5b8282b Compare October 23, 2025 08:50
@tungulin
Copy link
Contributor Author

@EldarMuhamethanov. Привет! Ребейзнул MR

@SevereCloud
Copy link
Contributor

Привет, изображения загрузились как бинарные файлы, нужно включить у себя git lfs и перекомитить изображения

git lfs install

Или можно вообще удалить скриншоты мы их сами обновим

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

e2e tests

⚠️ Some screenshots were failed. See Playwright Report.

Playwright Report

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

📊 Найдены изменения в собранных файлах: Отчет

Commit e5c7a78

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

👀 Docs deployed

📦 Package ✅

yarn add @vkontakte/vkui@https://vkui-screenshot.hb.bizmrg.com/pull/9083/e5c7a7882d98f2e9e98e0288f8a7d98e76a4c84e/pkg/@vkontakte/vkui/_pkg.tgz

Commit e5c7a78

@tungulin tungulin force-pushed the tungulin/8559/search-fix-placeholder branch 2 times, most recently from 8f017cb to e5c7a78 Compare October 28, 2025 10:50
@tungulin
Copy link
Contributor Author

@SevereCloud. Привет! lfs штука платная - дропнул скриншоты) Еще раз ребейзнул

@vkcom-publisher vkcom-publisher added the pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности label Nov 6, 2025
@tungulin tungulin force-pushed the tungulin/8559/search-fix-placeholder branch from e5c7a78 to 3f87207 Compare November 7, 2025 15:06
@vkcom-publisher vkcom-publisher removed the pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности label Nov 8, 2025
@tungulin tungulin force-pushed the tungulin/8559/search-fix-placeholder branch from 3f87207 to e00c34e Compare November 16, 2025 12:27
@tungulin tungulin force-pushed the tungulin/8559/search-fix-placeholder branch from e00c34e to e302811 Compare November 16, 2025 12:30
@tungulin tungulin requested a review from SevereCloud November 16, 2025 17:23
@vkcom-publisher vkcom-publisher added the pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности label Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][Search]: placeholder оберазается скрытыми кнопками справа

4 participants