itdominator
/
Easy-Session-Manager
1
0
Fork 0
Code Issues 3 Pull Requests Projects Releases Wiki Activity

Adding a "Deselect All" button #9

Merged
CRImier merged 3 commits from master into master 2020-08-25 01:45:47 +00:00
Conversation 11 Commits 3 Files Changed 3 +2 -3
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 18c257d60c - Show all commits

3
src/scripts/events.js
View File

@ -66,9 +66,8 @@ document.addEventListener("click", (e) => {
hideModal("loadModal"); hideModal("loadModal");
maximstewart commented 2020-08-25 00:35:53 +00:00 (Migrated from github.com)
Review
Copy Link

Please change from "var" to "let"

Please change from "var" to "let"
} }
} }
else if (action.includes("deselectAll")) { else if (action.includes("deselectAll")) {
var container = document.getElementById("editSelectionContainer"); let container = document.getElementById("editSelectionContainer");
deselectAll(container); deselectAll(container);
} }

2
src/scripts/utils.js
View File

@ -194,7 +194,7 @@ const getSelectionData = (container = null, keys = null, keysLength = 0) => {
} }
const deselectAll = (container = null) => { const deselectAll = (container = null) => {
var selAlls = container.querySelectorAll('.selAll'); let selAlls = container.querySelectorAll('.selAll');
for (var i = 0; i < selAlls.length; i++) { for (var i = 0; i < selAlls.length; i++) {
var selAll = selAlls[i]; var selAll = selAlls[i];
selAll.checked = false; selAll.checked = false;