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

Playlist description #5561

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

KyNarec
Copy link

@KyNarec KyNarec commented Jan 31, 2025

I redid #4586 (which comes from #4566) but I am not sure whether I need to add my code from LocalPlaylistSongsModern.kt to the other Playlist files like LocalPlaylistSongsModern.kt , PlaylistScreen.kt or PlaylistSongList.kt.

This feature did not work for a long while and I couldn't get my head around why this was the case until i found out, that @fast4x set description = null in 4507678#diff-54495b67b89955311e53b5b88e6ab47e6f4b1a496d784bbf8acda0c2579c8e87R187 and I don't know why...

Also, the fetching is pretty slow, so you will notice it like shown here:
Screen_recording_20250131_185059.webm

And I think, that I should implement a toggle for this in the settings, because I can see people get annoyed with it.

KyNarec and others added 6 commits November 20, 2024 10:34
…nto database.

TODO: Display the playlist description
…ist-Description

# Conflicts:
#	composeApp/src/androidMain/kotlin/it/fast4x/rimusic/ui/screens/localplaylist/LocalPlaylistSongs.kt
#	composeApp/src/androidMain/res/values/strings.xml
@fast4x
Copy link
Owner

fast4x commented Jan 31, 2025

Wait new version and try again, also because some files are changed or deleted.

@KyNarec
Copy link
Author

KyNarec commented Feb 1, 2025

I added a setting for it in Settings -> UI -> PlAYLISTS -> Show playlist description
Screenshot_20250201_122642

@KyNarec
Copy link
Author

KyNarec commented Feb 5, 2025

@fast4x this is ready to be merged

@fast4x
Copy link
Owner

fast4x commented Feb 5, 2025

yes i know,
solve this conflict meanwhile

@KyNarec
Copy link
Author

KyNarec commented Feb 5, 2025

Sorry, didn't see that, but it is resolved now

@fast4x
Copy link
Owner

fast4x commented Feb 6, 2025

This code not convinced me, i mean description for personal online playlist and where is placed, actually many code in playlist was changed, i want to wait.

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