search

bluerail / twitter-bootstrap-rails-confirm

Confirm dialogs using Twitter Bootstrap
https://bluerail.nl
MIT License
85 stars 34 forks source link

Prevent dual modal #23

Closed xibman closed 2 years ago

xibman commented 10 years ago

If you have a confirm button on a modal, you open a new modal for the confirm action, it's not very cute and rpsec don't love it !

martijn commented 10 years ago

I'm hesitant to accept this patch since requesting confirmation inside a confirmation a bad idea in the first place. The current behavior is as expected.

xibman commented 10 years ago

To show you the problem image

The red button used to delete and pop-up an alert.

martijn commented 10 years ago

Right, I misunderstood. Seems like a valid point. If someone can confirm the change as not breaking anything we can merge it. Thanks for your input!

xibman commented 10 years ago

I can do something more cleaner with a popover on the button ! Like this : http://ethaizone.github.io/Bootstrap-Confirmation/