Menu: Reset the mouseHandled flag after triggering select events. Fixes ...

[tjvantoll]

...#9469: on( "menuselect" ) not firing every time.

[tjvantoll]

@kborchers Is this sane? I'm still not quite clear on the purpose of the mouseHandled flag and whether it makes sense to reset it here. See http://bugs.jqueryui.com/ticket/9469#comment:4.

Thanks.

[kborchers]

Yeah, I think this makes sense. The mouseHandled flag prevents the click event from firing on the parent menu items in nested menus as the event bubbles up through the menus.

I would also probably remove the setting of mouseHandled on https://github.com/tjvantoll/jquery-ui/blob/d7d1aadb5e7fb14f0980844c2b297c33f949d411/ui/jquery.ui.menu.js#L78 since it will be handled in the call to select just below that.

[tjvantoll]

I would also probably remove the setting of mouseHandled on https://github.com/tjvantoll/jquery-ui/blob/d7d1aadb5e7fb14f0980844c2b297c33f949d411/ui/jquery.ui.menu.js#L78 since it will be handled in the call to select just below that.

Ah, good call. It probably makes sense just to move that line under this.select( event ); instead of doing it in the method.

I feel better about this now. Thanks.

[tjvantoll]

Rebased and landed in 484e382.