diff --git a/docs/memory/session-log.md b/docs/memory/session-log.md index 01e5c97..94c4f99 100644 --- a/docs/memory/session-log.md +++ b/docs/memory/session-log.md @@ -26,6 +26,42 @@ Each entry follows this structure: ## Sessions +### Session 2026-03-30e — Page feedback: VenueServices, Crematorium, Cemetery rewrites + +**Agent(s):** Claude Opus 4.6 (1M context) + +**Work completed:** +- **VenueServicesStep (new page):** Built step 7c — venue-specific service toggles after VenueDetailStep. Compact venue card at top, availability notices (info cards for unavailable services), AddOnOption toggles with maxDescriptionLines for "Read more" expansion, conditional tally showing total of selected priced services. Flow: VenueStep → VenueDetailStep → VenueServicesStep. +- **AddOnOption price colour fix:** Changed from `text.secondary` (grey) to `color="primary"` (copper) for consistency with all other price displays in the system. +- **CrematoriumStep rewrite:** Completely rewritten to match live site. Two variants based on funeral type: + - Service & Cremation: stacked crematorium card (image top, details below) + witness Yes/No/Decide later ToggleButtonGroup + - Cremation Only: stacked card + "Cremation Only" badge + "Included in Package" info notice, no witness question + - Removed: multi-card selection grid, priority dropdown, special instructions, crematoriums array prop + - Card iterated from compact horizontal → more generous horizontal → stacked layout based on user feedback +- **CemeteryStep rewrite:** Rewritten to match live site: + - RadioGroups replaced with ToggleButtonGroups (Yes/No/Not sure) + - Card selection grid replaced with freetext search input (Input atom with search icon) + - `searchSlot` prop for parent to inject Google Places or geocoding autocomplete + - Removed static `cemeteries` array prop and `CemeteryOption` type + - Progressive disclosure preserved: own plot → locate it; no plot → preference? → search + +**Decisions made:** +- Venue services belong on a separate page (not inline on VenueDetailStep or in a dialog) due to content volume per service (long descriptions, future images) +- Crematorium is always pre-selected by provider — no multi-select needed, just confirmation +- "Decide later" / "Not sure" options added as explicit third choice on questions where users may be uncertain — grief-sensitive pattern +- Cemetery search should use real location search (Google Places style), not a static dropdown — we don't maintain a cemetery database +- Stacked card layout (image top) works better than compact horizontal for single confirmation cards with long text + +**Open questions:** +- None + +**Next steps:** +- Continue page feedback: CoffinsStep, AdditionalServicesStep, SummaryStep, PaymentStep, ConfirmationStep +- Retroactive review Phase 3 (organisms) still pending +- Batch a11y fix (aria-describedby + aria-invalid) deferred + +--- + ### Session 2026-03-30d — Steps 8-15 consistency pass **Agent(s):** Claude Opus 4.6 (1M context)