feat(shopping-list) Show the choose-a-default hint with an info icon #53

Merged
araemer merged 1 commit from overview-default-prompt-icon into stage 2026-10-03 16:10:35 +00:00
Owner

The "choose a default list" hint on the shopping list overview was a bare text card. It now has an info icon.

Changes

  • basics/InfoBanner: an inline message box with a leading info icon, optionally dismissible, in two looks:
    • info: a neutral white card with an accent-coloured icon, for hints
    • warning: amber, for notices about something that happened
  • ShoppingListOverviewPage: the default-list hint uses the info look; the "list deleted" / "access lost" notice uses the warning look instead of its own markup. The notice looks and behaves as before.
  • docs/architecture.md lists the new component.

The hint is deliberately not tinted in the accent colour: the app's accent is a warm red, and a tinted box reads as an error. AiImportDraftNotice keeps its own tinted look and is not changed.

Verification

  • npm run lint (only the 2 existing warnings in MealPlanDayProvider), npm run test (372 tests) and vite build pass.
  • npm run build fails on its type check, on stage itself, not because of this PR. The contract on recipe-platform main already has spec 002's change (CompactShoppingListDto requires openItemsPreview and openItemCount), so src/mappers/tests/CompactShoppingListMapper.test.ts no longer compiles. A clean stage (f697a95) has exactly this one error; this branch has the same one and no other. It goes away when the app implements spec 002.
  • /verify-ui as user, from a dev server running this branch:
    • the hint with its icon on all five viewports, no horizontal overflow
    • the amber notice (reached via router state, as the list page sends it) with its 44 px close button; dismissing removes it
    • the test list was deleted afterwards

🤖 Generated with Claude Code

The "choose a default list" hint on the shopping list overview was a bare text card. It now has an info icon. ## Changes - **`basics/InfoBanner`:** an inline message box with a leading info icon, optionally dismissible, in two looks: - `info`: a neutral white card with an accent-coloured icon, for hints - `warning`: amber, for notices about something that happened - **`ShoppingListOverviewPage`:** the default-list hint uses the `info` look; the "list deleted" / "access lost" notice uses the `warning` look instead of its own markup. The notice looks and behaves as before. - `docs/architecture.md` lists the new component. The hint is deliberately not tinted in the accent colour: the app's accent is a warm red, and a tinted box reads as an error. `AiImportDraftNotice` keeps its own tinted look and is not changed. ## Verification - `npm run lint` (only the 2 existing warnings in `MealPlanDayProvider`), `npm run test` (372 tests) and `vite build` pass. - **`npm run build` fails on its type check, on `stage` itself, not because of this PR.** The contract on recipe-platform `main` already has spec 002's change (`CompactShoppingListDto` requires `openItemsPreview` and `openItemCount`), so `src/mappers/tests/CompactShoppingListMapper.test.ts` no longer compiles. A clean `stage` (f697a95) has exactly this one error; this branch has the same one and no other. It goes away when the app implements spec 002. - `/verify-ui` as `user`, from a dev server running this branch: - the hint with its icon on all five viewports, no horizontal overflow - the amber notice (reached via router state, as the list page sends it) with its 44 px close button; dismissing removes it - the test list was deleted afterwards 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat(shopping-list) show the choose-a-default hint with an info icon
Some checks failed
/ test (pull_request) Failing after 46s
e0a948552c
The hint on the shopping list overview was a bare text card. It is now an
InfoBanner: the same neutral white card with an accent-coloured info icon.
The overview's deleted / access-lost notice uses the same component in its
amber warning look, so both boxes share one implementation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
araemer merged commit 2bbe21c343 into stage 2026-10-03 16:10:35 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
araemer/recipe-app!53
No description provided.