Skip to content

docs(help): update in-app controls help#175

Merged
benvinegar merged 1 commit into
mainfrom
docs/update-controls-help
Apr 7, 2026
Merged

docs(help): update in-app controls help#175
benvinegar merged 1 commit into
mainfrom
docs/update-controls-help

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • update the in-app help modal/menu label from Keyboard help to Controls help
  • document horizontal code scrolling more explicitly, including left/right wording
  • add mouse wheel controls to the in-app help, with regular wheel as vertical scrolling and Shift+Wheel as horizontal code scrolling

Context

Reviewed commits since v0.8.1 for new shortcuts/behaviors that should be reflected in the in-app help.

Testing

  • bun run typecheck
  • bun test
  • bun run test:tty-smoke
  • real TTY smoke run on a sample diff

@greptile-apps

greptile-apps Bot commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the in-app controls help to accurately reflect features shipped since v0.8.1. The modal title and help-menu label are renamed from "Keyboard help" to "Controls help" (appropriate since the dialog now covers mouse as well as keyboard). A new Mouse section documents Wheel (vertical scroll) and Shift+Wheel (horizontal code scroll), and the ← / → Navigation row is updated to explicitly call out horizontal code-column scrolling with a Shift = faster note.

  • All documented shortcuts are verified as implemented: { / }moveToAnnotatedHunk (useAppKeyboardShortcuts.ts), Shift+WheelonScrollCodeHorizontally (App.tsx), ← / →scrollCodeHorizontally (useAppKeyboardShortcuts.ts).
  • Tests in both ui-components.test.tsx (snapshot rows + blank-row spacing assertions) and AppHost.interactions.test.tsx (menu label assertion) are updated to match.
  • No dead code introduced; no overlapping implementations.

Confidence Score: 5/5

Safe to merge — documentation-only changes with no logic modifications

All four files are purely additive documentation and label updates. Every newly documented shortcut was verified as already implemented in the codebase. Tests were updated in parallel and cover both the new content and spacing invariants.

No files require special attention

Important Files Changed

Filename Overview
src/ui/components/chrome/HelpDialog.tsx Adds Mouse section (Wheel/Shift+Wheel) and left/right arrow scroll row; renames modal title to 'Controls help'
src/ui/lib/appMenus.ts Renames help menu entry label from 'Keyboard help' to 'Controls help'
src/ui/AppHost.interactions.test.tsx Updates menu navigation integration test to assert 'Controls help' label
src/ui/components/ui-components.test.tsx Expands HelpDialog render test to cover new Mouse section rows, updated arrow-key description, and blank-row spacing

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User presses ? or F10 → Help menu] --> B[HelpDialog renders]
    B --> C[Navigation section]
    B --> D[Mouse section NEW]
    B --> E[View section]
    B --> F[Review section]
    D --> G[Wheel → scroll vertically]
    D --> H[Shift+Wheel → scroll code horizontally]
    C --> I[← / → → scroll code left/right]
    C --> J[{ / } → previous/next comment]
Loading

Reviews (1): Last reviewed commit: "docs(help): update in-app controls help" | Re-trigger Greptile

@benvinegar
benvinegar merged commit c7bc1ba into main Apr 7, 2026
3 checks passed
@benvinegar
benvinegar deleted the docs/update-controls-help branch April 7, 2026 02:10
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