Skip to content

Label help overlay keys by what they do - #68

Merged
pcarrier merged 4 commits into
fix/audio-disable-bitratefrom
fix/help-overlay-labels
Oct 10, 2026
Merged

pcarrier merged 4 commits into
fix/audio-disable-bitratefrom
fix/help-overlay-labels

Conversation

@mgasner

@mgasner mgasner commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

Stacked on #67. Replaces #39, which was opened from indent-com/yas.

The help overlay now labels Ctrl+B r "Preview panel" and Tab "Prev / Next pane", which matches what those bindings do.

View in Indent
Tag @indent to continue the conversation here.

🤖 Generated with Claude Code

@indent

indent Bot commented Sep 28, 2026

Copy link
Copy Markdown

Warning

This organization's free trial has ended, so Indent couldn't start this review. Manage billing to resume reviews on this repository.

Ctrl+B r only toggles the preview panel, and Tab/Shift+Tab cycle panes, as
the prefix map already says. Drop the two unused strings.
@mgasner
mgasner force-pushed the fix/audio-disable-bitrate branch from e7afe8d to 82908a4 Compare September 29, 2026 00:01
@mgasner
mgasner force-pushed the fix/help-overlay-labels branch from 3b9a755 to 0fd51c2 Compare September 29, 2026 00:01
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Coverage

Crate Lines Functions Regions
alacritty-driver 85.6% (1081/1263) 89.4% (84/94) 89.3% (1746/1955)
browser 25.5% (309/1212) 30.4% (34/112) 27.7% (605/2183)
cli 34.1% (6733/19768) 36.2% (638/1762) 35.2% (10006/28433)
client 68.3% (5800/8497) 68.5% (708/1033) 66.4% (7526/11331)
composite-transport 96.3% (526/546) 98.4% (60/61) 96.2% (884/919)
compositor 55.0% (11456/20820) 66.2% (838/1265) 54.9% (15870/28909)
desktop 78.4% (4460/5691) 71.6% (393/549) 75.1% (6211/8267)
edge 62.9% (798/1268) 50.9% (82/161) 57.7% (1038/1799)
fonts 77.3% (1257/1626) 82.7% (129/156) 78.9% (2424/3071)
fssync 86.0% (1609/1872) 85.8% (182/212) 87.1% (2834/3255)
git 70.5% (4472/6344) 68.5% (337/492) 67.3% (6334/9407)
guest 67.3% (6829/10148) 67.0% (488/728) 66.6% (8935/13416)
lsp 78.7% (3573/4542) 80.8% (336/416) 76.7% (5408/7054)
proxy 63.1% (1902/3012) 56.3% (184/327) 63.4% (2931/4626)
runtime-dir 93.6% (117/125) 100.0% (14/14) 94.8% (218/230)
sd-notify 73.9% (68/92) 100.0% (6/6) 83.2% (109/131)
server 71.1% (85743/120546) 73.7% (6223/8446) 69.2% (114973/166252)
ssh 67.2% (708/1054) 75.9% (85/112) 67.2% (1105/1645)
terminal-model 49.9% (314/629) 62.3% (38/61) 50.3% (505/1004)
uplink 94.2% (582/618) 92.6% (50/54) 93.1% (1062/1141)
webrtc-forwarder 33.6% (1321/3932) 44.9% (146/325) 36.0% (2279/6336)
webserver 80.7% (1490/1846) 81.1% (193/238) 83.2% (2551/3067)
website 35.1% (355/1012) 34.4% (53/154) 35.8% (607/1694)
xtask 0.0% (0/5149) 0.0% (0/131) 0.0% (0/9157)
yas 88.9% (28036/31522) 94.1% (2151/2286) 84.1% (44035/52388)
Total 67.0% (169539/253134) 70.1% (13452/19195) 65.3% (240196/367670)

mgasner and others added 3 commits September 29, 2026 08:54
yas open always serves the local server's UI, which lists configured
remotes; a command-line target was silently dropped. Credentials in the
echoed --on value are masked.

@ghost ghost 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.

Ultimator's review of 24b7179

Relabels the help overlay entries (Tab → "Prev / Next pane", r → "Preview panel") and drops the now-unused i18n keys. It also carries the stacked #69 change, where yas open refuses --on and --hub instead of ignoring them. I found no issues in the diff, but I couldn't check out the repository, so I didn't grep for other uses of the removed i18n keys.

No issues found.

Reviewed in 19s · Session · @ultimator-agent review reviews again · @ultimator-agent fix fixes the open issues

@pcarrier
pcarrier merged commit 1916484 into fix/audio-disable-bitrate Oct 10, 2026
11 checks passed
@pcarrier
pcarrier deleted the fix/help-overlay-labels branch October 10, 2026 15: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.

2 participants