From: dbs <dbs@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Thu, 23 Sep 2010 03:45:16 +0000 (+0000)
Subject: Remove most annoying UI "feature" of the new authority browse list interface
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=833ef624ed34f8c840d353977ad2abf0ebdc8a5d;p=evergreen%2Ftadl.git

Remove most annoying UI "feature" of the new authority browse list interface

When I added the new authority browse list interface, I used dojo.xhrGet()
to retrieve records from the authority browse backend - but because I didn't
specify sync:true, when you first right-clicked on a subfield, the function
would return immediately and default to showing the context menu. You would
then need to click two more times to show the authority list.

Now you get it on your first right-click, as it should be.


git-svn-id: svn://svn.open-ils.org/ILS/trunk@17910 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/xul/staff_client/server/cat/marcedit.js b/Open-ILS/xul/staff_client/server/cat/marcedit.js
index 980ffffa5c..58dcb78ad4 100644
--- a/Open-ILS/xul/staff_client/server/cat/marcedit.js
+++ b/Open-ILS/xul/staff_client/server/cat/marcedit.js
@@ -2326,7 +2326,7 @@ function browseAuthority (sf_popup, menu_id, target, sf, limit, page) {
     ;
 
     // would be good to carve this out into a separate function
-    dojo.xhrGet({"url":url, "handleAs":"xml", "load": function(records) {
+    dojo.xhrGet({"url":url, "sync": true, "handleAs":"xml", "load": function(records) {
         var create_menu = createMenu({ label: $('catStrings').getString('staff.cat.marcedit.create_authority.label')});
 
         var cm_popup = create_menu.appendChild(