From 5ca8d7c2a79ed3d9a6def61ba0fbd0acd5d404dd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Hannes=20Walln=C3=B6fer?= <hannesw@openjdk.org>
Date: Wed, 20 Aug 2025 14:52:04 +0000
Subject: [PATCH] 8284499: Add the ability to right-click and open in new tab
 JavaDoc Search results

Reviewed-by: liach
---
 .../doclets/formats/html/resources/search.js.template  | 10 ++++++++--
 .../doclets/formats/html/resources/stylesheet.css      | 10 ++++++----
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/search.js.template b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/search.js.template
index 5a36ee06819..16ae5a4d221 100644
--- a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/search.js.template
+++ b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/search.js.template
@@ -423,7 +423,8 @@ $.widget("custom.catcomplete", $.ui.autocomplete, {
         var label = getResultLabel(item);
         var resultDesc = getResultDescription(item);
         return $("<li/>")
-            .append($("<div/>")
+            .append($("<a/>")
+                .attr("href", item.indexItem ? pathtoroot + getURL(item.indexItem, item.category) : null)
                 .append($("<span/>").addClass("search-result-label").html(label))
                 .append($("<span/>").addClass("search-result-desc").html(resultDesc)))
             .appendTo(ul);
@@ -515,7 +516,7 @@ $(function() {
             this.menu.previousFilter = "_";
             this.menu.filterTimer = this.menu._delay(function() {
                 delete this.previousFilter;
-            }, 1000);
+            }, 500);
             return doSearch(request, response);
         },
         response: function(event, ui) {
@@ -531,6 +532,11 @@ $(function() {
             collision: "flip"
         },
         select: function(event, ui) {
+            for (var e = event.originalEvent; e != null; e = e.originalEvent) {
+                if (e.type === "click") {
+                    return;
+                }
+            }
             if (ui.item.indexItem) {
                 var url = getURL(ui.item.indexItem, ui.item.category);
                 window.location.href = pathtoroot + url;
diff --git a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/stylesheet.css b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/stylesheet.css
index 587121ad582..778ee5f03b3 100644
--- a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/stylesheet.css
+++ b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/stylesheet.css
@@ -1235,11 +1235,11 @@ li.ui-static-link a, li.ui-static-link a:visited {
     grid-template-columns: auto auto;
 }
 .ui-autocomplete > li,
-.ui-autocomplete > li > div {
+.ui-autocomplete > li > a {
     grid-column: 1 / 3;
 }
 .ui-autocomplete > li.result-item,
-.ui-autocomplete > li.result-item > div {
+.ui-autocomplete > li.result-item > a {
     display: grid;
     grid-template-columns: subgrid;
 }
@@ -1250,6 +1250,7 @@ li.ui-static-link a, li.ui-static-link a:visited {
 }
 .ui-autocomplete .search-result-label {
     padding: 1px 4px;
+    color: var(--block-text-color);
     overflow: hidden;
     text-overflow: ellipsis;
 }
@@ -1263,6 +1264,7 @@ li.ui-static-link a, li.ui-static-link a:visited {
 .ui-autocomplete .result-highlight {
     font-weight:bold;
 }
+.ui-menu .ui-state-active .search-result-label,
 .ui-menu .ui-state-active .search-result-desc {
     color: var(--selected-text-color);
 }
@@ -1843,9 +1845,9 @@ table.striped > tbody > tr > th {
         grid-template-columns: none;
     }
     .ui-autocomplete > li,
-    .ui-autocomplete > li > div,
+    .ui-autocomplete > li > a,
     .ui-autocomplete > li.result-item,
-    .ui-autocomplete > li.result-item > div {
+    .ui-autocomplete > li.result-item > a {
         grid-column: unset;
         display: block;
         grid-template-columns: none;