It ended up being more complicated than that. Cascading menus proved to be a bit tricky. I committed a fix which I think resolves it. (There was a declaration missing in CSS which caused a RTE.)
Harbs > On Sep 13, 2018, at 1:15 PM, Harbs <[email protected]> wrote: > > I’m guessing you’re talking about > MenuSelectionMouseController.hideMenu_internal(). > > I’d fix it by just using setTimeout() > >> On Sep 13, 2018, at 12:30 PM, Harbs <[email protected] >> <mailto:[email protected]>> wrote: >> >> >>> On Sep 13, 2018, at 11:57 AM, Alex Harui <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> Why would renderers dispatch the same event on both mouseUp and click? >>> Seems like they should use one or the other. >> >> I don’t get the question. They shouldn’t. Right now they do. They both >> dispatch itemClicked. >> >>> It might be that we should change Menu to rely on click and not mouseUp, >>> but that seems like something other people will trip over. >> >> Where’s the Menu code you are talking about? >
