Skip to content

project: Sanitize single-line completions from trailing newlines#44965

Merged
MrSubidubi merged 4 commits intozed-industries:mainfrom
prayanshchh:fix/completions_overlap
Dec 19, 2025
Merged

project: Sanitize single-line completions from trailing newlines#44965
MrSubidubi merged 4 commits intozed-industries:mainfrom
prayanshchh:fix/completions_overlap

Conversation

@prayanshchh
Copy link
Contributor

@prayanshchh prayanshchh commented Dec 16, 2025

Closes #43991
trim documentation string to prevent completion overlap

previous
Screencast from 2025-12-16 14-55-58.webm
after change
Screencast from 2025-12-16 14-50-05.webm
Release Notes:

  • Fixed an issue where completions in the completion menu would span multiple lines.

@cla-bot
Copy link

cla-bot bot commented Dec 16, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @prayanshchh on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@prayanshchh
Copy link
Contributor Author

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented Dec 16, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @prayanshchh on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot
Copy link

cla-bot bot commented Dec 16, 2025

The cla-bot has been summoned, and re-checked this pull request!

@prayanshchh
Copy link
Contributor Author

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented Dec 16, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @prayanshchh on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot
Copy link

cla-bot bot commented Dec 16, 2025

The cla-bot has been summoned, and re-checked this pull request!

@prayanshchh
Copy link
Contributor Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 16, 2025
@cla-bot
Copy link

cla-bot bot commented Dec 16, 2025

The cla-bot has been summoned, and re-checked this pull request!

@MrSubidubi MrSubidubi changed the title Fix/completions overlap project: Fix single line completions spanning multiple lines Dec 16, 2025
@prayanshchh
Copy link
Contributor Author

please review if there is any change in approach required, will fix failing tests

@MrSubidubi MrSubidubi changed the title project: Fix single line completions spanning multiple lines project: Sanitize single-line completions from trailing newlines Dec 16, 2025
Copy link
Member

@MrSubidubi MrSubidubi left a comment

Choose a reason for hiding this comment

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

Nice, looks good, thank you!

And congratulations to your first contribution! 🎉

@MrSubidubi MrSubidubi merged commit 6d776c3 into zed-industries:main Dec 19, 2025
25 checks passed
@prayanshchh
Copy link
Contributor Author

thank u so much!!

rtfeldman pushed a commit that referenced this pull request Jan 5, 2026
)

Closes #43991
trim documentation string to prevent completion overlap


previous
[Screencast from 2025-12-16
14-55-58.webm](https://github.com/user-attachments/assets/d7674d82-63b0-4a85-a90f-b5c5091e4a82)
after change
[Screencast from 2025-12-16
14-50-05.webm](https://github.com/user-attachments/assets/109c22b5-3fff-49c8-a2ec-b1af467d6320)
Release Notes:

- Fixed an issue where completions in the completion menu would span
multiple lines.
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…-industries#44965)

Closes zed-industries#43991
trim documentation string to prevent completion overlap


previous
[Screencast from 2025-12-16
14-55-58.webm](https://github.com/user-attachments/assets/d7674d82-63b0-4a85-a90f-b5c5091e4a82)
after change
[Screencast from 2025-12-16
14-50-05.webm](https://github.com/user-attachments/assets/109c22b5-3fff-49c8-a2ec-b1af467d6320)
Release Notes:

- Fixed an issue where completions in the completion menu would span
multiple lines.
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
…-industries#44965)

Closes zed-industries#43991
trim documentation string to prevent completion overlap


previous
[Screencast from 2025-12-16
14-55-58.webm](https://github.com/user-attachments/assets/d7674d82-63b0-4a85-a90f-b5c5091e4a82)
after change
[Screencast from 2025-12-16
14-50-05.webm](https://github.com/user-attachments/assets/109c22b5-3fff-49c8-a2ec-b1af467d6320)
Release Notes:

- Fixed an issue where completions in the completion menu would span
multiple lines.
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Feb 15, 2026
…-industries#44965)

Closes zed-industries#43991
trim documentation string to prevent completion overlap


previous
[Screencast from 2025-12-16
14-55-58.webm](https://github.com/user-attachments/assets/d7674d82-63b0-4a85-a90f-b5c5091e4a82)
after change
[Screencast from 2025-12-16
14-50-05.webm](https://github.com/user-attachments/assets/109c22b5-3fff-49c8-a2ec-b1af467d6320)
Release Notes:

- Fixed an issue where completions in the completion menu would span
multiple lines.
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.

Completions overlap

2 participants