Bug 429737 - view-media-playlist icon could stand to be improved
Summary: view-media-playlist icon could stand to be improved
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 20.08.3
Platform: Other Other
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-27 18:09 UTC by postix
Modified: 2021-11-15 19:43 UTC (History)
3 users (show)

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


Attachments
Awesome icon (11.06 KB, image/png)
2020-11-30 23:02 UTC, postix
Details

Note You need to log in before you can comment on or make changes to this bug.
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.