1. 19 Oct, 2013 3 commits
  2. 18 Oct, 2013 3 commits
  3. 17 Oct, 2013 1 commit
  4. 14 Oct, 2013 1 commit
  5. 10 Oct, 2013 10 commits
  6. 09 Oct, 2013 15 commits
  7. 08 Oct, 2013 2 commits
  8. 05 Oct, 2013 1 commit
    • csillag's avatar
      Change comments implementation to avoid fake highlights · 983b6719
      csillag authored
      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.
      983b6719
  9. 03 Oct, 2013 3 commits
  10. 02 Oct, 2013 1 commit
    • csillag's avatar
      Bridge: don't skip old data in the 'loadAnnotations' notifications · 9fd9abe4
      csillag authored
      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.
      9fd9abe4