Add "veryLongDuration" animation duration
ClosedPublic

Authored by ngraham on Thu, Mar 19, 5:22 PM.

Details

Summary

As discussed in D28033#630649.

Test Plan

Compiles. Not used anywhere yet, so no testing possible.

Diff Detail

Repository
R169 Kirigami
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.Thu, Mar 19, 5:22 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptThu, Mar 19, 5:22 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Thu, Mar 19, 5:22 PM
ngraham updated this revision to Diff 78037.Thu, Mar 19, 6:29 PM

Forgot one thing

mart added a comment.Fri, Mar 20, 3:03 PM

a problem tough.. this one won't follow system settngs, so globally disabling anoimations won't disable this one
it also needs implementing in plasma-framework/src/declarativeimports/kirigamiplasmadesktopstyle/ (where takes values from units.cpp in plasma units, which should get a very long duration as well.. those do follow global system durations, so would solve the problem)

mart requested changes to this revision.Fri, Mar 20, 3:03 PM
This revision now requires changes to proceed.Fri, Mar 20, 3:03 PM
davidre added a subscriber: davidre.EditedFri, Mar 20, 3:04 PM
In D28143#631225, @mart wrote:

a problem tough.. this one won't follow system settngs, so globally disabling anoimations won't disable this one
it also needs implementing in plasma-framework/src/declarativeimports/kirigamiplasmadesktopstyle/ (where takes values from units.cpp in plasma units, which should get a very long duration as well.. those do follow global system durations, so would solve the problem)

See D28144

mart accepted this revision.Fri, Mar 20, 3:38 PM

sorry, didn't see the other one

This revision is now accepted and ready to land.Fri, Mar 20, 3:38 PM
This revision was automatically updated to reflect the committed changes.