Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Syncing Between Music Video & Player, Decrease Network Bandwidth & Transfer, Repetitive Authentication and Styled Components #150

Merged
merged 55 commits into from
Nov 25, 2024

Conversation

WillKirkmanM
Copy link
Owner

No description provided.

Implements a new dialog component for editing song metadata including:
- Song name, artist, and contributing artists
- Track details like number, path, and duration
- Real-time fetch of existing song data
- Save functionality with error handling

Uses Radix UI components for accessibility and consistent styling
Organizes imports by grouping related modules together
Removes unused imports for Popover, FastAverageColor, Input, and Label
Standardizes import ordering for better maintainability
…dling

Improves dialog state management with cleanup handling
Adds error handling for playlist and song info fetches
Implements proper component unmounting with useRef
Separates dialog content and adds footer with close button

Improves user experience by preventing state updates during unmount
Enhances music video interaction by replacing direct YouTube links with an in-app modal player
- Integrates ReactPlayer for YouTube video playback
- Adds dialog component for modal presentation
- Improves user experience by keeping users within the application
- Adds hover state management for better interaction feedback
@WillKirkmanM WillKirkmanM self-assigned this Nov 25, 2024
Copy link

vercel bot commented Nov 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
parsonlabs-music-documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 7:23pm

@WillKirkmanM WillKirkmanM merged commit 4088654 into master Nov 25, 2024
6 checks passed
@WillKirkmanM WillKirkmanM deleted the nightly branch November 25, 2024 19:25
@WillKirkmanM WillKirkmanM restored the nightly branch November 25, 2024 19:25
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