feat(ui): complete ABox app experience redesign #10

2026-08-27T23:22:50+08:00

KremeCN commented

2026-08-27 23:22:50 +08:00

Summary

rebuild the authenticated shell, device dashboard, device detail pages, scenes, activity timeline, account sheet, and settings experience
redesign infrared remote management and controls, including custom remotes, learn-state handling, slot isolation, and safe default content
add persisted theme controls with internal accent, color-style, and color-standard resolution while keeping dynamic color authoritative
add ABox profile defaults for the three exact gateway device names, editable starter scenes, and custom infrared remotes
persist real activity events for device commands, infrared actions, and scene execution
stabilize navigation and back behavior, including duplicate top-back protection and removal of the unfinished predictive-back option
update launcher icons and bilingual application name

Verification

:android:app:assembleDebug
installed the debug APK on emulator-5554
manually exercised device detail navigation, settings/theme navigation, infrared management, activity records, and repeated back actions

## Summary - rebuild the authenticated shell, device dashboard, device detail pages, scenes, activity timeline, account sheet, and settings experience - redesign infrared remote management and controls, including custom remotes, learn-state handling, slot isolation, and safe default content - add persisted theme controls with internal accent, color-style, and color-standard resolution while keeping dynamic color authoritative - add ABox profile defaults for the three exact gateway device names, editable starter scenes, and custom infrared remotes - persist real activity events for device commands, infrared actions, and scene execution - stabilize navigation and back behavior, including duplicate top-back protection and removal of the unfinished predictive-back option - update launcher icons and bilingual application name ## Verification - `:android:app:assembleDebug` - installed the debug APK on `emulator-5554` - manually exercised device detail navigation, settings/theme navigation, infrared management, activity records, and repeated back actions

KremeCN added 40 commits 2026-08-27 23:22:50 +08:00

build: modernize Android Compose toolchain 94e3e394af

test: add Compose screenshot baseline b39b3f95c7

feat: persist global UI preferences 79b47fc354

fix: rethrow UI preference transform IO failures 686176b148

feat: add dual-renderer design tokens 4055c53362

fix: contain Material renderer adapter cae2ea9ce0

feat: add semantic ABox UI components 063a3f8dab

fix: correct ABox switch semantics and motion b24ca98e14

feat: add persistent interface settings 283f56a3b6

feat: separate session and credential actions c1c21551fb

feat: add adaptive authenticated navigation 54eef78a46

fix: stabilize adaptive navigation state 05d44578b9

feat: redesign account and provider dialog 6e78b71d31

feat: add profile scoped device catalog e14b4cee27

feat: replace device shortcuts with device center 876acd1e7b

feat: confirm socket controls by readback a30ab9e376

feat: render partial sensor readings 9ba17c7d72

feat: unify device detail presentation 171d1daa50

fix: isolate infrared slots by hardware b537cd1824

feat: preserve infrared ownership in schema 4798ae32ba

feat: route infrared through selected device 26ab5095e5

fix: clean up infrared learning cancellation 176a70cf7f

feat: define activity event contracts 25ed255ddf

feat: add automation persistence foundation 867096d7ef

feat: add observable scene execution foundation ca4c34f21c

feat: stream scene execution progress 713106df14

feat: evaluate automation conditions safely cce0441476

feat: add fail closed automation evaluation fd67820c63

feat: guard automation scheduling by profile 5a25c31b39

fix: reject unconfigured scene device fallbacks 1139606f9f

feat: select configured devices in scene editor 8e3d7e604b

feat: add persisted activity timeline 7afcea296e

feat: add isolated activity event writer 0e4a50b3dd

docs: record device data capability boundaries 7a107cc55a

fix: remove legacy text icons and placeholders d8d9eb4f28

test: align infrared UI instrumentation coverage cd4a5ea04c

docs: update release hardware validation 816606234b

feat(ui): complete app experience redesign 54aa74126f

