Bug 429737

Summary: view-media-playlist icon could stand to be improved
Product: [Applications] Elisa Reporter: postix <postix>
Component: generalAssignee: Matthieu Gallien <matthieu_gallien>
Status: RESOLVED FIXED    
Severity: normal CC: kainz.a, nate, postix
Priority: NOR    
Version First Reported In: 20.08.3   
Target Milestone: ---   
Platform: Other   
OS: Other   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Awesome icon

Description postix 2020-11-27 18:09:33 UTC 
SUMMARY

The very only toolbar button in Elisa which has text is the "show/hide playlist" button. I don't think it's necessary, a mouse hover hint would be enough. 

This would also make space for an "+" "add to playlist" button if one listens to a song before deciding whether to add it to the playlist or not.

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux
KDE Plasma Version: 5.20.3
KDE Frameworks Version: 5.76.0
Qt Version: 5.15.2
Comment 1 Nate Graham 2020-11-30 22:23:52 UTC 
The reason why we use text here is because the icon is not really recognizable without it. In general icons-only-buttons without 100% recognizable icons have poor usability.

Can you think, of find, or create a better icon for us to use?
Comment 2 postix 2020-11-30 23:02:14 UTC 
Created attachment 133761 [details]
Awesome icon

> Can you think, of find, or create a better icon for us to use?

OC hand drawn by me. It has passed the recognition test by my better half. ;)
Comment 3 Nate Graham 2020-11-30 23:12:15 UTC 
That is a much better icon for `view-media-playlist`!

If we update it to look more like that, I'll happily remove the text from the button. Moving this to breeze-icons.

Once this bug report is fixed, I'll push a quick commit to Elisa to do that.
Comment 4 postix 2020-12-01 11:13:57 UTC 
That's great! :)
Comment 5 andreas 2021-11-13 07:40:11 UTC 
icon was updated see https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/159
Comment 6 postix 2021-11-13 12:33:25 UTC 
(In reply to andreas from comment #5)
> icon was updated see
> https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/159

Thank you, looks great! Fixed in 21.12?
Comment 7 Nate Graham 2021-11-15 19:43:57 UTC 
Much better, thanks!

The icon actually lives in frameworks, so it will look prettier in Frameworks 5.89.