feat(command-center): put daily controls first and tidy Doctor, Stream, and HUD copy - #281
Merged
Merged
Conversation
…m, and HUD copy Command Center opens on every Back press, so the things a player touches every session now come first: Quick Keys, Controls, Session, then Overlays. The session strip keeps the health verdict at the top, and the Doctor, Stream, Sync, and Advanced cards follow it. Close is the primary header button; Disconnect stays quiet and End Session reads as destructive through a new destructive tint on NovaActionButton. The Doctor card leads with the finding and keeps the action in its chip, so Recheck no longer appears as title, chip, and button at once. Evidence comes from the first item Doctor grades as actionable, which drops the host's passing "a stream is active" check, and Confidence only shows beside real evidence. The card no longer repeats as the first row of the Overlays panel. The stability card is titled Stream and drops the sentence the strip and Doctor already show. The session pill names the mode and encoder; capture path, mode source, and role move to a detail line. Overlays gain a HUD Mode row that cycles Minimal, Performance, and Debug (Guide + Y still works), and Stats Overlay says it is the legacy Moonlight text. The host safe profile card moves under Advanced, says it is observational, and stays hidden when the host has nothing to report. NovaHUD reads SAFE instead of REC for recovery, shows the 1% low value once, names control channel retries as Link retries, drops the accent color from bitrate and codec, and uses the short mode name in the Debug header. The source guard that pinned the old Command Center order is re-pinned with the new intent. Verified on the Retroid Pocket 6 with a live Control stream.
3 tasks
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
destructiveflag onNovaActionButton.diagnosisActionfor callers and tests.sessionDetailline so the health summary keeps its room.Game.cycleNovaHudMode(); the Nova HUD caption is one sentence; Stats Overlay says it is the legacy Moonlight text.RECreadsSAFE; the 1% low tile shows the number once;Control retries observedand the status labelControl retriesbecomeLink retries; bitrate and codec no longer use the accent tone; the Debug header uses a newstreamModeShortLabel.Exact candidate
Commit:adea5b8Tree:29638b39d43f80377ca5440c0641ff188cd5ec8aParent:e8b35eaBase:master at e8b35eaVerification
:app:testNonRoot_gameDebugUnitTest: 1602 tests, 0 failures (re-pinnedNovaComposeSourceGuardTest,NovaQuickMenuUiStateTest+2 tests,NovaHudUiStateTest,AutoQualityUiStateTest,NovaDisplayResolutionPlannerTest,PolarisApiClientParsingTest):app:lintNonRoot_gameRelease: clean:app:assembleNonRoot_gameDebug: three ABIs builtcom.papi.nova.debug, live Control stream on pc-papi): header emphasis, strip pill and detail line, Quick Keys first, HUD Mode row cycling Minimal to Performance from the menu, Performance HUD showingLink retrieswith neutral bitrate, Doctor and Stream cards, Host safe profile inside Advanced, End Session flow.Not covered:
:app:assembleNonRoot_gameDebugAndroidTestfails on master already (NovaGameDetailComposeTestandNovaPolarisSyncSheetComposeTestreference parameters that no longer exist); nothing in this change touches those files. The two-line Doctor title landed after the device pass and is covered by the guard suite only.