feat(ui): update app launcher icons and bilingual app name bdeddf46e2

fix(ui): stabilize theme and navigation behavior a5f31486b4

KremeCN merged commit 86b7e5849b into main

2026-08-27 23:25:08 +08:00

KremeCN deleted branch feat/ui-ux-redesign

2026-08-27 23:25:08 +08:00

pr-agent-bot bot reviewed 2026-08-27 23:27:38 +08:00

pr-agent-bot bot left a comment

Inline comment

android/app/src/main/kotlin/com/flagship/abox/manager/ui/navigation/AuthenticatedNavigation.kt

						
				@@ -314,5 +314,5 @@

				) {

				    val activeBackStackEntry by navController.currentBackStackEntryAsState()

				    val activeRoute = activeBackStackEntry?.actualRoute()

				    var backNavigationInFlight by remember { mutableStateOf(false) }

				    LaunchedEffect(activeRoute) {

pr-agent-bot bot commented

2026-08-27 23:27:38 +08:00

返回锁死

backNavigationInFlight 只在 activeRoute 字符串变化时重置。快速双击同一场景或设备使相同路由连续入栈后，第一次返回仍会落在相同路由，LaunchedEffect 不会重新执行，导致该页面后续的顶部返回操作全部失效。应根据返回栈条目身份重置，或在弹栈完成后可靠地清除锁定状态。

**返回锁死** `backNavigationInFlight` 只在 `activeRoute` 字符串变化时重置。快速双击同一场景或设备使相同路由连续入栈后，第一次返回仍会落在相同路由，`LaunchedEffect` 不会重新执行，导致该页面后续的顶部返回操作全部失效。应根据返回栈条目身份重置，或在弹栈完成后可靠地清除锁定状态。

pr-agent-bot bot reviewed 2026-08-27 23:27:38 +08:00

pr-agent-bot bot left a comment

Inline comment

android/app/src/main/kotlin/com/flagship/abox/manager/ui/ABoxManagerApp.kt

						
				@@ -134,4 +111,0 @@

				                        onAddAutomatic = { navController.navigate(SceneRoutes.EDITOR_AUTOMATIC) },

				                        onEditScene = { sceneId -> navController.navigate(SceneRoutes.editorFor(sceneId)) },

				                        onExecuteScene = { sceneId -> navController.navigate(SceneRoutes.executionFor(sceneId)) },

				                    )

pr-agent-bot bot commented

2026-08-27 23:27:38 +08:00

设置不可达

账户弹窗是当前可见代码中唯一会打开设置界面的入口，但它始终调用 showTheme(returnToHome = false)。用户会直接进入主题详情，返回时又关闭整个设置层，因此设置首页中的渲染器、触感等选项无法访问。常规设置入口应先调用 showHome()，或允许主题详情返回设置首页。

**设置不可达** 账户弹窗是当前可见代码中唯一会打开设置界面的入口，但它始终调用 `showTheme(returnToHome = false)`。用户会直接进入主题详情，返回时又关闭整个设置层，因此设置首页中的渲染器、触感等选项无法访问。常规设置入口应先调用 `showHome()`，或允许主题详情返回设置首页。

pr-agent-bot bot reviewed 2026-08-27 23:27:39 +08:00

pr-agent-bot bot left a comment

Inline comment

android/app/src/main/kotlin/com/flagship/abox/manager/ui/ABoxManagerApp.kt

						
				@@ -80,4 +84,2 @@

				                    username = state.username,

				                    profile = state.profile,

				                    actionInProgress = actionState.inProgress,

				                    errorMessage = actionState.errorMessage,

pr-agent-bot bot commented

2026-08-27 23:27:39 +08:00

错误丢失

认证态不再接收或展示 actionState.errorMessage 和 inProgress，而账户操作会先关闭弹窗再发起切换提供方、结束会话或忘记凭据。若网络或存储操作失败且会话仍保持认证态，错误只被传给登录页，当前界面不会提供任何失败反馈或重试提示。

**错误丢失** 认证态不再接收或展示 `actionState.errorMessage` 和 `inProgress`，而账户操作会先关闭弹窗再发起切换提供方、结束会话或忘记凭据。若网络或存储操作失败且会话仍保持认证态，错误只被传给登录页，当前界面不会提供任何失败反馈或重试提示。

pr-agent-bot bot reviewed 2026-08-27 23:27:39 +08:00

pr-agent-bot bot left a comment

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 5 🔵🔵🔵🔵🔵

🧪 PR contains tests

🔒 No security concerns identified

⚠️ Review coverage: The following files were not included in this review because of the token budget:

android/app/src/main/kotlin/com/flagship/abox/manager/ui/devicecenter/DeviceCenterViewModel.kt
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/navigation/AuthenticatedNavigationRoutingTest.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/defaults/ABoxDefaultContentInitializer.kt
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/preferences/SettingsScreenTest.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/scene/SceneEditorViewModel.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/scene/RecordingSceneExecutor.kt
android/app/build.gradle.kts
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/shell/MainAppShellTest.kt
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/shell/AccountProviderDialogTest.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/devicecenter/DeviceCenterModels.kt
android/app/src/main/kotlin/com/flagship/abox/manager/di/DeviceGatewayModule.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/common/DomainErrorMapper.kt
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/scene/SceneChoiceFlowTest.kt
android/app/src/main/kotlin/com/flagship/abox/manager/di/SceneIntegrationModule.kt
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/temperaturehumidity/TemperatureHumidityDetailScreenTest.kt
android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/socket/SocketDetailScreenTest.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/devicecenter/DeviceCenterRoute.kt
android/app/src/main/kotlin/com/flagship/abox/manager/ui/navigation/Routes.kt
absdk-bridge/src/main/kotlin/com/flagship/abox/manager/absdk/bridge/AbsdkBridge.kt
android/app/src/main/kotlin/com/flagship/abox/manager/di/ProviderRoutingGateways.kt
android/app/src/main/kotlin/com/flagship/abox/manager/di/UiPreferencesModule.kt
android/app/src/main/AndroidManifest.xml

## PR Reviewer Guide 🔍 Here are some key observations to aid the review process: <table> <tr><td>⏱️ <strong>Estimated effort to review</strong>: 5 🔵🔵🔵🔵🔵</td></tr> <tr><td>🧪 <strong>PR contains tests</strong></td></tr> <tr><td>🔒 <strong>No security concerns identified</strong></td></tr> </table> <hr> ⚠️ **Review coverage:** The following files were not included in this review because of the token budget: - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/devicecenter/DeviceCenterViewModel.kt` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/navigation/AuthenticatedNavigationRoutingTest.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/defaults/ABoxDefaultContentInitializer.kt` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/preferences/SettingsScreenTest.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/scene/SceneEditorViewModel.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/scene/RecordingSceneExecutor.kt` - `android/app/build.gradle.kts` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/shell/MainAppShellTest.kt` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/shell/AccountProviderDialogTest.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/devicecenter/DeviceCenterModels.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/di/DeviceGatewayModule.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/common/DomainErrorMapper.kt` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/scene/SceneChoiceFlowTest.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/di/SceneIntegrationModule.kt` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/temperaturehumidity/TemperatureHumidityDetailScreenTest.kt` - `android/app/src/androidTest/kotlin/com/flagship/abox/manager/ui/socket/SocketDetailScreenTest.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/devicecenter/DeviceCenterRoute.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/ui/navigation/Routes.kt` - `absdk-bridge/src/main/kotlin/com/flagship/abox/manager/absdk/bridge/AbsdkBridge.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/di/ProviderRoutingGateways.kt` - `android/app/src/main/kotlin/com/flagship/abox/manager/di/UiPreferencesModule.kt` - `android/app/src/main/AndroidManifest.xml`

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Flagship/abox-mgr#10