fix: Prevent context menu from closing prematurely on mouse movement and hover interactions #1

pxlman · 2026-05-06T01:17:44Z

pxlman commented

2026-05-06 01:17:44 +00:00

Problem

The context menu would disappear immediately after opening or when the user moved the mouse to interact with menu items. This was caused by overly aggressive menu dismissal logic that closed the menu on any message except ShowContextMenu.

Root Cause

The preserve_context_menu flag only preserved the menu for ShowContextMenu messages, causing:

CursorMove events (mouse movement) to close the menu instantly
TableMsg::CellEnter events (hovering over menu items) to close the menu
The menu to be unusable in practice

Solution

Updated the menu preservation logic to keep the menu open for all interaction messages that occur while the menu is active:

let preserve_context_menu = matches!(
    &message,
    Message::ShowContextMenu { .. }      // Opening the menu
    | Message::TableMsg(_, TableMessage::CursorMove(_,_)) 
    | Message::TableMsg(_, TableMessage::CellEnter(_,_))
    | Message::HideContextMenu           // Explicit close action
);

## Problem The context menu would disappear immediately after opening or when the user moved the mouse to interact with menu items. This was caused by overly aggressive menu dismissal logic that closed the menu on **any** message except `ShowContextMenu`. ## Root Cause The `preserve_context_menu` flag only preserved the menu for `ShowContextMenu` messages, causing: - `CursorMove` events (mouse movement) to close the menu instantly - `TableMsg::CellEnter` events (hovering over menu items) to close the menu - The menu to be unusable in practice ## Solution Updated the menu preservation logic to keep the menu open for all interaction messages that occur while the menu is active: ```rust let preserve_context_menu = matches!( &message, Message::ShowContextMenu { .. } // Opening the menu | Message::TableMsg(_, TableMessage::CursorMove(_,_)) | Message::TableMsg(_, TableMessage::CellEnter(_,_)) | Message::HideContextMenu // Explicit close action ); ```

pxlman added 2 commits 2026-05-06 01:17:45 +00:00

6f1a5d31bf replace on_right_press with on_right_release for better experience

46ca844a93 fixing auto hiding context menu and adding a message passing debuging line

jess merged commit 15415db1af into main

2026-05-06 02:06:58 +00:00

jess referenced this issue from a commit

2026-05-06 02:07:00 +00:00

Merge pull request 'fix: Prevent context menu from closing prematurely on mouse movement and hover interactions' (#1) from pxlman/Acord:main into main

Sign in to join this conversation.

No reviewers

No Label

No Milestone