-
-
Notifications
You must be signed in to change notification settings - Fork 924
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
[Task] Adjust icons to make pixel perfect #1613
Comments
@mpodciwinski Because this is going to be a lengthy and time-consuming task, I suggest you better fork the repo and make a branch, that you will eventually use for a PR, add your suggested icons there and post the link here. It will be better to track the progress this way. |
Looking at your icons I have the following requests:
|
@JimiC Ok, but now can you send me folder with examples? or write me how I can easily create own?it's help me a lot with create new icon. You guidelines is not correct. I will describe them exactly tomorrow. |
We have no folder with examples. All icons are in the icons folder. |
I know, but how i want see how icon look in VsCode, must create folder (or file) with this extension. This is problematic. |
Yeah. you see the examples generator is a legacy tool. I made it when we switched from There is though a workaround by using the customization feature. This works if you follow these and these instructions. |
Don't be angry for me, but i can not do this. It's too hard for me now. 😓 I would be grateful if you do this himself and publish. |
I can help you further tomorrow. |
Thank's ✌️ |
@mpodciwinski Turns out that So until I get the fix in, you can start with the folders manually in
The icon should show up. Now if you want to test your icons simply replace the existing icon (make a backup first just in case) in the icons folder. Then start |
If you have any link I can read or watch on why our guidelines aren't correct, I would appreciate it much. |
I attach a detailed description of creating pixel perfect icons. Pixel Perfect guidelines
After create new artboard go to the Transform window and select both "Scale Strokes & Effects" and "Align to pixel grid"
Enjoy! 😉 |
@JimiC ok, give me a sign if you fix it and generate examples. |
@JimiC i did as you wrote:
but some errors appear to me Error screencast And when I preview the file's file it still leads to the installation folder. Even i disable vscode-icons in vscode and launch extensions (F5) and choose VSCode Icons in "Preferences:File Icon Theme" then too nothing happens (icons dont shows up). I'm doing something wrong? |
That means that the project did not build. Try |
output from shell: PS D:\Dropbox\Folder zespołu Inspect\Repo\github\vscode-icons> npm run build
> vscode-icons@7.23.0 prebuild D:\Dropbox\Folder zespołu Inspect\Repo\github\vscode-icons
> npm run lint && npm run test
> vscode-icons@7.23.0 lint D:\Dropbox\Folder zespołu Inspect\Repo\github\vscode-icons
> tslint -p .
'tslint' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! vscode-icons@7.23.0 lint: `tslint -p .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the vscode-icons@7.23.0 lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! C:\Users\mpodc\AppData\Roaming\npm-cache\_logs\2018-06-04T10_44_33_250Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! vscode-icons@7.23.0 prebuild: `npm run lint && npm run test`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the vscode-icons@7.23.0 prebuild script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! C:\Users\mpodc\AppData\Roaming\npm-cache\_logs\2018-06-04T10_44_33_288Z-debug.log I've installed all the recommended packages. |
Delete |
It also fails. PS D:\Dropbox\Folder zespołu Inspect\Repo\github\vscode-icons> npm i
> vscode-icons@7.23.0 preinstall D:\Dropbox\Folder zespołu Inspect\Repo\github\vscode-icons
> git submodule update --init --recursive
fatal: 'submodule' appears to be a git command, but we were not
able to execute it. Maybe git-submodule is broken?
npm ERR! code ELIFECYCLE
npm ERR! errno 128
npm ERR! vscode-icons@7.23.0 preinstall: `git submodule update --init --recursive`
npm ERR! Exit status 128
npm ERR!
npm ERR! Failed at the vscode-icons@7.23.0 preinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR! A complete log of this run can be found in:
npm ERR! C:\Users\mpodc\AppData\Roaming\npm-cache\_logs\2018-06-04T13_23_14_309Z-debug.log |
You are in trouble. The |
Do me a favor and tell me what |
Maybe, but other project works correct with git. PS C:\Users\mpodc\Desktop\vscode-icons> node -v
v8.11.2
PS C:\Users\mpodc\Desktop\vscode-icons> npm -v
6.1.0
PS C:\Users\mpodc\Desktop\vscode-icons>
|
Can you perform |
Reinstal |
You need to revert any changes. Make a copy of your changes first if you want to keep them (or stash them). You should not change anything in the |
I do not make any changes to the entire |
I'm at a loss here. I don't know how to help you further. |
Thanks anyway, i give up too. |
Still, you will have to resolve the issue with |
@mpodciwinski I figured out your issue. It's a bug with Since some commits have been made to the project it's prudent to fetch those commits before you try the following workaround. There are two ways to bypass the error:
or
Then proceed with P.S.: In case you install |
I did as you said, install node 7.9.0 and npm 5.10.0. ❯ node -v
v7.9.0
❯ npm -v
5.10.0 but still 😞 > vscode-icons@7.23.0 preinstall /Users/michal_mac/Dropbox/Folder zespołu Inspect/Repo/github/vscode-icons
> git submodule update --init --recursive
> nodegit@0.18.3 install /Users/michal_mac/Dropbox/Folder zespołu Inspect/Repo/github/vscode-icons/node_modules/nodegit
> node lifecycleScripts/preinstall && node lifecycleScripts/install
[nodegit] Running pre-install script
[nodegit] Configuring libssh2.
{ Error: spawn /bin/sh ENOENT
at exports._errnoException (util.js:1050:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:367:16)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickCallback (internal/process/next_tick.js:104:9)
code: 'ENOENT',
errno: 'ENOENT',
syscall: 'spawn /bin/sh',
path: '/bin/sh',
spawnargs:
[ '-c',
'/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/libssh2/configure --with-libssl-prefix=/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/openssl/openssl' ],
cmd: '/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/libssh2/configure --with-libssl-prefix=/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/openssl/openssl' }
[nodegit] ERROR - Could not finish preinstall
{ Error: spawn /bin/sh ENOENT
at exports._errnoException (util.js:1050:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:367:16)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickCallback (internal/process/next_tick.js:104:9)
code: 'ENOENT',
errno: 'ENOENT',
syscall: 'spawn /bin/sh',
path: '/bin/sh',
spawnargs:
[ '-c',
'/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/libssh2/configure --with-libssl-prefix=/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/openssl/openssl' ],
cmd: '/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/libssh2/configure --with-libssl-prefix=/Users/michal_mac/Dropbox/Folder\\ zespołu\\ Inspect/Repo/github/vscode-icons/node_modules/nodegit/vendor/openssl/openssl' }
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! nodegit@0.18.3 install: `node lifecycleScripts/preinstall && node lifecycleScripts/install`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the nodegit@0.18.3 install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! /Users/michal_mac/.npm/_logs/2018-06-09T15_52_34_852Z-debug.log Even how i delete preinstall script. Still this same error.
|
|
I missed it. |
@mpodciwinski Have you abandoned the task? |
No, I'm still working. there is a lot to do. |
Good to know. |
@mpodciwinski FYI, you can update |
Does this task still in progress? Because there is a option to make every icon pixel perfect for their current size (32x32) without having to rely on those steps. Just right click on the selection and select "Generate Pixel Perfect" and adobe Illustrator will automatically modify the svg sizes to make it fit the grid as stated by the guidelines. |
Illustrator version? |
@JimiC Adobe Illustrator CC 2019 :) |
As pointed out by @mpodciwinski, our icons aren't pixel perfect and it's about time to fix that.
A good reference about pixel perfect icon designing: https://www.youtube.com/watch?v=KYpVtDj-xQs
The text was updated successfully, but these errors were encountered: