Changeset 38326 for main/trunk/greenstone3/web

Timestamp:

2023-10-20T17:01:06+13:00 (8 months ago)

Author:

anupama

Message:

1. Confirmation dialog when deleting selected user comments. 2. The Delete Selected button should be unavailable if no user comments are checked.

File:

• main/trunk/greenstone3/web/interfaces/default/js/user_comments.js (modified) (5 diffs)

main/trunk/greenstone3/web/interfaces/default/js/user_comments.js

@@ -171 +171 @@
     // Arduous way: deletes each metadata individually
     //delCommentsButton.addEventListener("click", gs.usercomments.deleteSelectedComments);
-    
+    delCommentsButton.disabled = true;
     delButtonDiv.appendChild(delCommentsButton);
 
@@ -217 +217 @@
     //deleteCheckBox.setAttribute("data-comment", comment);
     divgroup.appendChild(deleteCheckBox);
+
+    //deleteCheckBox.setAttribute("checked", false); // why does this check the box?
+    deleteCheckBox.addEventListener('change', function() {
+        var delCommentsButton = document.getElementById("del-selected-comments-button");
+        if(delCommentsButton.disabled && this.checked) {
+        delCommentsButton.disabled = false;
+        } else if(!this.checked) { //if we just unchecked current checkbox
+        // only disable DeleteSelected button if all other checkboxes are all unchecked
+        var selectedComments = document.querySelectorAll('input[class=del-me-comment][type=checkbox]:checked');
+        if(selectedComments.length <= 0) {
+             delCommentsButton.disabled = true;
+        }
+        }
+    });
     }
     
@@ -244 +258 @@
 gs.usercomments.removeSelectedComments = function(eventObject, opt_metaname_username, opt_metaname_usertimestamp, opt_metaname_usercomment)
 {
+    // https://sabe.io/blog/javascript-yes-no-confirmation-box
+    var confirmResponse = confirm("Are you sure you want to delete the selected user comment(s)?");
+    if (!confirmResponse) { // user cancelled
+    return;
+    }
+
+    
     // https://stackoverflow.com/questions/590018/getting-all-selected-checkboxes-in-an-array
     // https://stackoverflow.com/questions/6166763/jquery-multiple-checkboxes-array
@@ -497 +518 @@
     
     // whether there was an error or not, re-enable the submit button now
-    // that the removeMetada ajax operations have completed.
+    // that the removeMetadata ajax operations have completed.
     document.getElementById("usercommentSubmitButton").disabled = false;
+    
     var delCommentsButton = document.getElementById("del-selected-comments-button");
     if(delCommentsButton != undefined) {
-    delCommentsButton.disabled = false;
-    }
-    
+    var delCheckboxes = document.querySelectorAll('input[class=del-me-comment][type=checkbox]');
+    if(delCheckboxes.length <= 0) {
+        delCommentsButton.disabled = true;
+    }
+    }    
 }
 
@@ -641 +665 @@
     var delCommentsButton = document.getElementById("del-selected-comments-button");
     if(delCommentsButton != undefined) {
-    delCommentsButton.disabled = false;
-    }
-    
+    var selectedComments = document.querySelectorAll('input[class=del-me-comment][type=checkbox]:checked');
+    if(selectedComments.length <= 0) {
+        delCommentsButton.disabled = true;
+    } else {
+        delCommentsButton.disabled = false;
+    }
+    }
 }