Skip to content

appim: consider resetStateWhenFocusIn#375

Merged
eagleoflqj merged 1 commit into
masterfrom
appim
May 28, 2026
Merged

appim: consider resetStateWhenFocusIn#375
eagleoflqj merged 1 commit into
masterfrom
appim

Conversation

@eagleoflqj

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the macOS frontend focus-in behavior to respect Fcitx global state-reset policy, ensuring the expected default input method is activated when switching between input fields that create separate input contexts within the same app.

Changes:

  • On focus-in, also switch to the app’s default IM when resetStateWhenFocusIn is set to PropertyPropagatePolicy::All, even if the focused app hasn’t changed.
  • Expands the in-code comment to document the rationale and provide example apps (Chrome, iTerm) where this matters.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread macosfrontend/macosfrontend.cpp
@github-actions

Copy link
Copy Markdown

arm64 comparison

No difference.

x86_64 comparison

No difference.

@eagleoflqj eagleoflqj merged commit 730c0fd into master May 28, 2026
8 checks passed
@eagleoflqj eagleoflqj deleted the appim branch May 28, 2026 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants