From 136cabedb4dc9b8bd639274a1e012bea70ee4e5a Mon Sep 17 00:00:00 2001 From: paxcut <53811119+paxcut@users.noreply.github.com> Date: Mon, 4 Aug 2025 03:23:24 -0700 Subject: [PATCH] fix: Menu items greyed out when they shouldn't (#2378) Problem was that a recent commit allowed any subview window to become the current subview with focus which also included the context menu popup. The fix is to list all the subviews that can acquire focus so that they are the only ones that will be detected by the menu condition. --- plugins/builtin/source/content/views/view_pattern_editor.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/plugins/builtin/source/content/views/view_pattern_editor.cpp b/plugins/builtin/source/content/views/view_pattern_editor.cpp index c0ad180ba..297b46af8 100644 --- a/plugins/builtin/source/content/views/view_pattern_editor.cpp +++ b/plugins/builtin/source/content/views/view_pattern_editor.cpp @@ -351,7 +351,8 @@ namespace hex::plugin::builtin { oldHeight = height; if (g.NavWindow != nullptr) { std::string name = g.NavWindow->Name; - m_focusedSubWindowName = name; + if (name.contains(textEditorView) || name.contains(consoleView) || name.contains(variablesView) || name.contains(settingsView) || name.contains(virtualFilesView) || name.contains(debuggerView)) + m_focusedSubWindowName = name; } fonts::CodeEditor().push();