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

Reply via email to