Skip to content

fix(budget): suppress browser print chrome and render inner item separators (#1328)#1329

Merged
steilerDev merged 1 commit into
betafrom
fix/1328-print-chrome-inner-boxes
Apr 20, 2026
Merged

fix(budget): suppress browser print chrome and render inner item separators (#1328)#1329
steilerDev merged 1 commit into
betafrom
fix/1328-print-chrome-inner-boxes

Conversation

@steilerDev
Copy link
Copy Markdown
Owner

Summary

  • Expanded @page rule in BudgetOverviewPage.module.css with explicit empty content for all 10 CSS Paged Media margin boxes (@top-*/@bottom-*) — suppresses Chrome's injected header (date, page title) and footer (URL, page number) while preserving @page { margin: 1.5cm 1cm } for per-page whitespace
  • Replaced non-rendering box-shadow: inset inner-box edges in CostBreakdownTable.module.css with border-top: 1pt solid var(--color-border) on .rowLevel2 td — creates visible item-group separators nested inside the outer area box

Fixes #1328

Test plan

  • Unit tests pass (95%+ coverage)
  • Integration tests pass
  • Pre-commit hook quality gates pass

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

…rators

- Expanded `@page` rule with explicit empty content for all 10 CSS Paged Media
  margin boxes (`@top-*`/`@bottom-*`) — suppresses Chrome's injected header
  (date, page title) and footer (URL, page number) while preserving
  `@page { margin: 1.5cm 1cm }` for per-page whitespace
- Replaced non-rendering `box-shadow: inset` inner-box edges with a simple
  `border-top: 1pt solid var(--color-border)` on `.rowLevel2 td` — creates
  visible item-group separators nested inside the outer area box

Fixes #1328

Co-Authored-By: Claude dev-team-lead (Sonnet 4.6) <noreply@anthropic.com>
Co-Authored-By: Claude frontend-developer (Haiku 4.5) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your submission! We require all contributors to sign our Contributor License Agreement before we can accept your contribution.

To sign, please comment on this PR with:
I have read the CLA Document and I hereby sign the CLA


I have read the CLA Document and I hereby sign the CLA


Frank Steiler seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@steilerDev steilerDev merged commit 6a3d476 into beta Apr 20, 2026
30 of 34 checks passed
@steilerDev steilerDev deleted the fix/1328-print-chrome-inner-boxes branch April 20, 2026 10:24
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant