Stop highlighting indicators

The way that commands were highlighted when selected, highlighted
both the indicators and the buttons. Change to only hightlight
the tabbed buttons.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

diff --git a/Open-ILS/xul/staff_client/server/patron/display.js b/Open-ILS/xul/staff_client/server/patron/display.js
index 03c350f..33fead8 100644 (file)
--- a/Open-ILS/xul/staff_client/server/patron/display.js
+++ b/Open-ILS/xul/staff_client/server/patron/display.js
@@ -692,14 +692,12 @@ patron.display.prototype = {
                 this.skip_hide_summary = false;
                 dont_hide_summary = true;
             }
-            this.controller.view.cmd_patron_checkout.setAttribute('style','');
-            this.controller.view.cmd_patron_items.setAttribute('style','');
-            this.controller.view.cmd_patron_edit.setAttribute('style','');
-            this.controller.view.cmd_patron_other.setAttribute('style','');
-            this.controller.view.cmd_patron_holds.setAttribute('style','');
-            this.controller.view.cmd_patron_bills.setAttribute('style','');
-            this.controller.view.cmd_standing_penalties.setAttribute('style','');
-            this.controller.view[ btn ].setAttribute('style','background: blue; color: white;');
+            var buttons = document.getElementsByTagName('button');
+            for(var i = 0; i < buttons.length; i++) {
+                var command = buttons[i].getAttribute('command');
+                if(command == btn) buttons[i].setAttribute('style','background: blue; color: white;');
+                else buttons[i].setAttribute('style','');
+            }
             var auto_hide_patron_sidebar = String( this.OpenILS.data.hash.aous['circ.auto_hide_patron_summary'] ) == 'true';
             var x = document.getElementById('splitter_grippy'); 
             if (x && auto_hide_patron_sidebar && ! dont_hide_summary) {