Skip to content

Comments

Add jump-to-section sticky headers to match list#1076

Open
gregmarra wants to merge 2 commits intomainfrom
match-list-sticky-headers
Open

Add jump-to-section sticky headers to match list#1076
gregmarra wants to merge 2 commits intomainfrom
match-list-sticky-headers

Conversation

@gregmarra
Copy link
Contributor

Summary

  • Extract SectionHeader composable and SectionHeaderInfo data class from EventsScreen.kt into a shared ui/components/SectionHeader.kt
  • Reuse SectionHeader in MatchList.kt — the stuck header now shows a dropdown arrow that lets users jump between sections (Qualifications, Rounds, Finals, etc.)
  • Fix key mismatch bug in MatchList that prevented stuck-header detection from working (headerKeys used "match_header_$it" but stickyHeader used "match_header ${group.label}")

Test plan

  • ./gradlew :app:assembleDebug builds clean
  • Navigate to event with many matches (2026test) → Matches tab — stuck header shows dropdown arrow, tap jumps to selected section
  • Navigate to event with playoff rounds (2025alhu) → Matches tab — multiple section headers (Qualifications, Round 1-4, Finals, Q8+) all work with jump navigation
  • Events List still works identically (no regression from extraction)

🤖 Generated with Claude Code

gregmarra and others added 2 commits February 21, 2026 21:46
Extract SectionHeader composable from EventsScreen into a shared
component and reuse it in MatchList. Fixes a key mismatch bug that
prevented stuck-header detection from working (headerKeys used
"match_header_$it" but stickyHeader used "match_header ${group.label}").

The match list now shows an interactive dropdown arrow on the stuck
header, letting users jump between Qualifications, Rounds, and Finals
sections — matching the behavior already present on the Events list.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The AllianceTeams composable accepted a color parameter but only
applied it to the title — individual team numbers were hardcoded to
MaterialTheme.colorScheme.primary (blue). Now uses the passed-in
alliance color for team numbers too.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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