Merge pull request #101 from larryng/fix-tap-focus-crash

Fix crash on tap to focus due to miscalculated focus area
pull/1/head
Dylan McIntyre 8 years ago committed by GitHub
commit bc0ad8cd50
  1. 28
      camerakit/src/main/api16/com/flurgle/camerakit/Camera1.java

@ -636,28 +636,28 @@ public class Camera1 extends CameraImpl {
} }
private Rect calculateFocusArea(float x, float y) { private Rect calculateFocusArea(float x, float y) {
int centerX = clamp(Float.valueOf((x / mPreview.getView().getWidth()) * 2000 - 1000).intValue(), getFocusAreaSize()); int buffer = getFocusAreaSize() / 2;
int centerY = clamp(Float.valueOf((y / mPreview.getView().getHeight()) * 2000 - 1000).intValue(), getFocusAreaSize()); int centerX = calculateCenter(x, mPreview.getView().getWidth(), buffer);
int centerY = calculateCenter(y, mPreview.getView().getHeight(), buffer);
return new Rect( return new Rect(
centerX - getFocusAreaSize() / 2, centerX - buffer,
centerY - getFocusAreaSize() / 2, centerY - buffer,
centerX + getFocusAreaSize() / 2, centerX + buffer,
centerY + getFocusAreaSize() / 2 centerY + buffer
); );
} }
private int clamp(int touchCoordinateInCameraReper, int focusAreaSize) { private static int calculateCenter(float coord, int dimen, int buffer) {
int result; int normalized = (int) ((coord / dimen) * 2000 - 1000);
if (Math.abs(touchCoordinateInCameraReper) + focusAreaSize / 2 > 1000) { if (Math.abs(normalized) + buffer > 1000) {
if (touchCoordinateInCameraReper > 0) { if (normalized > 0) {
result = 1000 - focusAreaSize / 2; return 1000 - buffer;
} else { } else {
result = -1000 + focusAreaSize / 2; return -1000 + buffer;
} }
} else { } else {
result = touchCoordinateInCameraReper - focusAreaSize / 2; return normalized;
} }
return result;
} }
} }

Loading…
Cancel
Save