Ticket #791 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

Numeric fields in UI need a spinner

Reported by: aivar Owned by: aivar
Priority: major Milestone: slimplugin1.2
Component: slim-plugin Severity: serious
Keywords: Cc:
Blocked By: Blocking:

Description

Nice to have a JSpinner for things like adjusting the threshold and adjusting cursors.

Change History

comment:1 Changed 7 years ago by aivar

Added JSpinners to SLIM Plugin UI. The cursor setting fields are problematic as these need to support a double time mode and an integer bins mode.

comment:2 follow-up: ↓ 4 Changed 7 years ago by curtis

ImageJ2 already has a JSpinner-based numeric widget, and many others. I would avoid focused effort on a custom UI component that may become irrelevant once SLIM Plugin has been converted to ImageJ2.

comment:3 Changed 7 years ago by aivar

The cursor spinners default to the floating point time mode; when I swap in an integer bins model the spinners stop working and I can't even type in the value. Dragging the cursors on the decay graph still works.

Having two separate models and swapping doesn't work (double and integer models weren't compatible, so I tried both doubles). Changing min/max/inc on a single model doesn't.

As a compromise I now disable these spinners in bin mode. This way you can still tweak the cursors using a spinner arrow and still get to see what the bin values are. You can't edit in bin mode but you can still drag the cursors.

Another UI element would be useful: If you check the box to fix a fitted parameter at that point you could use spinners to tweak the value.

comment:4 in reply to: ↑ 2 Changed 7 years ago by aivar

Replying to curtis:

ImageJ2 already has a JSpinner-based numeric widget, and many others. I would avoid focused effort on a custom UI component that may become irrelevant once SLIM Plugin has been converted to ImageJ2.

This will change in IJ2 but to support it we'll still have to figure out how to do it in raw Swing. I was just revisiting this issue as I'm putting out a new update and had to tidy up the cursor spinner code.

comment:5 Changed 7 years ago by aivar

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.