[Folder View] Fix action button hover and pressed state
ClosedPublic

Authored by broulik on Mar 3 2017, 11:09 AM.

Details

Summary

The hover state didn't work because it referenced "pressed" which it didn't find in the scope.
The pressed graphic suffix was wrong.

Test Plan

The hovered and pressed icons aren't part of Breeze Plasma theme (will fix that as part of Bug 376321) but when using Air/Oxygen it works nicely now. Tested both add, remove, and popup buttons.

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Mar 3 2017, 11:09 AM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 3 2017, 11:09 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
hein accepted this revision.Mar 3 2017, 4:17 PM
This revision is now accepted and ready to land.Mar 3 2017, 4:17 PM
This revision was automatically updated to reflect the committed changes.


This broke hover in FolderView at all (Qt 5.8.0, Kf5.32, Plasma 5.9.3 + this patch)

This change has nothing to do with that and we already fixed it for the next Frameworks and Plasma release.