Skip to content

Do not add pages dynamically unless autoPaging is enabled#3915

Merged
HackbrettXXX merged 1 commit into
parallax:masterfrom
anmiles:master
Oct 28, 2025
Merged

Do not add pages dynamically unless autoPaging is enabled#3915
HackbrettXXX merged 1 commit into
parallax:masterfrom
anmiles:master

Conversation

@anmiles

@anmiles anmiles commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Do not add pages dynamically unless autoPaging is enabled

🐛 Problem

JsPDF might add a blank page in advance even if autoPaging option is not enabled.
The reason is that some branches of autoPaging-related logic are executed despite the autoPaging option.

✅ Solution

  • Ensure that the appropriate branches of code are covered by checking the autoPaging option.

🧪 Pre-push checks

  • npm run build passes
  • npm run test-unit passes
  • npm run test-local passes
  • npm run lint passes

@HackbrettXXX HackbrettXXX left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks. Looks good.

@HackbrettXXX
HackbrettXXX merged commit e080935 into parallax:master Oct 28, 2025
7 checks passed
@anmiles

anmiles commented Oct 28, 2025

Copy link
Copy Markdown
Contributor Author

@HackbrettXXX thanks for merging! Do you have any plans on when the new version might be published to npm?

@HackbrettXXX

Copy link
Copy Markdown
Collaborator

I'll wait until all Hacktober PRs have been merged. So probably around mid November.

heikofolkerts pushed a commit to heikofolkerts/jsPDF-UA that referenced this pull request Dec 11, 2025
Upstream changes:
- Fix Context2d font regex (parallax#3906)
- Fix autoPaging in Context2d (parallax#3915)
- Fix TypeScript Margins interface (parallax#3816)
- Bump GitHub Actions versions (parallax#3907)
- Version bump to 3.0.4

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
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.

2 participants