Bug 406476 - Improve grid delegates' hover button usability
Summary: Improve grid delegates' hover button usability
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-04-12 15:18 UTC by Nate Graham
Modified: 2019-04-23 02:56 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Grid delegates: Elisa vs Lollypop (172.94 KB, image/png)
2019-04-12 15:18 UTC, Nate Graham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2019-04-12 15:18:07 UTC
Created attachment 119381 [details]
Grid delegates: Elisa vs Lollypop

Elisa from git master.

The grid delegates in genre/artist/album views have three buttons that appear on mouse hover. However they suffer from some issues:
- The grid delegates are quite small
- The entire grid delegate is not clickable; you need to click on one of the individual icons
- All three icons are the same size
- When the icons appear, the art behind it becomes washed out and you can no longer see what it is

I really like the way Lollypop handles this same situation. There, the grid delegates are larger, and the whole thing is always clickable; a click takes you into that item to the next level of navigation. However on hover, additional buttons appear allowing you to play all items in it or add it to the current playlist. These buttons cover up some of the art, but most of it is still visible.

See attached comparison screenshot.
Comment 1 Nate Graham 2019-04-23 02:56:40 UTC
Git commit 761e3af65347072f12b770529a5effb6b39b6e58 by Nate Graham.
Committed on 23/04/2019 at 02:56.
Pushed by ngraham into branch 'master'.

Revamp grid browser delegates to improve UI

Summary:
This patch changes the grid browser delegates in the following ways:
- Labels are centered
- Album art size increases from 100px to 170px, making the album view more attractive and leaving more room for labels
- Hover buttons now appear in the corner, have borders, and no longer make the album art get washed out when the delegate is hovered
- Some minor layout and positioning bugs are fixed

As a result, the following bugs are fixed:
Related: bug 406468

Also, the last usage of QQC1 in this component is removed as a side effect.

Test Plan:
Everything still works and there are no behavioral changes.

{F6780036}

Reviewers: mgallien, #elisa

Reviewed By: mgallien, #elisa

Differential Revision: https://phabricator.kde.org/D20581

M  +83   -63   autotests/qmltests/tst_GridBrowserDelegate.qml
M  +2    -2    src/qml/BaseTheme.qml
M  +62   -61   src/qml/GridBrowserDelegate.qml

https://commits.kde.org/elisa/761e3af65347072f12b770529a5effb6b39b6e58