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

Added extension icon #43

Merged
merged 4 commits into from
Mar 12, 2019
Merged

Added extension icon #43

merged 4 commits into from
Mar 12, 2019

Conversation

coliff
Copy link
Contributor

@coliff coliff commented Mar 1, 2019

Fixes: #39
If you'd like to suggest any changes, please let me know.
I think the path in package.json might need to be different, but I'm not certain.

coliff and others added 3 commits March 1, 2019 19:52
/src/icon.png -- 12.62kb -> 10.88kb (13.76%)
[ImgBot] Optimize images
Tom-Bonnike
Tom-Bonnike previously approved these changes Mar 3, 2019
package.json Show resolved Hide resolved
@Tom-Bonnike Tom-Bonnike dismissed their stale review March 3, 2019 09:36

Actually no, the img should be at the root of the project.

@Tom-Bonnike
Copy link
Owner

Tom-Bonnike commented Mar 3, 2019

Hey, thank you for this PR! ❤️
Could you move the icon to the root folder of the project? That’s what vscode-prettier does (and then the path in package.json will be correct).
Can you also make sure the circle is perfect? It seems cut from the GitHub preview.

screenshot 2019-03-03 at 10 29 14

I’m guessing the icon itself will look more crisp when scaled down to 128x128 by VSCode, thanks for uploading a retina version.

@coliff
Copy link
Contributor Author

coliff commented Mar 3, 2019

Hey. Thanks for the feedback. I’ll update the icon’s outer circle and the path in the package.json tomorrow morning.

@Tom-Bonnike
Copy link
Owner

Thaaanks! Just for clarity, the path in the package.json is correct but you put the icon in the src folder and it should be at the root instead.

@Tom-Bonnike
Copy link
Owner

Thank you!

@Tom-Bonnike Tom-Bonnike merged commit 04ce207 into Tom-Bonnike:master Mar 12, 2019
@Tom-Bonnike Tom-Bonnike changed the title Added icon Added extension icon Mar 12, 2019
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.

3 participants