feat: Modernize Android samples UI, edge-to-edge insets, and demo verification improvements - #2389
Draft
dkhawk wants to merge 4 commits into
Draft
feat: Modernize Android samples UI, edge-to-edge insets, and demo verification improvements#2389dkhawk wants to merge 4 commits into
dkhawk wants to merge 4 commits into
Conversation
…ML and Multilayer snippets - Use safe fallback `?: ""` instead of force-unwrap `!!` when accessing KML container properties. - Use `lateinit var` for map and context fields in Multilayer snippet to eliminate unnecessary nullable assertions (`!!`).
…ion improvements - Modernize CameraClampingDemo with Material 3 RangeSlider, clamp toggle group, and live camera telemetry - Fix SaveStateDemoActivity parcelable state retention across orientations - Improve dark/light theme contrast and custom InfoWindow layouts in MarkerDemoActivity - Add 60fps OnCameraMoveListener and clean coordinate formatting in EventsDemoActivity and CameraClampingDemoActivity - Initialize LayersDemoActivity to Sydney CBD with visible 3D buildings, traffic, and indoor floorplans - Apply edge-to-edge window insets consistently across Kotlin and Java samples
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…ign WearOS Java toolchain
…mat, and borderless button styles
| tapTextView.text = "tapped, point=$point" | ||
| val lat = String.format(Locale.US, "%.6f", point.latitude) | ||
| val lng = String.format(Locale.US, "%.6f", point.longitude) | ||
| tapTextView.text = "Tapped: ($lat, $lng)" |
| tapTextView.text = "tapped, point=$point" | ||
| val lat = String.format(Locale.US, "%.6f", point.latitude) | ||
| val lng = String.format(Locale.US, "%.6f", point.longitude) | ||
| tapTextView.text = "Tapped: ($lat, $lng)" |
| tapTextView.text = "long pressed, point=$point" | ||
| val lat = String.format(Locale.US, "%.6f", point.latitude) | ||
| val lng = String.format(Locale.US, "%.6f", point.longitude) | ||
| tapTextView.text = "Long pressed: ($lat, $lng)" |
| tapTextView.text = "long pressed, point=$point" | ||
| val lat = String.format(Locale.US, "%.6f", point.latitude) | ||
| val lng = String.format(Locale.US, "%.6f", point.longitude) | ||
| tapTextView.text = "Long pressed: ($lat, $lng)" |
| val zoom = String.format(Locale.US, "%.1f", pos.zoom) | ||
| val tilt = String.format(Locale.US, "%.1f", pos.tilt) | ||
| val bearing = String.format(Locale.US, "%.1f", pos.bearing) | ||
| cameraTextView.text = "Position: ($lat, $lng)\nzoom=$zoom, tilt=$tilt°, bearing=$bearing°" |
|
|
||
| override fun onStreetViewPanoramaChange(location: StreetViewPanoramaLocation) { | ||
| panoChangeTimesTextView.text = "Times panorama changed=" + ++panoChangeTimes | ||
| panoChangeTimesTextView.text = "Times panorama changed=${++panoChangeTimes}" |
|
|
||
| override fun onStreetViewPanoramaCameraChange(camera: StreetViewPanoramaCamera) { | ||
| panoCameraChangeTextView.text = "Times camera changed=" + ++panoCameraChangeTimes | ||
| panoCameraChangeTextView.text = "Times camera changed=${++panoCameraChangeTimes}" |
|
|
||
| override fun onStreetViewPanoramaCameraChange(camera: StreetViewPanoramaCamera) { | ||
| panoCameraChangeTextView.text = "Times camera changed=" + ++panoCameraChangeTimes | ||
| panoCameraChangeTextView.text = "Times camera changed=${++panoCameraChangeTimes}" |
| point?.let { | ||
| panoClickTimes++ | ||
| panoClickTextView.text = "Times clicked=$panoClickTimes : $point" | ||
| panoClickTextView.text = "Times clicked=$panoClickTimes : $it" |
| point?.let { | ||
| panoClickTimes++ | ||
| panoClickTextView.text = "Times clicked=$panoClickTimes : $point" | ||
| panoClickTextView.text = "Times clicked=$panoClickTimes : $it" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR modernizes the Google Maps Platform Android ApiDemos suite with Material 3 components, comprehensive edge-to-edge window inset handling, dark/light theme contrast fixes, state retention bugfixes, and real-time telemetry improvements.
Key Changes
CameraClampingDemoActivity):RangeSlider([2.0, 21.0], step0.5).MaterialButtonToggleGroupfor Adelaide, Pacific, and Reset bounds with visible active states.GoogleMap.setOnCameraMoveListener.IllegalStateExceptionbounds mismatch crash on Reset Zoom.SaveStateDemoActivity):MarkerDemoActivity):CameraDemoActivity):EventsDemoActivity):OnCameraMoveListenertelemetry tracking.LayersDemoActivity):LatLng(-33.8688, 151.2093), zoom 16.5, tilt 40°) so 3D buildings, traffic overlays, and indoor floorplans are immediately visible.MaterialCheckBox.SamplesBaseActivityacross both Kotlin and Java samples.