[OMPT-svn-commit] OpenMPT r22168 - trunk/OpenMPT/mptrack

[<sv...@op...>]

Author: sagamusix
Date: Tue Nov 12 19:00:41 2024
New Revision: 22168
URL: https://source.openmpt.org/browse/openmpt/?op=revision&rev=22168

Log:
[Fix] Keyboard config: Don't allow shortcut editor to become active by double-clicking when no valid shortcut is selected.

Modified:
   trunk/OpenMPT/mptrack/KeyConfigDlg.cpp

Modified: trunk/OpenMPT/mptrack/KeyConfigDlg.cpp
==============================================================================
--- trunk/OpenMPT/mptrack/KeyConfigDlg.cpp	Tue Nov 12 18:57:25 2024	(r22167)
+++ trunk/OpenMPT/mptrack/KeyConfigDlg.cpp	Tue Nov 12 19:00:41 2024	(r22168)
@@ -650,9 +650,9 @@
 	{
 		m_cmbKeyChoice.SetWindowText(_T(""));
 		m_eCustHotKey.SetWindowText(_T(""));
-		m_bKeyDown.SetCheck(0);
-		m_bKeyHold.SetCheck(0);
-		m_bKeyUp.SetCheck(0);
+		m_bKeyDown.SetCheck(BST_UNCHECKED);
+		m_bKeyHold.SetCheck(BST_UNCHECKED);
+		m_bKeyUp.SetCheck(BST_UNCHECKED);
 		m_curCommand = kcNull;
 	}
 
@@ -695,9 +695,9 @@
 		m_curKeyChoice = choice;
 		m_forceUpdate = true;
 		m_eCustHotKey.SetKey(ModNone, 0);
-		m_bKeyDown.SetCheck(0);
-		m_bKeyHold.SetCheck(0);
-		m_bKeyUp.SetCheck(0);
+		m_bKeyDown.SetCheck(BST_UNCHECKED);
+		m_bKeyHold.SetCheck(BST_UNCHECKED);
+		m_bKeyUp.SetCheck(BST_UNCHECKED);
 		return;
 	}
 
@@ -772,7 +772,7 @@
 	ClientToScreen(&point);
 	CRect rect;
 	m_eCustHotKey.GetWindowRect(rect);
-	if(m_eCustHotKey.IsBypassed() && rect.PtInRect(point))
+	if(m_curCommand != kcNull && m_eCustHotKey.IsBypassed() && rect.PtInRect(point))
 		EnableKeyChoice(true);
 	else
 		CPropertyPage::OnLButtonDblClk(flags, point);