AngularJS refactored, added unit tests #4

Open
wants to merge 10 commits into
from

Conversation

Projects
None yet
3 participants
Collaborator

vojtajina commented May 16, 2012

No description provided.

This is really weird since w-up actually binds lots of keyboard handling that's not "up". I'm not sure this follows best practice you'd want in a demo app. Feels like a hack.

Owner

vojtajina replied May 16, 2012

Yep, I know :-D

I could either add another directive (say w-keydown) or have five separate directives (but then I would have to register five keydown listeners, which I don't wanna). So I'm changing to first one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment