Skip to content

terminal: Prevent terminal cursor stretching by always using cell width#45545

Merged
Veykril merged 7 commits intozed-industries:mainfrom
tacshi:fix/cursor_stretch
Feb 2, 2026
Merged

terminal: Prevent terminal cursor stretching by always using cell width#45545
Veykril merged 7 commits intozed-industries:mainfrom
tacshi:fix/cursor_stretch

Conversation

@tacshi
Copy link
Contributor

@tacshi tacshi commented Dec 23, 2025

Closes #8516

Release Notes:

  • Prevent terminal cursor stretching by always using cell width.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 23, 2025
@tacshi tacshi changed the title Prevent terminal cursor stretching by always using cell width. Prevent terminal cursor stretching by always using cell width Dec 23, 2025
@Veykril Veykril self-assigned this Dec 27, 2025
@tacshi tacshi force-pushed the fix/cursor_stretch branch from 537afcb to d1fed3c Compare December 27, 2025 10:45
Copy link
Member

@Veykril Veykril left a comment

Choose a reason for hiding this comment

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

I don't think this is quite right, that will look funny for emojis and other non-standard cell characters. Let's do a check if the text fragment consists solely of whitespace instead, and if that is the case fall back to the cell width instead

@tacshi tacshi force-pushed the fix/cursor_stretch branch from 64b2447 to 2de00f7 Compare January 24, 2026 12:15
@tacshi tacshi requested a review from Veykril January 24, 2026 12:17
Copy link
Member

@Veykril Veykril left a comment

Choose a reason for hiding this comment

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

As assumed this still breaks the cursor for bigger non whitespace glyphs

Image

@tacshi
Copy link
Contributor Author

tacshi commented Jan 27, 2026

As assumed this still breaks the cursor for bigger non whitespace glyphs

Image

It should work this time

@tacshi tacshi requested a review from Veykril January 27, 2026 05:16
Copy link
Member

@Veykril Veykril left a comment

Choose a reason for hiding this comment

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

Thanks!

@Veykril Veykril changed the title Prevent terminal cursor stretching by always using cell width terminal: Prevent terminal cursor stretching by always using cell width Jan 30, 2026
@Veykril Veykril enabled auto-merge (squash) January 30, 2026 10:31
@Veykril
Copy link
Member

Veykril commented Jan 30, 2026

needs a cargo fmt

auto-merge was automatically disabled January 30, 2026 11:07

Head branch was pushed to by a user without write access

@tacshi tacshi requested a review from Veykril January 30, 2026 11:07
@Veykril Veykril enabled auto-merge (squash) January 30, 2026 11:26
@Veykril Veykril merged commit af0e45e into zed-industries:main Feb 2, 2026
43 of 45 checks passed
@tacshi tacshi deleted the fix/cursor_stretch branch February 2, 2026 09:01
adb-sh pushed a commit to adb-sh/zed that referenced this pull request Feb 5, 2026
…th (zed-industries#45545)

Closes zed-industries#8516

Release Notes:

- Prevent terminal cursor stretching by always using cell width.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Terminal cursor "stretched" during shell script execution

3 participants