[Bugfix] Fix character encoding issue #1
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Trello
To fix this issue, I updated the code based on what was change in this PR for fixing the same issue.
To verify
https://ychen-hosted-envs.web.app/pdfjs-encoding/
Steps the above demo was created (and I hope they are correct):
src/pdfjs/src/src
& replace the files thereAPI change discussion link
No changes to API
UI/UX change discussion link
No changes to UIUX
Effect on pdf.js express
This fixes a pdfjs specific issue
Checklist
@method Core.AnnotationManager#exportAnnotations
npm run host-env
) and send it over Xodo to Carolyn Chen (@caroly) or Joshua Sun (@JoshuaSun) for review.Related issues
pdfjs express community topic link