material-extensions / vscode-material-icon-theme

Available on the VSCode Marketplace
https://marketplace.visualstudio.com/items?itemName=PKief.material-icon-theme
MIT License
1.95k stars 607 forks source link

The 5.0+ icons looks worse on my system than the 4.34 icons #2311

Open samualtnorman opened 2 months ago

samualtnorman commented 2 months ago

4.34.0: image 5.1.0: image

I don't mind the new designs, but where the old icons were clear and sharp, the new icons look blurry. The obvious examples are the Node.js logo where the "JS" is clearly readable in the first screenshot but very blurry in the second, .env's appears darker due to the blurring, Tailwind's logo looks sort of squished, the .d.ts icon has become a square with 2 border colours (one darker, one lighter), and PNPM's squares are now blurring into each other.

This is not a criticism of the designs, I just think something is misconfigured. My instance of VSCode is running default settings for the explorer window and my zoom is set to 100% so I don't think this is an issue on my end.

I've been sticking to version 4.34 but I thought I'd make this issue because I'm sure other people are having this problem as well (whether they realise it or not).

Computer information (please complete the following information):

mallowigi commented 2 months ago

I also don't understand why the "pixel perfect" branch was merged. It's obviously a degradation over the previous SVGs.

lucas-labs commented 2 months ago

Are you sure your zoom settings are 100%?

This is how it was in 4.34:

image

This is how 5.x looks for me:

image

Yours looks a bit smaller, althogh it's just the icon, the text is exactly the same as it is on my end:

image

samualtnorman commented 2 months ago

are you on macos?

lucas-labs commented 2 months ago

After some forensic investigation (😅) over your screenshots, I see your icons are showing at 15x15px for some reason:

image

Instead of the vscode default, which is 16x16px (the pixel perfect PR made icons align with a 16x16 grid):

image

Default vscode css:

image

I honestly don't know of any vscode setting that allow us to change the size of the icon alone without loading a custom css that specifically changes the icon's background-size property (you would need the loader extension and set "vscode_custom_css.imports" setting in vscode) or changing the zoom level.

lucas-labs commented 2 months ago

are you on macos?

I'm on Windows and Ubuntu.

samualtnorman commented 2 months ago

Are you sure your zoom settings are 100%?

I just setup a fresh vscode on my PC and did nothing but install Material Icon Theme and switch to it. The issue persists. My monitor is 1920x1080 and I'm running KDE if that helps.

I'm on Windows and Ubuntu.

And the icons look identical on both OS's to you?

lucas-labs commented 2 months ago

And the icons look identical on both OS's to you?

Yes, they look sharp on my end on both Windows and Ubuntu.

The only way I could make it look exactly like in your screenshot was by manually changing the icons to 15px using devtools.

image

samualtnorman commented 2 months ago

Is that the desktop version of VS Code or the browser version? I've just tested it in a browser and all the icons look proper. This might be an issue that only affects desktop users (which is the majority).

samualtnorman commented 2 months ago

I enabled Developer: Toggle Developer Tools, and inspected the css of an icon in the desktop version of VS Code and this is what I got: image

lucas-labs commented 2 months ago

Is that the desktop version of VS Code or the browser version? I've just tested it in a browser and all the icons look proper. This might be an issue that only affects desktop users (which is the majority).

I'm using the Desktop version.

I enabled Developer: Toggle Developer Tools, and inspected the css of an icon in the desktop version of VS Code...

Looks the same here. Weird.

image

I'm honestly clueless 😒 I've been reading antialiasing, GPU acceleration and some GPU configs might cause issues like this one, but idk... I think the fonts should look blurry too if that was the issue and it doesn't seem to be the case here.

lucas-labs commented 2 months ago

Btw, I also noticed that your icons in v4.34 are also being displayed 1px smaller than what they really were. Which strangely make them look a bit better compared to mine.

image

(above is yours, below is mine)

samualtnorman commented 2 months ago

this is turning into a rabbit hole