| Summary: | Enter doesn't wait the game out in Killbots | ||
|---|---|---|---|
| Product: | [Applications] killbots | Reporter: | David Benjamin <davidben> |
| Component: | general | Assignee: | Parker Coates <coates> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | kde-games-bugs |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
|
Description
David Benjamin
2008-12-11 01:01:36 UTC
Did you try both the main enter key and the numeric keypad enter key? This might be a case of Qt treating them differently. I'm out of the country and away from my dev machine, so I can't confirm this behaviour at this time. My intention was to have two sets of shortcuts: one using only the keypad (the primary set) and one using only letter keys (the alternate set), but if this is going to cause confusion over the enter keys, then maybe I should be rethinking that. (I'm on a laptop and don't actually have a numpad enter key, but Fn+Enter seems to send the right keycode.) Using the "numpad"'s enter does indeed work. Hmm, here if i press numeric Enter it enters an infinite waiting state until i'm killed, it's that the intended behaviour? @Albert Yes, that's the intended behaviour, except that you're supposed to be skilled enough to not die. ;) The idea is that you get the hero into a situation where you can complete the round without having to move. You get bonus points and/or energy for each enemy destroyed while that happens, so the goal is to get yourself into such a position as soon as possible. Ok, then i think you can close the bug, unless you want it to work with return too. I'm going to leave it open until I've had a chance to play with Qt and KShortcutsDialog's handling of the enter/return keys. The shortcuts certainly won't change before 4.2. Okay. So Qt (and presumably X) treats the return key (above the left shift) and the enter (on the numpad) as entirely different entities. The problem is that most modern keyboards label both keys as 'Enter'. The only way I see to clear this up is to change the shortcut. SVN commit 896497 by coates: Change the primary shortcut for 'Wait Out Round' to '*' to avoid confusion between around the naming of the 'enter' and 'return' keys. The enter key was a bit to easy to hit for such a dangerous action anyway. Hopefully I don't yet have enough users for this change to bother anyone. BUG: 177449 M +13 -13 mainwindow.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=896497 |