Skip to content

fix(reporter): lower max dots per line from 100 to 80 - #40

Merged
timmywil merged 1 commit into
mainfrom
mgol-reporter-dots-per-line-80
Sep 29, 2026
Merged

timmywil merged 1 commit into
mainfrom
mgol-reporter-dots-per-line-80

Conversation

@mgol

@mgol mgol commented Sep 21, 2026

Copy link
Copy Markdown
Member

The original version from PR gh-31 used 100 dots per line, which doesn't fit in all terminals; 80 is a more sensible default.

If the terminal is wider, it will just produce more dotted lines than necessary. If the terminal is narrower and the whole line doesn't fit, though, the dots wrap and then end mid-line which looks much worse.

Ref gh-31

The original version from PR gh-31 used 100 dots per line, which doesn't
fit in all terminals; 80 is a more sensible default.

If the terminal is wider, it will just produce more dotted lines than
necessary. If the terminal is narrower and the whole line doesn't fit,
though, the dots wrap and then end mid-line which looks much worse.
@mgol
mgol requested a review from timmywil September 21, 2026 20:30
@mgol mgol self-assigned this Sep 21, 2026
@mgol

mgol commented Sep 21, 2026

Copy link
Copy Markdown
Member Author

I've started hitting it after switching to cmux which has a sidebar, when I use two terminals side by side in a single group; 100 columns doesn't fit then for me.

@timmywil timmywil changed the title reporter: lower max dots per line from 100 to 80 fix(reporter): lower max dots per line from 100 to 80 Sep 29, 2026
@timmywil
timmywil merged commit d148220 into main Sep 29, 2026
8 checks passed
@timmywil
timmywil deleted the mgol-reporter-dots-per-line-80 branch September 29, 2026 17:01
@mgol mgol removed the Needs review label Sep 30, 2026
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