Skip to content

Refactor printer card overlays into component - #77

Merged
ichwars merged 1 commit into
mainfrom
dev-split-printer-card-ui
Aug 4, 2026
Merged

Refactor printer card overlays into component#77
ichwars merged 1 commit into
mainfrom
dev-split-printer-card-ui

Conversation

@ichwars

@ichwars ichwars commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • extract the contiguous printer-card modal and overlay region into a typed PrinterCardOverlays component
  • reduce PrinterCard.tsx from 3,817 to 2,981 lines without changing the rendered JSX flow
  • lower the source-size allowlist ceiling accordingly; the extracted component is 876 lines and below budget

Local verification

  • npm.cmd exec -- tsc -b
  • npm.cmd run lint
  • 10 focused printer/modal test files: 222 tests passed
  • full frontend suite: 244 files / 2,848 tests passed
  • all 11 locales in parity
  • isolated Vite production build: 3,044 modules transformed
  • source size budget: 1,572 files passed
  • source budget unit tests: 14 passed

@ichwars
ichwars merged commit df99291 into main Aug 4, 2026
30 checks passed
@ichwars
ichwars deleted the dev-split-printer-card-ui branch August 4, 2026 03:43
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.

1 participant