[haiku-development] Re: New menu tracking implementation

2007/11/16, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>:
> Hi Stefano,
>
> "Stefano Ceccherini" <stefano.ceccherini@xxxxxxxxx> wrote:
> > I'll post a more updated patch in the next hours, although currently
> > it already works, although I'm having some problems with BMenuFields.
> > Any suggestion is appreciated.
>
> If you would go a bit more into the details, I'm sure we can find a way
> to make them work :-)
>

Okay :)
With the latest patch, menufields work, but there is still a problem:
clicking and releasing the mouse on a menufield doesn't keep the menu
opened, because the newly opened menu receives a B_MOUSE_UP event.
I've been looking for a way to avoid adding stuff to BMenu to handle
that, but I think it's inevitable: I guess we'll end up adding a
special case to BMenu for when it's attached to a BMenuField, like
with the old implementation.

Another problem of the current patch is this: the only menu with the
event mask set is the root one, but this implies having lots of
special casing to check if the mouse movement happens in the root menu
or not, because it needs to be handled differently.
In fact, now that I worked on this implementation, I think that a
better option would be to have the event mask set only on the last
child, although doing that client side would mean: every time a child
menu is opened, reset the event mask on the current menu, set the
event mask on the newly opened menu.
In this case, having the event mask set, server side, always on the
last opened menu would be simpler, maybe.

Other related posts: