enhanced the "discard post" confirmation message to only trigger if

something in the post body actually changed
v1.18.x
Julian Lam 11 years ago
parent 64e35c734e
commit 1ee24517e3

@ -150,7 +150,8 @@ define(['taskbar'], function(taskbar) {
cid: threadData.cid, cid: threadData.cid,
pid: threadData.pid, pid: threadData.pid,
title: threadData.title || '', title: threadData.title || '',
body: threadData.body || '' body: threadData.body || '',
modified: false
}; };
composer.load(uuid); composer.load(uuid);
} else { } else {
@ -179,6 +180,9 @@ define(['taskbar'], function(taskbar) {
var uuid = $(this).parents('.post-window')[0].getAttribute('data-uuid'); var uuid = $(this).parents('.post-window')[0].getAttribute('data-uuid');
if (this.nodeName === 'INPUT') composer.posts[uuid].title = this.value; if (this.nodeName === 'INPUT') composer.posts[uuid].title = this.value;
else if (this.nodeName === 'TEXTAREA') composer.posts[uuid].body = this.value; else if (this.nodeName === 'TEXTAREA') composer.posts[uuid].body = this.value;
// Mark this post window as having been changed
composer.posts[uuid].modified = true;
}); });
jPostContainer.on('click', '.action-bar button', function() { jPostContainer.on('click', '.action-bar button', function() {
@ -188,7 +192,7 @@ define(['taskbar'], function(taskbar) {
case 'post': composer.post(uuid); break; case 'post': composer.post(uuid); break;
case 'minimize': composer.minimize(uuid); break; case 'minimize': composer.minimize(uuid); break;
case 'discard': case 'discard':
if (postContentEl.value.length > 0) { if (composer.posts[uuid].modified) {
bootbox.confirm('Are you sure you wish to discard this post?', function(discard) { bootbox.confirm('Are you sure you wish to discard this post?', function(discard) {
if (discard) composer.discard(uuid); if (discard) composer.discard(uuid);
}); });

Loading…
Cancel
Save