Skip to content

[WIP] Video Card UI Overhaul#437

Draft
dammitjeff wants to merge 1 commit intoShaneIsrael:developfrom
dammitjeff:video-card-overhaul
Draft

[WIP] Video Card UI Overhaul#437
dammitjeff wants to merge 1 commit intoShaneIsrael:developfrom
dammitjeff:video-card-overhaul

Conversation

@dammitjeff
Copy link
Contributor

Mentioned here: #380

With the dates system sorted, and more game information being available, work on the video card overhaul is officially underway.
Screenshot 2026-02-06 at 8 20 36 PM

This is still heavily in progress and won't be done for a while, but progress is officially started. Most of the code will simply be brought over from the existing CompactVideoCards.js . I distinctly recall that one-click options for tasks such as toggling visibility and sharing links were a big priority in this redesign. I think I manged to figure out a solution. By default these options are hidden from the main dashboard view, but revealed upon hovering on the thumbnail of the video. I think it looks really nice, and makes for a much cleaner interface.
Screen Recording 2026-02-06 at 8 05 13 PM 2026-02-06 20_08_03

There's still a lot that needs to tested, and some things need to be reconsidered to work with the new design:
• Selecting multiple videos
• Editing game information (Date, Titles, etc)
• Appearances in GameVideo.js (since we won't need the game name in each thumbnail anymore)
• Appearance of Game Detection Card

I'll keep working at it, just wanted to inform that its officially being worked on haha

@ShaneIsrael
Copy link
Owner

Definitely an improvement over the existing styling. I think editing game information could be an option in a hamburger menu somewhere? Could also be done in the preview modal, possibly reworking some of control layout in that.

Overall though I think it's a fantastic start.

@dammitjeff
Copy link
Contributor Author

Awesome! Yeah this is the perfect time to implement hamburger menu now with the new overhaul, Makes for a much cleaner layout.

I recall something was said about spaces being converted to underscores to prevent injection attacks, will there need to be any tweaking made for this to still work with our new design? Since these show spaces in between words.

@ShaneIsrael
Copy link
Owner

That's only for uploaded file names. Doesn't matter for files that are scanned in (at least it shouldn't) and definitely doesn't matter for setting the name in the UI as that is an admin action.

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