swcs/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Pop up prompt when the user tries to leave/refresh the page while editing in infinite mode.

Browse Source
This commit is contained in:
Robert
2018-04-24 15:09:49 +02:00
parent d0f9021bcf
commit f2dd63c280
2 changed files with 24 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Umbraco.Web.UI.Client/src/common/services/editor.service.js
+2 -1
View File
@@ -80,7 +80,8 @@
var service = {
open: open,
close: close
close: close,
getEditors: getEditors
};
return service;
src/Umbraco.Web.UI.Client/src/controllers/main.controller.js
+22 -4
View File
@@ -8,7 +8,9 @@
* The main application controller
*
*/
function MainController($scope, $rootScope, $location, $routeParams, $timeout, $http, $log, appState, treeService, notificationsService, userService, navigationService, historyService, updateChecker, assetsService, eventsService, umbRequestHelper, tmhDynamicLocale, localStorageService, tourService, editorService) {
function MainController($scope, $rootScope, $location, $routeParams, $timeout, $http, $log, $window,
appState, treeService, notificationsService, userService, navigationService, historyService,
updateChecker, assetsService, eventsService, umbRequestHelper, tmhDynamicLocale, localStorageService, tourService, editorService) {
//the null is important because we do an explicit bool check on this in the view
$scope.authenticated = null;
@@ -20,6 +22,22 @@ function MainController($scope, $rootScope, $location, $routeParams, $timeout, $
notificationsService.remove(index);
};
//listening for unload event to warn users of unsaved changes in case they refresh/leave the page while
//editing in infinite mode http://issues.umbraco.org/issue/U4-11225
$window.onbeforeunload = function (event) {
var editorsOpen = editorService.getEditors();
console.log(editorsOpen.length, editorsOpen);
//most browser stopped supporting custom messages for the pop up window, but I guess is a good
//idea to return one just in case some browser still support it
var message = "You have " +
editorsOpen.length +
" open which might have unsaved changes, are you sure you want to leave/refresh the page?";
//returning null or undefine will skip the pop up and procede to refresh/leave the page.
return editorsOpen.length > 0 ? message : null;
};
$scope.closeDialogs = function (event) {
//only close dialogs if non-link and non-buttons are clicked
var el = event.target.nodeName;
@@ -49,8 +67,8 @@ function MainController($scope, $rootScope, $location, $routeParams, $timeout, $
$scope.user = null;
}));
evts.push(eventsService.on("app.userRefresh", function(evt) {
userService.refreshCurrentUser().then(function(data) {
evts.push(eventsService.on("app.userRefresh", function (evt) {
userService.refreshCurrentUser().then(function (data) {
$scope.user = data;
//Load locale file
@@ -134,7 +152,7 @@ function MainController($scope, $rootScope, $location, $routeParams, $timeout, $
evts.push(eventsService.on("appState.overlay", function (name, args) {
$scope.overlay = args;
}));
// events for tours
evts.push(eventsService.on("appState.tour.start", function (name, args) {
$scope.tour = args;