Spinners are missing up/down arrows with FlatLaf

[ctrueden]

As reported on the Image.sc Forum:

Unfortunately, spinner buttons in ImageJ2 style ContextCommand plugins are missing. Other L&Fs work.

To reproduce:

• From the Script Editor, choose Templates → Intro → Widgets (JavaScript).
• Delete the BigInteger and BigDecimal parameters (there's currently a bug affecting them)
• When the dialog box appears, observe that none of the numeric parameters have spinner arrows, only text fields

[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

[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.

[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)