Skip to content

C
coopwire-hypothesis
Commit 5dab3c36 authored by Randall Leeds's avatar Randall Leeds
Browse files
Options

Merge pull request #1753 from hypothesis/better-pdf-test 

Consolidate code paths around deciding about PDF.js
parents 9aac31c2 4977bfa9
Hide whitespace changes
Inline Side-by-side
Showing with 8 additions and 4 deletions
h/static/scripts/guest.coffee
View file @ 5dab3c36
...@@ -22,8 +22,6 @@ class Annotator.Guest extends Annotator ...@@ -22,8 +22,6 @@ class Annotator.Guest extends Annotator
TextPosition: {} TextPosition: {}
TextQuote: {} TextQuote: {}
FuzzyTextAnchors: {} FuzzyTextAnchors: {}
PDF: {}
Document: {}
# Internal state # Internal state
tool: 'comment' tool: 'comment'
...@@ -37,8 +35,14 @@ class Annotator.Guest extends Annotator ...@@ -37,8 +35,14 @@ class Annotator.Guest extends Annotator
super super
delete @options.noScan delete @options.noScan
if window.PDFJS # Are going to be able to use the PDF plugin here?
delete @options.Document if window.PDFTextMapper?.applicable()
# If we can, let's load the PDF plugin.
@options.PDF = {}
else
# If we can't use the PDF plugin,
# let's load the Document plugin instead.
@options.Document = {}
@frame = $('<div></div>') @frame = $('<div></div>')
.appendTo(@wrapper) .appendTo(@wrapper)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment