Skip to content

fix: button release on blur#1101

Merged
imaginarny merged 2 commits into
kaplayjs:masterfrom
Stanko:release-keys-on-window-blur
Jun 25, 2026
Merged

fix: button release on blur#1101
imaginarny merged 2 commits into
kaplayjs:masterfrom
Stanko:release-keys-on-window-blur

Conversation

@Stanko

@Stanko Stanko commented May 31, 2026

Copy link
Copy Markdown
Contributor

Relates to #1094

I would like you folks to take a look before I convert it to a "real" pull request.

I added "release all" buttons for keyboard and mouse. These events can "drop" when focus leaves the canvas. On the other hand gamepad is polling for events so I made sure to skip those (if there are any).

@imaginarny imaginarny left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This was on todo so it would be great help again as well. 🙂

Will test some more once its updated and could merge it before #1106 so you have less work merging conflicts. Plus, I have one another in progress that could cause troubles as well, so would make sense to merge this one first.

Comment thread src/app/app.ts Outdated
Comment thread src/app/app.ts
@Stanko
Stanko force-pushed the release-keys-on-window-blur branch from 107a45d to 62d75eb Compare June 12, 2026 19:13
@imaginarny imaginarny linked an issue Jun 25, 2026 that may be closed by this pull request
2 tasks
@imaginarny

Copy link
Copy Markdown
Member

Would merge once you update the branch and changelog it.

@Stanko
Stanko force-pushed the release-keys-on-window-blur branch from 62d75eb to 7a6ef35 Compare June 25, 2026 12:44
@Stanko
Stanko marked this pull request as ready for review June 25, 2026 12:44
@Stanko

Stanko commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Done and done 🙌

@imaginarny imaginarny left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you. 👍

@imaginarny
imaginarny merged commit 61a429b into kaplayjs:master Jun 25, 2026
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.

bug: isKeyDown and isButtonDown get stuck when cmd-tabbing on mac

3 participants