google / google/ground-android

[Code health] Cleanup CameraPosition object

Open
#1,712 0 comments 0 reactions 0 assignees View on GitHub
type: code health
Dominant language
Kotlin
Stars
293
Forks
149
Avg merge
1d 16h
Merged PRs (30d)
42

Description

I didn't realize we have bounds here; essentially (`target`, `zoomLevel`) and `bounds` model the same info. Also, `isAllowZoomOut` is an arg to an event, not to the current camera state. Can you file a bug (against me?) to clean this up and add a TODO here?

_Originally posted by @gino-m in https://github.com/google/ground-android/pull/1711#discussion_r1232525088_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.