Modifying the breadcrumb context menu so that it also requires clicking 'v'.

core/src/main/resources/hudson/widgets/HistoryWidget/entry.jelly

@@ -35,7 +35,7 @@ THE SOFTWARE.
       ${build.displayName}
     </td>
     <td style="padding-right:0">
-      <a class="tip model-link tl-tr inside" href="${link}">
+      <a class="tip model-link inside" href="${link}">
         <i:formatDate value="${build.timestamp.time}" type="both" dateStyle="medium" timeStyle="medium"/>
       </a>
     </td>

core/src/main/resources/lib/layout/breadcrumb.jelly

@@ -40,7 +40,7 @@ THE SOFTWARE.
 
   <j:if test="${mode=='breadcrumbs'}">
     <li id="${attrs.id}">
-      <a href="${attrs.href}">
+      <a href="${attrs.href}" class="model-link inside">
         ${attrs.title}
       </a>
     </li>

core/src/main/resources/lib/layout/breadcrumbBar.jelly

@@ -58,8 +58,8 @@ THE SOFTWARE.
           <ul id="breadcrumbs">
             <j:forEach var="anc" items="${request.ancestors}">
               <j:if test="${h.isModel(anc.object) and anc.prev.url!=anc.url}">
-                <li class="${h.isModelWithContextMenu(anc.object)?null:'no-context-menu'}">
-                  <a href="${anc.url}/">
+                <li class="">
+                  <a href="${anc.url}/" class="${h.isModelWithContextMenu(anc.object)?'model-link':null} inside">
                     ${anc.object.displayName}
                   </a>
                 </li>

core/src/main/resources/lib/layout/breadcrumbs.css

@@ -29,14 +29,15 @@
 #breadcrumbs LI {
     float:left;
     line-height:2em;
-    padding-left:.75em;
     color:#555753;
+    padding-right: 15px;
+    background:url('breadcrumb.gif') no-repeat right center;
 }
 
 #breadcrumbs LI A {
-    display:block;
-    padding:0 15px 0 0;
-    background:url('breadcrumb.gif') no-repeat right center;
+    display: block;
+    padding-left: 0.75em;
+    line-height: 2em;
 }
 
 #breadcrumbs LI A:link, #breadcrumbs LI A:visited {

core/src/main/resources/lib/layout/breadcrumbs.js

@@ -114,7 +114,6 @@ var breadcrumbs = (function() {
             this.style.visibility = "hidden";
         };
         menuSelector.observe("click",function () {
-            this.hide();
             handleHover(this.target);
         });
 
@@ -190,11 +189,11 @@ var breadcrumbs = (function() {
         return false;
     }
 
-    Behaviour.specify("#breadcrumbs LI", 'breadcrumbs', 0, function (e) {
-        // when the mouse hovers over LI, activate the menu
-        if (e.hasClassName("no-context-menu"))  return;
-        e.observe("mouseover", function () { handleHover(e.firstChild) });
-    });
+//    Behaviour.specify("#breadcrumbs LI", 'breadcrumbs', 0, function (e) {
+//        // when the mouse hovers over LI, activate the menu
+//        if (e.hasClassName("no-context-menu"))  return;
+//        e.observe("mouseover", function () { handleHover(e.firstChild) });
+//    });
 
     Behaviour.specify("A.model-link", 'breadcrumbs', 0, function (a) {
         // ditto for model-link, but give it a larger delay to avoid unintended menus to be displayed