Modify caps lock behaviour of shift method

We really shouldn't be merging these two checks. Shift behaviour is independent of Caps Locks for most control key combinations. For typing and capitalization, it should be XOR of Shift and Caps, not the OR of them.
lairworks · Jan 28, 2024 · 43bb9d4 · 43bb9d4
1 parent 629aaed
commit 43bb9d4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/NAS2D/EventHandler.cpp b/NAS2D/EventHandler.cpp
@@ -750,7 +750,7 @@ bool EventHandler::textInputMode()
  */
 bool EventHandler::shift(KeyModifier mod)
 {
-	return KeyModifier::None != (mod & (KeyModifier::Shift | KeyModifier::Caps));
+	return KeyModifier::None != (mod & (KeyModifier::Shift));
 }
 
 

diff --git a/test/EventHandler.test.cpp b/test/EventHandler.test.cpp
@@ -8,7 +8,7 @@ TEST(EventHandler, shift) {
 	EXPECT_TRUE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::ShiftLeft));
 	EXPECT_TRUE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::ShiftRight));
 	EXPECT_TRUE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::ShiftLeft | NAS2D::EventHandler::KeyModifier::CtrlLeft));
-	EXPECT_TRUE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::Caps));
+	EXPECT_FALSE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::Caps));
 	EXPECT_FALSE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::Ctrl));
 	EXPECT_FALSE(NAS2D::EventHandler::shift(NAS2D::EventHandler::KeyModifier::Alt));
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -750,7 +750,7 @@ bool EventHandler::textInputMode() @@
      */
     bool EventHandler::shift(KeyModifier mod)
     {
-    	return KeyModifier::None != (mod & (KeyModifier::Shift | KeyModifier::Caps));
+    	return KeyModifier::None != (mod & (KeyModifier::Shift));
     }
@@ Expand Down @@