Skip to content

Conversation

@inomdzhon
Copy link
Contributor


  • [ ] Unit-тесты
  • [ ] e2e-тесты
  • [ ] Дизайн-ревью
  • [ ] Документация фичи
  • Release notes

Описание

В v7.11.0 в Tooltip внесли 2 изменения:

  1. Свойство overflowPadding, чтобы можно было добавить отступы и избавиться от прилипания к краю области видимости.

  2. Пересчёт ширины, чтобы maxWidth не вызлезал за viewportWidthv7.11.2 был фикс округления).

Но п.2, к сожалению, создал другая проблему, что ширина стала статичной, поэтому для решения проблемы ничего не остается как откатить п.2.

Как воркэраунд, проблему с maxWidth > viewportWidth необхоидмо решать локально – задавать maxWidth с вычетом overflowPadding, учитывая, что эти данные должны быть заведомо известны. Также стоит учитывать какой минимальной ширины устройства у пользователей приложения.

К будущей мажорной версии v9, должна будет появится поддержка min() в CSS. Начиная с поддержки min(), будет возможность добавить новое свойство fitViewport, чтобы решить проблему с п.2 и в тоже время решить конфликт свойства с maxWidth.

Release notes

Исправления

  • Tooltip: откатили изменения из v7.11.0 и v7.11.2, связанные с пересчётом ширины

@inomdzhon inomdzhon added this to the 7.11.3 milestone Jan 29, 2026
@inomdzhon inomdzhon self-assigned this Jan 29, 2026
@inomdzhon inomdzhon requested a review from a team as a code owner January 29, 2026 12:09
@inomdzhon inomdzhon added this to VKUI Jan 29, 2026
@github-actions github-actions bot added the ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча label Jan 29, 2026
@inomdzhon inomdzhon added ci:cherry-pick:v7 and removed ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча labels Jan 29, 2026
@github-actions
Copy link
Contributor

size-limit report 📦

Path Size
JS 411.24 KB (-0.05% 🔽)
JS (gzip) 126.18 KB (-0.06% 🔽)
JS (brotli) 104.32 KB (+0.02% 🔺)
JS import Div (tree shaking) 811 B (0%)
CSS 373.51 KB (0%)
CSS (gzip) 46.28 KB (0%)
CSS (brotli) 36.57 KB (0%)

@github-actions
Copy link
Contributor

e2e tests

Playwright Report

@github-actions
Copy link
Contributor

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

Commit 973cb94

@github-actions
Copy link
Contributor

👀 Docs deployed

📦 Package ✅

yarn add @vkontakte/vkui@https://development.s3.prodcloud.vk.team/pull/9410/973cb94e792acb281d58f7326992d4a2355dab53/pkg/@vkontakte/vkui/_pkg.tgz

Commit 973cb94

@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.72%. Comparing base (058d50d) to head (973cb94).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9410      +/-   ##
==========================================
- Coverage   94.73%   94.72%   -0.01%     
==========================================
  Files         440      440              
  Lines       11900    11897       -3     
  Branches     4387     4387              
==========================================
- Hits        11273    11269       -4     
- Misses        627      628       +1     
Flag Coverage Δ
unittests 94.72% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@inomdzhon inomdzhon merged commit aac3afc into master Jan 29, 2026
28 of 29 checks passed
@inomdzhon inomdzhon deleted the imirdzhamolov/fix/tooltip-revert-sizeMiddleware branch January 29, 2026 15:28
@github-project-automation github-project-automation bot moved this to ✅ Done in VKUI Jan 29, 2026
vkcom-publisher pushed a commit that referenced this pull request Jan 29, 2026
<!-- Если этот PR закрывает Issue, то укажи ссылку на него. Используй доступные ключевые слова (см. https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests). -->
- caused by #9281
- related to #9389
inomdzhon added a commit that referenced this pull request Jan 30, 2026
<!-- Если этот PR закрывает Issue, то укажи ссылку на него. Используй доступные ключевые слова (см. https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests). -->
- caused by #9281
- related to #9389
@inomdzhon inomdzhon mentioned this pull request Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants