Update session log for 2026-03-30d
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -26,6 +26,37 @@ Each entry follows this structure:
|
|||||||
|
|
||||||
## Sessions
|
## 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 `<Divider>` 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
|
### Session 2026-03-30c — HomePage build + Chromatic + Story to Design setup
|
||||||
|
|
||||||
**Agent(s):** Claude Opus 4.6 (1M context)
|
**Agent(s):** Claude Opus 4.6 (1M context)
|
||||||
|
|||||||
Reference in New Issue
Block a user