Interrupt / delverengine

Delver game engine and editor
zlib License
803 stars 81 forks source link

Replace T-key rotation in the editor with ActionListeners #111

Closed evrimoztamur closed 4 years ago

evrimoztamur commented 4 years ago

With regards to #96, the current behaviour was defined in the tick() function of the EditorApplication which made it ignore the input and focus checks.

This PR moves the behaviour to MenuItems and assigns new commands, Ctrl + Left and Ctrl + Right for rotation, CW and CCW respectively (as the original T key binding is reserved for texture rotation actions).

evrimoztamur commented 4 years ago

Although the changed key binding is not optimal, the alternative to this would be hacking into the EditorInput and InputProcessor logic and skip the MenuAccelerator usage. That's also viable, of course, but would result in hardcoded overridden behaviour.

Let me know what you think on the remapping. If undesirable to remap, we can bypass in EditorInput.