remove unused call to hide loading indicator
This commit is contained in:
parent
e833225ab3
commit
b41ae5f483
2 changed files with 12 additions and 9 deletions
|
@ -34,8 +34,8 @@ define(["loading", "dialogHelper", "dom", "jQuery", "components/libraryoptionsed
|
||||||
dialogHelper.close(dlg);
|
dialogHelper.close(dlg);
|
||||||
}, function() {
|
}, function() {
|
||||||
require(["toast"], function(toast) {
|
require(["toast"], function(toast) {
|
||||||
toast(Globalize.translate("ErrorAddingMediaPathToVirtualFolder"))
|
toast(Globalize.translate("ErrorAddingMediaPathToVirtualFolder"));
|
||||||
})
|
});
|
||||||
isCreating = false;
|
isCreating = false;
|
||||||
loading.hide();
|
loading.hide();
|
||||||
});
|
});
|
||||||
|
@ -70,7 +70,12 @@ define(["loading", "dialogHelper", "dom", "jQuery", "components/libraryoptionsed
|
||||||
$(".collectionTypeFieldDescription", dlg).html(folderOption.message || "")
|
$(".collectionTypeFieldDescription", dlg).html(folderOption.message || "")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}), page.querySelector(".btnAddFolder").addEventListener("click", onAddButtonClick), page.querySelector("form").addEventListener("submit", onSubmit), page.querySelector(".folderList").addEventListener("click", onRemoveClick), page.querySelector(".chkAdvanced").addEventListener("change", onToggleAdvancedChange)
|
});
|
||||||
|
|
||||||
|
page.querySelector(".btnAddFolder").addEventListener("click", onAddButtonClick);
|
||||||
|
page.querySelector("form").addEventListener("submit", onSubmit);
|
||||||
|
page.querySelector(".folderList").addEventListener("click", onRemoveClick);
|
||||||
|
page.querySelector(".chkAdvanced").addEventListener("change", onToggleAdvancedChange);
|
||||||
}
|
}
|
||||||
|
|
||||||
function onToggleAdvancedChange() {
|
function onToggleAdvancedChange() {
|
||||||
|
@ -131,9 +136,6 @@ define(["loading", "dialogHelper", "dom", "jQuery", "components/libraryoptionsed
|
||||||
}
|
}
|
||||||
|
|
||||||
function onDialogClosed() {
|
function onDialogClosed() {
|
||||||
// I can't see any corresponding call to loading.show,
|
|
||||||
// so I think this is not supposed to be here.
|
|
||||||
loading.hide();
|
|
||||||
currentResolve(hasChanges);
|
currentResolve(hasChanges);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -176,7 +178,7 @@ define(["loading", "dialogHelper", "dom", "jQuery", "components/libraryoptionsed
|
||||||
initLibraryOptions(dlg);
|
initLibraryOptions(dlg);
|
||||||
};
|
};
|
||||||
xhr.send();
|
xhr.send();
|
||||||
})
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -184,7 +186,7 @@ define(["loading", "dialogHelper", "dom", "jQuery", "components/libraryoptionsed
|
||||||
var currentResolve;
|
var currentResolve;
|
||||||
var currentOptions;
|
var currentOptions;
|
||||||
|
|
||||||
var hasChanges;
|
var hasChanges = false;
|
||||||
var isCreating = false;
|
var isCreating = false;
|
||||||
|
|
||||||
return editor
|
return editor
|
||||||
|
|
|
@ -193,8 +193,9 @@ define(["loading", "dialogHelper", "dom", "components/libraryoptionseditor/libra
|
||||||
}
|
}
|
||||||
|
|
||||||
var currentDeferred;
|
var currentDeferred;
|
||||||
var hasChanges;
|
|
||||||
var currentOptions;
|
var currentOptions;
|
||||||
|
|
||||||
|
var hasChanges = false;
|
||||||
|
|
||||||
return editor;
|
return editor;
|
||||||
});
|
});
|
Loading…
Add table
Add a link
Reference in a new issue