FAForever / fa

Lua code for FAF
221 stars 228 forks source link

Clarify hotbuild keybind collision warning #6217

Closed lL1l1 closed 1 month ago

lL1l1 commented 1 month ago

Issue

Hotbuild automatically assigns shift and alt modifiers for its hotbuild actions, and gives warnings if another action is already bound to that key. The warning is hard to understand because it didn't specify what hotbuild key the autogenerated modifier came from.

WARNING: Hotbuild key Alt-H is already bound to action "Toggle intel and shield" under "ORDERS" category

Description of the proposed changes

Add the other side of the key collision to the warning, and the resolution of the collision.

WARNING: Alt modifier for Hotbuild action "build shields + mobiles Shields & shield disruptors + shield boat" (H)  is already bound to action "Toggle intel and shield" (Alt-H)
WARNING: The Alt modifier of the Hotbuild action will not work!

Testing done on the proposed changes

Checked for errors in log when closing the keybindings menu (this is when the keybindings are auto generated).

Checklist