-
Notifications
You must be signed in to change notification settings - Fork 185
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(PanelHeaderButton): refactor pressets #7874
feat(PanelHeaderButton): refactor pressets #7874
Conversation
size-limit report 📦
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
e2e tests |
👀 Docs deployed
Commit 907e50d |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7874 +/- ##
==========================================
+ Coverage 95.16% 95.30% +0.13%
==========================================
Files 376 375 -1
Lines 11008 11002 -6
Branches 3653 3681 +28
==========================================
+ Hits 10476 10485 +9
+ Misses 532 517 -15
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏 💅
packages/vkui/src/components/PanelHeaderBack/PanelHeaderBack.tsx
Outdated
Show resolved
Hide resolved
packages/codemods/src/transforms/v7/__tests__/__snapshots__/panel-header-back.ts.snap
Outdated
Show resolved
Hide resolved
…and hideLabelOnIOS
Кажется, платформа vkcom не умрёт 🌝 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Есть микроконсёрн, что у нас в дизайне при отсутствии текста ширина кнопки немного меньше: 30×44 на iOS, 44×44 на Android.
Было бы классно засинкать, чтобы не приходилось потом объяснять, но в целом не критикал именно для этой задачи.
В остальном feels good.
Co-authored-by: Inomdzhon Mirdzhamolov <[email protected]>
907e50d
✅ v7.0.0-beta.2 🎉 |
✅ v7.0.0 🎉 |
Описание
В настоящее время пресеты компонента PanelHeaderButton демонстрируют несогласованное поведение при использовании пропсов label и children на разных платформах.
Изменения
Унифицировано поведение пресетов для обеспечения предсказуемого отображения на всех платформах.
Основные изменения
Поведение label:
Удаление children:
Особый случай: PanelHeaderBack
Документация
Дополнительные изменения
UPD
Пришлось оставить исключение PanelHeaderButton, так как у него есть особенная логика - label иногда должен отображаться не только на IOS, но и на VKCOM. Также было бы неплохо, чтобы можно было управлять тем, нужно ли скрывать label на разных платформах, поэтому добавил свойства hideLabelOnVKCom и hideLabelOnIOS для более точно найстройки в зависимости от кейса.
Release notes
BREAKING CHANGE
PanelHeaderClose
удалено свойствоchildren
. Теперь для прокидывания текста дляa11y
нужно прокидывать его в свойствоlabel
PanelHeaderSubmit
удалено свойствоchildren
. Теперь для прокидывания текста дляa11y
нужно прокидывать его в свойствоlabel
PanelHeaderEdit
удалены свойстваchildren
иlabel
. Вместоlabel
можно использовать свойстваdoneLabel
иeditLabel
. Свойствоchildren
не использовалось.PanelHeaderBack
удалено свойствоchildren
. Теперь для прокидывания текста дляa11y
нужно прокидывать его в свойствоlabel
. Логика отображенияlabel
осталась как была, в отличие от других пресетов. Также для более точно настройкиlabel
были добавлены свойстваhideLabelOnVKCom
иhideLabelOnIOS
, чтобы можно было скрыватьlabel
на соответствующей платформе.