Too big icon on start menu

[linsui]

Describe the bug
Too big icon on start menu.

Please confirm that this problem is VSCodium-specific

• This bug doesn't happen if I use Microsoft's Visual Studio Code. It only happens in VSCodium.

Please confirm that the issue/resolution isn't already documented

• I checked the Docs page and my issue is not mentioned there.

Screenshots

Desktop (please complete the following information):

• OS: Windows 10
• Architecture x64
• Version 1.48.0

[Bufkin]

Same issue on Windows 10 Pro

[linsui]

https://github.com/VSCodium/vscodium/blob/master/src/resources/win32/code_150x150.png
https://github.com/VSCodium/vscodium/blob/master/src/resources/win32/code_70x70.png
These icons should be fixed.

Here is the recommended padding:
https://docs.microsoft.com/en-us/windows/uwp/design/style/app-icons-and-logos#tile-padding-recommendations

@estatra Could you please help?

[layday]

The new icon is oversized on Mac as well.

[atraest]

Thanks for the reports. Here's the fix for Windows.

For the 70x70 one I couldn't do a perfect 66% padding because the image editor doesn't scale to floating points, so I got it to as close as possible at 65.719% padding.

---

@layday Can you specify which icon(s) please? Dock? Tab? Other?

@linsui @stripedpajamas Can you guys help me locate the problematic icons for Mac? I don't use neither Windows nor Mac. I recommend we solve this before merging, so I can append the Mac fix to the PR.

Can you guys confirm the location please?
Mac: /vscodium/src/resources/darwin/code.icns
Linux: /vscodium/src/resources/linux/code.png

In case this is correct, are there any other directories I should work on?

[layday]

It appears to be the case that it is slightly oversized in the dock as well as in the Finder and the tab switcher in comparison to other icons.

[atraest]

Thanks for the quick reply layday. Seems to be a padding issue also. I'll wait for linsui and stripedpajamas help. I'll check the Linux one as well, looks OK on GNOME but it could use better padding too.

[linsui]

I don't use mac. The icons looks perfectly on Windows. Thanks! @estatra

[atraest]

I don't use mac. The icons looks perfectly on Windows. Thanks! @estatra

No problem!

I'll wait for stripedpajamas when he have the time to help and I'll fix the Mac icons. I'll do some padding for the Linux ones also for good measure, to make it even across all systems.

[sefsh]

VS Code is having the same issue and have also had that in past rebrandings.

[atraest]

VS Code is having the same issue and have also had that in past rebrandings.

Oh thanks for that info @yusf. Good know, I was unaware of that. I blindly followed VS Code's hierarchy and icon sizes.

For VSCodium, one PR is up and done. The Windows one is fixed. Currently working on a second PR to fix the macOS one, and improve the Linux one a little bit.

[stripedpajamas]

Mac: /vscodium/src/resources/darwin/code.icns

Correct @estatra

[atraest]

Mac: /vscodium/src/resources/darwin/code.icns

Correct @estatra

Cool. I'll work on it soon and submit a PR.

[sefsh]

For macOS you may want to make it adhere to the Big Sur icon guidelines. Otherwise you’ll be having similar issues within a few months.

[atraest]

For macOS you may want to make it adhere to the Big Sur icon guidelines. Otherwise you’ll be having similar issues within a few months.

Thanks for the heads up.

[atraest]

@stripedpajamas Done! The PR is ready for review.

[layday]

Perhaps someone on Big Sur can try it out but on Mojave it still looks too big.

[atraest]

Perhaps someone on Big Sur can try it out but on Mojave it still looks too big.

Hello @layday thanks for the feedback. It should be fixed now, can you please check again after the new commit?

I'm strictly following:
https://developer.apple.com/design/resources/
https://developer.apple.com/design/human-interface-guidelines/macos/icons-and-images/app-icon/
https://developer.apple.com/design/human-interface-guidelines/macos/overview/whats-new-in-macos/

[layday]

It is the correct size now comparing it to the Photoshop template from your first link. Minor perhaps, but you will notice that the corners don't line up with the template's exactly. This is because the Apple shape is not a rounded rectangle but a squircle - you will notice it begins to taper inward close to the centre.

[atraest]

It is the correct size now comparing it to the Photoshop template from your first link.

Good. That was the main problem I wanted to solve right now. P.S.: I use the Sketch files.

Minor perhaps, but you will notice that the corners don't line up with the template's exactly.

I know. I simply do not have the time to patch such minor details right now. I wanted to solve the major problem which was the padding issue.

This is because the Apple shape is not a rounded rectangle but a squircle - you will notice it begins to taper inward close to the centre.

The technical term is Lamé's special quartic. You don't need to teach a designer about geometry math ; ) Don't worry, it's not a lack of knowledge, it's a lack of time.

Maybe I'll give the macOS icon more time to do Apple's specific Lamé's math in the future, can't right now, sorry. As the main issue is solved I'm satisfied.

Anyone is free to submit a patch for Apple's specific Lamé's math. I doing this between snacks and study, I need to get back to work (day job) soon.

--

There is also some autogenerated mistakes (I'm guessing from the CI process?) for the icons on the Windows side of things, if anyone wants to dive into that and have the free time to do so, please go ahead. I did what I could/had the time to.

[layday]

Yes, well, I don't know what your credentials are - I did not mean to offend. And we all do things pro bono in our free time we might take an interest in - just like our time is precious to all of us - we only have a limited supply of it after all.

[github-actions]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment, and we'll keep it open. If you have any new additional information, please include it with your comment!

[github-actions]

This issue has been closed due to inactivity, and will not be monitored. If this is a bug and you can reproduce this issue, please open a new issue.

[github-actions]

This issue has been closed due to inactivity, and will not be monitored. If this is a bug and you can reproduce this issue, please open a new issue.