Skip to content

Refactor PDFJS global var usage and move it initialization into single file #6779

Description

@yurydelendik

for v1.x we are going to keep supporting accessing PDFJS.xxxx members, however for future (e.g. v2.0) we shall probably rely only on module exports keeping that single file as compatibility feature.

TODO:

  • move all preferences from api.js into parameter object for related methods (e.g. disableRange is parameter for getDocument, and externalLinkTarget for AnnotationLayer.render)
  • refactor global PDFJS usage/assignments from all core/, shared/ and display/ , keep them in the single file e.g. src/display/globals.js

Activity

  1. shikhar-scs commented on Jan 20, 2018

    @shikhar-scs
    Contributor

    Is this issue still up @yurydelendik ?

    Though I guess either of the tasks have been achieved in the respective PRs

  2. timvandermeij commented on Jan 20, 2018

    @timvandermeij
    Collaborator

    You're right, both tasks should be done already when the last pull request above is merged.

  3. shikhar-scs commented on Jan 20, 2018

    @shikhar-scs
    Contributor

    Thanks @timvandermeij for confirming. I'll look up for something else 👍

  4. Snuffleupagus commented on Mar 4, 2018

    @Snuffleupagus
    Collaborator

    Closing, since the remaining TODO was fixed by PR #9493.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Fields

    Priority

    None yet

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions