Skip to content

fix(catppuccin.ts): remove duped t.link.#37

Merged
sgoudham merged 1 commit intocatppuccin:mainfrom
ninetailedtori:t-link
Apr 10, 2026
Merged

fix(catppuccin.ts): remove duped t.link.#37
sgoudham merged 1 commit intocatppuccin:mainfrom
ninetailedtori:t-link

Conversation

@ninetailedtori
Copy link
Copy Markdown
Contributor

@ninetailedtori ninetailedtori commented Apr 9, 2026

image

@uncenter
Copy link
Copy Markdown
Member

uncenter commented Apr 9, 2026

Please explain your reasoning for the change/pull request.

Copy link
Copy Markdown
Contributor

@sgoudham sgoudham left a comment

Choose a reason for hiding this comment

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

You mention duped t.link in the PR title, but duplicated where?

@ninetailedtori
Copy link
Copy Markdown
Contributor Author

You mention duped t.link in the PR title, but duplicated where?

Please see: https://github.com/catppuccin/codemirror/pull/37/changes#diff-7821420741b76c60ecf22c6e281ef763fbcb501067e992952b5c2a746e8f6c83R138, where it's also defined, again. This removes the original t.link styling entirely, and overwrites it with its later redefinition of the color.

@ninetailedtori ninetailedtori requested a review from sgoudham April 9, 2026 22:47
@sgoudham sgoudham merged commit 9b495f7 into catppuccin:main Apr 10, 2026
1 check passed
@ninetailedtori ninetailedtori deleted the t-link branch April 10, 2026 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants