-
Notifications
You must be signed in to change notification settings - Fork 290
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
Replace the java file icon with the old java cup icon #538
Comments
Hi, this is a common question I have heard from a number of VS Code users, as the J icon (when small) looks like a red clock, which, anecdotally, is often confused with an error. In the original PR that changed to this icon (#527), a second proposal was given which uses a less complicated cup. Is this something that we could perhaps consider for Java files? |
I'm pretty open to it - @nickzhums (I just grabbed you as someone who will know folks using Java in vs code) do you have an opinions on the default java file icon? |
thanks for including me on this one. Let me also mention our engineering team here : @Eskibear @CsCherrYY @jdneo @testforstephen @akaroml |
+1 to use the coffee cup. The current one is easily confused with an error indicator. |
@orta We have discussed internally and decided we should go ahead with the icon replacement. How should we go about this? |
Someone should send a PR 👍🏻 |
tbh, blue seems a bit weird here - @orta do you have any suggestions? I'm ok with the red one (first picture) |
No suggestions on color, I don't use Java so I don't know what colour is culturally close. @sanket-bhalerao - that's probably going to look out of placed and unbalanced compared to the other icons which are visually centered 👍🏻 |
@orta for the icon, do we need a 16x16 or 20x20 ? |
Recommendations are in the README: https://github.com/jesseweed/seti-ui#adding-file-icons and then compare with others for feel |
We had a survey on Twitter: https://twitter.com/NickZhu9/status/1517053859025195008 . Turns out users are more fond of a simple J icon. So I just created a PR for it. Below is the comparison with current behavior. |
Issue Type: Feature Request
change java icon to the java cup icon.
VS Code version: Code - Insiders 1.37.0-insider (0bd57ac9efae79f9eb1aedc3818ebbb570b9bf6b, 2019-08-02T21:22:13.578Z)
OS version: Windows_NT x64 10.0.17134
The text was updated successfully, but these errors were encountered: