Skip to content

Fix tasted badge positioning and remove redundant want_to_try indicator - #205

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

Fix tasted badge positioning and remove redundant want_to_try indicator#205
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 tasted status badge was overlapping with drink names and the unchecked circle indicator for "want to try" status was redundant with the existing bookmark button.

Changes

  • Repositioned tasted badge: Moved from top-left to bottom-right corner to eliminate overlap with drink name
  • Removed want_to_try badge: Only show badge for status == 'tasted' since bookmark button already indicates want_to_try status
  • Simplified badge logic: Removed switch statement handling multiple statuses, now only renders green checkmark with try count for tasted drinks

Visual Impact

Before (top-left):                   After (bottom-right):
┌──────────────────────────┐        ┌──────────────────────────┐
│ [✓] Beer Name       [📖] │        │ Beer Name           [📖] │
│ Brewery • Location       │        │ Brewery • Location       │
│ [Beer] [IPA] [5.5%]      │        │ [Beer] [IPA] [5.5%] [✓] │
└──────────────────────────┘        └──────────────────────────┘

The badge now appears only when a drink has been tasted and sits in the bottom-right corner clear of both the title and info chips.


💡 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.

Co-authored-by: richardthe3rd <573334+richardthe3rd@users.noreply.github.com>
Copilot AI changed the title [WIP] Implement My Festival favourites feature Fix tasted badge positioning and remove redundant want_to_try indicator Jan 3, 2026
Copilot AI requested a review from richardthe3rd January 3, 2026 10:16
@richardthe3rd
richardthe3rd marked this pull request as ready for review January 3, 2026 10:19
@richardthe3rd
richardthe3rd merged commit f726d39 into claude/implement-my-festival-3diaO Jan 3, 2026
@richardthe3rd
richardthe3rd deleted the copilot/sub-pr-200 branch January 3, 2026 10:19
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