rattail/tailbone
2
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

Disable a focus() call in menubar.js which messed with search filter focus

Browse source 
Hopefully this is a good idea..?  Probably should look for a replacement
lib at some point...
This commit is contained in:
Lance Edgar 2016-10-16 21:40:04 -05:00
parent 6cb4b86fd0
commit 383d714214
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
tailbone/static/js/lib/jquery.ui.menubar.js vendored
View file

@ -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;
},