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

feat: icons for desktop os #1635

Merged
merged 5 commits into from Sep 3, 2023
Merged

feat: icons for desktop os #1635

merged 5 commits into from Sep 3, 2023

Conversation

bart-krakowski
Copy link
Contributor

@bart-krakowski bart-krakowski commented Aug 7, 2022

This PR adds icons for the desktop OSs (Linux, Windows and MacOS).
Closes: #1612

Frame 31

@md-siam
Copy link

md-siam commented Aug 10, 2022

Still same as before:
Screen Shot 2022-08-10 at 6 13 10 AM

@yozachar
Copy link
Contributor

Hi, I want this association too!

Thanks for contributing @bart-krakowski! I tested it, but something's not right.

image

The folder icons vanish! Where am I going wrong?

cc: @PKief

Copy link
Owner

@PKief PKief left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The proposed icons look quite nice but are not that consistent to the other folder icons in the theme. Usually a folder icon has a colored folder icon with a symbol which has one (!) brighter color. In this PR only the windows icons looks consistent. There are some exceptions, but only if the recognition value would suffer greatly if the symbol were monochrome. For example the aurelia or bower folder icons look really ugly (I created them long time ago and should probably fix them someday 😅). I would not like to deviate from this consistency with new icons.

I will add this rule to the contributions guide very soon, as I see that this is happening quite often 🙈

@yozachar
Copy link
Contributor

yozachar commented Oct 7, 2022

@PKief, thanks for the update. So, I hopped on to Figma and did a few tweaks.

Linux Windows MacOS
Close folder-linux folder-windows folder-macos
Open folder-linux-open folder-windows-open folder-macos-open

Are these icons okay? (Open to suggestions!)


I use the gruvbox color theme. Setting custom icons on settings.json makes 'em disappear:

    "material-icon-theme.folders.associations": {
        "linux": "../../../../../../../.vscode/extensions/icons/folder-linux",
        "windows": "../../../../../../../.vscode/extensions/icons/folder-windows",
        "macos": "../../../../../../../.vscode/extensions/icons/folder-macos",
    }

But if I change to default dark color theme, the folder icons return, but they remain the default ones. (Should I open an issue?)

@PKief PKief self-requested a review October 10, 2022 19:13
@yozachar
Copy link
Contributor

image

Hi @PKief, I understand it boils down to personal taste, but the Linux folder icons on the right seems better.

If Windows and MacOS gets their latest official icons, it's fair that Linux does too. Plus "yellow" relates to the current kernel icon, sort of 😉 See: https://kernel.org/

image

SVG files 👇🏻

  • folder-linux
  • folder-linux-open

@md-siam
Copy link

md-siam commented Jul 15, 2023

183957919-d1ebfd17-4682-4d99-8a69-98865fcd0809

joe733 still not fixed. 😒

@yozachar
Copy link
Contributor

Hi @md-siam, it will only be fixed if the PR is merged. I'm neither the PR author nor a repository maintainer. I'm afraid I cannot 'fix' it.

What you can do as a workaround is, download these SVGs:

Linux Windows MacOS
Close folder-linux folder-windows folder-macos
Open folder-linux-open folder-windows-open folder-macos-open

and add the following to your .vscode/setting.json

{
    "material-icon-theme.folders.associations": {
        "linux": "../icons/folder-linux",
        "windows": "../icons/folder-windows",
        "macos": "../icons/folder-macos",
    }
}

But, in my system it's still vanishing, maybe try at your end a see.

image


@PKief I understand now, that the icon I suggested earlier was not part of Material Design Icon pack. But the ones I've put forward in this comments (see above table) conforms the both to material icons and colors. Please review.

@md-siam
Copy link

md-siam commented Jul 15, 2023

joe733 Thanks for the temporary solution. 👍

@Keeryel
Copy link
Contributor

Keeryel commented Aug 30, 2023

Surprising this branch hasn’t been pushed to the base branch yet

@PKief
Copy link
Owner

PKief commented Sep 3, 2023

Hi @md-siam, it will only be fixed if the PR is merged. I'm neither the PR author nor a repository maintainer. I'm afraid I cannot 'fix' it.

What you can do as a workaround is, download these SVGs:
...

I've just added these icons to this PR. I think they look great. Thanks for the creation of these SVGs and thanks for being patient 🙂

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.

[Icon Request] <linux, macos, windows>
5 participants