D13931: [KCM] Add speaker placement test

[David Rosca]

drosca added inline comments.

INLINE COMMENTS

> nicolasfella wrote in Advanced.qml:117
> When I remove this line and switch profile e.g. from Stereo to 5.1 the UI doesn't get updated. I'm no QML expert but it looks like a change to sinkmodel does not result in reevaluation of data(). I'm overriding the binding with an identical one, so I guess it should be fine

What you do here is assignment to the property, not property binding. So here you actually remove the binding.

Binding would be: `grid.pulseObject = Qt.binding(function() { return sinkmodel .... })`

In any case, dealing with C++ models outside of delegates is stupid in QML, and QQC1 ComboBox doesn't help.
I've tried to fix it myself and the only thing I came up with is this "brute-force" approach: https://paste.kde.org/po8v22yzp

REPOSITORY
  R115 Plasma Audio Volume Applet

REVISION DETAIL
  https://phabricator.kde.org/D13931

To: nicolasfella, drosca
Cc: ngraham, #vdg, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180714/ab60ff9d/attachment.html>