-
Notifications
You must be signed in to change notification settings - Fork 5.9k
markdown_preview: Stylize links using accented text color #39149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
markdown_preview: Stylize links using accented text color #39149
Conversation
danilo-leal
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking great, good catch! Added one comment there before we move on.
danilo-leal
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great, thanks for picking this up!
|
Hey @Simek, nice work on this! Should we also stylize raw, non-embedded links too?
|
|
Hey @JosephTLyons, thanks! And yeah, I agree that would be nice to have unified appearance for all possible links within Markdown, had to miss this case, can look at this tomorrow and work on the follow-up PR if you want. 🙂 |
|
I'm not sure what that mind end up entailing, if itll be straight forward or not, but if you're up for it, that would be rad. Of course, no pressure. Really appreciate the help improving markdown preview. The last 2-3 releases added a lot of good fixes. |
…ries#39149) # How Emphasize links in Markdown Preview text using accented text color. > [!note] > I have chosen the accent color for links since it was looking fine with all bundled by default themes, but I'm happy to alter the color to use different theme value, if you have better candidates. Release Notes: - Stylize links using accented text color in Markdown Preview # Preview ### Before <img width="1606" height="1066" alt="Screenshot 2025-09-29 at 22 19 38" src="https://github.com/user-attachments/assets/59b6ee72-4523-42fb-a468-9c694d30b5df" /> ### After <img width="1652" height="1066" alt="Screenshot 2025-09-29 at 22 18 20" src="https://github.com/user-attachments/assets/e00e3742-6435-4c1d-aaaa-e6332719db17" /> <img width="1652" height="1066" alt="Screenshot 2025-09-29 at 22 18 47" src="https://github.com/user-attachments/assets/a1b76f4a-c4d2-4ca8-ae3c-fc4dc5d55e01" /> **Release notes** <img width="2090" height="582" alt="Screenshot 2025-09-29 at 22 36 33" src="https://github.com/user-attachments/assets/81d6df12-83bd-4794-b71e-5a1fd40f0140" /> <img width="2090" height="582" alt="Screenshot 2025-09-29 at 22 40 41" src="https://github.com/user-attachments/assets/aa820767-b82b-42a5-aa5b-b0d3d22ac5e3" />
Closes #39838 Refs: * #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" />
…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" />

How
Emphasize links in Markdown Preview text using accented text color.
Note
I have chosen the accent color for links since it was looking fine with all bundled by default themes, but I'm happy to alter the color to use different theme value, if you have better candidates.
Release Notes:
Preview
Before
After
Release notes