Yeh, adding event in a directive is bad... although I thought that it will react to tap events as well, need to test.

What worries me more is is the fact that the class name (`active`) is hard-coded here.
I like the idea of this directive less and less :-)

Maybe we really should just do a generic `class-toggle` and be done with it? WDYT?