Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Mutliple modals, double click closes two modals instead top. #6061

Closed
sponkrashin opened this issue Jun 29, 2016 · 6 comments
Closed

Mutliple modals, double click closes two modals instead top. #6061

sponkrashin opened this issue Jun 29, 2016 · 6 comments

Comments

@sponkrashin
Copy link

sponkrashin commented Jun 29, 2016

When there are two nested modals, i.e. one modal is shown from another, double click on backdrop of the top modal results in closing of both modals instead only top.

Problem is reproduced even if double click is done when cursor is above first modal. As in the attached image from plunker, need to click in selected area.

image

http://plnkr.co/edit/Fidgh0aBW6LMi9nUaetg

@icfantv
Copy link
Contributor

icfantv commented Jul 2, 2016

@Advarius this is expected behavior because you have the modals configured to close when the backdrop is clicked. You can turn this off via the backdrop property.

@sponkrashin
Copy link
Author

sponkrashin commented Jul 3, 2016

This is not expected behavior, because as I mentioned, first modal will be closed even if double click is above selected area on the screenshot. In other words, first click is on the backdrop of the top modal, but second click is on the content of the first modal. Why modal is closed when I click it's content?

P.S.:
image
New screenshot with better example. If you click in the selected area, which is above the content of first modal, first modal will also be closed.

@icfantv
Copy link
Contributor

icfantv commented Jul 3, 2016

@advarius, ah, thank you for the better example. I'll flag for investigation.

@icfantv icfantv reopened this Jul 3, 2016
@daimoonis
Copy link

Hello, i'm facing the same issue.
Both modals are simply taken out of its stack and it happens twice as there happen two click actions on backdrop.

@jkoontz2010
Copy link

I didn't realize this, but this issue was corrected by #6104

@wesleycho
Copy link
Contributor

wesleycho commented Jul 24, 2016

Closing as fixed in master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants