VolumeSlider: Ignore all value changes until Component is completed
ClosedPublic

Authored by drosca on Jan 24 2017, 12:58 PM.

Details

Summary

BUG: 375495
FIXED-IN: 5.9.0

Test Plan

Mute is no longer changed on first open of popup, feedback is also
no longer played in this case.

Diff Detail

Repository
R115 Plasma Audio Volume Applet
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
drosca retitled this revision from to VolumeSlider: Ignore all value changes until Component is completed.Jan 24 2017, 12:58 PM
drosca updated this object.
drosca edited the test plan for this revision. (Show Details)
drosca added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptJan 24 2017, 12:58 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson added inline comments.
applet/contents/ui/ListItemBase.qml
163

can you not just add

if (Component.status != Ready) to the top of this method

and get rid of the bool?

drosca added inline comments.Jan 24 2017, 11:07 PM
applet/contents/ui/ListItemBase.qml
163

No, because Component.status is not attached property and so is not accessible from there.

davidedmundson accepted this revision.Jan 25 2017, 12:05 AM
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.Jan 25 2017, 12:05 AM
This revision was automatically updated to reflect the committed changes.