Skip to content

Add Apple map presentation outside Compose UI - #1384

Merged
sargunv merged 4 commits into
mainfrom
feat/apple-map-presentation
Sep 13, 2026
Merged

Add Apple map presentation outside Compose UI#1384
sargunv merged 4 commits into
mainfrom
feat/apple-map-presentation

Conversation

@sargunv

@sargunv sargunv commented Sep 10, 2026

Copy link
Copy Markdown
Member

Description

Add AppleMapPresentation and MaplibreMapView for hosting maps in UIKit, SwiftUI, and CarPlay on iOS. The view owns its Metal layer and layout; custom hosts can attach a caller-owned layer. The iOS MaplibreMap composable uses the same presentation path while retaining its caller's composition locals and frame clock.

Standalone Apple and Android presentations share the runtime composition host. Closing a presentation disposes its style effects and detaches rendering while retaining the caller's map state. Tests cover ownership, failure cleanup, detached recomposition, layer replacement, and UIKit layout.

Add a CarPlay scene and share a small map demo with Android Auto. Zoom buttons use eased camera animations. Simplify Wear OS to a map and a credits button. Consolidate hosting documentation and fix Metal-layer KDoc references so docs and publishing pass on Linux.

Closes #1373.

Validation

  • mise run test:ios: 774 map tests and 14 location tests passed.
  • mise run build:docs passed, including API generation and internal link checks.
  • Android Auto and Wear OS debug APKs built successfully.
  • Built and launched the iOS demo with Xcode 26.5 on the iOS 26.5 simulator; visually verified map rendering through the shared presentation path.
  • mise run check passed.

CarPlay appears in the simulator launcher, but external-display touch automation could not open it; end-to-end CarPlay interaction and physical CarPlay remain unverified. Hardware CarPlay requires an approved signing profile. Fresh remote CI is pending.

AI assistance

GPT-6 in Codex implemented the original design and performed the code review, rebase, fixes, and validation. This description was drafted with the same assistance.

@sargunv
sargunv force-pushed the feat/apple-map-presentation branch from 1b7be18 to f3a980d Compare September 12, 2026 19:51
@sargunv
sargunv marked this pull request as ready for review September 12, 2026 19:51
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-12T19:58:46.221173Z f3a980d Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f3a980dda1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread demo-app/ios/iosApp/CarPlaySceneDelegate.swift
Comment thread demo-app/ios/iosApp/CarPlaySceneDelegate.swift
@sargunv
sargunv merged commit 914b583 into main Sep 13, 2026
21 checks passed
@sargunv
sargunv deleted the feat/apple-map-presentation branch September 13, 2026 09: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.

Add AppleMapPresentation for maps outside Compose UI

1 participant