Skip to content

Clarify app detail actions - #41

Merged
kasnder merged 4 commits into
codex/fix-android-sdk-cifrom
codex/app-detail-actions
Sep 25, 2026
Merged

kasnder merged 4 commits into
codex/fix-android-sdk-cifrom
codex/app-detail-actions

Conversation

@kasnder

@kasnder kasnder commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Concern addressed: Opening-screen selection is hidden behind the blocking picker in app detail and missing from Custom rules; the pause action and secondary picker control are visually unclear.

What changed and why: Added a separate “Change opening screen” action in app detail and Custom rules. The Custom rules action carries navigation mode through its notification so it can select a destination in another app; the hide action remains scoped to blocking. Matched the pause heading to existing section labels and made the secondary action a white, navy-outlined pill on both screens.

Evidence: ./gradlew test passed across both flavours. ./gradlew assembleStandardDebug lintStandardDebug and git diff --check passed. The updated layouts were installed and visually checked on a Pixel 8. The final PR CI run passed.

Not in this PR: Android SDK CI setup is in PR #42, on which this PR is stacked. Picker interaction has not been exercised on-device.

Expose a separate opening-screen picker action and label the pause durations so the controls describe their effect. Keep each picker button scoped to its matching mode.
@kasnder
kasnder force-pushed the codex/app-detail-actions branch from b33b13d to e5cd2e8 Compare September 25, 2026 09:29
@kasnder
kasnder changed the base branch from main to codex/fix-android-sdk-ci September 25, 2026 09:29
@kasnder
kasnder added this pull request to stack #43 September 25, 2026 09:33
Carry the requested picker mode through the notification flow so the new Custom rules action selects a destination. Align the pause heading with section labels and use the existing text-button style for the secondary action.
The picker controls are intentionally hidden in gmwaylite by resource override. Verify both hidden controls there while retaining the notification routing assertions in standard.
Use a white pill with a navy outline and sentence-case text on both entry screens so the secondary action remains visible against the warm background.
@kasnder
kasnder merged commit 8465ae1 into main Sep 25, 2026
1 check passed
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