Skip to content

Conversation

@Simek
Copy link
Contributor

@Simek Simek commented Oct 9, 2025

Closes #39838

Refs:

How

After digging a bit more to find out why raw links are not colored in Markdown renderer I have found a simpler approach to applying color decoration, which also fixed the lack of colors on raw links mentioned in issue and comment above.

Release Notes:

  • Improved decoration logic for links in Markdown

Preview

Screenshot 2025-10-09 at 23 39 09

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Oct 9, 2025
@Simek Simek changed the title markdown_preview: improve the link decoration logic markdown_preview: Improve the link decoration logic Oct 9, 2025
danilo-leal added a commit that referenced this pull request Oct 14, 2025
# Why

Follow up to:
* #39905

# How

Apply accented color to links in message content inside Blame tooltip,
to match appearance in Markdown Preview panel.

Release Notes:

- Improved appearance of links in message content inside Blame tooltip.

# Preview

### Before

<img width="1186" height="798" alt="Screenshot 2025-10-13 at 19 33 37"
src="https://github.com/user-attachments/assets/33ab4fb5-7910-4d28-9152-c692d6ddeaa6"
/>

### After

<img width="1186" height="798" alt="Screenshot 2025-10-13 at 19 33 10"
src="https://github.com/user-attachments/assets/38082c5c-50d6-4fb3-90ca-410accff9aad"
/>

---------

Co-authored-by: Danilo Leal <[email protected]>
Copy link
Member

@danilo-leal danilo-leal left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for following up!

@danilo-leal danilo-leal merged commit 287314f into zed-industries:main Oct 17, 2025
23 checks passed
P1n3appl3 pushed a commit to P1n3appl3/zed that referenced this pull request Oct 18, 2025
…s#40124)

# Why

Follow up to:
* zed-industries#39905

# How

Apply accented color to links in message content inside Blame tooltip,
to match appearance in Markdown Preview panel.

Release Notes:

- Improved appearance of links in message content inside Blame tooltip.

# Preview

### Before

<img width="1186" height="798" alt="Screenshot 2025-10-13 at 19 33 37"
src="https://github.com/user-attachments/assets/33ab4fb5-7910-4d28-9152-c692d6ddeaa6"
/>

### After

<img width="1186" height="798" alt="Screenshot 2025-10-13 at 19 33 10"
src="https://github.com/user-attachments/assets/38082c5c-50d6-4fb3-90ca-410accff9aad"
/>

---------

Co-authored-by: Danilo Leal <[email protected]>
P1n3appl3 pushed a commit to P1n3appl3/zed that referenced this pull request Oct 18, 2025
…9905)

Closes zed-industries#39838

Refs:
*
zed-industries#39149 (comment)

# How

After digging a bit more to find out why raw links are not colored in
Markdown renderer I have found a simpler approach to applying color
decoration, which also fixed the lack of colors on raw links mentioned
in issue and comment above.

Release Notes:

- Improved decoration logic for links in Markdown

# Preview

<img width="1712" height="820" alt="Screenshot 2025-10-09 at 23 39 09"
src="https://github.com/user-attachments/assets/3864cb6c-3fc6-4110-8067-6158cd4b58f5"
/>
@JosephTLyons
Copy link
Collaborator

Thank you @Simek!

@JosephTLyons
Copy link
Collaborator

JosephTLyons commented Oct 18, 2025

Not sure what's up, but I'm still missing the styling on my end on the latest nightly.

Zed: v0.210.0 (Zed Nightly 6c3a7f6ddb9746ed36c8b1397b5c63a6dd2f8a92) 
OS: macOS 26.0
Memory: 64 GiB
Architecture: aarch64

I can do into it more later on:

SCR-20251017-tvyn

@JosephTLyons
Copy link
Collaborator

JosephTLyons commented Oct 18, 2025

Oops! Sorry, I looked at the timestamp of the PR, and not when it was merged. All looks great on my end with a fresh compile! My apologies!

@Simek Simek deleted the @simek/markdown-preview-improve-link-decoration branch October 18, 2025 09:40
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.

Markdown Preview: Raw links are styled differently from embedded links

4 participants