Skip to content

Commit 677850f

Browse files
Merge pull request #328 from QuietMisdreavus/fix-search-nav
fix keyboard nav on releases pages
2 parents 53f67d1 + 18ab76e commit 677850f

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

templates/releases.hbs

+5-3
Original file line numberDiff line numberDiff line change
@@ -62,15 +62,17 @@
6262
if (ev.ctrlKey || ev.altKey || ev.metaKey || document.activeElement.tagName === "INPUT") {
6363
return;
6464
}
65-
if (ev.which === 38) { // Down arrow
65+
if (ev.which === 40) { // Down arrow
66+
ev.preventDefault();
6667
if (active === null) {
6768
active = document.getElementsByClassName("recent-releases-container")[0].getElementsByTagName("li")[0];
6869
} else if (active.nextElementSibling) {
6970
active.classList.remove("selected");
7071
active = active.nextElementSibling;
7172
}
7273
active.classList.add("selected");
73-
} else if (ev.which === 40) { // Up arrow
74+
} else if (ev.which === 38) { // Up arrow
75+
ev.preventDefault();
7476
if (active === null) {
7577
active = document.getElementsByClassName("recent-releases-container")[0].getElementsByTagName("li")[0];
7678
} else if (active.previousElementSibling) {
@@ -79,7 +81,7 @@
7981
}
8082
active.classList.add("selected");
8183
active.focus();
82-
} else if (e.which === 13) { // Return
84+
} else if (ev.which === 13) { // Return
8385
if (active !== null) {
8486
document.location.href = active.getElementsByTagName("a")[0].href;
8587
}

0 commit comments

Comments
 (0)