Skip to content

Prevent underline and strikethrough artifacts on final glyph in text runs#50934

Merged
smitbarmase merged 2 commits intozed-industries:mainfrom
prertik:markdown-preview-underline-bug-fix
Apr 8, 2026
Merged

Prevent underline and strikethrough artifacts on final glyph in text runs#50934
smitbarmase merged 2 commits intozed-industries:mainfrom
prertik:markdown-preview-underline-bug-fix

Conversation

@prertik
Copy link
Copy Markdown
Contributor

@prertik prertik commented Mar 6, 2026

Closes #50587

Before you mark this PR as ready for review, make sure that you have:

  • Added a solid test coverage and/or screenshots from doing manual testing
  • Done a self-review taking into account security and performance aspects
  • Aligned any UI changes with the UI checklist

Release Notes:

  • Fixed issue that showed underline and strikethrough when markdown preview screen was scaled to a smaller size

Previously:

image

Now:

image

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Mar 6, 2026
@zed-community-bot zed-community-bot Bot added the first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions label Mar 6, 2026
@prertik
Copy link
Copy Markdown
Contributor Author

prertik commented Mar 14, 2026

Hi @dinocosta.

Can you please review this as well?

Copy link
Copy Markdown
Member

@smitbarmase smitbarmase left a comment

Choose a reason for hiding this comment

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

Thanks!

@smitbarmase smitbarmase force-pushed the markdown-preview-underline-bug-fix branch from 61eca52 to 3440b80 Compare April 8, 2026 15:53
@smitbarmase smitbarmase merged commit a7e677e into zed-industries:main Apr 8, 2026
30 checks passed
Caio-Ze pushed a commit to Caio-Ze/postprod-ide that referenced this pull request Apr 8, 2026
… text runs (zed-industries#50934)

Closes zed-industries#50587 

Before you mark this PR as ready for review, make sure that you have:
- [x] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:
- Fixed issue that showed underline and strikethrough when markdown
preview screen was scaled to a smaller size


Previously:

<img width="741" height="457" alt="image"
src="https://github.com/user-attachments/assets/f127cf44-c679-4140-b246-e0756c9fca32"
/>



Now:

<img width="742" height="592" alt="image"
src="https://github.com/user-attachments/assets/a29fdc24-cac0-438f-8538-4e85c4df995a"
/>

---------

Signed-off-by: Pratik Karki <[email protected]>
Co-authored-by: Smit Barmase <[email protected]>
AzureZee added a commit to AzureZee/gpuix that referenced this pull request Apr 9, 2026
AzureZee added a commit to AzureZee/gpuix that referenced this pull request Apr 9, 2026
piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
… text runs (zed-industries#50934)

Closes zed-industries#50587 

Before you mark this PR as ready for review, make sure that you have:
- [x] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:
- Fixed issue that showed underline and strikethrough when markdown
preview screen was scaled to a smaller size


Previously:

<img width="741" height="457" alt="image"
src="https://github.com/user-attachments/assets/f127cf44-c679-4140-b246-e0756c9fca32"
/>



Now:

<img width="742" height="592" alt="image"
src="https://github.com/user-attachments/assets/a29fdc24-cac0-438f-8538-4e85c4df995a"
/>

---------

Signed-off-by: Pratik Karki <[email protected]>
Co-authored-by: Smit Barmase <[email protected]>
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 first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Markdown preview underline wrap-around bug

3 participants