Bug 474756 - Album headers in playlist view are completely messed up when shuffling
Summary: Album headers in playlist view are completely messed up when shuffling
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (other bugs)
Version First Reported In: 23.08.1
Platform: NixOS Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-09-21 04:32 UTC by [object Object]
Modified: 2024-04-13 20:55 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
Before (without shuffle) (107.79 KB, image/jpeg)
2023-09-21 04:32 UTC, [object Object]
Details
After (with shuffle) (151.12 KB, image/jpeg)
2023-09-21 04:32 UTC, [object Object]
Details
New Example (118.71 KB, image/webp)
2024-02-03 05:33 UTC, [object Object]
Details

Note You need to log in before you can comment on or make changes to this bug.
Description [object Object] 2023-09-21 04:32:30 UTC
Created attachment 161767 [details]
Before (without shuffle)

SUMMARY
See screenshots

STEPS TO REPRODUCE
1. Have some music with albums
2. Shuffle the music and look at the playlist view

OBSERVED RESULT
See screenshots

EXPECTED RESULT
See screenshots

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 6.4.15
(available in About System)
KDE Plasma Version: 5.27.8
KDE Frameworks Version: 5.110.0
Qt Version: 5.15.10

ADDITIONAL INFORMATION
Comment 1 [object Object] 2023-09-21 04:32:59 UTC
Created attachment 161768 [details]
After (with shuffle)
Comment 2 [object Object] 2023-09-21 04:33:38 UTC
Maybe the album headers should just be removed in shuffle mode?
Comment 3 Nate Graham 2023-09-21 15:57:25 UTC
This is in fact intended. If we removed the header, how would you know what album the song came from? Or is it that you don't care what album it came from in shuffle mode? I don't actually use shuffle mode much myself so I'm unsure of what the best UX for it would be.
Comment 4 [object Object] 2023-09-21 16:33:23 UTC
(In reply to Nate Graham from comment #3)
> This is in fact intended. If we removed the header, how would you know what
> album the song came from? Or is it that you don't care what album it came
> from in shuffle mode? I don't actually use shuffle mode much myself so I'm
> unsure of what the best UX for it would be.

If you really want to know what album something came from, you can just click the info button on it. Also, a massive header every 2 songs, along with half the songs having thumbnails and the other half not, makes this seem unintended to me.
Comment 5 [object Object] 2023-09-21 16:34:31 UTC
The album headers don't even work correctly here - songs not from that album are right below the header.
Comment 6 Bug Janitor Service 2023-10-06 03:46:28 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 7 [object Object] 2024-02-03 05:29:40 UTC
Actually, the album is already displayed as a subtitle below the text. The random headers aren't even needed here, and they don't match the albums of the songs under them.
Comment 8 [object Object] 2024-02-03 05:33:55 UTC
Created attachment 165490 [details]
New Example

I've added a new screenshot to show what I mean. This is clearly broken - some of the songs under the War Thunder album header aren't even from the same artist...
Comment 9 [object Object] 2024-04-13 20:55:49 UTC
Not sure what was changed, but it seems to be fixed now.