From b5a56b649782c0c6aefa47d6704cb37d84fcc61f Mon Sep 17 00:00:00 2001 From: Richie Date: Mon, 30 Mar 2026 18:23:26 +1100 Subject: [PATCH] Update session log for 2026-03-30d Co-Authored-By: Claude Opus 4.6 (1M context) --- docs/memory/session-log.md | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/docs/memory/session-log.md b/docs/memory/session-log.md index 7a6d55a..c0fc61b 100644 --- a/docs/memory/session-log.md +++ b/docs/memory/session-log.md @@ -26,6 +26,37 @@ Each entry follows this structure: ## Sessions +### Session 2026-03-30d — Steps 8-15 consistency pass + +**Agent(s):** Claude Opus 4.6 (1M context) + +**Work completed:** +- **Consistency pass across Steps 8-15** — aligned all remaining wizard steps with the conventions established during the Steps 1-6 feedback iteration: + - **Subheading variant:** CrematoriumStep, CoffinDetailsStep changed from `body2` → `body1` (match IntroStep/DateTimeStep centered-form convention) + - **Subheading spacing:** CrematoriumStep, CemeteryStep, CoffinDetailsStep, AdditionalServicesStep, SummaryStep, PaymentStep all changed from `mb: 4` → `mb: 5` + - **Section dividers:** CrematoriumStep gained 3 `` between question sections (crematorium → witness, conditional priority, special instructions) + - **Link atom:** PaymentStep terms links and ConfirmationStep phone number changed from `Box component="a"` → `Link` atom + - **CoffinsStep:** No changes needed (wide-form layout has its own spacing convention) +- **Audited CrematoriumStep (15/20) and PaymentStep (16/20)** — findings are pre-existing, not introduced by the pass +- **Visual verification:** Playwright screenshots of CrematoriumStep, PaymentStep, ConfirmationStep, AdditionalServicesStep, SummaryStep — all pass + +**Decisions made:** +- Centered-form heading convention now formally: `display3` → `body1` subheading → `mb: 5` gap (documented via consistent implementation across all 10+ centered-form steps) +- Wide-form layout (CoffinsStep) keeps its own tighter spacing convention (`body2` + `caption`, `mb: 3`) + +**Open questions:** +- **Checkbox atom:** PaymentStep uses MUI Checkbox directly — only form control without an FA atom wrapper. Create Checkbox atom? +- **aria-describedby/aria-invalid:** All wizard steps lack error field association. Batch normalize fix recommended. +- **Retroactive review Phase 2.2** still pending (audit priority molecules: ServiceOption, AddOnOption, ProviderCardCompact) + +**Next steps:** +- Batch a11y fix: add aria-describedby + aria-invalid to error fields across all wizard steps +- Create Checkbox atom if prioritised +- Retroactive review Phase 2.2 (morning housekeeping) +- Continue any further page-specific feedback from user review + +--- + ### Session 2026-03-30c — HomePage build + Chromatic + Story to Design setup **Agent(s):** Claude Opus 4.6 (1M context)