Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

Replace current operation confirm prompts with SweetAlert #142

Open
alariva opened this issue Nov 25, 2016 · 7 comments
Open

Replace current operation confirm prompts with SweetAlert #142

alariva opened this issue Nov 25, 2016 · 7 comments

Comments

@alariva
Copy link
Member

alariva commented Nov 25, 2016

Timegrid is currently using js prompt for getting user confirmation upon some deletion operations.

It would be nice-to-have, these confirmations with SweetAlert component.

Note that the JS code that handles confirmations might also require some refactor, and it is currently not implemented in a nicely reusable fashion.

@sahilsharma011
Copy link
Contributor

Where exactly in the code is this enhancement required? Please explain.

@alariva
Copy link
Member Author

alariva commented Dec 19, 2016

Sure, I've updated the description.

@sahilsharma011
Copy link
Contributor

Thank you for that. I'll try to work on this 👍

@alariva
Copy link
Member Author

alariva commented Dec 19, 2016

I don't see this being big change, but neither it's a small one. Please let me know whatever I may help with! :)

@alariva alariva changed the title Replace current confirm alerts with SweetAlert Replace current operation confirm prompts with SweetAlert Dec 19, 2016
@softnayr
Copy link

Any development on this?

@alariva
Copy link
Member Author

alariva commented Jun 29, 2017

@softnayr All quiet here, you may take it if you'd like to.

@softnayr
Copy link

Ok. I'll take a look. This is a nice ux addition.

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

3 participants