Skip to content

Add Videos tab to bottom navigation for viewing match videos#19

Draft
Copilot wants to merge 5 commits intomainfrom
copilot/fix-18
Draft

Add Videos tab to bottom navigation for viewing match videos#19
Copilot wants to merge 5 commits intomainfrom
copilot/fix-18

Conversation

Copy link
Contributor

Copilot AI commented Aug 24, 2025

This PR adds a new Videos tab to the bottom navigation that allows users to browse and watch available match videos from fixtures.

What's Added

New Videos Tab

  • Added a 4th tab to the bottom navigation with a videocam icon
  • Updated MainNavigationView to support 4 tabs using BottomNavigationBarType.fixed
  • Created dedicated VideosView for displaying video content

Videos Interface

The new Videos tab displays:

  • List of fixtures that have available videos
  • Match details including teams, scores, date/time, and venue
  • Support for multiple videos per match
  • Video playback through existing VideoPlayerDialog
  • Pull-to-refresh functionality

Data Integration

  • Added getFixturesWithVideos() method to DataService
  • Filters fixtures to show only those with non-empty videos list
  • Sorts videos by most recent first (past matches only)
  • Currently uses mock data for demonstration

User Experience

Users can now:

  1. Tap the Videos tab (videocam icon) in bottom navigation
  2. Browse recent matches that have videos available
  3. Tap video buttons to watch match footage
  4. See multiple video options when available for a single match

The interface follows FIT brand guidelines with consistent styling and provides proper loading states and error handling.

Technical Implementation

The implementation is minimal and surgical:

  • 3 files modified: main_navigation_view.dart, videos_view.dart (new), data_service.dart
  • No breaking changes: Existing functionality preserved
  • Reuses existing components: Integrates with VideoPlayerDialog and FIT styling
  • Future-ready: Structure supports real API/database integration when mock data is replaced

Fixes #18.


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

Copilot AI and others added 2 commits August 24, 2025 10:26
Co-authored-by: goodtune <286798+goodtune@users.noreply.github.com>
Co-authored-by: goodtune <286798+goodtune@users.noreply.github.com>
Copilot AI changed the title [WIP] Add 🎥 Videos tab Add Videos tab to bottom navigation for viewing match videos Aug 24, 2025
Copilot AI requested a review from goodtune August 24, 2025 10:38
goodtune and others added 2 commits August 24, 2025 20:42
- Add `_getDivisionName()` helper method to look up division names from API
- Update `_playVideo()` method to pass required `homeTeamName`, `awayTeamName`, and `divisionName` parameters
- Replace mock data in `getFixturesWithVideos()` with real API data fetching across all events/divisions
- Filter for completed fixtures with videos and sort by most recent first

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <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.

Add 🎥 Videos tab

2 participants