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 vfl and kl support. #72

Closed
wants to merge 3 commits into from
Closed

Add vfl and kl support. #72

wants to merge 3 commits into from

Conversation

melMass
Copy link
Contributor

@melMass melMass commented May 26, 2017

preview

@PKief
Copy link
Owner

PKief commented May 27, 2017

Thank you for the pull request. The icons look great! But I would like to improve it a little bit:

Instead of...

image

...I'd like to change it to this

image

because it matches better to the other icons and the contrast of the icon will be a little bit better.

Is it ok for you if I make the three inner parts of the .kl-icon transparent?

@PKief PKief changed the title PULL REQUEST: Add vfl and kl support. Add vfl and kl support. May 28, 2017
@melMass
Copy link
Contributor Author

melMass commented May 29, 2017

Looks great to me !

@PKief PKief closed this May 30, 2017
PKief added a commit that referenced this pull request May 30, 2017
@melMass
Copy link
Contributor Author

melMass commented May 31, 2017

Thanks!
can you push to vs ?

@PKief
Copy link
Owner

PKief commented May 31, 2017

@melMass I'll update the extension on the marketplace soon. I'm currently working on the next update and in a few days it will be ready!

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.

None yet

2 participants