commit 2958702524348e9e4fcbdf490be731e92b353dad Author: Nate Graham Date: Tue Mar 24 08:10:54 2020 -0600 Stop multiplying duration values Summary: After D28144, some animations are now too long because they were inappropriately multiplying a duration value. That's no longer necessary, so let's remove the multiplication. Depends on D28144 Test Plan: Various things that were a bit too slow before feel just right now Reviewers: #vdg, #plasma, cblack Reviewed By: #vdg, #plasma, cblack Subscribers: cblack, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D28238 diff --git a/applets/devicenotifier/package/contents/ui/FullRepresentation.qml b/applets/devicenotifier/package/contents/ui/FullRepresentation.qml index d15e47ca7..80b9c73cd 100644 --- a/applets/devicenotifier/package/contents/ui/FullRepresentation.qml +++ b/applets/devicenotifier/package/contents/ui/FullRepresentation.qml @@ -225,7 +225,7 @@ MouseArea { popupIconTimer.restart() } } - Behavior on height { NumberAnimation { duration: units.shortDuration * 3 } } + Behavior on height { NumberAnimation { duration: units.shortDuration } } } } } diff --git a/applets/systemtray/package/contents/ui/ExpanderArrow.qml b/applets/systemtray/package/contents/ui/ExpanderArrow.qml index 9baf654b0..729f56b23 100644 --- a/applets/systemtray/package/contents/ui/ExpanderArrow.qml +++ b/applets/systemtray/package/contents/ui/ExpanderArrow.qml @@ -37,7 +37,7 @@ PlasmaCore.ToolTipArea { anchors.fill: parent onClicked: root.expanded = !root.expanded - readonly property int arrowAnimationDuration: units.shortDuration * 3 + readonly property int arrowAnimationDuration: units.shortDuration PlasmaCore.Svg { id: arrowSvg diff --git a/lookandfeel/contents/components/ActionButton.qml b/lookandfeel/contents/components/ActionButton.qml index 9fbd2a8ab..7a66c5642 100644 --- a/lookandfeel/contents/components/ActionButton.qml +++ b/lookandfeel/contents/components/ActionButton.qml @@ -44,7 +44,7 @@ Item { opacity: activeFocus || containsMouse ? 1 : 0.85 Behavior on opacity { PropertyAnimation { // OpacityAnimator makes it turn black at random intervals - duration: units.longDuration * 2 + duration: units.longDuration easing.type: Easing.InOutQuad } } @@ -59,7 +59,7 @@ Item { opacity: activeFocus || containsMouse ? (softwareRendering ? 0.8 : 0.15) : (softwareRendering ? 0.6 : 0) Behavior on opacity { PropertyAnimation { // OpacityAnimator makes it turn black at random intervals - duration: units.longDuration * 3 + duration: units.longDuration easing.type: Easing.InOutQuad } } diff --git a/lookandfeel/contents/components/UserDelegate.qml b/lookandfeel/contents/components/UserDelegate.qml index 8882e39ae..cdb55f1a7 100644 --- a/lookandfeel/contents/components/UserDelegate.qml +++ b/lookandfeel/contents/components/UserDelegate.qml @@ -72,7 +72,7 @@ Item { Behavior on width { PropertyAnimation { from: faceSize - duration: units.longDuration * 2; + duration: units.longDuration; } } width: isCurrent ? faceSize : faceSize - units.largeSpacing