Skip to content

fix: app crashes on linux - don't set cursor-ignore state before the window is realized - #147

Merged
karansinghgit merged 1 commit into
karansinghgit:mainfrom
badmuffin:fix/pill-cursor-linux-crash
Sep 22, 2026
Merged

karansinghgit merged 1 commit into
karansinghgit:mainfrom
badmuffin:fix/pill-cursor-linux-crash

Conversation

@badmuffin

@badmuffin badmuffin commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

this pr is to fix the issue #140

@badmuffin badmuffin changed the title fix: don't set cursor-ignore state before the window is realized fix: app crashes on linux - don't set cursor-ignore state before the window is realized Sep 19, 2026
@karansinghgit
karansinghgit merged commit e48ce73 into karansinghgit:main Sep 22, 2026
@karansinghgit

Copy link
Copy Markdown
Owner

Merged, thank you. The guard in set_interactive is the right shape: the pill's cursor state only matters while it's on screen, and show() reapplies it every time the pill appears, so nothing is lost by skipping it while hidden.

Verified locally on macOS: format, clippy and the 76 tests all pass. Your Fedora test covers the platform this actually affects.

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