Skip to content

fix(useFloatingWithInteractions): ref for props#9216

Merged
SevereCloud merged 1 commit intomasterfrom
SevereCloud/fix/useFloatingWithInteractions/ref-for-props
Nov 26, 2025
Merged

fix(useFloatingWithInteractions): ref for props#9216
SevereCloud merged 1 commit intomasterfrom
SevereCloud/fix/useFloatingWithInteractions/ref-for-props

Conversation

@SevereCloud
Copy link
Copy Markdown
Contributor

@SevereCloud SevereCloud commented Nov 25, 2025


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

Описание

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

Release notes

Улучшения

  • Изменен внутренний код всплывающих элементов для поддержки React компилятора

Удалил рефы для свойств. Для чего нужны рефы не понял. При переключении типа триггера обработчики сохранялись, что гипотетически могло привести к багам
@SevereCloud SevereCloud requested a review from a team as a code owner November 25, 2025 15:44
@SevereCloud SevereCloud self-assigned this Nov 25, 2025
@SevereCloud SevereCloud added this to VKUI Nov 25, 2025
@github-actions github-actions bot added the ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча label Nov 25, 2025
@SevereCloud SevereCloud moved this to 👀 In Review in VKUI Nov 25, 2025
@github-actions
Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
JS 426.4 KB (-0.04% 🔽)
JS (gzip) 129.06 KB (-0.02% 🔽)
JS (brotli) 105.66 KB (+0.04% 🔺)
JS import Div (tree shaking) 1.67 KB (0%)
CSS 370.5 KB (0%)
CSS (gzip) 45.84 KB (0%)
CSS (brotli) 36.32 KB (0%)

@github-actions
Copy link
Copy Markdown
Contributor

e2e tests

Playwright Report

@github-actions
Copy link
Copy Markdown
Contributor

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

Commit dfcc2bc

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.03%. Comparing base (580dbe4) to head (dfcc2bc).
⚠️ Report is 92 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9216      +/-   ##
==========================================
- Coverage   95.03%   95.03%   -0.01%     
==========================================
  Files         420      420              
  Lines       11208    11206       -2     
  Branches     4221     4221              
==========================================
- Hits        10652    10650       -2     
  Misses        556      556              
Flag Coverage Δ
unittests 95.03% <100.00%> (-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.

@github-actions
Copy link
Copy Markdown
Contributor

👀 Docs deployed

📦 Package ✅

yarn add @vkontakte/vkui@https://development.s3.prodcloud.vk.team/pull/9216/dfcc2bcaee3f0d75567662e828c93313b1dc4084/pkg/@vkontakte/vkui/_pkg.tgz

Commit dfcc2bc

@SevereCloud SevereCloud removed the ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча label Nov 26, 2025
@SevereCloud SevereCloud added this to the v8.0.0 milestone Nov 26, 2025
@SevereCloud SevereCloud merged commit bd86a1b into master Nov 26, 2025
29 checks passed
@SevereCloud SevereCloud deleted the SevereCloud/fix/useFloatingWithInteractions/ref-for-props branch November 26, 2025 08:26
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in VKUI Nov 26, 2025
@inomdzhon inomdzhon added the v8 label Dec 19, 2025
@vkcom-publisher
Copy link
Copy Markdown
Contributor

v8.0.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants