[BMenu] doesn't close on focus lost

[diver]

Click attribute menu in tracker window and then click in container window, attribute menu should close, but it won't.

[jackburton]

That's weird. Especially because it only happens inside tracker. It happens, though, so something might be wrong indeed.

[jackburton]

By the way... can you tell/remember if it's a problem introduced only recently ? Have you got the chance to try a revision older than 15876 or 15877 ?

[jackburton]

Axel, I noticed that BPoseView::MouseDown() also uses a synchronous loop to check mouse buttons, but with a delay which is much smaller than the one used in BMenu. Since both use the GetMouse() variation which checks the message queue, I think PoseView "steals" the B_MOUSE_DOWN message, and bmenu never gets the clicked state.

Should I use the other variation of GetMouse(), the one which only gets the current mouse state ? Couldn't that lose clicks as well, if the menu tracking loop doesn't execute fast enough ?

Does what I wrote make sense ?

[jackburton]

I just committed a change which should've fixed this. Can you check ?

[jackburton]

Apparently this bug is fixed, but I introduced a new problem: closing menus this way makes them open again when you move the cursor over the menubar.

[jackburton]

Fixed in hrev17654