This was a CoffeeScript-generated variable.

This was a CoffeeScript-generated variable.

These were generated by CoffeeScript (due to its implicit returns) but
don't actually appear to be necessary.

Minimal changes to the CoffeeScript-generated annotation-sync.js to
stop `make lint` complaining about it, while still having `make test`
pass.

annotation-sync.js is the unmodified output of
`coffee -bc src/annotator/annotation-sync.coffee`.

Fix Mobile Safari Scrolling

* master:
  Add tests for sidebar closing when user clicks or taps in document
  Fixing tapping anchored annotations not opening sidebar
  Fixing sidebar not closing on outside content click issue
  eslint: Ignore docs/_build
  Update ESLint to v3.18.0

Since the `<thread-list>` is now contained within a scrollable element
which is not `document.body`, it needs to scroll that element rather
than the window in order to scroll the element into view.

 * Remove the hard-coded selector used to find the scroll root and
   instead search for the closest ancestor element to the
   `<thread-list>` whose computed style has `overflowY` set to `scroll`.
 * Set the scroll root Element's `scrollTop` to scroll the thread list
   instead of calling `window.scroll`
 * Compute the maximum scroll offset based on the `scrollTop` and
   `scrollHeight` properties of the scroll root element instead of
   using `document.body.scrollHeight`.
 * Update tests for `<thread-list>` to check for the scrollable
   ancestor being scrolled instead of the window.

Fixing sidebar not closing on outside content click issue

Fixing tapping anchored annotations not opening sidebar

eslint: Ignore docs/_build

Update ESLint to v3.18.0

 - Update ESLint from v3.2.0 to v3.18.0
 - Add a little extra configuration for the "indent" rule which best
   matches our existing practice in the codebase
 - Fix a few places that were inconsistent about indentation

Convert `<top-bar>` to a component

Add tests that callbacks and properties are passed down correctly to
child components.

`frame` is never assigned anywhere and `ng-cloak` is unnecessary since
the HTML for the `<top-bar>` component is not present in the page when
it loads but is added by Angular after the app has booted.

Fix incorrect additions of 'vm.' prefix that were added when converting
`<top-bar>` to a component and a missing 'vm.' prefix in the call to log
out.

Convert `<hypothesis-app>` to a component

Rename the modules and templates for the `<hypothesis-app>` component to
match the component name, following our conventions.