From: Jason Etheridge <jason@esilibrary.com>
Date: Thu, 20 Dec 2012 21:10:57 +0000 (-0500)
Subject: internal: default_focus not called
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=8bd78a3e0f4d910591208a0845a2d05df43b4270;p=working%2FEvergreen.git

internal: default_focus not called

Minor bug. Was being called in an else clause dependent on some help context
logic for new tabs, but should be called for every new tab. Shouldn't result
in any end-user visible change, but if interfaces wrapped in browser.js were
to have a default_focus function, then this could come into play.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
---

diff --git a/Open-ILS/xul/staff_client/chrome/content/main/menu.js b/Open-ILS/xul/staff_client/chrome/content/main/menu.js
index 29f09010dd..80ea58bdc2 100644
--- a/Open-ILS/xul/staff_client/chrome/content/main/menu.js
+++ b/Open-ILS/xul/staff_client/chrome/content/main/menu.js
@@ -2476,7 +2476,8 @@ commands:
                                         'src' : ''
                                     };
                                     obj.set_help_context(help_params);
-                                } else if (typeof cw.default_focus == 'function') {
+                                }
+                                if (typeof cw.default_focus == 'function') {
                                     cw.default_focus();
                                 }
                             } catch(E) {