Skip to content
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

0.3 ability to close individual/all notifications from code #267

Open
wants to merge 2 commits into
base: 0.3
Choose a base branch
from

Conversation

Kambaa
Copy link

@Kambaa Kambaa commented Feb 27, 2017

This pull request is improved from the pull request #239

Needed this functionality on my ajax requests. Open a notification, save that notification on a variable, do stuff and close it.

What i did was, changed the log,error and success notification methods' return value from alertify object to a 0-100 math random number. Random value is used as part of the class name of the notification to be created, and with the closeNow() method, i used it to close the dialog i wanted to close. And while i found some time to kill, i added a closeAll() method that closes all open notifications with the same idea.

After that i edited the example page to test, document/explain what i did. Used a minifier to minify the edited js file.

Quick usage:

tempNotification = alertify.log("A persistent nofitication (This one will be closed)", "", 0);
 ... // do stuff
alertify.closeNow(tempNotification); // closes spesific notification

alertify.closeAll(); // closes all of the open notifications.

Kambaa added a commit to Kambaa/alertify-refinements that referenced this pull request Oct 12, 2017
updated alertify.js with custom notification close capability. See fabien-d/alertify.js#267
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants