Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update pdf.js#183

Open
AVert wants to merge 47 commits into
modesty:mastermodesty/pdf2json:masterfrom
AVert:patch-1AVert/pdf2json:patch-1Copy head branch name to clipboard
Open

Update pdf.js#183
AVert wants to merge 47 commits into
modesty:mastermodesty/pdf2json:masterfrom
AVert:patch-1AVert/pdf2json:patch-1Copy head branch name to clipboard

Conversation

@AVert
Copy link
Copy Markdown

@AVert AVert commented Apr 13, 2019

can we use not hardcoded values?

modestysn and others added 30 commits February 28, 2016 13:45
…level pdf2json file; also updated readme with more code example
… renderer needs update. See readme for more details
[fix] Cannot find module './pdf2json/PDFParser'
Added parameters this and 1 to PDFParser in readme
Use .npmignore file to exclude "test" folder from NPM (Saves 150 MB)
@MatthewDLudwig
Copy link
Copy Markdown

The whole pdf.js library is in need of updating. It still treats names that are too long as an error whereas pdf.js updated to treat it as a warning some time ago:

mozilla/pdf.js#6151

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.

Morty Proxy This is a proxified and sanitized view of the page, visit original site.