Ubuntu Notes app

Merge lp:~dpm/reminders-app/fix-autofocus-1342252 into lp:reminders-app

  1. fix-autofocus-1342252
  2. Merge into trunk
Proposed by David Planella
Status: Merged
Approved by: Michael Zanetti
Approved revision: 196
Merged at revision: 197
Proposed branch: lp:~dpm/reminders-app/fix-autofocus-1342252
Merge into: lp:reminders-app
Diff against target: 12 lines (+1/-1)
1 file modified
src/app/qml/ui/CameraPage.qml (+1/-1)
To merge this branch: bzr merge lp:~dpm/reminders-app/fix-autofocus-1342252
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Michael Zanetti (community) Approve
Review via email: mp+228244@code.launchpad.net

Commit message

Make the autofocus "just work"

Description of the change

Makes the autofocus work. Having continuous autofocus might not be the best for the battery, but for the purposes of the app, and to keep it simple and most importantly avoid blurry pictures, should be a good solution.

To post a comment you must log in.
Revision history for this message
Michael Zanetti (mzanetti) wrote :

Yep, this is what I do in Tagger too.

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/app/qml/ui/CameraPage.qml'
2--- src/app/qml/ui/CameraPage.qml 2014-02-25 15:07:24 +0000
3+++ src/app/qml/ui/CameraPage.qml 2014-07-25 06:03:04 +0000
4@@ -46,7 +46,7 @@
5 Camera {
6 id: camera
7 flash.mode: Camera.FlashOff
8- focus.focusMode: Camera.FocusAuto
9+ focus.focusMode: Camera.FocusContinuous
10 focus.focusPointMode: Camera.FocusPointAuto
11
12 property alias currentZoom: camera.digitalZoom

Subscribers

People subscribed via source and target branches