[Merge] lp:~uriboni/camera-app/reset-zoom-ui into lp:camera-app
Ugo Riboni
ugo.riboni at canonical.com
Tue Nov 24 17:32:19 UTC 2015
> We have a binding one way.
>
> Binding { target: camera; property: "currentZoom"; value:
> zoomControl.value }
>
> We ideally would want one the other way no? This could probably be achieved
> without binding loop issues with something like:
>
> Connections {
> target: camera
> onCurrentZoomChanged: zoomControl.value = camera.currentZoom
> }
This actually causes lots of binding loop warnings when zooming, as the value changes:
ViewFinderOverlay.qml:743:13: QML Binding: Binding loop detected for property "value"
I don't think they are a problem, but it still not nice to have them. Should I revert to my previous implementation ?
--
https://code.launchpad.net/~uriboni/camera-app/reset-zoom-ui/+merge/277869
Your team Ubuntu Phablet Team is subscribed to branch lp:camera-app.
More information about the Ubuntu-reviews
mailing list