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

Interactive forms: rewrite AcroForms example #8030

Merged
merged 1 commit into from
Feb 6, 2017

Conversation

timvandermeij
Copy link
Contributor

This patch lets the AcroForms example make use of the built-in interactive forms functionality in PDF.js. This makes the example:

  • much easier to understand;
  • more feature-complete;
  • in sync with the core when new functionality is added;
  • similar to the other examples in terms of structure.

Fixes one of the items in #7613.

This patch lets the AcroForms example make use of the built-in interactive
forms functionality in PDF.js. This makes the example:

- much easier to understand;
- more feature-complete;
- in sync with the core when new functionality is added;
- similar to the other examples in terms of structure.
@Snuffleupagus Snuffleupagus self-requested a review February 6, 2017 09:59
Copy link
Collaborator

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, thank you for the patch!

@Snuffleupagus Snuffleupagus merged commit 3e5c6e4 into mozilla:master Feb 6, 2017
@timvandermeij timvandermeij deleted the acroforms-example branch February 6, 2017 21:54
movsb pushed a commit to movsb/pdf.js that referenced this pull request Jul 14, 2018
Interactive forms: rewrite AcroForms example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants