Differential D3828

[Plasma ScrollViewStyle] Show scroll bar background only on hover
ClosedPublic
Actions

Authored by broulik on Dec 27 2016, 9:49 PM.

Details

Reviewers
mart
Group Reviewers
Plasma: Design
Plasma
Commits
R242:8675a31d2a99: [Plasma ScrollViewStyle] Show scroll bar background only on hover
Summary

This makes it consistent with the Breeze widget style.
The other themes (Air and Oxygen) didn't even have a scroll bar background.

Test Plan

Works. Looks a bit weird when I move my mouse directly to the handle as the SVG changes to hover with no transition but you still get the background fading in.

Diff Detail

Repository
R242 Plasma Framework (Library)
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 9403.Dec 27 2016, 9:49 PM
broulik retitled this revision from to [Plasma ScrollViewStyle] Show scroll bar background only on hover.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added reviewers: Plasma, Plasma: Design.
broulik set the repository for this revision to R242 Plasma Framework (Library).
Restricted Application added a project: Plasma. · View Herald TranscriptDec 27 2016, 9:49 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Dec 28 2016, 11:15 AM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Dec 28 2016, 11:15 AM
Closed by commit R242:8675a31d2a99: [Plasma ScrollViewStyle] Show scroll bar background only on hover (authored by broulik). · Explain WhyDec 31 2016, 12:45 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 9545

View Options

src/declarativeimports/plasmastyle/ScrollViewStyle.qml

Loading...