-
-
Notifications
You must be signed in to change notification settings - Fork 63
Implementation Summary — Playlist System #174
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
base: master
Are you sure you want to change the base?
Conversation
…ts and multilingual support
|
Generally looking at the code, I think it is overcomplicated for a playlist feature. Is the activity necessary? is the category at home tab necessary? it could have been done within the audio player only (and maybe a single button in the bottom bar to launch the audio player). Another thing is the design. A lot of the UI elements don't really fit well with the rest of the app. I don't want to be picky, but I don't want the app to be a collection of random UI elements. I believe you used AI to do some of this, I'm not against that (I use AI too), but the generated code must always be carefully checked. Some of the original code has been changed in a way that led to problems, like the slider in the audio player, it has nothing to do with the playlist feature, yet it somehow has been modified, and now it has issues when manually dragged. I'll be busy starting form next week, so I leave this until I come back. You can still have the feature in your fork during this time, I don't think there is any code confliction. Thank you. |
|
Thank you for the detailed feedback, what you mentioned makes a lot of sense. You’re right, I got a bit too carried away with the design, which ended up making things more complicated than they needed to be. The extra tab was because I thought it could be interesting to have a quicker way to open the playlist management, but I understand your point — it really makes more sense to keep this inside the audio player, as you suggested. Regarding the design, I understand the concern and will review it to keep consistency with the rest of the app. As for the use of AI, yes, I did use it in some parts, but I fully agree that the generated code must always be carefully reviewed. I actually hadn’t noticed the issue with the slider in the audio player, so thank you for pointing that out — I’ll fix it and make sure it doesn’t affect other functionalities. In the meantime, I’ll keep the feature in my fork and try to keep it always up to date to avoid conflicts. |
This PR delivers the complete playlist system, providing all the
essential features for creating, organizing, and managing songs.
#155
🔹 Features
Creation & management
Song addition
Playback functions
UI/UX
🔧 Important fixes
⚡ The playlist system is now complete, stable, and ready to use.
💡 If there are any bugs or code improvements I may have overlooked,
please let me know — I’ll be happy to address them as quickly as
possible.