[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