Move scope.feature() from the annotation directive to vm.feature() in
the AnnotationController. Just trying to move as much as possible out of
the directive's scope and link function into the controller, if we're
gonna use a directive controller here try to just use it instead of
randomly using the link function and scope for some things and the
controller for others.

It doesn't appear to be possible for model to be null.

Move the DOM event listener function that saves an annotation on
Ctrl+Enter into AnnotationController. This is just for the sake of
keeping all the code in one place (in AnnotationController, not in the
directive's link function) and putting code where it can be tested more
easily.

Note that the controller's save() function was previously called with
scope.$evalAsync() and we're now just calling it synchronously. This
seems to be fine.

Remove AnnotationController's vm.preview as it doesn't appear to be
used.

Replace AnnotationController's vm.editing boolean with a vm.editing()
method.

This removes some duplicated state, as both vm.editing and vm.action
were recording whether or not the annotation was being edited.

Move scope.share() from the annotation directive to vm.share() in
the AnnotationController. Just trying to move as much as possible out of
the directive's scope and link function into the controller, if we're
gonna use a directive controller here try to just use it instead of
randomly using the link function and scope for some things and the
controller for others.

It does not change after directive linking, there's no reason to
$observe() it.

Translate annotation.coffee to JavaScript

Fixes a bug that crept in during the annotation.coffee -> annotation.js
translation.

Declare variables on the same line where they're first defined, instead
of at the top of the function.

This is a translation of commit 363134d28cb7ff8d2313278e1b4b6c09b5bf9bed
from CoffeeScript to JavaScript. Original commit message from
363134d28cb7ff8d2313278e1b4b6c09b5bf9bed:

When saving a draft to the drafts store, don't save changes for fields
which aren't actually set. This prevents us from accidentally later
restoring `null` to one of these fields.

Fixes an issue where creating an annotation when logged-out would result
in a null permissions field.

These were lost when translating annotation-test.coffee ->
annotation-test.js using the `coffee` command.

Tidy up the CoffeeScript-generated code in annotation-test.js.

These were lost when translate annotation.coffee to annotation.js using
the `coffee` command.

Tidy up the CoffeeScript-generated code in annotation.js.

annotation-test.js is just the output from `coffee -c ...` untouched.

directive/annotation.js is just the output from `coffee -c ...`,
untouched.

This is horrible, but while we're migrating this page to the app, this
is the quickest and easiest fix for the homepage typeface.

Remove Font Awesome and move add missing social media icons to H icon font

Font Awesome was used for the 4 icons on the homepage

 - Twitter, GitHub, RSS feed in the footer
 - 'Drag' icon in the bookmarklet installation instructions

Reduce page weight and simplify our lives by having all the icons
in a single icon font.

Validate incoming annotation data with jsonschema

Handle missing permissions on annotations more gracefully

Don't save null permissions/tags/text to drafts store