-
-
Notifications
You must be signed in to change notification settings - Fork 641
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 styled-components icon (https://styled-components.com) #657
base: main
Are you sure you want to change the base?
Conversation
51dd496
to
de04c27
Compare
Any possibility in merging this? |
b670644
to
387e8ff
Compare
@bart-krakowski do you want to merge the updated icon? |
Yes, I optimized and updated the icon as per your suggestion. |
Thanks! (I made the PR to be credited as a author or co-author in the update commit, but no worries) |
I'm looking forward for this icon. A directory icon would be cool too! |
Any updates on this? I'm don't know if the styled-component icon just wasn't already uploaded to the extension or if I'm just making some typo in the settings.json to make it work. |
The proposed icon does not fit so good into this extension in its current status. The black border is not really suitable and the icon. So we should at least remove this border and refine some colors a bit and then it might be a fit. Will check it soon. |
Closes: #656