Skip to content
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

Feat: added sway icon #2573

Merged
merged 8 commits into from
Sep 6, 2024
Merged

Feat: added sway icon #2573

merged 8 commits into from
Sep 6, 2024

Conversation

mohitchandel
Copy link
Contributor

@mohitchandel mohitchandel commented Sep 4, 2024

Description

Added file icon for sway language with sw extension

Contribution Guidelines

@github-actions github-actions bot added the icons PR with new icons label Sep 4, 2024
Copy link
Contributor

github-actions bot commented Sep 4, 2024

Preview

Thank you for creating a pull request. This preview shows you how your icon will look on the different themes:

Generated preview

Check how your icon fits in a 16x16 grid with our Pixel Perfect Checker by following this link.

You can find more information on how to contribute in the contribution guidelines.

icons/sway.svg Show resolved Hide resolved
src/core/icons/fileIcons.ts Outdated Show resolved Hide resolved
Copy link
Member

@lucas-labs lucas-labs left a comment

Choose a reason for hiding this comment

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

This suggestion makes the icon pixel perfect at 16x16 and replaces colors, using the material palette

icons/sway.svg Outdated Show resolved Hide resolved
@PKief PKief merged commit f8171ba into material-extensions:main Sep 6, 2024
4 checks passed
Copy link
Contributor

github-actions bot commented Sep 6, 2024

Merge Successful

Thanks for your contribution! 🎉

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

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.

3 participants