Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Modal inside modal. #429

Closed
damianprzygodzki opened this issue Feb 24, 2017 · 3 comments
Closed

Modal inside modal. #429

damianprzygodzki opened this issue Feb 24, 2017 · 3 comments

Comments

@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented Feb 24, 2017

Type of issue

Feature request

Current behavior

Now when we want to open modal (for example after process button click) inside another modal (i.e. Advanced Edit) the content from process is replacing content of advanced edit.

Expected behavior

After closing one modal (process) we need to go back to another (advanced edit)

.

@cadavre should we just cover one with another?

@cadavre
Copy link
Contributor

cadavre commented Mar 30, 2017

@damianprzygodzki isn't it already implemented?

@damianprzygodzki
Copy link
Contributor Author

damianprzygodzki commented Mar 30, 2017

Never was, for crying out loud.

I'm gonna rewrite thing one more time.

We have connected and not connected modal. Two different components. We can display them, over themselves, but when we want to display two modals of the same type, like connected modal over connected modal (advanced edit over add new modal i.e. [it is random example, but i am sure and we experienced that in some cases there is a need to display it]) it causes replacing old modal content with new, without ability to return.

It was never discussed to provide modal inside modal (lots of workarounds), to provide history (sic) inside modal, to navigate between different windows inside.

@cadavre
Copy link
Contributor

cadavre commented Apr 19, 2017

So far all cases while we needed "modal over modal" are covered with "connected" and "not connected" modals. Closing as resolved.

@cadavre cadavre closed this as completed Apr 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants