Skip to content

feat(nova): modernize session glass menus#96

Open
papi-ux wants to merge 1 commit into
masterfrom
feat/nova-glass-session-menus-cleanroom
Open

feat(nova): modernize session glass menus#96
papi-ux wants to merge 1 commit into
masterfrom
feat/nova-glass-session-menus-cleanroom

Conversation

@papi-ux

@papi-ux papi-ux commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds shared Nova sheet and alert chrome for session surfaces.
  • Replaces the legacy in-stream GameMenu AlertDialog list with Nova glass bottom-sheet rows.
  • Makes the verbose session progress overlay the startup owner, including cleanup for the no-AVC fatal path.
  • Adds stateful D-pad focused and pressed feedback for Nova sheet action rows.

Test Plan

  • PASS: ./gradlew testRootDebugUnitTest --tests com.papi.nova.ui.NovaThemeResourcesTest --tests com.papi.nova.ui.NovaStreamOverlayUiStateTest
  • PASS: ./gradlew assembleRootDebug
  • PASS: git diff --cached --check and git diff --check
  • PASS: added-line static scan hits 0
  • PASS: Retroid Pocket 6 root debug install and Library launch smoke; NovaLibraryActivity focused and Nova loaded 19 games

Notes

  • Clean-room branch extracted from origin/master to avoid unrelated dirty master changes.
  • Final independent reviewer returned passed=true with no security concerns or logic errors.
  • No merge requested.

- Add shared Nova sheet and alert chrome for session surfaces
- Replace legacy in-stream GameMenu list with Nova glass bottom sheet rows
- Make startup progress overlay the single owner and clean up fatal no-AVC path
- Add D-pad focus/pressed feedback for sheet action rows

Verification:
- ./gradlew testRootDebugUnitTest --tests com.papi.nova.ui.NovaThemeResourcesTest --tests com.papi.nova.ui.NovaStreamOverlayUiStateTest
- ./gradlew assembleRootDebug
- Retroid root debug install and Library launch smoke
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