mouseleave event handler returns false and prevents event propagation

[BrandonLWhite]

Fighting with this in 0.9.11 (jQuery). Why does chosen need to "return _this.mouse_leave(evt);" from the mouseleave event handler setup in Chosen.prototype.register_observers? The result is always false, due to:

AbstractChosen.prototype.mouse_leave = function() {
  return this.mouse_on_container = false;
};

This causes jQuery to StopPropagation, thus any parent elements will not get their mouseleave invoked. This is causing me grief (tooltip sort of stuff, FWIW).

If I eliminate the "return" then everything is cool. I'm scratching my head trying to figure out why its there to begin with. Can someone fix this? Or if it's as-designed please help me understand the purpose.

[BrandonLWhite]

Ok I get the 'why' now. It's the default Coffescript behavior to return the last expression from the function. You have to manually opt-out return values in Coffescript. Thus to resolve the issue, in abstract-chosen.coffee add a "return" to specify to return undefined. This probably should be applied to other event handlers as well.

mouse_leave: -> @mouse_on_container = false; return

If you guys will accept a pull request I'm happy to make the change.

[stof]

yeah, please send a PR doing it for all event handlers to avoid issues