Error when opening menu if a different menu is already open

[lmonilsson]

I have a few times got an error when trying to open a menu when a different menu is already open.

The following error occurs in the layerClick handler:

TypeError: Floating-point value is not finite:
  target = document.elementFromPoint(x - $win.scrollLeft(), y - $win.scrollTop())

x = e.pageX and y = e.pageY are undefined here, as the layer click actually comes from the following line in show:

// hide any open menus
$('#context-menu-layer').trigger('mousedown');

I haven't noticed the error before, so the behaviour (treating the situation as a non-silent error) may be a result of changes made in Firefox 20.

[rodneyrehm]

good catch! (hoping I get some time in May to fix a couple of these bugs…)

[lmonilsson]

Patch proposal: https://gist.github.com/Lajnold/5404556

It wraps a bunch of menu-repositioning code in layerClick() in a "fakeClick" condition.

[rodneyrehm]

can you re-post that without the white-space noise?

[lmonilsson]

I've updated the Gist.

[lmonilsson]

I previously used the appendTo option to make the menus children to other elements. I'm no longer using appendTo, and I can't seem to reproduce the problem. So it may only be reproducible when using appendTo.