Skip to content

Conversation

@neilcsmith-net
Copy link
Member

@neilcsmith-net neilcsmith-net commented Nov 25, 2021

Switch to FlatLaf Light as default look and feel, and editor profile, for the IDE. This is the simplest approach using nb/ide.branding, so keeps the existing defaults for the platform, and the laf used for the upgrader dialog.

Related to discussion at https://lists.apache.org/thread/9v3tlz8klqs93c1z6s0kvg03tr8w6kdx

@neilcsmith-net neilcsmith-net added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Nov 25, 2021
@neilcsmith-net neilcsmith-net added this to the NB13 milestone Nov 25, 2021
Copy link
Contributor

@lkishalmi lkishalmi left a comment

Choose a reason for hiding this comment

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

A really elegant solution, indeed!

@eirikbakke
Copy link
Contributor

Did this approach successfully avoid the problems from the "The Options ruban remains white with DarkLaf" email thread, including on the very first startup with a clean user directory?

(In my own NetBeans Platform app, I noticed that simply setting "--laf com.formdev.flatlaf.FlatLightLaf" in app.conf did not work, due to the FlatLAF module not being loaded early enough in the startup sequence.)

@neilcsmith-net
Copy link
Member Author

neilcsmith-net commented Nov 25, 2021

Thanks @lkishalmi - it got simpler the more I looked at it.

@eirikbakke try it! It's what review is for. 😉

The problem in that thread, and probably in your own platform application, is a missing dependency on the non-API org.netbeans.swing.laf.flatlaf. That module is autoload so doesn't kick in without one. We have that in core.kit. Having that module as autoload is a bug in my opinion. One perhaps to fix in this PR.

@neilcsmith-net neilcsmith-net removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Nov 29, 2021
@geertjanw
Copy link
Member

Great.

@eirikbakke
Copy link
Contributor

Sorry, I didn't get a chance to test this patch yet--but I'm sure it works fine!

(I'll apply it on my NetBeans Platform app eventually, though, and report problems if there are any.)

@neilcsmith-net neilcsmith-net merged commit 1aea549 into apache:master Nov 29, 2021
@lkishalmi
Copy link
Contributor

Looks Great!

image

The picture is taken from a new install of a freshly cut netbeans-dev Snap

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.

4 participants