Skip to content

Conversation

vlsi
Copy link
Collaborator

@vlsi vlsi commented Mar 15, 2020

Before:

logo on dark background

After:

logo on light background

By default, the background depends on the look and feel, however,
the logo does not look great with very dark background.
@vlsi
Copy link
Collaborator Author

vlsi commented Mar 15, 2020

@FSchumacher , do you have any idea on the way to display the logo with a dark background?

So far I'm inlined to enforce #f2f2f2 background for the splash screen.

@weisJ
Copy link
Contributor

weisJ commented Mar 16, 2020

Another possible solution would be to convert the image to svg and change the critical font colors based on the foreground of a JLabel. The font should be this. Though I don’t know if this would violate the Apache identity style guide.

@vlsi
Copy link
Collaborator Author

vlsi commented Mar 16, 2020

The logo is already in SVG: https://github.com/apache/jmeter/blob/master/xdocs/images/logo.svg

There's an alternative https://docs.oracle.com/javase/8/docs/api/java/awt/SplashScreen.html that might appear before any theme is actually loaded (== start with java.awt.SplashScreen and replace it with a more heavy-weight one later).

critical font colors based on the foreground

The question there is to figure out the appropriate colors as well.
For instance, the platform default LaF might have an unpredictable background color, so it is not clear if there's a way to automatically select the font colors.

@weisJ
Copy link
Contributor

weisJ commented Mar 16, 2020

The question there is to figure out the appropriate colors as well.
For instance, the platform default LaF might have an unpredictable background color, so it is not clear if there's a way to automatically select the font colors.

Shouldn’t it be safe to assume that the colour of a jlabel that is enabled/disabled can be read on its default background colour?

@vlsi
Copy link
Collaborator Author

vlsi commented Mar 16, 2020

Shouldn’t it be safe to assume that the colour of a jlabel that is enabled/disabled can be read on its default background colour?

Color bg = label.getBackground() works.

What do you suggest for

Color apacheFgColor = ???;
Color jmeterFgColor = ???;
Color tmFgColor = ???;

I'm not a designer, but I doubt something like 255-R, 225-G, 255-B would produce aesthetic results.

@weisJ
Copy link
Contributor

weisJ commented Mar 16, 2020

I think the apache red should stay the way it is. Would need a really weird background to produce something that isn’t readable.
Then the colours could be:

label.setEnabled(true);
Color jmeterFgColor = label.getForeground();
label.setEnabled(false);
Color tmFgColor = label. getForeground();

@vlsi
Copy link
Collaborator Author

vlsi commented Mar 16, 2020

I see what you mean.

I've created #565 as an alternative. The implementation is surprisingly simple thanks to com.github.weisj.darklaf.icons.ThemedSVGIcon.

@vlsi vlsi closed this Mar 16, 2020
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.

2 participants