Skip to content

fix(ui): add contextual keyboard status hint#104

Open
saurabhhhcodes wants to merge 3 commits into
pie-314:devfrom
saurabhhhcodes:fix/keyboard-help-status-hint-53
Open

fix(ui): add contextual keyboard status hint#104
saurabhhhcodes wants to merge 3 commits into
pie-314:devfrom
saurabhhhcodes:fix/keyboard-help-status-hint-53

Conversation

@saurabhhhcodes

@saurabhhhcodes saurabhhhcodes commented May 31, 2026

Copy link
Copy Markdown

What changed

  • Added a contextual keyboard help hint to the permanent bottom status bar in src/ui/draw.rs.
  • The normal status line now shows the primary app shortcuts for help, search, and quit while preserving the selected-count and package-manager indicators.
  • Search mode and settings edit mode show mode-specific guidance so the bottom bar remains useful while the user is typing or editing.

Why

Issue #53 asks for a persistent keyboard help hint in the status bar. This keeps the most important shortcuts visible without adding a modal, changing the layout structure, or touching package-manager logic.

Fixes #53

How it was tested

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings -A dead_code -A clippy::type_complexity
  • cargo test --workspace --all-targets --all-features
  • cargo run -- --help
  • Runtime TUI smoke check with cargo run, then quit with q
  • GitHub CI Verify passed
  • GitHub CI Build passed

UI evidence

Runtime TUI capture after cargo run:

Runtime status bar evidence

The bottom bar now renders:

NORMAL  | Selected: 0  | Manager: Homebrew (macOS/Linux)  | Press '?' for help  •  'e' to search  •  'q' to quit

Scope checklist

  • One issue only.
  • One file changed: src/ui/draw.rs.
  • No unrelated formatting or package-manager changes.
  • PR targets the required dev branch.

@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the pie's projects Team on Vercel.

A member of the Team first needs to authorize it.

@saurabhhhcodes

Copy link
Copy Markdown
Author

Thanks for the guidance on the previous PR. I reopened this as a fresh PR against dev and expanded the description with the required what/why/testing/UI-evidence details.

Current status:

  • GitHub CI Verify passed
  • GitHub CI Build passed
  • Remaining Vercel status is the external team authorization preview, not a code failure

If accepted, could you please add the scoring labels from the issue: gssoc:approved, level:beginner, and type:ui?

@pie-314

pie-314 commented May 31, 2026

Copy link
Copy Markdown
Owner

@saurabhhhcodes you need to present screenshots wherever applicable, go through docs

@saurabhhhcodes

saurabhhhcodes commented May 31, 2026

Copy link
Copy Markdown
Author

Thanks, noted. I went through CONTRIBUTING.md, installed the Rust toolchain locally, and added runtime UI evidence to the PR body.

Updated verification:

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings -A dead_code -A clippy::type_complexity
  • cargo test --workspace --all-targets --all-features
  • cargo run -- --help
  • runtime TUI smoke check with cargo run, then quit via q

Runtime status-bar evidence is now included in the PR body, showing the bottom hint: Press '?' for help • 'e' to search • 'q' to quit.

@pie-314 pie-314 added the ai-slop label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants