Skip to content
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

Merged
merged 5 commits into from
Feb 12, 2024

Conversation

TitanKuzmich
Copy link
Contributor

@TitanKuzmich TitanKuzmich commented Feb 6, 2024

Overlay

  • добавлен в новой архитектуре
  • добавлен в поставку для plasma-{b2c/web}
  • написаны тесты и документация
  • интегрирован в компоненты Toast, Modal

What/why changed

Добавлен компонент Overlay.

Будет использоваться в всплывающих компонентах.

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
npm install @salutejs/[email protected]
# or 
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]
yarn add @salutejs/[email protected]

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1033/

Copy link
Contributor

github-actions bot commented Feb 6, 2024

⚡ Component performance testing

Result: 🟢 OK

@neretin-trike
Copy link
Collaborator

допиши пожалуйста в описании к PR (до What/why changed), в каких компонентах в итоге добавил компонент Overlay

@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1033/

Copy link
Contributor

github-actions bot commented Feb 8, 2024

⚡ Component performance testing

Result: 🟢 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';
Copy link
Collaborator

Choose a reason for hiding this comment

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

а мы точно хотим его в наружу отдавать? По идее это же техническая штука, которую мы в своих компонентах используем 🤔

Copy link
Contributor Author

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/
Мне кажется, что может пригодиться как отдельный компонент

Copy link
Contributor

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');
Copy link
Collaborator

Choose a reason for hiding this comment

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

колбэк отрабатывает, но при включенном свойстве closeOnOverlayClick не закрывает сам оверлей

Copy link
Collaborator

Choose a reason for hiding this comment

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

возможно просто нужно пример поправить

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Логически неправильно назвал проп. Должно быть isClickable

Copy link
Collaborator

Choose a reason for hiding this comment

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

а получилось в итоге поправить? У меня пока всё ещё не закрывается

Copy link
Contributor Author

@TitanKuzmich TitanKuzmich Feb 9, 2024

Choose a reason for hiding this comment

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

Проп переименуется в isClickable, чтобы не было путаницы

Copy link
Collaborator

Choose a reason for hiding this comment

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

а этот колбэк не хотим оставить? Вроде же полезная штука может быть

Copy link
Contributor Author

@TitanKuzmich TitanKuzmich Feb 9, 2024

Choose a reason for hiding this comment

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

Колбэк остается, просто логика в нем прописывается пользователем

Copy link
Contributor

github-actions bot commented Feb 9, 2024

⚡ Component performance testing

Result: 🟢 OK

@TitanKuzmich TitanKuzmich requested a review from Yakutoc February 9, 2024 10:06
@Salute-Eva
Copy link
Contributor

Theme Builder app deployed!

http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1033/

Copy link
Contributor

⚡ Component performance testing

Result: 🟢 OK

@TitanKuzmich TitanKuzmich added this pull request to the merge queue Feb 12, 2024
Merged via the queue into dev with commit cf4b84b Feb 12, 2024
25 checks passed
@TitanKuzmich TitanKuzmich deleted the plasma-1764/overlay-component branch February 12, 2024 09:48
@Salute-Eva
Copy link
Contributor

🚀 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] 🚀

@Salute-Eva
Copy link
Contributor

🚀 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
@Salute-Eva
Copy link
Contributor

🚀 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] 🚀

@Salute-Eva
Copy link
Contributor

🚀 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] 🚀

@Salute-Eva
Copy link
Contributor

🚀 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 was referenced Jul 24, 2024
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.

4 participants