From c1f339c99bf20f9bf51bba811ed77e46a655d7a9 Mon Sep 17 00:00:00 2001 From: Adam Moussa Date: Fri, 31 Jul 2026 11:33:03 -0400 Subject: [PATCH] docs(form): record frontend delivery decisions --- README.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/README.md b/README.md index 46d001d..070167d 100644 --- a/README.md +++ b/README.md @@ -30,6 +30,15 @@ reminders to employees └────── who haven't ordered ``` +### Form frontend decisions + +- **Theme:** Retain the current order-form palette and typography. There is no shared Sea Haven web design system to adopt, and changing the theme without one would be an isolated visual redesign. Future theme changes should remap the existing CSS custom properties instead of adding scattered color values or inline styles. +- **Deployment:** Keep generating and publishing one self-contained HTML file. Inlining the template CSS and JavaScript at build time preserves the current atomic S3 upload and avoids introducing asset versioning, cache coordination, and additional CloudFront invalidation paths. Revisit a multi-file deploy only when asset size, cross-page sharing, or independent caching provides a concrete benefit. +- **Visual regression:** Do not add visual-regression CI now. Structural and Playwright coverage remain the active safeguards. Revisit snapshots, computed-style assertions, or a hosted service only after a layout or token regression reaches production without those checks catching it. + +ESLint and Prettier check the JavaScript template sources in CI. They do not lint +the generated HTML, and the generated deployment artifact remains self-contained. + ## Weekly Flow | When | What | How |