-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added confirmation popups for deletes #32
Conversation
This looks good - I'm going to change some of the CSS though, as it doesn't really match the look and feel of the current design. |
Merged this into the develop branch - thanks for the contribution! Once I tweak the styles, I'll merge it into master. I also fixed a small bug in the default blade, so once I merge this in, you should probably grab a fresh copy. In the future, please submit pull requests to the develop branch to make my life a little easier. Thanks! |
Oh..Sorry, totally forgot about the develop branch for the fix branch and pull request. My bad ... |
No worries, I appreciate the help :) |
I actually ended up using a method that uses bootstrap's native modal functionality. It made it easier to inherit the styles, etc and not have to manage styles in separate stylesheets, etc. You can check it out at 34bbf46. Merged everything into master as well. |
Added confirmation popup for the delete functionality.
Used alertify for the confirmation popups.
Addresses #30