Skip to content

Commit

Permalink
feat: added alert.timeoutfn
Browse files Browse the repository at this point in the history
Also, various alert methods now pass two arguments,
alert and params
  • Loading branch information
julianlam committed Aug 26, 2020
1 parent bacee6b commit cd8e796
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions public/src/modules/alerts.js
Expand Up @@ -39,15 +39,15 @@ define('alerts', ['translator', 'components', 'benchpress'], function (translato
}

if (params.timeout) {
startTimeout(alert, params.timeout);
startTimeout(alert, params);
}

if (typeof params.clickfn === 'function') {
alert
.addClass('pointer')
.on('click', function (e) {
if (!$(e.target).is('.close')) {
params.clickfn();
params.clickfn(alert, params);
}
fadeOut(alert);
});
Expand All @@ -69,7 +69,7 @@ define('alerts', ['translator', 'components', 'benchpress'], function (translato

clearTimeout(parseInt(alert.attr('timeoutId'), 10));
if (params.timeout) {
startTimeout(alert, params.timeout);
startTimeout(alert, params);
}

alert.children().fadeOut(100);
Expand Down Expand Up @@ -99,9 +99,15 @@ define('alerts', ['translator', 'components', 'benchpress'], function (translato
});
}

function startTimeout(alert, timeout) {
function startTimeout(alert, params) {
var timeout = params.timeout;

var timeoutId = setTimeout(function () {
fadeOut(alert);

if (typeof params.timeoutfn === 'function') {
params.timeoutfn(alert, params);
}
}, timeout);

alert.attr('timeoutId', timeoutId);
Expand Down

0 comments on commit cd8e796

Please sign in to comment.