Skip to content

feat(carplay): wire Car Tabs setting to the browse-grid category list#610

Open
mvanhorn wants to merge 1 commit into
music-assistant:devfrom
mvanhorn:feat/576-carplay-car-settings-wiring
Open

feat(carplay): wire Car Tabs setting to the browse-grid category list#610
mvanhorn wants to merge 1 commit into
music-assistant:devfrom
mvanhorn:feat/576-carplay-car-settings-wiring

Conversation

@mvanhorn

Copy link
Copy Markdown

Summary

CarPlay now honors the Car Settings tab/action preferences configured in Settings → Car.

  • Browse grid tabs: The Browse grid now reads SettingsRepository.carTabsConfig (the same store Android Auto uses) via a new carBrowseCategories() bridge method in KmpHelper. Categories are shown in the user-configured order; disabled categories are hidden; unconfigured categories appear in the default order.
  • Item tap actions: Already wired in a prior PR via KmpHelper.playCarDefaultTap. No change needed.
  • Bulk actions: Already wired via KmpHelper.carBulkActionNames / playCarAction. No change needed.

Note: a full iOS/CarPlay build requires Xcode and the KMP toolchain; CI is the verification gate for compilation.

Why this matters

Issue #576 reported that changing the Car Settings tabs and item action preferences had no effect on the CarPlay UI. The settings were persisted but the CarPlay layer did not read them. Maintainer seadowg confirmed 2025-06-11 the issue is purely about the settings not affecting the CarPlay UI.

Fixes #576

CarPlay's Browse grid was hardcoded to show all 7 categories regardless
of the Settings → Car → Tabs preference. This adds `carBrowseCategories()`
to KmpHelper (reads `SettingsRepository.carTabsConfig`, same source Android
Auto uses) and threads it through CarPlayContentManager to pushBrowseGrid,
which now builds the grid from the user-configured order/visibility instead
of a fixed list.

Item tap actions and bulk actions (Settings → Car → Item Actions) were
already wired in a prior PR and remain unchanged.

Fixes music-assistant#576
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.

Update CarPlay to make use of Car Settings (Tabs and Item actions)

1 participant