Skip to content

Improve macOS layout shortcuts for v0.3.4 - #3

Merged
andy5090 merged 1 commit into
mainfrom
agent/macos-layout-shortcuts-v0.3.4
Aug 1, 2026
Merged

Improve macOS layout shortcuts for v0.3.4#3
andy5090 merged 1 commit into
mainfrom
agent/macos-layout-shortcuts-v0.3.4

Conversation

@andy5090

@andy5090 andy5090 commented Aug 1, 2026

Copy link
Copy Markdown
Owner

What changed

  • added a one-row top margin when the big-font zone is disabled without starving compact terminals
  • added Shift+F5 as a platform-neutral line-spacing shortcut while keeping Option+L / Alt+L as alternates
  • clarified macOS Option, Control, Command, Windows, and function-key notation in help and README
  • kept primary reading and sound shortcuts visible in adaptive footer guidance
  • bumped Termleaf to v0.3.4

Why

macOS terminals may not deliver Option+L consistently, and the document started flush against the top edge when big text was disabled. Modifier labels also needed to match the physical keys users see on each operating system.

Validation

  • cargo fmt --all -- --check
  • cargo test --all-features (82 tests)
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo build --release
  • git diff --check

@andy5090
andy5090 marked this pull request as ready for review August 1, 2026 18:43
@cursor

cursor Bot commented Aug 1, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@andy5090
andy5090 merged commit 7f007b3 into main Aug 1, 2026
8 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4b09d1d2fa

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/renderer/terminal.rs
"View: F3 Focus · F4 Big text · F6 Theme · F7/F8 Size 1–5",
"Reading: Alt+L Line spacing 1–3 · F5 Page width (Alt+P alternate)",
"Reading: F5 Page width · Shift+F5 Line spacing 1–3",
"Alternates: macOS Option+P/L · Windows/Linux Alt+P/L",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep alternate shortcuts from displacing primary help

On terminals 15–16 rows high, draw_help can display only 8–9 content lines and always truncates from the end. Inserting this optional alternate-shortcut line before the Sound and Other lines therefore makes the F10 guidance disappear at 15 rows and the quit/help guidance disappear at 16 rows; previously those lines were visible at those respective heights. Move or combine the alternate notation so primary controls remain reachable in short-terminal help.

Useful? React with 👍 / 👎.

@andy5090
andy5090 deleted the agent/macos-layout-shortcuts-v0.3.4 branch August 1, 2026 18:50
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