Skip to content

Commit 30ff77e

Browse files
🪲 BUG: Fix null reference errors in custom.js
- Add null checks before calling addEventListener on sidebar-true and sidebar-false elements - Prevents 'Cannot read properties of null' error when elements don't exist in DOM - Applied fix to both docs/assets/custom.js and template/assets/js/custom.js
1 parent ead19a3 commit 30ff77e

File tree

2 files changed

+16
-12
lines changed

2 files changed

+16
-12
lines changed

‎docs/assets/custom.js‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ document.addEventListener("DOMContentLoaded", function () {
9191
});
9292
});
9393

94-
var e = document
95-
.getElementById("sidebar-true")
96-
.addEventListener("click", function () {
94+
var sidebarTrue = document.getElementById("sidebar-true");
95+
if (sidebarTrue) {
96+
sidebarTrue.addEventListener("click", function () {
9797
document.getElementById("component-sidebar").style.display = null;
9898
document
9999
.getElementById("component-sidebar")
@@ -102,10 +102,11 @@ document.addEventListener("DOMContentLoaded", function () {
102102
.getElementById("component-content")
103103
.classList.replace("col-12", "col-9");
104104
});
105+
}
105106

106-
var e = document
107-
.getElementById("sidebar-false")
108-
.addEventListener("click", function () {
107+
var sidebarFalse = document.getElementById("sidebar-false");
108+
if (sidebarFalse) {
109+
sidebarFalse.addEventListener("click", function () {
109110
document.getElementById("component-sidebar").style.display = "none";
110111
document
111112
.getElementById("component-sidebar")
@@ -114,4 +115,5 @@ document.addEventListener("DOMContentLoaded", function () {
114115
.getElementById("component-content")
115116
.classList.replace("col-9", "col-12");
116117
});
118+
}
117119
});

‎template/assets/js/custom.js‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ document.addEventListener("DOMContentLoaded", function () {
9191
});
9292
});
9393

94-
var e = document
95-
.getElementById("sidebar-true")
96-
.addEventListener("click", function () {
94+
var sidebarTrue = document.getElementById("sidebar-true");
95+
if (sidebarTrue) {
96+
sidebarTrue.addEventListener("click", function () {
9797
document.getElementById("component-sidebar").style.display = null;
9898
document
9999
.getElementById("component-sidebar")
@@ -102,10 +102,11 @@ document.addEventListener("DOMContentLoaded", function () {
102102
.getElementById("component-content")
103103
.classList.replace("col-12", "col-9");
104104
});
105+
}
105106

106-
var e = document
107-
.getElementById("sidebar-false")
108-
.addEventListener("click", function () {
107+
var sidebarFalse = document.getElementById("sidebar-false");
108+
if (sidebarFalse) {
109+
sidebarFalse.addEventListener("click", function () {
109110
document.getElementById("component-sidebar").style.display = "none";
110111
document
111112
.getElementById("component-sidebar")
@@ -114,4 +115,5 @@ document.addEventListener("DOMContentLoaded", function () {
114115
.getElementById("component-content")
115116
.classList.replace("col-9", "col-12");
116117
});
118+
}
117119
});

0 commit comments

Comments
 (0)