Instead of using `scope` here, you could use `controller` if you change the controller definition. (adding the controller functions to the `this`object instead of the `scope` object. It might be confusing, because scope is implicitely shared between controller and scope.

In fact, the link function gets a controller injected, which is never used. But removing it would break the directive's functionality, which might look strange.

Shouldn't hold up a merge though...