supertuxkart-sourceforge-migration / stk-migration-test2

0 stars 0 forks source link

Detect conflicts between keyboard configs #39

Open supertuxkart-sourceforge-migration opened 10 years ago

supertuxkart-sourceforge-migration commented 10 years ago

Author: hikerstk

Migrated-From: https://sourceforge.net/apps/trac/supertuxkart/ticket/39

supertuxkart-sourceforge-migration commented 10 years ago

Author: hikerstk Deleting configurations added as of r5929. However there currently is no confirmation dialog shown when asking to delete, this should probably be added

Confirm dialog added as of r5931.

As such, lowering the priority of this bug since the important part of it was implemented.

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria Renaming because topic (1) is already done, and point (3) is handled in a separate ticket

supertuxkart-sourceforge-migration commented 10 years ago

Author: hikerstk I think that ticket can be closed assuming that a clash of keyboard configuration does not prevent a single user to use the keyboard. E.g. if someone has two keyboards set up, later changes the keybinding for the first keyboard (so that they clash with the 2nd), it should still be possible to play in single player mode. When more than one person is player, they will obviously detect that they try to press the same key :)

Cheers, Joerg

supertuxkart-sourceforge-migration commented 10 years ago

Author: hikerstk A discussion on irc ended with the idea of adding two different warnings: 1) using the same fire key in different configurations This can cause problems in split-screen (only), since the game might assign the wrong input configuration for a player. 2) use the same key for different actions in different configs This can cause a problem in single player (which uses all input configs at the same time), since it's not obvious which of the various actions are supposed to be used. I can't see this particularly useful in split screen either ... since two people would have to press the same key ;)

supertuxkart-sourceforge-migration commented 10 years ago

Author: red-skull Fixed in r9656.