Stop multiplying duration values
ClosedPublic

Authored by ngraham on Mar 24 2020, 2:16 PM.

Details

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

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Mar 24 2020, 2:16 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 24 2020, 2:16 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Mar 24 2020, 2:16 PM
cblack accepted this revision as: Plasma, cblack.Mar 24 2020, 2:21 PM
cblack added a subscriber: cblack.

Code looks good to me, but I haven't looked at how these changes look visually.
These are where these changes will be visible, right?

  • Clicking a device in the system tray device list
  • System tray arrow

I'm not sure where you can see the changes to ActionButton.qml and UserDelegate.qml.

This revision is now accepted and ready to land.Mar 24 2020, 2:21 PM

I'm not sure where you can see the changes to ActionButton.qml and UserDelegate.qml.

the lock, logout, and login screens

cblack accepted this revision.Mar 24 2020, 5:21 PM
This revision was automatically updated to reflect the committed changes.