[Device Notifier] Fix transition animation

Authored by thsurrel on Dec 6 2018, 9:50 PM.



In the specific case of having the device notifier opened
and inserting a device that has been configured to be
automatically mounted, the transition animation could be
interrupted resulting in the opacity being wrong.

Test Plan

In the setup discribed in the summary, you should never
see something like the attached screenshot where the list
of action opacity is not 1.

See http://doc.qt.io/qt-5/qml-qtquick-viewtransition.html#handling-interrupted-animations
for more information.

Diff Detail

R120 Plasma Workspace
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
thsurrel created this revision.Dec 6 2018, 9:50 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 6 2018, 9:50 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
thsurrel requested review of this revision.Dec 6 2018, 9:50 PM
thsurrel edited the test plan for this revision. (Show Details)Dec 6 2018, 9:51 PM
ngraham accepted this revision.Dec 7 2018, 12:36 AM
ngraham added a subscriber: ngraham.

Nice catch! This works and looks good to me, but let's make sure the Plasma folks agree. :)

This revision is now accepted and ready to land.Dec 7 2018, 12:36 AM
davidedmundson added inline comments.

alwaysRunToEnd : true

Is this Bug 379759?

thsurrel added inline comments.Dec 7 2018, 11:38 AM

That does not work by itself.

davidedmundson accepted this revision.Dec 7 2018, 12:06 PM

Is this Bug 379759?

I don't think so, but I will have a look at that one too.

If any of you guys can have a look at my other device notifier patches, that would be really nice!

This revision was automatically updated to reflect the committed changes.