adds no_english_mode #1
Conversation
|
Thanks for your contribution! Looks good to me. However I suggest not adding the The reason is, other configuration items do not have such command. The |
|
Ok so you want to remove In my opinion this option is aggresive and may cause breaks or malfunctions in some applications, depends on app runtimes, focus capturing method even Windows version. You can have a try on a virtualize machine or a remote desktop with agent which usually contained keyboard/mouse key escaping and you will know my concerning immediately. In this case the option could provide a single shot while starting so you dont need to edit and reload config, simple but working. I would like to provide a commit to apply the change if you keep your ideas still. |
|
I'm not quite following you cuz the |
|
Noted. Which proposal you prefer:
|
|
I'd suggest picking one of these two options you prefer:
|
Adds No English Mode, some impls from https://github.com/mbbill/no_english_mode
Add no_en as a config option and --no-en <enable|disable> as a CLI entry, enabled by default.
Add a focus hook to handle window/control focus changes.
Add a mouse-up fallback to cover cases where Windows switches back to English without a clear focus event.
Add simple request coalescing and foreground-window checks to reduce stale or incorrect IME switching, especially around Alt+Tab.
No significant performance impact or potential bugs have been confirmed yet.