CleverRaven / Cataclysm-DDA

Cataclysm - Dark Days Ahead. A turn-based survival game set in a post-apocalyptic world.
http://cataclysmdda.org
Other
10.29k stars 4.12k forks source link

CBM and sort can share keybind #70186

Closed Rabadash94 closed 7 months ago

Rabadash94 commented 9 months ago

Describe the bug

When I was trying to activate the integrated dosimeter I noticed that a sort menu was displayed instead because both things were sharing keybind (s)

Attach save file

N/A

Steps to reproduce

  1. Install enought CBMs in your body until s is used for keybind some CBM
  2. Try to activate the CBM that uses s
  3. See the sort menu

Expected behavior

Keys already used for predefined functions should not be assigned to anything else in the same window.

Screenshots

Captura de pantalla de 2023-12-13 10-39-59

Versions and configuration

Additional context

I see the keys can be reassigned, but it's still a bug. I also see that passive CBMs are also assigned a key. Isn't that a bug?

github-actions[bot] commented 8 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.