-
Notifications
You must be signed in to change notification settings - Fork 864
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
Update NetBeans icon to match macOS 11 look and feel #3168
Conversation
For the record: the icns file is in no way mac os specific. I have it integrated for example in gnome-shell. Given that everybody seems to want to have dark desktops, I'm surprised, that an icon, that works on both is put onto a light background and that is the future? |
Is this something that should be raised on the mailing lists? To get the communities feedback on? I have no issues approving this, I do however prefer the current icon style over the Apple style but ultimately not pushed either way. |
@matthiasblaesing I do not believe this is correct, I think the icns file is only used on macOS. [Edit] But it seems you can change the desktop entry to use the icns and it looks much better. https://github.com/apache/netbeans/search?q=netbeans.icns The gnome desktop entry uses the png
@mcdonnell-john I'll send an email to the mailing list for additional feedback. |
Something like the last image, with a less drastic gradient, the dark side on the bottom, and a slight drop shadow for the NB logo. |
That’s the one, fits in with the general look and feel of icons on macOS. Beautiful! |
That last one is the best one imo @oyarzun... Still not overly fussed on changing the icon personally, but I do like that one better then the previous ones you've done :) |
673e83c
to
402261b
Compare
@eirikbakke I think you are correct. Here is an updated svg. |
402261b
to
de3e1b3
Compare
the more this PR progresses the larger the icons get :) |
Thanks! Could we still get the nice drop shadow on the logo as was in https://user-images.githubusercontent.com/2364606/133636825-ef6804ee-d4a5-4941-9821-5ce9151c2bb7.png ? Could we see how the latest version looks in the Dock together with other standard MacOS apps? |
de3e1b3
to
b04280a
Compare
@eirikbakke added back the drop shadow. As seen in the screenshot of the dock, when NetBeans is minimized it uses the original icon since this is coming from the netbeans/platform/core.windows/src/org/netbeans/core/windows/view/ui/MainWindow.java Lines 506 to 523 in f566859
|
Thanks! Perhaps size the logo down a little bit, so that there's a bit of margin to the edge of the rounded square? See the other icons in the doc: Ideally we'd get the same icon showing in minimized state. What happens if f.setIconImages is commented out in MainWindow, and the application is run from an application bundle? Does the main window and any dialog boxes then end up with the icon specified in -Xdoc:icon? (We could add some MacOS specific behavior in MainWindow, probably enabled by a system property that's passed in from the same script that sets -Xdock:icon.) |
I agree with @eirikbakke that scaling down the logo would enhance its appearance, as it currently extends to the top and bottom of the icon. Additionally, maintaining a white background color would ensure consistency with both the splash screen and the about dialog, as in both cases the logo is presented on a white background with no effects applied. Perhaps we could consider using the small side element as a reference for scaling? It might help maintain proportion and balance across the design: Here's how it would appear in the dock, alongside other icons: |
The installer changes here are effectively dead. If this change goes in to the IDE, will need a matching PR to update https://github.com/apache/netbeans-nbpackage/tree/master/src/main/resources/org/apache/netbeans/nbpackage for the macOS installer. |
Thanks. I will prepare an appropriate PR for this with just one more tiny adjustment: aligning the logo to the icon grid, as both the inner and outer visual components fit perfectly. PS: I'll provide an update once the icon set is ready, as it requires some adjustments for the smaller sizes. In the meantime, any other comments and suggestions are welcome. |
PR in is now available: apache/netbeans-nbpackage#51 |
Closing in favor of PR #7132 using @mhalachev's icon |
Update NetBeans macOS icon according to https://developer.apple.com/design/human-interface-guidelines/macos/icons-and-images/app-icon