Skip to content

feat(java): restore Java UiControlsActivity and UiControlSnippets for SDK 16.0.1 - #84

Open
dkhawk wants to merge 1 commit into
mainfrom
feat/java-ui-controls
Open

dkhawk wants to merge 1 commit into
mainfrom
feat/java-ui-controls

Conversation

@dkhawk

@dkhawk dkhawk commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

Restores the Java Map3DControl UI controls showcase (UiControlsActivity.java) and snippets (UiControlSnippets.java) that were split out of #78 while awaiting the play-services-maps3d:16.0.1 release.

Context

  • In play-services-maps3d:16.0.0, com.google.android.gms.maps3d.Map3DControl.class contained a malformed RuntimeInvisibleParameterAnnotations attribute (num_parameters = 2 on a 1-parameter constructor), which caused javac on JDK 21+ to crash with java.lang.IndexOutOfBoundsException whenever Java source referenced Map3DControl.
  • The bytecode issue is fixed in play-services-maps3d:16.0.1 (maps3d.android.sdk_20260831_RC03). Once 16.0.1 is published to Google Maven and gradle/libs.versions.toml is bumped, this PR can be merged to complete 3-way Java/Kotlin/Compose parity for the UI Controls showcase.

Changes Included

  • Maps3DSamples/ApiDemos/java-app:
    • Restores UiControlsActivity.java demonstrating interactive placement and orientation controls for all 6 Map3DControl widgets (MAP_MODE_PICKER, COMPASS, TILT, ROTATE, ZOOM, DEFAULT_CAMERA).
    • Registers .uicontrols.UiControlsActivity in AndroidManifest.xml, MainActivity.java, and MainActivityTest.java.
  • snippets/java-app:
    • Restores UiControlSnippets.java and registers the 5 UI control snippets in SnippetRegistry.java.

@dkhawk
dkhawk requested review from LoyalAbbas and kikoso October 14, 2026 23:24
Base automatically changed from feat/quick-settings-widget to main October 14, 2026 23:28
@dkhawk
dkhawk force-pushed the feat/java-ui-controls branch from 6c23706 to 31680de Compare October 14, 2026 23:33
@dkhawk
dkhawk marked this pull request as ready for review October 14, 2026 23:34
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.

1 participant