Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Directly override the pdfJsViewer plugin version of pdf.js #10

Open
ctgraham opened this issue Mar 5, 2020 · 3 comments
Open

Directly override the pdfJsViewer plugin version of pdf.js #10

ctgraham opened this issue Mar 5, 2020 · 3 comments

Comments

@ctgraham
Copy link
Contributor

ctgraham commented Mar 5, 2020

Nobody wants to:

To apply it...

cd plugins/generic/pdfJsViewer/pdf.js
patch -p2 < /path/to/ab3f25.diff

ctgraham added a commit to ulsdevteam/hypothesis that referenced this issue Mar 5, 2020
@asmecher
Copy link
Owner

asmecher commented Mar 5, 2020

There are different instructions in the master branch README, since Hypothes.is recently updated their package and no longer have a single commit that'll turn a non-Hypothes.is-enabled pdf.js into a Hypothes.is-enabled one. But this part of the issue still applies:

Nobody wants to

However, I'm not sure about just shipping this plugin with the hypothes.is alterations included, as it would seem that the Hypothes.is plugin should do that. On the flip side, it's not clear to me how to allow the Hypothes.is plugin to intervene in this plugin's behaviour gracefully. I'm open to ideas.

@asmecher
Copy link
Owner

asmecher commented Mar 5, 2020

@ctgraham answers my question at the moment I finish asking it :) https://github.com/asmecher/hypothesis/pull/11/files

asmecher added a commit that referenced this issue Mar 5, 2020
#10: Modify delivery of pdfJsViewer template to use hypothes.is version of pdf.js
@ctgraham
Copy link
Contributor Author

ctgraham commented Mar 5, 2020

@asmecher, are you also cherry-picking to master, or do you want to get the PR I've almost formulated?

Note that I have not tested this against 3.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants