diff --git a/glfw/cocoa_window.m b/glfw/cocoa_window.m index 0f5344959..a2f79e809 100644 --- a/glfw/cocoa_window.m +++ b/glfw/cocoa_window.m @@ -1185,7 +1185,6 @@ is_ascii_control_char(char x) { } if (([self hasMarkedText] || previous_has_marked_text) && !_glfw.ns.text[0]) { // do not pass keys like BACKSPACE while there's pre-edit text, let IME handle it - UPDATE_PRE_EDIT_TEXT; debug_key("Ignoring key press as IME is active and it generated no text\n"); return; } @@ -1218,7 +1217,7 @@ is_ascii_control_char(char x) { switch(key) { case GLFW_FKEY_CAPS_LOCK: - mod_name = "caps"; + mod_name = "capslock"; action = modifierFlags & NSEventModifierFlagCapsLock ? GLFW_PRESS : GLFW_RELEASE; break; case GLFW_FKEY_LEFT_SUPER: case GLFW_FKEY_RIGHT_SUPER: @@ -1240,7 +1239,7 @@ is_ascii_control_char(char x) { return; } GLFWkeyevent glfw_keyevent = {.key = key, .native_key = keycode, .native_key_id = keycode, .action = action, .mods = mods}; - debug_key("\x1b[33mflagsChanged:\x1b[m modifer: %s native_key: 0x%x (%s) glfw_key: 0x%x %s\n", + debug_key("\x1b[33mflagsChanged:\x1b[m modifier: %s native_key: 0x%x (%s) glfw_key: 0x%x %s\n", mod_name, keycode, safe_name_for_keycode(keycode), key, format_mods(mods)); marked_text_cleared_by_insert = false; NSTextInputContext *inpctx = [NSTextInputContext currentInputContext]; @@ -1250,14 +1249,13 @@ is_ascii_control_char(char x) { [inpctx handleEvent:event]; in_key_handler = 0; if (marked_text_cleared_by_insert) { - debug_key("Clearing pre-edit text"); + debug_key("Clearing pre-edit text because insertText called from flagsChanged\n"); CLEAR_PRE_EDIT_TEXT; if (_glfw.ns.text[0]) glfw_keyevent.text = _glfw.ns.text; else _glfw.ns.text[0] = old_first_char; } } glfw_keyevent.ime_state = GLFW_IME_NONE; - debug_key("\n"); _glfwInputKeyboard(window, &glfw_keyevent); } @@ -1380,8 +1378,8 @@ is_ascii_control_char(char x) { if (string == nil || !s[0]) { bool had_marked_text = [self hasMarkedText]; [self unmarkText]; - if (had_marked_text && !in_key_handler) { - debug_key("clearing pre-edit because setMarkedText called from event loop\n"); + if (had_marked_text && (!in_key_handler || in_key_handler == 2)) { + debug_key("Clearing pre-edit because setMarkedText called from event loop or flagsChanged\n"); GLFWkeyevent glfw_keyevent = {.ime_state = GLFW_IME_PREEDIT_CHANGED}; _glfwInputKeyboard(window, &glfw_keyevent); _glfw.ns.text[0] = 0; @@ -1397,8 +1395,8 @@ is_ascii_control_char(char x) { [markedText release]; markedText = [[NSMutableAttributedString alloc] initWithString:string]; } - if (!in_key_handler) { - debug_key("updating IME text in kitty from setMarkedText called from event loop: %s\n", _glfw.ns.text); + if (!in_key_handler || in_key_handler == 2) { + debug_key("Updating IME text in kitty from setMarkedText called from event loop or flagsChanged: %s\n", _glfw.ns.text); GLFWkeyevent glfw_keyevent = {.text=[[markedText string] UTF8String], .ime_state = GLFW_IME_PREEDIT_CHANGED}; _glfwInputKeyboard(window, &glfw_keyevent); _glfw.ns.text[0] = 0; @@ -1463,11 +1461,11 @@ void _glfwPlatformUpdateIMEState(_GLFWwindow *w, const GLFWIMEUpdateEvent *ev) { { const char *utf8 = polymorphic_string_as_utf8(string); debug_key("\n\tinsertText: %s replacementRange: (%lu, %lu)\n", utf8, replacementRange.location, replacementRange.length); - if ([self hasMarkedText]) { + if ([self hasMarkedText] && !is_ascii_control_char(utf8[0])) { [self unmarkText]; marked_text_cleared_by_insert = true; if (!in_key_handler) { - debug_key("clearing pre-edit because insertText called from event loop\n"); + debug_key("Clearing pre-edit because insertText called from event loop\n"); GLFWkeyevent glfw_keyevent = {.ime_state = GLFW_IME_PREEDIT_CHANGED}; _glfwInputKeyboard(window, &glfw_keyevent); _glfw.ns.text[0] = 0; @@ -1478,9 +1476,11 @@ void _glfwPlatformUpdateIMEState(_GLFWwindow *w, const GLFWIMEUpdateEvent *ev) { snprintf(s, sizeof(_glfw.ns.text) - (s - _glfw.ns.text), "%s", utf8); _glfw.ns.text[sizeof(_glfw.ns.text) - 1] = 0; if ((!in_key_handler || in_key_handler == 2) && _glfw.ns.text[0]) { - debug_key("sending text to kitty from insertText called from event loop: %s\n", _glfw.ns.text); - GLFWkeyevent glfw_keyevent = {.text=_glfw.ns.text, .ime_state=GLFW_IME_COMMIT_TEXT}; - _glfwInputKeyboard(window, &glfw_keyevent); + if (!is_ascii_control_char(_glfw.ns.text[0])) { + debug_key("Sending text to kitty from insertText called from called from event loop or flagsChanged: %s\n", _glfw.ns.text); + GLFWkeyevent glfw_keyevent = {.text=_glfw.ns.text, .ime_state=GLFW_IME_COMMIT_TEXT}; + _glfwInputKeyboard(window, &glfw_keyevent); + } _glfw.ns.text[0] = 0; } }