| Summary: | Add hover and pressed states in list view's buttons | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] frameworks-kirigami | Reporter: | matheus4551 |
| Component: | general | Assignee: | Marco Martin <notmart> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | nate |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | Not decided | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/kirigami/bdb5465181b6e03ff0141111d3da95791121e15d | Version Fixed/Implemented In: | 5.56 |
| Sentry Crash Report: | |||
|
Description
matheus4551
2019-01-26 21:55:53 UTC
Huge +1. Looks like the problem is that these buttons are basically custom-synthesized fake toolbuttons instead of real toolbuttons. I'll see what I can do. Wasn't as hard as I thought. Here's a patch: https://phabricator.kde.org/D18908 Git commit bdb5465181b6e03ff0141111d3da95791121e15d by Nate Graham.
Committed on 11/02/2019 at 16:02.
Pushed by ngraham into branch 'master'.
[SwipeListItem] Use a real toolbutton to improve usability
Summary:
Use a real toolbutton instead of a custom icon + mouse area, which
provides hovered and clicked states automatically and simplifies
the code a bit.
FIXED-IN: 5.56
Test Plan:
Hovered state: {F6605906, size=full}
Pressed state: {F6605907, size=full}
Reviewers: mart, #kirigami
Reviewed By: mart, #kirigami
Subscribers: apol, plasma-devel
Tags: #kirigami
Differential Revision: https://phabricator.kde.org/D18908
M +15 -23 src/controls/templates/SwipeListItem.qml
https://commits.kde.org/kirigami/bdb5465181b6e03ff0141111d3da95791121e15d
|