[open-ils-commits] r17911 - branches/rel_2_0/Open-ILS/xul/staff_client/server/cat (dbs)

[svn at svn.open-ils.org]

Author: dbs
Date: 2010-09-22 23:46:09 -0400 (Wed, 22 Sep 2010)
New Revision: 17911

Modified:
   branches/rel_2_0/Open-ILS/xul/staff_client/server/cat/marcedit.js
Log:
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.


Modified: branches/rel_2_0/Open-ILS/xul/staff_client/server/cat/marcedit.js
===================================================================
--- branches/rel_2_0/Open-ILS/xul/staff_client/server/cat/marcedit.js	2010-09-23 03:45:16 UTC (rev 17910)
+++ branches/rel_2_0/Open-ILS/xul/staff_client/server/cat/marcedit.js	2010-09-23 03:46:09 UTC (rev 17911)
@@ -2326,7 +2326,7 @@
     ;
 
     // 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(