Skip to content

feat(ActionSheet): rename onClose to onClosed and add onClose#9285

Merged
EldarMuhamethanov merged 9 commits intomasterfrom
e.muhamethanov/action-sheet-rename-onClose-to-onClosed
Dec 23, 2025
Merged

feat(ActionSheet): rename onClose to onClosed and add onClose#9285
EldarMuhamethanov merged 9 commits intomasterfrom
e.muhamethanov/action-sheet-rename-onClose-to-onClosed

Conversation

@EldarMuhamethanov
Copy link
Contributor

@EldarMuhamethanov EldarMuhamethanov commented Dec 11, 2025

  • Unit-тесты
  • Release notes

Описание

Переименовал обработчик onClose на onClosed и добавил onClose, который срабатывает перед анимацией закрытия

Release notes

Улучшения

  • ActionSheet:
    • добавлен обработчик закрытия onClose, который срабатывает перед началом анимации закрытия.
    • экспортирован тип ActionSheetOnCloseReason - причина закрытия, передаваемая в onClose.

BREAKING CHANGE

  • ActionSheet:
    • обработчик onClose переименован на onClosed.
    • параметр передаваемый в onClosed помечен как устаревший и будет удален в v10. Для получения причины закрытия лучше использовать reason, передаваемый в onClose.
    • экспортируемый тип ActionSheetOnCloseOptions устарел и будет удален в v10.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

e2e tests

⚠️ Some screenshots were failed. See Playwright Report.

Playwright Report

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

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

Commit 69b5dab

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

👀 Docs deployed

📦 Package ✅

yarn add @vkontakte/vkui@https://development.s3.prodcloud.vk.team/pull/9285/69b5dab96c5f32a0b11429f6d3095ea4b414cea4/pkg/@vkontakte/vkui/_pkg.tgz

Commit 69b5dab

@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.91%. Comparing base (67efe75) to head (69b5dab).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9285   +/-   ##
=======================================
  Coverage   94.91%   94.91%           
=======================================
  Files         433      433           
  Lines       11769    11775    +6     
  Branches     4342     4343    +1     
=======================================
+ Hits        11170    11176    +6     
  Misses        599      599           
Flag Coverage Δ
unittests 94.91% <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.

@inomdzhon inomdzhon added the v8 label Dec 19, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 22, 2025

size-limit report 📦

Path Size
JS 408.49 KB (+0.03% 🔺)
JS (gzip) 125.13 KB (+0.01% 🔺)
JS (brotli) 103.67 KB (+0.1% 🔺)
JS import Div (tree shaking) 811 B (0%)
CSS 372.71 KB (0%)
CSS (gzip) 46.21 KB (0%)
CSS (brotli) 36.59 KB (0%)

Copy link
Contributor

@inomdzhon inomdzhon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@EldarMuhamethanov EldarMuhamethanov merged commit 50455c2 into master Dec 23, 2025
29 checks passed
@EldarMuhamethanov EldarMuhamethanov deleted the e.muhamethanov/action-sheet-rename-onClose-to-onClosed branch December 23, 2025 15:37
@github-project-automation github-project-automation bot moved this to ✅ Done in VKUI Dec 23, 2025
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.

2 participants