Code review comment for lp:~widelands-dev/widelands/bug-1616661_messagebox_hotkeys

Revision history for this message
GunChleoc (gunchleoc) wrote :

I have found out what happens:

if (code.mod & KMOD_CTRL) { triggers once, toggling the button off or on.

if (code.mod & KMOD_ALT) { triggers twice, toggling the button off and on.

The next question to solve is why / how to circumvent the bug.

The intention is that the user can hit the same key twice in order to toggle the button on and off, rather than having to use ALT + 0 to toggle off, so I'd rather not reprogram the filter_messages function.

« Back to merge proposal