Skip to content

Add A3 and Tabloid paper sizes - #74

Merged
jasonmadigan merged 1 commit into
tracefinity:mainfrom
noobydp:codex/a3-page-size-support
Jun 14, 2026
Merged

Add A3 and Tabloid paper sizes#74
jasonmadigan merged 1 commit into
tracefinity:mainfrom
noobydp:codex/a3-page-size-support

Conversation

@noobydp

@noobydp noobydp commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add A3 and Tabloid as supported paper-size presets for trace calibration.
  • Keep backend constants, request/session schemas, frontend types, and the trace-page selector in sync.
  • Preserve saved session paper size when reloading the trace page so A3/Tabloid sessions do not fall back to A4.
  • Fix corrected-image downscale ratio reporting to use the actual integer resize ratio.
  • Update user-facing docs and onboarding copy for the expanded paper-size list.

Validation

  • git diff --check
  • D:\Development\tracefinity\backend\venv\Scripts\python.exe -m pytest backend\tests\test_paper_sizes.py backend\tests\test_image_downscale.py -q
    • 8 passed
  • corepack pnpm@10.29.2 --dir frontend exec tsc --noEmit
  • corepack pnpm@10.29.2 --dir frontend build

Code and PR drafted with Codex

@jasonmadigan jasonmadigan 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.

nice @noobydp, thank you

stale comment in image_processor.py (aspect ratio list) -- can fix in a follow-up.

@jasonmadigan
jasonmadigan force-pushed the codex/a3-page-size-support branch from 1feff01 to 70bfde5 Compare June 14, 2026 17:44
@jasonmadigan
jasonmadigan marked this pull request as ready for review June 14, 2026 17:46
@jasonmadigan
jasonmadigan merged commit c403f58 into tracefinity:main Jun 14, 2026
1 check failed
shanetinklenberg pushed a commit to shanetinklenberg/tracefinity that referenced this pull request Jun 28, 2026
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