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

Fix PDF.js.js #90

Merged
merged 2 commits into from Oct 28, 2015
Merged

Fix PDF.js.js #90

merged 2 commits into from Oct 28, 2015

Conversation

nobita4176
Copy link
Contributor

だいぶ前からPDF.js.jsが動作していなかった(TypeError: content.window.wrappedJSObject.PDFView is undefinedが表示されていた)ため,修正しました.
なお,:pdfjs indexのcompletionは未だ動作していませんが,正常動作がわからないので修正していません.

@Jagua
Copy link
Member

Jagua commented Oct 28, 2015

content.window.wrappedJSObject.PDFView から content.wrappedJSObject.PDFViewerApplication への変更で .window を取り除いたのは何か意図ありますか?
特に理由がなければ修正お願いできますか?

@nobita4176
Copy link
Contributor Author

特に意図はありませんでしたので,.windowを戻しました.

Jagua added a commit that referenced this pull request Oct 28, 2015
@Jagua Jagua merged commit ec7669c into vimpr:master Oct 28, 2015
@nobita4176 nobita4176 deleted the pdfjsjs branch October 28, 2015 16:10
@Jagua
Copy link
Member

Jagua commented Oct 28, 2015

Thank you.

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

Successfully merging this pull request may close these issues.

None yet

2 participants