Enter as done shortcut is conflict with CJK input method #2360

Closed
opened 2026-03-22 14:04:23 -05:00 by GiteaMirror · 2 comments
Owner

Originally created by @maggch97 on GitHub (Sep 20, 2025).

Description

Image

Enter is a keyboard shortcut to close the candidate words window for Chinese, Janpanse, Korean input methods.
When the candidate window is displayed, Vikunja still captures the enter key.

I will help later when I have time.

Vikunja Version

latest

Browser and version

No response

Can you reproduce the bug on the Vikunja demo site?

Please select

Screenshots

No response

Originally created by @maggch97 on GitHub (Sep 20, 2025). ### Description <img width="474" height="145" alt="Image" src="https://github.com/user-attachments/assets/ad29774a-80b1-426d-b849-6c60a2712873" /> Enter is a keyboard shortcut to close the candidate words window for Chinese, Janpanse, Korean input methods. When the candidate window is displayed, Vikunja still captures the enter key. I will help later when I have time. ### Vikunja Version latest ### Browser and version _No response_ ### Can you reproduce the bug on the Vikunja demo site? Please select ### Screenshots _No response_
Author
Owner

@kolaente commented on GitHub (Sep 22, 2025):

Shouldn't the input method tool make sure that the input can't be captured further? Do you have the same problem on other websites as well, like GitHub?

@kolaente commented on GitHub (Sep 22, 2025): Shouldn't the input method tool make sure that the input can't be captured further? Do you have the same problem on other websites as well, like GitHub?
Author
Owner

@maggch97 commented on GitHub (Sep 22, 2025):

https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/isComposing

It can be fixed easily, just add isComposing check to all keydown.enter. I will have a PR soon

	if (e. isComposing) {
		return
	}
@maggch97 commented on GitHub (Sep 22, 2025): https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/isComposing It can be fixed easily, just add isComposing check to all keydown.enter. I will have a PR soon ``` if (e. isComposing) { return } ```
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#2360