VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard
verified publisher icon VKCOM

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

Open stoope opened this issue 3 years ago • 7 comments

Текущая реализация модальных окон требует много времени на поддержку со стороны разработчиков и содержит много багов. Необходимо подумать как можно упростить модальные окна

UPD Необходимо задепрекейтить ModalRoot и создать вместо него ModalManager.

stoope avatar Apr 28 '22 15:04 stoope

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

k-egor-smirnov avatar Apr 28 '22 15:04 k-egor-smirnov

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

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

stoope avatar Apr 28 '22 15:04 stoope

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

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

k-egor-smirnov avatar Apr 28 '22 15:04 k-egor-smirnov

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

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

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

stoope avatar Apr 28 '22 15:04 stoope

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

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

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

inomdzhon avatar May 05 '22 07:05 inomdzhon

Напомню что я уже делал подход к этому вопросику: https://github.com/VKCOM/VKUI/pull/2182

thoughtspile avatar Dec 29 '22 09:12 thoughtspile