Skip to content

docs(studio): shrink the Escape section to what the app does not show - #2314

Merged
kantord merged 1 commit into
mainfrom
studio-followups/04-escape-docs
Oct 15, 2026
Merged

kantord merged 1 commit into
mainfrom
studio-followups/04-escape-docs

Conversation

@kantord

@kantord kantord commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Summary

The Escape section of the Studio docs restated the order the in-app Keyboard shortcuts page already shows.

Part of #2303 ("Tiny follow-up fixes"); it does not close it. This is PR 4 of 14 in a stack of draft PRs, reviewed and merged in order.

What changed

  • Removed the numbered list and the lines that restate it, and one sentence in the key-bindings paragraph that repeated it.
  • Kept what a reader cannot infer: each press handles one action; a pending review stops Escape even when its controls are unavailable; clearing a draft needs a release between two presses within half a second; Enter or Space on a focused approval button selects that decision without sending the draft.
  • Points to the in-app Keyboard shortcuts page for the full order.

Behavior changes

Docs only.

Verification

task docs:check passes (0 broken links, 0 orphans). The pending-review and draft-timing claims were checked against chat-escape.ts. "Enter or Space selects without sending the draft" is native button behavior; no dedicated test exists for it.

Not included

The "Review tool calls and plans" and "Review external authorization" headings, which are already in the right order.

Fully or partially written by an AI agent.

🤖 Generated with Claude Code

@peppescg peppescg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Daniel. This trims the Escape section to what the in-app Keyboard shortcuts page doesn't already cover, and I checked the remaining claims against chat-escape.ts. LGTM.

@kantord
kantord force-pushed the studio-followups/04-escape-docs branch from 21819ee to d1d45de Compare October 15, 2026 09:25
@kantord
kantord force-pushed the studio-followups/04-escape-docs branch from d1d45de to d164c7e Compare October 15, 2026 09:26
Base automatically changed from studio-followups/03-knowledge-statuses to main October 15, 2026 09:26
The in-app Keyboard shortcuts page already lists the Escape order. Keep
only the behavior a reader cannot infer from it.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kantord
kantord force-pushed the studio-followups/04-escape-docs branch from d164c7e to a77c342 Compare October 15, 2026 09:26
@kantord
kantord merged commit 77637fa into main Oct 15, 2026
29 checks passed
@kantord
kantord deleted the studio-followups/04-escape-docs branch October 15, 2026 09:29
@toolhive-release-app toolhive-release-app Bot mentioned this pull request Oct 20, 2026
2 tasks

This branch was successfully deployed

1 active deployment
Preview — a77c3427 Deployed Oct 15, 2026 by vercel[bot]
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.

2 participants