internal: default_focus not called
authorJason Etheridge <jason@esilibrary.com>
Thu, 20 Dec 2012 21:10:57 +0000 (16:10 -0500)
committerJason Etheridge <jason@esilibrary.com>
Tue, 26 Feb 2013 16:31:43 +0000 (11:31 -0500)
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>
Open-ILS/xul/staff_client/chrome/content/main/menu.js

index 29f0901..80ea58b 100644 (file)
@@ -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) {