commit 2958702524348e9e4fcbdf490be731e92b353dad
Author: Nate Graham <nate@kde.org>
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