Skip to content

Finished video grid main elements#23

Merged
logbasem merged 19 commits into
mainfrom
video-grid
May 8, 2026
Merged

Finished video grid main elements#23
logbasem merged 19 commits into
mainfrom
video-grid

Conversation

@logbasem
Copy link
Copy Markdown

@logbasem logbasem commented May 4, 2026

  • Implemented video grid with 5 cameras
  • User clicks on dropdown to select camera path, then clicks "Add Camera" to add the associated feed to the first available (non-populated) container.
  • Placeholder text for no camera
  • Loading indicator for when video is still loading after add
  • Start of camera bar at top of screen
  • Works with fake backend, has not been tested with real feeds

Figma Design

image

UI Screenshot

Screenshot 2026-05-04 at 5 15 13 PM

@logbasem logbasem requested a review from onkoe May 7, 2026 00:00
Copy link
Copy Markdown
Member

@onkoe onkoe left a comment

Choose a reason for hiding this comment

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

lgtm! please feel free to solve these kinda-minor nits if you have time.

Comment thread react-app/src/App.css
Comment thread react-app/public/loading_indicator.png Outdated
Comment thread react-app/src/App.tsx Outdated
Comment thread react-app/src/App.tsx
Comment thread react-app/src/App.tsx Outdated
Comment thread react-app/src/App.tsx
Comment thread react-app/src/CameraGrid.tsx
@logbasem logbasem merged commit c6ce64c into main May 8, 2026
4 checks passed
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