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.

Sign up for GitHub

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

Jump to bottom

[Feature] Переписать модальные окна #2449

Open
stoope opened this issue Apr 28, 2022 · 6 comments · May be fixed by #5092
Open

[Feature] Переписать модальные окна #2449

stoope opened this issue Apr 28, 2022 · 6 comments · May be fixed by #5092
Assignees
@SevereCloud
Labels
cmp:modal type:feature

Comments

@stoope
Copy link
Contributor

stoope commented Apr 28, 2022

Текущая реализация модальных окон требует много времени на поддержку со стороны разработчиков и содержит много багов. Необходимо подумать как можно упростить модальные окна
@stoope stoope self-assigned this Apr 28, 2022
@k-egor-smirnov
Copy link
Contributor

Хороший пример в AntD есть: https://ant.design/components/modal/. Модалка вызывается из любого места. Конечно, тут нужно придумать, как предотвратить открытие сразу нескольких модалок одновременно, но это точно решаемая проблема

@stoope
Copy link
Contributor Author

stoope commented Apr 28, 2022

Хороший пример в AntD есть: https://ant.design/components/modal/. Модалка вызывается из любого места. Конечно, тут нужно придумать, как предотвратить открытие сразу нескольких модалок одновременно, но это точно решаемая проблема

Ага, что то подобное планирую. А открытие нескольких модалок одновременно стоит контролировать со стороны VKUI или дать возможность разработчику это контролировать? У нас есть какие то кейсы когда это нужно?

@k-egor-smirnov
Copy link
Contributor

Текущая дизайн система не предусматривает открытие сразу нескольких модалок. Так что, как мне кажется, тут есть два варианта:

  1. При открытии второй модалки скрывать предыдущую в стэк и возвращать при закрытии текущей.
  2. Кидать warn в консоль и ничего не делать, пока не закроется предыдущая

@stoope
Copy link
Contributor Author

stoope commented Apr 28, 2022

Текущая дизайн система не предусматривает открытие сразу нескольких модалок. Так что, как мне кажется, тут есть два варианта:

  1. При открытии второй модалки скрывать предыдущую в стэк и возвращать при закрытии текущей.
  2. Кидать warn в консоль и ничего не делать, пока не закроется предыдущая

Я за первый вариант

@inomdzhon
Copy link
Contributor

Текущая дизайн система не предусматривает открытие сразу нескольких модалок. Так что, как мне кажется, тут есть два варианта:

  1. При открытии второй модалки скрывать предыдущую в стэк и возвращать при закрытии текущей.
  2. Кидать warn в консоль и ничего не делать, пока не закроется предыдущая

Возможно стоит с дизайном обсудить этот момент

@inomdzhon inomdzhon added type:feature cmp:modal design Нужно участие команды дизайна labels May 5, 2022
@stoope stoope removed their assignment Sep 7, 2022
@SevereCloud SevereCloud mentioned this issue Nov 23, 2022
Open
@thoughtspile
Copy link
Contributor

Напомню что я уже делал подход к этому вопросику: #2182

@SevereCloud SevereCloud linked a pull request May 18, 2023 that will close this issue
Open
@SevereCloud SevereCloud added this to the v6 milestone May 18, 2023
@SevereCloud SevereCloud removed the design Нужно участие команды дизайна label May 18, 2023
@SevereCloud SevereCloud mentioned this issue Jun 20, 2023
Open
@SevereCloud SevereCloud self-assigned this Jun 26, 2023
@inomdzhon inomdzhon modified the milestones: v6, v6.1.0 Jan 16, 2024
@SevereCloud SevereCloud removed this from the v6.1.0 milestone Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SevereCloud SevereCloud

Labels
cmp:modal type:feature
Projects
VKUI
Status: ⛔️ Blocked
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add ModalSheet VKCOM/VKUI
5 participants
@thoughtspile @inomdzhon @SevereCloud @stoope @k-egor-smirnov