Skip to content
This repository has been archived by the owner on Jan 17, 2019. It is now read-only.

Add nested option #74

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Add nested option #74

wants to merge 6 commits into from

Conversation

Deuteu
Copy link
Contributor

@Deuteu Deuteu commented Oct 3, 2016

To allow nesting of confirmation modal as requested in #54

@mnapoli
Copy link
Member

mnapoli commented Oct 6, 2016

Great! But is the option necessary? If it works well then why not allow nested confirms? Do you think it could cause issues in terms of user experience sometimes?

@Deuteu
Copy link
Contributor Author

Deuteu commented Oct 6, 2016

Oooh, like change default behaviour and always nested if there is all ready a .confirmation-modal opened ?
I didn't think about that (I was driven by the other options I've added so I followed this path) but can be a good thing.
Will see that. (Might not be soon but will let you know asap.)

Conflicts:
	jquery.confirm.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants