Skip to content

fix(macOS) "fix" toggling IME on appkit backend#4512

Merged
madsmtm merged 1 commit intorust-windowing:masterfrom
Random-Scientist:master
Mar 14, 2026
Merged

fix(macOS) "fix" toggling IME on appkit backend#4512
madsmtm merged 1 commit intorust-windowing:masterfrom
Random-Scientist:master

Conversation

@Random-Scientist
Copy link
Copy Markdown
Contributor

@Random-Scientist Random-Scientist commented Mar 13, 2026

  • Tested on all platforms changed
  • Added an entry to the changelog module if knowledge of this change could be valuable to users
  • [N/A] Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • [N/A] Created or updated an example program if it would help users understand this functionality

Fixes #4432, although there are still countless other bugs in the implementation. For context, I'm planning on experimenting with a rewrite of the appkit IME backend, based loosely on Chromium's workarounds, and this makes the internal api a bit nicer for that. It's also an easy "harm reduction" fix for a simple bug (unlike the rest of the NSTextInputClient related issues).

@Random-Scientist Random-Scientist marked this pull request as ready for review March 13, 2026 23:48
@madsmtm madsmtm added DS - appkit Affects the AppKit/macOS backend B - bug Dang, that shouldn't have happened labels Mar 14, 2026
Copy link
Copy Markdown
Member

@madsmtm madsmtm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I'd probably be fine with trimming things further here, a lot of these things really should be unnecessary and I'm not as worried about breaking enable/disable as I am about the rest, but also fine with keeping it for now.

@madsmtm madsmtm merged commit 0b6b794 into rust-windowing:master Mar 14, 2026
57 checks passed
junglie85 pushed a commit to junglie85/winit that referenced this pull request Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B - bug Dang, that shouldn't have happened DS - appkit Affects the AppKit/macOS backend

Development

Successfully merging this pull request may close these issues.

[0.31-beta.2] Disabling IME is broken on macOS

2 participants