Skip to content

Refactor Festival Log UI: bookmark metaphor and diary-style layout - #204

Merged
richardthe3rd merged 2 commits into
claude/implement-my-festival-3diaOfrom
copilot/sub-pr-200
Jan 3, 2026
Merged

Refactor Festival Log UI: bookmark metaphor and diary-style layout#204
richardthe3rd merged 2 commits into
claude/implement-my-festival-3diaOfrom
copilot/sub-pr-200

Conversation

Copilot AI commented Jan 3, 2026

Copy link
Copy Markdown
Contributor

The Festival Log UI had four issues: confusing "favorite/heart" action semantics, status badge overlapping with action button, unclear empty circle indicator, and flat list view not matching diary/log mental model.

Icon Changes

  • Bookmark replaces heart - Icons.bookmark/Icons.bookmark_border for "Want to Try" action
  • Badge repositioned - Status badge moved from right: 8 to left: 8 to avoid button overlap
  • Status indicators - Bookmark icon for "Want to Try", green checkmark for "Tasted"

Festival Log Screen

  • Day-based grouping - Tasted drinks grouped by "Today", "Yesterday", weekday names, or date strings
  • Inline timestamps - Each tasting shows time (e.g., "2:30 PM")
  • Section headers - "Want to Try" section at top, then tasted sections by day with count badges
  • Compact cards - Tasted drinks use condensed layout with inline metadata
// Before: flat list with simple divider
SliverList(drinks.map((d) => DrinkCard(d)))

// After: grouped by day with headers
widgets.add(_buildSectionHeader('Want to Try', count));
for (drink in wantToTry) widgets.add(_buildDrinkCard(drink));
for (dayKey in dayKeys) {
  widgets.add(_buildDayHeader(dayKey, count));
  for (drink in tastedByDay[dayKey]) 
    widgets.add(_buildTastedDrinkCard(drink, timestamp));
}

All tests updated to use bookmark icons. 579 tests passing.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

- Replace heart icon with bookmark icon for "Want to Try" action
- Move status badge to left side to avoid overlap with bookmark button
- Use bookmark icon for "Want to Try" status (instead of empty circle)
- Redesign My Festival screen with diary-style layout
  - Group tasted drinks by day (Today, Yesterday, weekday names, dates)
  - Show timestamps for each tasting
  - Visual separation between "Want to Try" and tasted sections
  - More compact tasted drink cards with inline timestamps
- Update all tests to use bookmark icons instead of heart icons

Co-authored-by: richardthe3rd <573334+richardthe3rd@users.noreply.github.com>
Copilot AI changed the title [WIP] Implement My Festival favourites feature Refactor Festival Log UI: bookmark metaphor and diary-style layout Jan 3, 2026
Copilot AI requested a review from richardthe3rd January 3, 2026 09:56
@richardthe3rd
richardthe3rd marked this pull request as ready for review January 3, 2026 09:58
@richardthe3rd
richardthe3rd merged commit 42af7a3 into claude/implement-my-festival-3diaO Jan 3, 2026
@richardthe3rd
richardthe3rd deleted the copilot/sub-pr-200 branch January 3, 2026 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants