1. 26 Feb, 2015 4 commits
    • Nick Stenning's avatar
      Bump the browserNoActivityTimeout · 5d842b61
      Nick Stenning authored
      Travis is being especially slow at the moment.
      5d842b61
    • Nick Stenning's avatar
      Browserify the test harness · 253e4334
      Nick Stenning authored
      Use karma-browserify to run the tests, rather than relying on prebuilt
      assets. This commit alters our test harness so that it no longer relies
      (at all) on the webassets pipeline. This means:
      
      - test isolation is improved, as each test file runs in its own module
      - bringing objects into test scope is now simply a matter of requiring
        the correct module, rather than bolting more and more objects onto the
        Annotator namespace
      - autoreloading of tests when either test files or their dependencies
        are modified is now possible (and enabled by default)
      253e4334
    • Nick Stenning's avatar
      ba6280b3
    • Nick Stenning's avatar
      Add browserify config to package.json · a6aec003
      Nick Stenning authored
      This includes files that need to be shimmed under the "browserify-shim"
      key. Annotator plugins that have no dependencies do not need to be
      shimmed at the moment as they just attach themselves to the global
      Annotator object.
      
      jQuery is assigned as a global, this means that it is assumed to be
      included separately and bound to the global scope.
      
      Both jQuery and Annotator have aliases under the "browsers" key to
      make them easier to reference.
      a6aec003
  2. 25 Feb, 2015 2 commits
    • csillag's avatar
      Simply PDF text extraction · 27da8253
      csillag authored
      Removing some obsolete conditional code,
      which was necessary for some obsolete version
      of PDF.js.
      
      Fortunately, by now, all supported versions
      of PDF.js uses the same data format, so this
      workaround can go.
      27da8253
    • csillag's avatar
      Simply PDF text extraction · f42c3656
      csillag authored
      Earlier, on some versions of PDF.js, we used to
      use the PDF.js's FindController as a data source
      for the text extraction.
      
      However, at some point, we stopped using the
      routines shipped with it, since it didn't always
      provide use adequate spacing between the various
      pieces of texts. So we ended up just contenating
      the various pieces of text ourselves.
      
      Then, for new versions of PDF.js, we introduced
      other means of accessing the same information,
      completely bypassing the PDFFindController.
      
      This change simply unifies the access; now we can
      do the same an all PDF.js versions.
      f42c3656
  3. 24 Feb, 2015 2 commits
  4. 23 Feb, 2015 3 commits
  5. 22 Feb, 2015 3 commits
    • gergely-ujvari's avatar
      Merge pull request #1949 from hypothesis/1928-fix-realtime-delete · e009194e
      gergely-ujvari authored
      Fix real-time delete
      e009194e
    • csillag's avatar
      Adding tests for applyUpdate · 56de2455
      csillag authored
      56de2455
    • csillag's avatar
      Fix real-time delete · 5b488a77
      csillag authored
      When receiving a real-time notification about the removal
      of an annotation, we emit an annotationDeleted event,
      so that all components can react.
      
      Earlier, we emited this event simply with the Annotation
      bject that arrived on the wire.
      
      However, some components couldn't deal with this, because
      they were expecting to see the 'real' Annotation object,
      the ones they already knew about.
      
      So now we do a lookup before emiting the event, and use
      the locally found objects instead.
      5b488a77
  6. 20 Feb, 2015 6 commits
  7. 18 Feb, 2015 1 commit
    • csillag's avatar
      Change the scrolling API · 06152dca
      csillag authored
      The goal of this change is to avoid any false appearance
      that we are implementing the proposed scrollIntoView() API [1],
      while we clearly don't.
      
      So, changes:
      
       - Rename scrollIntoView to scrollToView
         on both Anchor and Highlight
       - Stop returning promises at both methods
      
      I also updated all the tests to reflect this change.
      
      [1] http://dev.w3.org/csswg/cssom-view/#dom-element-scrollintoview
      06152dca
  8. 17 Feb, 2015 1 commit
  9. 16 Feb, 2015 4 commits
  10. 14 Feb, 2015 2 commits
  11. 13 Feb, 2015 3 commits
  12. 12 Feb, 2015 3 commits
  13. 11 Feb, 2015 6 commits