Skip to content

fix(WeaselTSF): restore UIElement updates in non-UILess mode - #1951

Merged
fxliang merged 1 commit into
rime:masterfrom
cary-rowen:fix/tsf-uielement-updates
Oct 11, 2026
Merged

fxliang merged 1 commit into
rime:masterfrom
cary-rowen:fix/tsf-uielement-updates

Conversation

@cary-rowen

@cary-rowen cary-rowen commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

After #1927, UIElement updates are skipped when the normal candidate window is displayed. This regresses the behavior introduced by #1801 and prevents screen readers such as NVDA from receiving candidate updates.

_pbShow describes the candidate window display state and is not a reliable indicator of whether the TSF UIElement is available. Use _uiStarted to guard _UpdateUIElement() instead, so updates are sent in both UILess and normal display modes while the UIElement lifecycle is active.

@cary-rowen

Copy link
Copy Markdown
Contributor Author

cc @fxliang
Could you take a look at this? Let me know if any further action is required.

@cysworld

Copy link
Copy Markdown

Just tested weasel-artifact-0.17.4.81.9f3dd58 with NVDA, and candidate announcements are working great on my end now! 🎉
Really glad to see this getting fixed. Thanks for the work, and hope this gets merged soon! 👍

@frank-fuyun

Copy link
Copy Markdown

这是个不错的输入法,但愿能早日合并这个修复。

@fxliang
fxliang merged commit 9e3047c into rime:master Oct 11, 2026
3 checks passed
@cary-rowen
cary-rowen deleted the fix/tsf-uielement-updates branch October 15, 2026 22:05
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.

4 participants