Bug 16900 - Hold suspend button incorrectly styled in patron holds list
authorOwen Leonard <oleonard@myacpl.org>
Mon, 11 Jul 2016 19:03:37 +0000 (15:03 -0400)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 15 Jul 2016 15:06:44 +0000 (15:06 +0000)
This patch updates the style of the suspend/resume button in the list of
holds displayed on the checkout page and on the patron detail page. The
button is now a Bootstrap-styled button with a Font Awesome icon.

To test, apply the patch and locate a patron who has holds.

- On the checkout page, view the 'Holds' tab and confirm that there is a
  correctly-styled 'Suspend' button for each hold.
- Verify that the button works correctly and looks correct when a hold
  has been suspended.

Signed-off-by: Arslan Farooq <arslanone@gmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/js/holds.js

index 3182a30..7ddd5ea 100644 (file)
@@ -133,11 +133,11 @@ $(document).ready(function() {
                             if ( oObj.found ) {
                                 return "";
                             } else if ( oObj.suspend == 1 ) {
-                                return "<a class='hold-resume btn btn-link' id='resume" + oObj.reserve_id + "' style='display: inline; white-space: nowrap;'>"
-                                     + "<i class='icon-play'></i> " + RESUME + "</a>";
+                                return "<a class='hold-resume btn btn-mini' id='resume" + oObj.reserve_id + "'>"
+                                     + "<i class='fa fa-play'></i> " + RESUME + "</a>";
                             } else {
-                                return "<a class='hold-suspend btn btn-link' id='suspend" + oObj.reserve_id + "' style='display: inline; white-space: nowrap;'>"
-                                     + "<i class='icon-pause'></i> " + SUSPEND + "</a>";
+                                return "<a class='hold-suspend btn btn-mini' id='suspend" + oObj.reserve_id + "'>"
+                                     + "<i class='fa fa-pause'></i> " + SUSPEND + "</a>";
                             }
                         }
                     }