Skip to content

feat: add icon for commitizen#2874

Merged
PKief merged 3 commits intomaterial-extensions:mainfrom
juliolmuller:commitizen-icon
Feb 9, 2025
Merged

feat: add icon for commitizen#2874
PKief merged 3 commits intomaterial-extensions:mainfrom
juliolmuller:commitizen-icon

Conversation

@juliolmuller
Copy link
Copy Markdown
Contributor

@juliolmuller juliolmuller commented Feb 4, 2025

Description

Add icon for commitizen configuration files.

Clsoes #2016

Contribution Guidelines

@github-actions github-actions bot added the icons PR with new icons label Feb 4, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 4, 2025

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.

@lucas-labs
Copy link
Copy Markdown
Member

Thanks for fixing the margins!

I will add some changes to improve pixel perfectness later today.

@lucas-labs
Copy link
Copy Markdown
Member

I removed the "cz" inside the logo, because it was too small to be able to see it at 16px.
Also aligned the internal lines to the 16x16 grid to make it look sharper at that size.

@lucas-labs lucas-labs requested a review from PKief February 8, 2025 13:09
Copy link
Copy Markdown
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.

@lucas-labs thank you for the adjustments. It looks nice now!

@PKief PKief merged commit 1c404cd into material-extensions:main Feb 9, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 9, 2025

Merge Successful

Thanks for your contribution! 🎉

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

@juliolmuller juliolmuller deleted the commitizen-icon branch February 10, 2025 15:37
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