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

SettingsDialog: LAF fix #1827

Merged
merged 2 commits into from Aug 13, 2022

Conversation

Scrappers-glitch
Copy link
Contributor

Fixed issue #1824 swing look and feel to be defaulted to swing LAF, if you want to change the LAF on your own use this at your SimpleApplication:

static {
  try {
      UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
  } catch (Exception e) {
      Logger.getLogger(MyClass.class.getName()).severe("Cannot set the LAF, defaulting back to System LAF");
  }
}

This is a test for this PR:
Screenshot from 2022-05-30 16-36-43

When setting the LAF at the static initializer (as a user code):
Screenshot from 2022-05-30 16-39-17

@tonihele tonihele requested a review from pspeed42 August 13, 2022 08:42
@tonihele
Copy link
Contributor

I assigned @pspeed42 the swing master to review. For this seemingly trivial thing to go forward.

@pspeed42 pspeed42 merged commit 9bcfc46 into jMonkeyEngine:master Aug 13, 2022
@pspeed42
Copy link
Contributor

Yeah, I think that's better. Let the app decide. Merged.

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.

None yet

5 participants