Added logic to un check the select-all-checkbox on event CLOSE_BANNER…

… and Remove Item from Multi Select
hlxsites · Mar 4, 2024 · 36a5e58 · 36a5e58
1 parent cef86c3
commit 36a5e58
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 6 deletions.
diff --git a/blocks/adp-collection-header/adp-collection-header.js b/blocks/adp-collection-header/adp-collection-header.js
@@ -39,10 +39,10 @@ function createCollectionInfoHeader(collectionInfoHeader, collection) {
 
   document.getElementById('select-all-checkbox').addEventListener('click', function(event) {
       // Get the checked state of the select all checkbox
-      var isChecked = event.target.checked;
+      const isChecked = event.target.checked;
 
       // Get all the checkboxes within the cards
-      var checkboxes = document.querySelectorAll('.checkbox-container input[type="checkbox"], .filetype-video .checkbox-container input[type="checkbox"]');
+      const checkboxes = document.querySelectorAll('.checkbox-container input[type="checkbox"], .filetype-video .checkbox-container input[type="checkbox"]');
       // isChecked then selectAllAssets() else deselectAllAssets
       isChecked ? selectAllAssets() : deselectAllAssets();
   });

diff --git a/blocks/adp-infinite-results-collection/CollectionDatasource.js b/blocks/adp-infinite-results-collection/CollectionDatasource.js
@@ -88,6 +88,8 @@ export default class CollectionsDatasource {
         },
         removeItemFromMultiSelectionHandler: () => {
           infiniteResultsContainer.removeItemFromMultiSelection(assetId);
+          //Uncheck select all checkbox
+          document.getElementById('select-all-checkbox').checked = false;
         },
       },
     );

diff --git a/blocks/adp-infinite-results-collection/adp-infinite-results-collection.js b/blocks/adp-infinite-results-collection/adp-infinite-results-collection.js
@@ -16,6 +16,8 @@ export default async function decorate(block) {
     infiniteResultsContainer.deselectItem(e.detail.assetId);
   });
   addEventListener(EventNames.CLOSE_BANNER, () => {
+    //Uncheck select all checkbox
+    document.getElementById('select-all-checkbox').checked = false;
     infiniteResultsContainer.clearAllSelections();
   });
 }

diff --git a/blocks/adp-infinite-results-linkshare/LinkShareDatasource.js b/blocks/adp-infinite-results-linkshare/LinkShareDatasource.js
@@ -53,6 +53,8 @@ export default class LinkShareDatasource {
         },
         removeItemFromMultiSelectionHandler: () => {
           infiniteResultsContainer.removeItemFromMultiSelection(assetId);
+          //Uncheck select all checkbox
+          document.getElementById('select-all-checkbox').checked = false;
         },
       },
     );

diff --git a/blocks/adp-infinite-results-linkshare/adp-infinite-results-linkshare.js b/blocks/adp-infinite-results-linkshare/adp-infinite-results-linkshare.js
@@ -13,6 +13,8 @@ export default async function decorate(block) {
     infiniteResultsContainer.deselectItem(e.detail.assetId);
   });
   addEventListener(EventNames.CLOSE_BANNER, () => {
+    //Uncheck select all checkbox
+    document.getElementById('select-all-checkbox').checked = false;
     infiniteResultsContainer.clearAllSelections();
   });
 }
@@ -30,5 +32,3 @@ export function selectAllAssets() {
 export function deselectAllAssets() {
   infiniteResultsContainer.deselectAllItems();
 }
-
-
diff --git a/blocks/adp-share-header/adp-share-header.js b/blocks/adp-share-header/adp-share-header.js
@@ -23,10 +23,10 @@ function createShareInfoHeader(shareInfoHeader) {
 
   document.getElementById('select-all-checkbox').addEventListener('click', function(event) {
       // Get the checked state of the select all checkbox
-      var isChecked = event.target.checked;
+      const isChecked = event.target.checked;
 
       // Get all the checkboxes within the cards
-      var checkboxes = document.querySelectorAll('.checkbox-container input[type="checkbox"], .filetype-video .checkbox-container input[type="checkbox"]');
+      const checkboxes = document.querySelectorAll('.checkbox-container input[type="checkbox"], .filetype-video .checkbox-container input[type="checkbox"]');
       // isChecked then selectAllAssets() else deselectAllAssets
       isChecked ? selectAllAssets() : deselectAllAssets();
   });