From 833ef624ed34f8c840d353977ad2abf0ebdc8a5d Mon Sep 17 00:00:00 2001 From: dbs Date: Thu, 23 Sep 2010 03:45:16 +0000 Subject: [PATCH] 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 --- Open-ILS/xul/staff_client/server/cat/marcedit.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/xul/staff_client/server/cat/marcedit.js b/Open-ILS/xul/staff_client/server/cat/marcedit.js index 980ffffa5..58dcb78ad 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( -- 2.11.0