1. 23 Jun, 2016 1 commit
    • Nick Stenning's avatar
      Fix missing PDF highlights · 8f4cc121
      Nick Stenning authored
      fda63b4 unfortunately broke highlighting in PDFs, because the PDF
      anchoring code first finds the text in the document, and then uses a
      TextPositionSelector and the HTML anchoring code to select a range in
      the rendered page for highlighting.
      
      This commit fixes that issue by moving the TextQuoteSelector check up a
      level into `Guest#anchor`.
      
      I've added two tests for this in `guest-test.coffee` -- one tests that
      annotations where the target contains no TextQuoteSelector are marked as
      orphans, and the second tests that the low-level anchoring code is never
      called in this case.
      
      I have also removed the integration test, on the basis that I don't
      think integration testing failure cases is a good habit to get into.
      
      Fixes #3530.
      8f4cc121
  2. 21 Jun, 2016 7 commits
  3. 20 Jun, 2016 9 commits
  4. 17 Jun, 2016 9 commits
  5. 16 Jun, 2016 6 commits
  6. 15 Jun, 2016 8 commits