internal: default_focus not called
authorJason Etheridge <jason@esilibrary.com>
Thu, 20 Dec 2012 21:10:57 +0000 (16:10 -0500)
committerMike Rylander <mrylander@gmail.com>
Thu, 12 Sep 2013 17:50:39 +0000 (13:50 -0400)
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>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/xul/staff_client/chrome/content/main/menu.js

index cc7a91a..805dcdb 100644 (file)
@@ -2502,7 +2502,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) {