Skip to content

fix(ui): inset the bar line one column from the left edge - #27

Merged
jayhesselberth merged 1 commit into
mainfrom
bar-left-inset
Aug 29, 2026
Merged

jayhesselberth merged 1 commit into
mainfrom
bar-left-inset

Conversation

@jayhesselberth

Copy link
Copy Markdown
Member

The bar's first cell sat hard against the terminal's left edge, so the
severity glyph and job id read as if they had been clipped.

render now prints the line one column in and builds each mode line —
status, notices, help — for the columns left over, so the inset belongs to
the bar rather than to any one mode and the status line's drop-to-fit logic
stays honest about the pane width. The accent rule still spans edge to edge:
it is a fence, and a gap at its end would read as a break in it.

The monitor panel rows are unchanged and still start at column 0.

Tested: cargo fmt --all --check, cargo clippy --workspace --all-targets -D warnings and cargo test --workspace all clean; a new
the_bar_line_is_inset_from_the_left_edge checks all three bar modes at
40/80/200 columns start with a space and never exceed cols.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GDCMtWbUps1fu3JMhyh5y6

The bar's first cell sat hard against the terminal's left edge, where the
glyph and job id read as if they had been clipped. The inset belongs to
the bar rather than to each mode, so `render` pads the line it prints and
builds every mode line — status, notices, help — for the columns left
over, which keeps the status line's drop-to-fit logic honest about the
pane width. The accent rule still spans edge to edge: it is a fence, and
a gap at its end would read as a break in it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GDCMtWbUps1fu3JMhyh5y6
@jayhesselberth
jayhesselberth merged commit c2d38e9 into main Aug 29, 2026
3 checks passed
@jayhesselberth
jayhesselberth deleted the bar-left-inset branch August 29, 2026 01:33
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