diff --git a/docs/memory/session-log.md b/docs/memory/session-log.md index ab21098..8ce649b 100644 --- a/docs/memory/session-log.md +++ b/docs/memory/session-log.md @@ -26,6 +26,46 @@ Each entry follows this structure: ## Sessions +### Session 2026-03-31d — Phase 4 adapt fixes + ProvidersStep filter panel + +**Agent(s):** Claude Opus 4.6 (1M context) + +**Work completed:** +- **Phase 4 /adapt reviews:** Ran responsive adaptation audits on Navigation, Footer, ProviderCard, VenueCard. Fixed P0/P1 touch target issues in Navigation (hamburger + drawer items minHeight 44px) and Footer (all links minHeight 44px, tagline responsive maxWidth). ProviderCard/VenueCard skipped — meta rows aren't interactive, card itself is the touch target. +- **ProvidersStep structured filters:** Replaced generic `ProviderFilter[]` chip array with `ProviderFilterValues` interface containing 5 filter controls inside FilterPanel/DialogShell: + - **Location:** Autocomplete `multiple` + `freeSolo` with chip-in-input pattern (real estate site UX — chip sits inside the search field with X to clear) + - **Service tradition:** Autocomplete with type-to-search, 21 options ('None' first, then alphabetical traditions) + - **Funeral type:** Chip multi-select with wrapping layout (6 types from spec) + - **Verified providers / Online arrangements:** Switch toggles + - **Price range:** Dual-knob slider + compact editable inputs (matching CoffinsStep) +- **FilterPanel molecule updates:** "Done" → "Apply", "Clear all" moved to footer as "Reset filters" text button +- **DialogShell padding:** px 3 → 5 (12px → 20px) for breathing room +- **Filter panel UX iterations (3 rounds):** + - Section headings: `labelLg` + `fontWeight: 600` for visual hierarchy + - Price inputs: compact `fontSize: 0.875rem` + tighter padding + - Funeral type chips: wrap layout (not horizontal scroll) — stays within bounds + - Price slider: `px: 2.5` accommodates MUI thumb radius, no overflow + - Active filter count includes location search; reset clears search too + +**Decisions made:** +- "Service tradition" as the label for religion/faith filter — neutral, user-centric, covers religious and non-religious +- 'None' as first tradition option +- Location uses Autocomplete chip-in-input pattern (Domain/realestate.com.au style) +- Funeral type chips wrap (stack) rather than horizontal scroll — clearer at dialog width +- DialogShell px: 5 is a global change — all dialogs get more breathing room +- FilterPanel footer: "Reset filters" + "Apply" replaces header "Clear all" + footer "Done" + +**Open questions:** +- None currently + +**Next steps:** +- Continue user feedback on remaining pages +- Phase 4 remaining: `/typeset` sample, `/preflight` full codebase +- Update component-registry.md with filter panel changes +- Add progress bar + cart stories to more page stories + +--- + ### Session 2026-03-31c — PaymentStep/ConfirmationStep review + progress bar + CartButton **Agent(s):** Claude Opus 4.6 (1M context) diff --git a/docs/reference/retroactive-review-plan.md b/docs/reference/retroactive-review-plan.md index a76c9ec..3dd5173 100644 --- a/docs/reference/retroactive-review-plan.md +++ b/docs/reference/retroactive-review-plan.md @@ -102,13 +102,13 @@ retained as alternatives. ## Phase 4: Cross-cutting concerns After individual components are clean: -1. **Form error colour normalisation (D034):** Override MuiFormHelperText, MuiOutlinedInput, and ToggleButtonGroup error styles in `src/theme/index.ts` to use copper instead of red. Keep `palette.error.main` red for non-form uses. This unifies atom-level error styling with the page-level copper already applied in the wizard grooming pass. -2. Run `/adapt` on all organisms + ProviderCard/VenueCard (responsive check) +1. ~~**Form error colour normalisation (D034):**~~ Done (2026-03-28). Copper error styling in MuiOutlinedInput, MuiFormHelperText, MuiFormLabel, ToggleButtonGroup. +2. ~~Run `/adapt` on all organisms + ProviderCard/VenueCard (responsive check)~~ Done (2026-03-31d). Navigation + Footer touch targets fixed (P0/P1). ProviderCard/VenueCard: no action needed (card is the touch target). 3. Run `/typeset` across a representative sample of each tier 4. Run `/preflight` to verify the full codebase 5. Commit all fixes -**Estimated effort:** 0.5 session. +**Estimated effort:** 0.5 session (remaining: typeset + preflight only). ---