Skip to content

Move notes below video titles in desktop table - #19

Merged
daggerok merged 9 commits into
mainfrom
arena/01a0bc49-youtube
Sep 20, 2026
Merged

daggerok merged 9 commits into
mainfrom
arena/01a0bc49-youtube

Conversation

@arena-ai-coding-agent

@arena-ai-coding-agent arena-ai-coding-agent Bot commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • show only the filtered and total video counts in the inventory heading
  • switch from the desktop table to mobile-first cards below 1440px, before columns become clipped
  • split datetimes into date and time lines only in the compact desktop table
  • keep mobile-card datetimes on one non-wrapping line and preserve one-line dates on wide desktops
  • use the same compact 1rem outer gutter at every screen width
  • keep only the add/edit control in each desktop Note cell
  • use the same add-note pen icon for note controls on desktop and mobile
  • replace the pen with a green save check and red cancel control while editing
  • support Enter to save and Escape to cancel without saving
  • make editable note text slightly brighter than viewed note text
  • show existing note text directly below its video title using the existing small text style
  • open the multiline note editor below the video title when the Note icon is clicked
  • make view and edit modes share the same text position, typography, and spacing
  • remove the editor border, background, padding, and resize handle, and auto-size it to its content
  • scale displayed notes and note editors proportionally with video titles on desktop and mobile
  • narrow the Note column so the title and note content have more room
  • inline the filter utilities in index.html and remove the standalone filters.js file

Tests

  • node --test tests/filters.test.js

daggerok and others added 2 commits September 20, 2026 03:57
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
@daggerok
daggerok merged commit 2ad3c77 into main Sep 20, 2026
3 checks passed
@daggerok
daggerok deleted the arena/01a0bc49-youtube branch September 20, 2026 04:27

This branch was successfully deployed

1 active deployment
github-pages — 0937e0ef Deployed Sep 20, 2026 by github-pages[bot] via deploy #46
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.

1 participant