Refine Deep Review action bar UX#692
Merged
Merged
Conversation
limityan
force-pushed
the
yanzhn/deep
branch
2 times, most recently
from
2a0e871 to
ffdbcc3
Compare
Keep the Deep Review action surface available throughout the review lifecycle by replacing the hard dismiss path with minimize/restore state and showing running reviews as a minimized entry. Scope review action bar state by child review session so simultaneous Deep Reviews do not steal each other's floating controls, capacity updates, restore actions, or remediation state. Reconcile the running placeholder when capacity-waiting state clears before the review finishes, and treat pending review turns as active so the minimized entry is created from the start. Keep the action bar minimized after starting a fix, restore it when fix submission fails, and mark user-cancelled fix turns as interrupted with the remaining remediation scope. Send subagent timeout controls with the desktop command request shape so the Ignore limit button actually reaches the running subagent timeout handle. Polish completed-report controls with clearer export actions, a footer Open as Markdown affordance, compact Markdown editor controls, and a copy button in Markdown view. Keep Deep Review reviewer task cards collapsed while they run so progress updates do not cause large height jumps in the audit timeline. Reserve bottom breathing room for floating review controls so the final lines of a review conversation stay readable when scrolled to the end.
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
1 participant
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
Product Impact
This makes Deep Review feel like one continuous task surface from launch through completion instead of a completion-only popover. Users can keep working while the review runs, reopen the minimized state when needed, and inspect or export the completed report without losing context. Concurrent Deep Reviews now keep their own floating action state, fix runs give immediate minimized feedback, cancelled fix runs surface a recoverable state, and timeout controls perform the expected backend action.
Validation
pnpm run lint:webpnpm run type-check:webpnpm --dir src/web-ui run test:rungit diff --check