From beb8c80787beadbfdb8b970368a3200f7d59f58e Mon Sep 17 00:00:00 2001 From: jvoisin Date: Wed, 20 Mar 2024 23:59:37 +0100 Subject: Replace a bunch of `let` with `const` According to https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const > Many style guides (including MDN's) recommend using const over let whenever a variable is not reassigned in its scope. This makes the intent clear that a variable's type (or value, in the case of a primitive) can never change. --- internal/ui/static/js/modal_handler.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'internal/ui/static/js/modal_handler.js') diff --git a/internal/ui/static/js/modal_handler.js b/internal/ui/static/js/modal_handler.js index 0fa55bfa..536cea3e 100644 --- a/internal/ui/static/js/modal_handler.js +++ b/internal/ui/static/js/modal_handler.js @@ -8,7 +8,7 @@ class ModalHandler { } static getFocusableElements() { - let container = this.getModalContainer(); + const container = this.getModalContainer(); if (container === null) { return null; @@ -18,14 +18,14 @@ class ModalHandler { } static setupFocusTrap() { - let focusableElements = this.getFocusableElements(); + const focusableElements = this.getFocusableElements(); if (focusableElements === null) { return; } - let firstFocusableElement = focusableElements[0]; - let lastFocusableElement = focusableElements[focusableElements.length - 1]; + const firstFocusableElement = focusableElements[0]; + const lastFocusableElement = focusableElements[focusableElements.length - 1]; this.getModalContainer().onkeydown = (e) => { if (e.key !== 'Tab') { @@ -57,13 +57,13 @@ class ModalHandler { this.activeElement = document.activeElement; - let container = document.createElement("div"); + const container = document.createElement("div"); container.id = "modal-container"; container.setAttribute("role", "dialog"); container.appendChild(document.importNode(fragment, true)); document.body.appendChild(container); - let closeButton = document.querySelector("button.btn-close-modal"); + const closeButton = document.querySelector("button.btn-close-modal"); if (closeButton !== null) { closeButton.onclick = (event) => { event.preventDefault(); @@ -89,7 +89,7 @@ class ModalHandler { } static close() { - let container = this.getModalContainer(); + const container = this.getModalContainer(); if (container !== null) { container.parentNode.removeChild(container); } -- cgit v1.2.3