Skip to content

feat: expose map zoom limits#949

Open
arpitagarwal1301 wants to merge 1 commit into
googlemaps:mainfrom
arpitagarwal1301:feat/472-camera-zoom-limits
Open

feat: expose map zoom limits#949
arpitagarwal1301 wants to merge 1 commit into
googlemaps:mainfrom
arpitagarwal1301:feat/472-camera-zoom-limits

Conversation

@arpitagarwal1301

Copy link
Copy Markdown

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the available local tests and linter pass
  • Code coverage does not decrease (focused instrumentation coverage is included)
  • Appropriate docs were updated (public API KDoc was added)

Fixes #472 🦕

Summary

  • expose minZoomLevel and maxZoomLevel as read-only nullable properties on CameraPositionState
  • delegate both values to the currently bound GoogleMap, returning null while unbound
  • add focused instrumentation coverage for the unbound and map-loaded states

Testing

  • ./gradlew :maps-compose:compileDebugKotlin :maps-app:compileDebugAndroidTestKotlin --stacktrace --no-daemon
  • ./gradlew build jacocoTestReport :maps-app:validateDebugScreenshotTest -x :maps-app:generateDebugScreenshotTestConfig -x :maps-app:generateReleaseScreenshotTestConfig --stacktrace --no-daemon
  • git diff --check

The API-key-dependent connected instrumentation test compiled locally but was not run because the checkout has only the placeholder key; the repository instrumentation workflow injects its Maps API secret.

AI disclosure

This focused change was prepared and independently reviewed with AI assistance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide access to map maxZoomLevel

2 participants