From 25f654231e0e520e68e137b859bc8470afd79396 Mon Sep 17 00:00:00 2001
From: Kathy Lussier <klussier@masslnc.org>
Date: Tue, 25 Jul 2017 10:38:41 -0400
Subject: [PATCH] LP#1706365: cursor for record menu items should be a pointer

When using the Mark For menu on the bib record page, the
cursor becomes an i-beam while hovering over the menu items instead of the
expected pointer. This commit adds an empty href attribute to the a tag for menu
items so that a pointer displays instead.

Test plan:
Log into the web client and retrieve a record. Click the 'Mark For' menu on the
record page. Pre-patch, as you scroll through the menu items, the cursor will
turn into an i-beam, indicating that text can be selected. Post-patch, the
cursor will be the expected pointer that typically displays for links.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
---
 Open-ILS/src/templates/staff/cat/catalog/t_catalog.tt2 | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Open-ILS/src/templates/staff/cat/catalog/t_catalog.tt2 b/Open-ILS/src/templates/staff/cat/catalog/t_catalog.tt2
index 0848890f8a..7ed8ddd4a1 100644
--- a/Open-ILS/src/templates/staff/cat/catalog/t_catalog.tt2
+++ b/Open-ILS/src/templates/staff/cat/catalog/t_catalog.tt2
@@ -34,31 +34,31 @@
         </button>
         <ul uib-dropdown-menu role="menu" aria-labelledby="mark-for-button">
              <li role="menuitem">
-                    <a ng-click="markConjoined()">
+                    <a ng-click="markConjoined()" href="">
                         [% l('Conjoined Items') %]
                     <span class="target-record-aside" ng-if="current_conjoined_target">[% l('(Currently [_1])', '{{current_conjoined_target}}') %]</span>
                 </a>
             </li>
            <li role="menuitem">
-                <a ng-click="markOverlay()">
+                <a ng-click="markOverlay()" href="">
                     [% l('Overlay Target') %]
                     <span class="target-record-aside" ng-if="current_overlay_target">[% l('(Currently [_1])', '{{current_overlay_target}}') %]</span>
                 </a>
             </li>
             <li role="menuitem">
-                <a ng-click="mark_hold_transfer_dest()">
+                <a ng-click="mark_hold_transfer_dest()" href="">
                    [% l('Title Hold Transfer') %]
                 <span class="target-record-aside" ng-if="current_hold_transfer_dest">[% l('(Currently [_1])', '{{current_hold_transfer_dest}}') %]</span>
                 </a>
             </li>
             <li role="menuitem">
-                <a ng-click="markVolTransfer()">
+                <a ng-click="markVolTransfer()" href="">
                     [% l('Volume Transfer') %]
                     <span class="target-record-aside" ng-if="current_voltransfer_target">[% l('(Currently [_1])', '{{current_voltransfer_target}}') %]</span>
                 </a>
             </li>
             <li role="menuitem">
-                <a ng-click="clearRecordMarks()">[% l('Reset Record Marks') %]</a>
+                <a ng-click="clearRecordMarks()" href="">[% l('Reset Record Marks') %]</a>
             </li>
         </ul>
     </div>
-- 
2.11.0