Skip to content

Commit 2f25b32

Browse files
committed
Fix: add keyboard accessibility to popup close icons
1 parent 6c175b3 commit 2f25b32

File tree

1 file changed

+44
-40
lines changed

1 file changed

+44
-40
lines changed

index.html

Lines changed: 44 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -198,13 +198,17 @@
198198
<div class="popupMsg" id="printText" tabindex="-1">
199199
<div class="contentWrapper">
200200
<span id="printTextContent"></span>
201-
<img class="msgCloseIcon" onclick="hidePrintText()" src="images/close.svg" alt="Close">
201+
<img class="msgCloseIcon" role="button" tabindex="0" onclick="hidePrintText()"
202+
onkeydown="if(event.key==='Enter'||event.key===' '){event.preventDefault();hidePrintText();}"
203+
src="images/close.svg" alt="Close">
202204
</div>
203205
</div>
204206
<div class="popupMsg" id="errorText" tabindex="-1">
205207
<div class="contentWrapper">
206208
<span id="errorTextContent"></span>
207-
<img class="msgCloseIcon" onclick="hideErrorText()" src="images/close.svg" alt="Close">
209+
<img class="msgCloseIcon" role="button" tabindex="0" onclick="hideErrorText()"
210+
onkeydown="if(event.key==='Enter'||event.key===' '){event.preventDefault();hideErrorText();}"
211+
src="images/close.svg" alt="Close">
208212
</div>
209213
</div>
210214

@@ -709,16 +713,16 @@
709713

710714
const languageDropdown = document.getElementById("languagedropdown");
711715
if (languageDropdown) {
712-
const langLinks = languageDropdown.querySelectorAll("a");
713-
langLinks.forEach(link => {
714-
link.addEventListener("click", function (e) {
715-
e.preventDefault(); // prevent default <a> behavior
716-
const selectedLang = this.id; // ID corresponds to language code
717-
localStorage.setItem("languagePreference", selectedLang);
718-
location.reload(); // automatically reload
719-
});
720-
});
721-
}
716+
const langLinks = languageDropdown.querySelectorAll("a");
717+
langLinks.forEach(link => {
718+
link.addEventListener("click", function (e) {
719+
e.preventDefault(); // prevent default <a> behavior
720+
const selectedLang = this.id; // ID corresponds to language code
721+
localStorage.setItem("languagePreference", selectedLang);
722+
location.reload(); // automatically reload
723+
});
724+
});
725+
}
722726
});
723727

724728
</script>
@@ -946,38 +950,38 @@
946950
window.addEventListener("resize", togglePlayOnlyMode);
947951
});
948952

949-
(function () {
950-
function setAppHeight() {
951-
const vh = window.innerHeight * 0.01;
952-
document.documentElement.style.setProperty('--vh', `${vh}px`);
953-
}
954-
955-
window.addEventListener('resize', setAppHeight);
956-
document.addEventListener('fullscreenchange', setAppHeight);
957-
setAppHeight();
958-
})();
959-
960-
(function () {
961-
function resizeCanvasesToScreen() {
962-
const canvases = document.querySelectorAll("canvas");
963-
const width = window.innerWidth;
964-
const height = window.innerHeight;
953+
(function () {
954+
function setAppHeight() {
955+
const vh = window.innerHeight * 0.01;
956+
document.documentElement.style.setProperty('--vh', `${vh}px`);
957+
}
965958

966-
canvases.forEach(c => {
967-
c.width = width;
968-
c.height = height;
969-
c.style.width = "100%";
970-
c.style.height = "100%";
971-
});
972-
}
959+
window.addEventListener('resize', setAppHeight);
960+
document.addEventListener('fullscreenchange', setAppHeight);
961+
setAppHeight();
962+
})();
963+
964+
(function () {
965+
function resizeCanvasesToScreen() {
966+
const canvases = document.querySelectorAll("canvas");
967+
const width = window.innerWidth;
968+
const height = window.innerHeight;
969+
970+
canvases.forEach(c => {
971+
c.width = width;
972+
c.height = height;
973+
c.style.width = "100%";
974+
c.style.height = "100%";
975+
});
976+
}
973977

974-
window.addEventListener("resize", resizeCanvasesToScreen);
975-
document.addEventListener("fullscreenchange", resizeCanvasesToScreen);
978+
window.addEventListener("resize", resizeCanvasesToScreen);
979+
document.addEventListener("fullscreenchange", resizeCanvasesToScreen);
976980

977-
setTimeout(resizeCanvasesToScreen, 150);
978-
})();
981+
setTimeout(resizeCanvasesToScreen, 150);
982+
})();
979983

980-
</script>
984+
</script>
981985

982986

983987
</body>

0 commit comments

Comments
 (0)