Skip to content

WC Selected match banner#2958

Merged
lvachon1 merged 11 commits intomainfrom
lev/feat/wc_selected_match
Feb 23, 2026
Merged

WC Selected match banner#2958
lvachon1 merged 11 commits intomainfrom
lev/feat/wc_selected_match

Conversation

@lvachon1
Copy link
Contributor

@lvachon1 lvachon1 commented Feb 19, 2026

Scope

Asana Ticket: Specific match - fancy label with selected match (design TBD)

Implementation

Created selected_match_banner function/component
Added some logic to hide/show UI elements depending on if a match is selected

Screenshots

Design
Screenshot 2026-02-20 at 10 44 39 AM

Implementation
Screenshot 2026-02-20 at 10 42 48 AM

How to test

http://localhost:4001/preview/schedules/CR-WorldCup Then select a match!

@lvachon1 lvachon1 marked this pull request as ready for review February 20, 2026 14:38
@lvachon1 lvachon1 requested a review from a team as a code owner February 20, 2026 14:38
@lvachon1 lvachon1 requested a review from joshlarson February 20, 2026 14:38
@lvachon1 lvachon1 enabled auto-merge (squash) February 20, 2026 15:14
Copy link
Contributor

@joshlarson joshlarson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I had originally had a different vision for how <.match_link /> and <.selected_match_banner /> would interplay, but honestly, I think your approach works better than what I had in mind 🥳

I left a bunch of LiveView style comments, and honestly a bunch of them are ones that I'll probably want to implement either as part of this PR or as follow-up work that we do pretty soon.

@lvachon1 lvachon1 merged commit 3cd0aa3 into main Feb 23, 2026
17 checks passed
@lvachon1 lvachon1 deleted the lev/feat/wc_selected_match branch February 23, 2026 14:52
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