- Wait until we have a token to finish on-going edits
- Make sure to reset the on-going highlight switch
- No need to cancel on route change now that draft discarding does that

Prevents losing track of drafts when switching from highlighter to
comment tool if replies have been started in the sidebar.

Refactored the way changing the tool triggers setting visible highlights.
The change should always ensure better interaction if other components
change the tool or highlight state in the future and trigger events.

For #798

For #798

This change may affect #803 and #805.

I believe this is no longer necessary since we started using
`event.preventDefault` more consistently in annotation actions.

That's good, because this timeout stuff was brittle.

Previously, we were adding fake highlights to comments, which
showed up on the last character of the document, when in always-on
or highlight mode.

This was needed so that the heatmap can find the comments.

The new implementatain is much cleaner; now we maintain a list of comments
independently of the DOM, and the heatmap works from that.

Therefore, the fake highlights are now gone.

Fixes #700.

Update INSTALL.OSX.rst

Bridge: don't skip old data in the 'loadAnnotations' notifications

Previously the fuzzyTime filter showed fuzzy value for dates no older than 30 days. After that it just displayed an unformatted new Date(date) which looks strange compared to the other displayed time strings.
Now it has been extended to show x. month ago and y years ago.

Fixes #799

Earlier when a 'loadAnnotations' notification was sent on the bridge plugin,
 * Only locally created annotations were included; annotations with
   existing bridge tags were never sent. This was true even if the
   'annotationsLoaded` event (which we are reacting to) contained such
   annotations.
 * On the receiving end, existing annotations were dropped without being
   parsed.

This made it impossible to propagate data via this notification.
However, when we do the actual anchoring in the host document, we are
trying to use this notification to spread the data which become available
as the result of the anchoring; for example, the visual diffs.

To fix this, I have made the following changes:
 * All annotations (coming from the 'annotationsLoaded' event are sent
   with the notifications
 * On the receiving end, all annotations are parsed, so that the updates
   in various fields can be merged in. Of course the already existing
   annotations are not included in the subsequent `loadAnnotations()` call.

This fixes #807.

Fixes #790

- Avoid double scan at boot by passing the intended option
- Don't call domMatcher.scan directly; call the appropriate wrapper on Annotator

When trying to reply without signing in, show warning.

- When entering page search, disable dynamic bucket mode
- When leaveing page search, enable dynamic bucket mode

Fixes #790.

This reverts commit 32f307074f8be09dd1fd853dac1afa432076dfeb.

Fixes #794.
Reopens #790.

Fixes #788

Closes #685

Fixes #790

Fixes #791

added info for setting the IP in development.ini

This is a workaround, until we can get the forced
log-in workflow properly working for these cases.

Closes #364.

Support for annotating multiple frames

Closes #779

This code was moved from the host to the guest but I forgot to remove
it from the host.

Regression.