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

Update NetBeans icon to match macOS 11 look and feel #3168

Closed
wants to merge 1 commit into from

Conversation

oyarzun
Copy link
Contributor

@oyarzun oyarzun commented Sep 15, 2021

@matthiasblaesing
Copy link
Contributor

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?

@mcdonnell-john
Copy link
Contributor

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.

@oyarzun
Copy link
Contributor Author

oyarzun commented Sep 15, 2021

For the record: the icns file is in no way mac os specific. I have it integrated for example in gnome-shell.

@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

[Desktop Entry]
Encoding=UTF-8
Name=Apache NetBeans IDE 12.5
Comment=The Smarter Way to Code
Exec=/bin/sh "/home/oyarzun/netbeans-12.5/netbeans/bin/netbeans"
Icon=/home/oyarzun/netbeans-12.5/netbeans/nb/netbeans.png
Categories=Application;Development;Java;IDE
Version=1.0
Type=Application
Terminal=0

@mcdonnell-john I'll send an email to the mailing list for additional feedback.

@oyarzun
Copy link
Contributor Author

oyarzun commented Sep 16, 2021

nb1

nb2

nb3

nb4

@swpalmer
Copy link
Contributor

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.

@oyarzun
Copy link
Contributor Author

oyarzun commented Sep 16, 2021

nb-ds

@swpalmer
Copy link
Contributor

That’s the one, fits in with the general look and feel of icons on macOS. Beautiful!

@mcdonnell-john
Copy link
Contributor

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 :)

@eirikbakke
Copy link
Contributor

I think the logo has been squashed slightly in the proposed icons above; be sure to maintain the aspect ratio when resizing the art. Here's the logo on the NetBeans website overlaid 50% with the icon:

overlay_nbicon

@oyarzun
Copy link
Contributor Author

oyarzun commented Jun 28, 2022

@eirikbakke I think you are correct. Here is an updated svg.
apache-netbeans-macos-ds2 copy

@oyarzun oyarzun force-pushed the macos-bigsur-icon branch from 402261b to de3e1b3 Compare June 29, 2022 02:10
@mbien
Copy link
Member

mbien commented Jun 29, 2022

the more this PR progresses the larger the icons get :)

@eirikbakke
Copy link
Contributor

eirikbakke commented Jun 30, 2022

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?

@oyarzun oyarzun force-pushed the macos-bigsur-icon branch from de3e1b3 to b04280a Compare June 30, 2022 21:20
@oyarzun
Copy link
Contributor Author

oyarzun commented Jun 30, 2022

@eirikbakke added back the drop shadow.

apache-netbeans-macos-ds2 copy

Screen Shot 2022-06-30 at 5 15 44 PM

As seen in the screenshot of the dock, when NetBeans is minimized it uses the original icon since this is coming from the MainWindow.

private static final String ICON_16 = "org/netbeans/core/startup/frame.gif"; // NOI18N
private static final String ICON_32 = "org/netbeans/core/startup/frame32.gif"; // NOI18N
private static final String ICON_48 = "org/netbeans/core/startup/frame48.gif"; // NOI18N
private static final String ICON_256 = "org/netbeans/core/startup/frame256.png"; // NOI18N
private static final String ICON_512 = "org/netbeans/core/startup/frame512.png"; // NOI18N
private static final String ICON_1024 = "org/netbeans/core/startup/frame1024.png"; // NOI18N
static void initFrameIcons(Frame f) {
List<Image> currentIcons = f.getIconImages();
if( !currentIcons.isEmpty() )
return; //do not override icons if they have been already provided elsewhere (JDev)
f.setIconImages(Arrays.asList(
ImageUtilities.loadImage(ICON_16, true),
ImageUtilities.loadImage(ICON_32, true),
ImageUtilities.loadImage(ICON_48, true),
ImageUtilities.loadImage(ICON_256, true),
ImageUtilities.loadImage(ICON_512, true),
ImageUtilities.loadImage(ICON_1024, true)));
}

@eirikbakke
Copy link
Contributor

eirikbakke commented Jul 1, 2022

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:

logosize

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.)

@mbien mbien added stale No recent activity - likely to be closed. UI User Interface os:macos labels Oct 14, 2023
@mhalachev
Copy link
Contributor

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:

scale

Here's how it would appear in the dock, alongside other icons:

dock

@neilcsmith-net
Copy link
Member

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.

@mhalachev
Copy link
Contributor

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.

icon-grid

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.

@mhalachev
Copy link
Contributor

PR in is now available: apache/netbeans-nbpackage#51

@oyarzun
Copy link
Contributor Author

oyarzun commented Mar 4, 2024

Closing in favor of PR #7132 using @mhalachev's icon

@oyarzun oyarzun closed this Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
os:macos stale No recent activity - likely to be closed. UI User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants