-
Notifications
You must be signed in to change notification settings - Fork 20
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(plasma-new-hope/web/b2c): overlay component #1033
Conversation
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1033/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-1033/ |
⚡ Component performance testingResult: 🟢 OK |
допиши пожалуйста в описании к PR (до What/why changed), в каких компонентах в итоге добавил компонент Overlay |
93e82ec
to
e8617d8
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1033/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-1033/ |
⚡ Component performance testingResult: 🟢 OK |
@@ -18,6 +18,7 @@ export * from './components/List'; | |||
export * from './components/Modal'; | |||
export * from './components/ModalBase'; | |||
export * from './components/Notification'; | |||
export * from './components/Overlay'; |
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.
а мы точно хотим его в наружу отдавать? По идее это же техническая штука, которую мы в своих компонентах используем 🤔
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.
В MUI поставляется как отдельный компонент https://mui.com/material-ui/react-backdrop/
Мне кажется, что может пригодиться как отдельный компонент
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.
я тоже думал, что это как отдельный компонент, который в поставку входить будет
|
||
import { Overlay } from '.'; | ||
|
||
const onOverlayClick = action('onOverlayClick'); |
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.
колбэк отрабатывает, но при включенном свойстве closeOnOverlayClick
не закрывает сам оверлей
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.
возможно просто нужно пример поправить
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.
Логически неправильно назвал проп. Должно быть isClickable
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.
а получилось в итоге поправить? У меня пока всё ещё не закрывается
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.
Проп переименуется в isClickable, чтобы не было путаницы
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.
а этот колбэк не хотим оставить? Вроде же полезная штука может быть
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.
Колбэк остается, просто логика в нем прописывается пользователем
e8617d8
to
dfb9116
Compare
⚡ Component performance testingResult: 🟢 OK |
packages/plasma-new-hope/src/components/Overlay/Overlay.types.ts
Outdated
Show resolved
Hide resolved
eaf553c
to
f7dcdd9
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1033/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-1033/ |
⚡ Component performance testingResult: 🟢 OK |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
3 similar comments
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
Overlay
plasma-{b2c/web}
Toast
,Modal
What/why changed
Добавлен компонент Overlay.
Будет использоваться в всплывающих компонентах.
📦 Published PR as canary version:
Canary Versions
✨ Test out this PR locally via: