fix(Button): fix hover state change with loading

[EldarMuhamethanov]

• Release notes

Описание

Исправлен баг: после перехода кнопки из состояния загрузки (loading={true}) обратно в обычное состояние кнопка оставалась отображаться в состоянии hover, если курсор не покидал её во время загрузки.

Причина: в Clickable при hasHover={false} (когда у Button включён loading) обработчики onPointerEnter/onPointerLeave подменяются на noop, но внутреннее состояние hoveredStateLocal в хуке useHover не сбрасывалось. Курсор остаётся над кнопкой, pointerleave не срабатывает — при возврате hasHover={true} значение hoveredStateLocal по-прежнему true, и hover-стиль показывается некорректно.

Решение: в useHover (Clickable/useState.tsx) добавлен useEffect: при переходе hasHover в false сбрасываются hoveredStateLocal, prevIsHoveredRef и вызывается setParentStateLock(false) для согласованного состояния родителя.

Изменения

• Clickable/useState.tsx: в useHover добавлен эффект, сбрасывающий локальное hover-состояние при hasHover === false, чтобы после повторного включения hover (например, по окончании loading) стиль применялся только при реальном наведении курсора.

Release notes

Исправления

• Button: после завершения загрузки (loading: true → false) кнопка больше не остаётся в визуальном состоянии hover

[github-actions]

size-limit report 📦

Path	Size
JS	411.66 KB (+0.02% 🔺)
JS (gzip)	126.16 KB (+0.02% 🔺)
JS (brotli)	104.13 KB (-0.17% 🔽)
JS import Div (tree shaking)	811 B (0%)
CSS	373.51 KB (0%)
CSS (gzip)	46.29 KB (0%)
CSS (brotli)	36.6 KB (0%)

[github-actions]

e2e tests

Playwright Report

[github-actions]

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

Commit a877176

[github-actions]

👀 Docs deployed

• ✅ Website
• ✅ Playground

📦 Package ✅

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

Commit a877176

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.78%. Comparing base (869f68c) to head (a877176).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #9478   +/-   ##
=======================================
  Coverage   94.78%   94.78%           
=======================================
  Files         441      441           
  Lines       11928    11933    +5     
  Branches     4394     4395    +1     
=======================================
+ Hits        11306    11311    +5     
  Misses        622      622           

Flag	Coverage Δ
unittests	94.78% <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.