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

Add UML icon #392

Merged
merged 18 commits into from
Apr 20, 2020
Merged

Add UML icon #392

merged 18 commits into from
Apr 20, 2020

Conversation

patrik-csak
Copy link
Contributor

@patrik-csak patrik-csak commented Feb 18, 2019

Fixes #323


fileIcons

@PKief PKief force-pushed the master branch 3 times, most recently from 8651a6d to 59d9f43 Compare March 24, 2019 14:13
src/icons/fileIcons.ts Outdated Show resolved Hide resolved
@patrik-csak
Copy link
Contributor Author

@PKief Any update on this PR?

@PKief PKief self-assigned this Apr 20, 2020
Update colors and spaces of the icon
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.

Hi @ptrkcsk, I've good news, I finally reviewed your icon and did some small changes on the colors and the spaces between the letters so that it now looks like this:

uml file icon on dark background

uml file icon on light background

Thanks for your contribution and your patience ;)
It will be part of the upcoming update!

@PKief PKief merged commit e0dd874 into material-extensions:master Apr 20, 2020
@patrik-csak
Copy link
Contributor Author

Looks great! Thanks, @PKief .

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.

[Request] PlantUML
2 participants