Simplified closeModal definition

The `closeModal` function had legacy parameters `action` and `namespace` which are no longer being used. The `action` hasn't been used in quite a while. The `namespace` was previously needed to determine the specific model window. This was replaced with the `[id^=""]` selector in a previous update, thus eliminating that need.

In addition, there's no need to call `modal('hide')` from _both_ the jQuery listener _and_ the `closeModal` function, so this is fixed. That was adding a bit of extra overhead which noticeably delayed the closing of the modal.

Author: JeremyCaney

OnTopic.Editor.AspNetCore/Shared/Scripts/NestedTopicList.js

@@ -72,10 +72,8 @@ initEditorModal = function (namespace, title, targetUrl, onCloseFunction) {
 \-----------------------------------------------------------------------------------------------------------------------------*/
 /**
  * Closes the current modal window
- * @param {string} action - Default parameter supplied by JavaScript event (unused).
- * @param {string} namespace - The parent key of the nested topic container.
  */
-window.closeModal = function (action, namespace) {
+window.closeModal = function () {
   $('[id^="EditorModal"]').modal('hide');
 };
 
@@ -88,8 +86,7 @@ window.closeModal = function (action, namespace) {
   | Event Handler: Close Button
   \---------------------------------------------------------------------------------------------------------------------------*/
   $('#ModalCloseButton').on('click', function (e) {
-    window.parent.closeModal('canceled', '');
-    $('[id^="EditorModal"]').modal('hide');
+    window.parent.closeModal();
   });
 
   /*----------------------------------------------------------------------------------------------------------------------------