Skip to content

Northmoor: body type one step up the scale - #68

Closed
danmolitor wants to merge 2 commits into
mainfrom
chore/northmoor-type-scale
Closed

Northmoor: body type one step up the scale#68
danmolitor wants to merge 2 commits into
mainfrom
chore/northmoor-type-scale

Conversation

@danmolitor

@danmolitor danmolitor commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Base body size 7.5pt → 8.25pt and line-height 1.5 → 1.6 in northmoor-shared.css. Every Northmoor document reflows at once.

This is a workload test for Forme Review: one stylesheet change, thirty documents to review. Watching the check title, the comment's length, and whether approving is one decision or thirty.

Base 7.5pt → 8.25pt, line-height 1.5 → 1.6, in the shared stylesheet, so
every Northmoor document reflows at once. A workload test for Forme
Review: one CSS change, thirty documents to review.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MKzAu3ytK4juTnPg1XHNRN
@forme-review

forme-review Bot commented Sep 8, 2026

Copy link
Copy Markdown

⛔ Changes rejected

27 documents changed structurally · 1 error, 3 warnings, 0 info · baseline bb5bc1a484c285

dist/review/northmoor-policy-acknowledgement.pdf — ⛔ Rejected by Dan Molitor · baseline stays at bb5bc1a

Severity Event Location Rule
error Page count changed 1 → 2 (3 events) document page-count-changed
warn Text "Retention: duration of employment plus …" moved from page 1 to page 2 p1 → p2 element-moved-to-different-page
warn Text "Ref. ACK-2026-20614" moved from page 1 to page 2 p1 → p2 element-moved-to-different-page
warn Added text "P A G E 2 O F 2" on page 2 p2 element-added

Page count changed 1 → 2 (+2 repeated header/footer elements on the new page); text "Retention: duration of employment plus …" moved from page 1 to page 2; text "Ref. ACK-2026-20614" moved from page 1 to page 2.

View full diff ↗

26 documents changed, none blocking

Document Causes Headline
dist/review/northmoor-cover-letter.pdf · approved 1 snapshot changed but no nameable event fired: 31 of 45 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-credit-note.pdf · approved 1 snapshot changed but no nameable event fired: 17 of 71 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-delivery-note.pdf · approved 1 snapshot changed but no nameable event fired: 44 of 99 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-employment-contract.pdf · approved 1 snapshot changed but no nameable event fired: 48 of 188 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-expense-report.pdf · approved 1 snapshot changed but no nameable event fired: 82 of 144 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-invoice-detailed.pdf · approved 1 snapshot changed but no nameable event fired: 237 of 342 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-invoice-standard.pdf · approved 1 snapshot changed but no nameable event fired: 31 of 93 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-lab-report.pdf · approved 1 snapshot changed but no nameable event fired: 71 of 136 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-letterhead.pdf · approved 1 snapshot changed but no nameable event fired: 62 of 99 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-meeting-minutes.pdf · approved 1 snapshot changed but no nameable event fired: 31 of 98 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-memo.pdf · approved 1 snapshot changed but no nameable event fired: 23 of 74 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-nda.pdf · approved 1 snapshot changed but no nameable event fired: 41 of 120 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-offer-letter.pdf · approved 1 snapshot changed but no nameable event fired: 46 of 59 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-packing-slip.pdf · approved 1 snapshot changed but no nameable event fired: 56 of 110 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-payslip.pdf · approved 1 snapshot changed but no nameable event fired: 72 of 133 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-product-catalog.pdf · approved 1 snapshot changed but no nameable event fired: 150 of 217 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-purchase-order.pdf · approved 1 snapshot changed but no nameable event fired: 37 of 103 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-quote.pdf · approved 1 snapshot changed but no nameable event fired: 37 of 97 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-receipt.pdf · approved 1 snapshot changed but no nameable event fired: 22 of 74 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-reference-letter.pdf · approved 1 snapshot changed but no nameable event fired: 26 of 44 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-remittance-advice.pdf · approved 1 snapshot changed but no nameable event fired: 44 of 100 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-report-annual.pdf · approved 1 snapshot changed but no nameable event fired: 89 of 205 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-report-monthly.pdf · approved 1 snapshot changed but no nameable event fired: 80 of 143 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-service-agreement.pdf · approved 1 snapshot changed but no nameable event fired: 132 of 207 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-statement.pdf · approved 1 snapshot changed but no nameable event fired: 72 of 129 matched elements moved or resized below the 24pt threshold view
dist/review/northmoor-termination-letter.pdf · approved 1 snapshot changed but no nameable event fired: 47 of 57 matched elements moved or resized below the 24pt threshold view

12 documents unchanged: dist/review/template-shipping-label.pdf, dist/review/template-report.pdf, dist/review/template-receipt.pdf, dist/review/template-letter.pdf, dist/review/template-invoice.pdf, dist/review/northmoor-shipping-label.pdf, dist/review/northmoor-inspection-report.pdf, dist/review/northmoor-certificate.pdf, dist/review/html-zebra-invoice.pdf, dist/review/html-statement.pdf, dist/review/html-letterhead.pdf, dist/review/html-dashed-borders.pdf

No page thumbnails here: GitHub proxies and caches comment images, which would put a rendered customer document in a third-party cache. Open the full diff to see the pages.

Forme Review · run 1pldv6, run fvvdqd, run yo1p5h, run 5g8648, run zmja9a, run opv732, run dx7ixb, run qojkeq, run m3u9ng, run 05ty7i, run uup9j6, run ai09js, run v7jmmv, run tm30yl, run 2qpr9f, run 5m2ptm, run p11gy9, run y85q5z, run 84zjii, run eggoox, run yrwczk, run vfa67q, run ibp9hm, run zgh27x, run 221w52, run azclmm, run qhtbcx, run 9ejeyb, run 10hles, run tftk3f, run j5s5gt, run dozn01, run wrohfl, run 46uumw, run hciexj, run 2aujou, run xepq0m, run e3jxji, run e9w4m3 · powered by pdf-testkit 0.2.3

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

📄 pdf-testkit — invoice-layout.json

✅ No semantic changes detected.

@danmolitor

Copy link
Copy Markdown
Owner Author

Closing: this PR existed to put a 39-document workload through Forme Review, and it did. The type bump is not wanted on its own, and the flex-row fragmentation it exposed in the policy acknowledgement deserves its own scoped work.

@danmolitor danmolitor closed this Sep 8, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 8, 2026
@danmolitor danmolitor reopened this Sep 8, 2026
Repository owner unlocked this conversation Sep 8, 2026
@danmolitor danmolitor closed this Sep 8, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 8, 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