fix: skip key combos with super modifier

closes #458
rime · Jan 25, 2023 · 80bef8f · 80bef8f
1 parent 2a12cb8
commit 80bef8f
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 5 deletions.
diff --git a/src/rime/gear/editor.cc b/src/rime/gear/editor.cc
@@ -58,7 +58,7 @@ ProcessResult Editor::ProcessKeyEvent(const KeyEvent& key_event) {
     }
   }
   if (char_handler_ &&
-      !key_event.ctrl() && !key_event.alt() &&
+      !key_event.ctrl() && !key_event.alt() && !key_event.super() &&
       ch > 0x20 && ch < 0x7f) {
     DLOG(INFO) << "input char: '" << (char)ch << "', " << ch
                << ", '" << key_event.repr() << "'";

diff --git a/src/rime/gear/punctuator.cc b/src/rime/gear/punctuator.cc
@@ -59,7 +59,8 @@ static bool punctuation_is_translated(Context* ctx) {
 }
 
 ProcessResult Punctuator::ProcessKeyEvent(const KeyEvent& key_event) {
-  if (key_event.release() || key_event.ctrl() || key_event.alt())
+  if (key_event.release() ||
+      key_event.ctrl() || key_event.alt() || key_event.super())
     return kNoop;
   int ch = key_event.keycode();
   if (ch < 0x20 || ch >= 0x7f)

diff --git a/src/rime/gear/recognizer.cc b/src/rime/gear/recognizer.cc
@@ -80,7 +80,8 @@ Recognizer::Recognizer(const Ticket& ticket) : Processor(ticket) {
 
 ProcessResult Recognizer::ProcessKeyEvent(const KeyEvent& key_event) {
   if (patterns_.empty() ||
-      key_event.ctrl() || key_event.alt() || key_event.release()) {
+      key_event.ctrl() || key_event.alt() || key_event.super() ||
+      key_event.release()) {
     return kNoop;
   }
   int ch = key_event.keycode();

diff --git a/src/rime/gear/shape.cc b/src/rime/gear/shape.cc
@@ -41,7 +41,8 @@ ProcessResult ShapeProcessor::ProcessKeyEvent(const KeyEvent& key_event) {
   if (!engine_->context()->get_option("full_shape")) {
     return kNoop;
   }
-  if (key_event.ctrl() || key_event.alt() || key_event.release()) {
+  if (key_event.ctrl() || key_event.alt() || key_event.super() ||
+      key_event.release()) {
     return kNoop;
   }
   int ch = key_event.keycode();

diff --git a/src/rime/gear/speller.cc b/src/rime/gear/speller.cc
@@ -90,7 +90,8 @@ Speller::Speller(const Ticket& ticket) : Processor(ticket),
 }
 
 ProcessResult Speller::ProcessKeyEvent(const KeyEvent& key_event) {
-  if (key_event.release() || key_event.ctrl() || key_event.alt())
+  if (key_event.release() ||
+      key_event.ctrl() || key_event.alt() || key_event.super())
     return kNoop;
   int ch = key_event.keycode();
   if (ch < 0x20 || ch >= 0x7f)  // not a valid key for spelling