threerings / playn

Legacy 1.x version of PlayN library.
http://playn.io/
Apache License 2.0
195 stars 66 forks source link

Added modifiers to Keyboard Events #81

Closed BenDol closed 9 years ago

BenDol commented 9 years ago

To resolve: #80

Keyboard.TypedEvent doesn't detect the modifiers at this stage (probably doesn't make sense anyway). Haven't written tests for this yet, but can verify it works in the JavaPlatform and HTMLPlatform so far. Let me know if there is a preferred way to do this, I just wanted this functionality so decided to implement it and provide it as PR encase it is helpful.

The pom files seem to be set to 1.10-SNAPSHOT for some reason, not sure if we want to update the version so the tests don't complain. I did it locally, but won't commit that since you guys will probably take care of it. Or maybe I've set it up wrong.

GwtTestHtmlJson.java test class can't see the core/tests/shared/json package for some reason.