Skip to content

SettingsDialog: LAF fix - #1827

Merged
pspeed42 merged 2 commits into
jMonkeyEngine:masterfrom
Electrostat-Lab:settings-laf-fix
Aug 13, 2022
Merged

SettingsDialog: LAF fix#1827
pspeed42 merged 2 commits into
jMonkeyEngine:masterfrom
Electrostat-Lab:settings-laf-fix

Conversation

@pavly-gerges

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@stephengold stephengold added this to the Future Release milestone Aug 14, 2022
@pavly-gerges
pavly-gerges deleted the settings-laf-fix branch August 15, 2022 16:23
@Ali-RS Ali-RS modified the milestones: Future Release, v3.6.0 Dec 31, 2022
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.

5 participants