Wrap around when navigating CPE suggestions via arrow keys

ra1nb0rn · Sep 2, 2024 · 4c0f556 · 4c0f556
1 parent b1d9185
commit 4c0f556
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/web_server_files/static/js/search_vulns.js b/web_server_files/static/js/search_vulns.js
@@ -1114,9 +1114,11 @@ function moveCPESuggestionUpDown(event) {
     else if (event.keyCode == 40)
         curSelectedCPESuggestion++;
 
-    // enforce lower and upper bounds
-    curSelectedCPESuggestion = Math.max(-1, curSelectedCPESuggestion);
-    curSelectedCPESuggestion = Math.min(curSelectedCPESuggestion, $("#cpeSuggestions").find('ul').children('li').length - 1);
+    // wrap around if moving below available suggestions or above them
+    if (curSelectedCPESuggestion < 0)
+        curSelectedCPESuggestion = $("#cpeSuggestions").find('ul').children('li').length - 1;
+    else if (curSelectedCPESuggestion > $("#cpeSuggestions").find('ul').children('li').length - 1)
+        curSelectedCPESuggestion = 0;
 
     if (curSelectedCPESuggestion > -1) {
         const suggestionElement = $('#cpe-suggestion-' + curSelectedCPESuggestion);