Skip to content

General: Polish overview and device settings, persist learned ANC#522

Merged
d4rken merged 10 commits into
mainfrom
more-polishing
Apr 17, 2026
Merged

General: Polish overview and device settings, persist learned ANC#522
d4rken merged 10 commits into
mainfrom
more-polishing

Conversation

@d4rken

@d4rken d4rken commented Apr 17, 2026

Copy link
Copy Markdown
Member

What changed

  • Overview dashboard polish: permission cards and the device-limit
    upgrade card now show a small title icon; inactive device cards
    collapse to reduce clutter; the upgrade card is full-width.
  • Device settings screen split into clean section cards (Noise Control,
    Reactions, Sound, Controls). ANC-with-one-pod is a dedicated toggle.
    The listening-mode cycle row no longer shows Edit/Upgrade chip labels.
  • ANC mode and listening-mode settings learned from a device now
    persist across reconnects, so your preferences stick.

Technical Context

  • AAP session components were moved into a new engine subpackage under
    pods.core.apple.aap for clearer boundaries; no behavior change.
  • DeviceSettingsScreen.kt was broken out from a 1048-line monolith
    into per-section card files and shared components.
  • New AapLearnedSettingsPersister captures AllowOffOption and
    ListeningModeCycle off the AAP stream and writes them onto the
    AppleDeviceProfile; on reconnect the profile rehydrates the session.
  • Permission-card icons are resolved via a private UI-layer
    Permission.iconVector() mapper; the Permission enum stays free of
    Compose types so JVM tests/non-UI code aren't affected.

Review checklist

  • Toggle ANC/Transparency, disconnect, reconnect — the previous
    mode is restored from the stored profile.
  • Inactive device cards on the overview collapse and re-expand.
  • Permission-card title icons render correctly on both the
    normal and scan-blocking (errorContainer) variants.
  • All device-settings sections render without regression on a
    real device (not just previews).

@d4rken d4rken added enhancement Add a new feature of improve an existing feature device support coms/AAP Uses Apples AirPod Protocol. Requires Android ROM with fixed L2CAP support on the Bluetooth sockets. labels Apr 17, 2026
@d4rken d4rken merged commit e7c280c into main Apr 17, 2026
10 checks passed
@d4rken d4rken deleted the more-polishing branch April 17, 2026 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

coms/AAP Uses Apples AirPod Protocol. Requires Android ROM with fixed L2CAP support on the Bluetooth sockets. device support enhancement Add a new feature of improve an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant