This was needed when this part of the app template lived in the `h`
repository. It is not needed any more.

Move the template which defines the structure of the top-level of the UI
from `app.html.jinja2` in the hypothesis/h repo into the client.

This means that the H service only needs to serve an app.html page
containing a `<hypothesis-app></hypothesis-app>` placeholder in the body
which the app will then load into.

Remove SASS variables and mixins which are not used by the client

ESLint - Enforce indentation, quote style and `this` style consistency

Since this module was imported with minor changes from
substack/frame-rpc, disable ESLint to avoid adding noise to the Git
history for this file.

See https://github.com/hypothesis/client/pull/32#discussion_r70957311

Use 2-space indents everywhere and fix up non-conforming code with
ESLint's --fix tool.

Only allow use of `this` or `self` to refer to the 'this' context
object in a method.

Add an ESLint rule to require this and fix up non-conforming code using
ESLint's --fix tool.

Refactor the streamer into a stand alone class

The group invite icon is only used in the H web service

* The corejs polyfills are not needed in the Gulpfile since we switched
   to a more recent version of Node

 * The Object.assign() polyfill does not need to be included in 'raven'
   or 'settings' because it is already included by the polyfills.js
   script

This was left over from when the Hypothesis client was extracted out
from the hypothesis/h repository, where the extension code currently
remains.

Enforce that all paths through a function either return a value
or do not return a value.

This cause several cases of spurious returns left in by CoffeeScript ->
JS conversion and return statements with missing values.

Fix 'time' tests in timezones behind UTC

Several test fixtures have expectations that the "current date", where
"current" is reset to the Unix epoch at the start of the tests, is 1st
Jan 1970. This is true in timezones equal to or ahead of UTC but not the
case for timezones behind UTC.

Resolve the issue by just advancing the fake clock forwards to midnight
on 01/01/1970 if the timezone is behind UTC at the start of the tests.

Annotation / Notes tabs copy update and style tweaks

Update UI state after search API request completes with no results

Remove sticky tab bar behavior

When a search API request returns results, the returned annotations are
added to the app state and this transition triggers a digest cycle.

When there are no results however, this was not happening and the value
of the 'isLoading()' function changed but no digest cycle was triggered
to pick this up and reflect it in the UI.

Since SearchClient's 'end' event is emitted asynchronously via a Promise
callback, $scope.$apply() or $evalAsync() must be used to ensure that
change detection runs.

The real solution will be to store all the information required to
determine the loading/not-loading state in the Redux store, so that any
transition automatically triggers a digest cycle. This requires some
more extensive refactoring however.

The sticky tab bar behavior has some UI issues that need to be resolved
before we could ship it. For example, when switching between tabs the
scroll position needs to be reset to the top.

For the time being, this commit removes the behavior.

Following feedback and design review, change the label from 'Notes' to
'Page Notes' to better distinguish notes from annotations and make the
empty state text for the Annotations tab more helpful by explaining to
users how to create a new annotation.

Shift the 'No annotations in this group' message down slightly to
prevent the border from being clipped.

Also prevent text selection in the tabs.

ESLint - Enable 'Best practices' rules

Usage of Hound has been replaced by a Travis task that runs `npm run
lint`

Enable additional rules from the 'Best Practices' list on ESLint's Rules
page.

Use `.delete` instead of `['delete']` etc. since all browsers we target
support ES5.

Add ESLint configuration and use it to replace JSCS / JSHint

These have been replaced by ESLint

For stylistic consistency, require semicolons rather than relying on
ASI.

Checking for return values in Array callbacks found several incorrect
uses of Array.map() where Array.forEach() should have been used.

This will aid migration to let/const in future