Spinners are missing up/down arrows with FlatLaf about scijava-ui-swing

[DevCharly]

Following lines cause the problem:

scijava-ui-swing/src/main/java/org/scijava/ui/swing/widget/SwingNumberWidget.java

Lines 131 to 133 in 40bd474

	Dimension spinnerSize = spinner.getSize();
	spinnerSize.width = 50;
	spinner.setPreferredSize(spinnerSize);

spinner.getSize() always returns 0, 0 because the spinner was just created and is not yet layouted.
So the preferred height of the spinner becomes 0, which FlatLaf does not like.
Probably a bug in FlatLaf. Will investigate...

Anyway seems that the intention of the above code is to limit the preferred width of the spinner.
Replacing spinner.getSize() with spinner.getPreferredSize() should fix the issue.

[imagejan]

Thanks for the suggestion, @DevCharly. I can confirm this fixes the issue, and opened #78.

[imagesc-bot]

This issue has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/missing-spinner-buttons-with-flatlaf-look-and-feel/72185/2

[tferr]

FYI, I actually don't think (I could be, of course, wrong - I did not test any other L&F) limiting the preferred width of the spinner is needed. Things seem to work fine if one leaves it to defaults. Maybe this is just some historic code that has been carried over!? On #79, I have a couple of proposals on things that I found when looking into this.

[tferr]

FYI, I actually don't think (I could be, of course, wrong - I did not test any other L&F) limiting the preferred width of the spinner is needed. Things seem to work fine if one leaves it to defaults

I was wrong of course. The rationale for this is explained here: #79 (comment)