Skip to content

refactor: remove icons for tex and ltx file extensions #3065

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

Merged
merged 1 commit into from
Jun 25, 2025

Conversation

marlondecol
Copy link
Contributor

Description

This pull request removes the icon associations for the .tex and .ltx file extensions to avoid a conflict with the icon assigned to the latex language ID.

The issue is that the .tex extension is used for both TeX and LaTeX files. However, since an icon was specifically associated with this extension, the icon would always be the same regardless of the language ID.

By removing the icon association from the extension, only the language ID will determine which icon is shown. Since VS Code recognizes the tex and latex IDs without needing a third-party extension, this should not cause any issues.

Contribution Guidelines

@github-actions github-actions bot added the icons PR with new icons label Jun 25, 2025
Copy link
Member

@PKief PKief left a comment

Choose a reason for hiding this comment

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

lgtm! Thank you for the update.

@PKief PKief merged commit b1daa97 into material-extensions:main Jun 25, 2025
3 checks passed
Copy link
Contributor

Merge Successful

Thanks for your contribution! 🎉

The changes will be part of the upcoming update on the Marketplace.

@marlondecol marlondecol deleted the latex-icons-adjust branch June 25, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icons PR with new icons
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants