Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes "Unbound" keybind slots sometimes not being removed #5720

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

SabreML
Copy link
Member

@SabreML SabreML commented Feb 14, 2024

About the pull request

Fixes keybinds sometimes getting 'Unbound' slots permanently stuck to them.
This was caused by the "clear_keybind" act code adding the keybind to the "Unbound" list for every key assigned to the keybind, rather than just once.
(I can't really explain this very well but there's an example video below)

Explain why it's good for the game

It doesn't actually change the keybind's behaviour at all, but it's a bit weird to have an extra unremovable "Unbound" slot on some keybinds.

Testing Photographs and Procedure

Screenshots & Videos

Before:

TGBkF4aH6N.mp4

After:

oPBBrraydL.mp4

Changelog

🆑
fix: Fixed the 'Clear' button not resetting the number of keybind assignment slots. (Keybinds menu)
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Feb 14, 2024
@harryob harryob added this pull request to the merge queue Feb 15, 2024
Merged via the queue into cmss13-devs:master with commit d33071d Feb 15, 2024
28 checks passed
cm13-github added a commit that referenced this pull request Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants