From 383d714214b96e4da591a2be9f15fb02aa8b4b44 Mon Sep 17 00:00:00 2001 From: Lance Edgar Date: Sun, 16 Oct 2016 21:40:04 -0500 Subject: [PATCH] Disable a focus() call in menubar.js which messed with search filter focus Hopefully this is a good idea..? Probably should look for a replacement lib at some point... --- tailbone/static/js/lib/jquery.ui.menubar.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tailbone/static/js/lib/jquery.ui.menubar.js b/tailbone/static/js/lib/jquery.ui.menubar.js index 41fa2a69..a1559091 100644 --- a/tailbone/static/js/lib/jquery.ui.menubar.js +++ b/tailbone/static/js/lib/jquery.ui.menubar.js @@ -267,8 +267,12 @@ .removeAttr( "aria-hidden" ) .attr( "aria-expanded", "true" ) .menu("focus", event, menu.children( ".ui-menu-item" ).first() ) - // TODO need a comment here why both events are triggered - .focus() + // TODO need a comment here why both events are triggered + // TODO: heh well given the above comment i'm not sure what the + // implications might be for disabling the focus() call..but it + // messes with text input focus in undesirable ways..so disable it + // we will..until we know why we shouldn't + // .focus() .focusin(); this.open = true; },