[Edit] Add clarifying comments

Add clarifying comments related to checking of conditions
on unload, WTD-1035.
This commit is contained in:
Victor Woeltjen 2015-03-31 14:39:30 -07:00
parent 8a8c84eacf
commit 5ef382d159
2 changed files with 16 additions and 4 deletions

View File

@ -31,9 +31,19 @@ define(
});
return {
/**
* Get the domain object which is navigated-to.
* @returns {DomainObject} the domain object that is navigated-to
*/
navigatedObject: function () {
return navigatedObject;
},
/**
* Get the warning to show if the user attempts to navigate
* away from Edit mode while unsaved changes are present.
* @returns {string} the warning to show, or undefined if
* there are no unsaved changes
*/
getUnloadWarning: function () {
var editorCapability = navigatedObject &&
navigatedObject.getCapability("editor"),

View File

@ -43,12 +43,14 @@ define(
}
}
// Include this instance of the directive's unload function
unloads.push(unload);
// If this is the first active instance of this directive,
// register as the window's beforeunload handler
$window.onbeforeunload = checkUnloads;
if (unloads.length === 0) {
$window.onbeforeunload = checkUnloads;
}
// Include this instance of the directive's unload function
unloads.push(unload);
// Remove it when the scope is destroyed
scope.$on("$destroy", removeUnload);