Jan Eglinger

Results 251 comments of Jan Eglinger

@stelfrich wrote: > Especially with respect to (macro) recording of Services? This is unrelated to visibility of parameters in UI widgets, no? In general, `Service` parameters _should_ not get recorded...

This issue here is just about `ObjectWidget`. @tischi I think https://github.com/scijava/scijava-common/issues/317 is more relevant for your issue.

While looking into this, I was surprised to find that there *is* a different (slightly better?) message already, located in `CheckInputsPreprocessor`: https://github.com/scijava/scijava-common/blob/3d64e2a2ea9f6eab5d75c7e778dc3da75bd03f90/src/main/java/org/scijava/module/process/CheckInputsPreprocessor.java#L56-L58 ... but it never/rarely gets executed because `AbstractInputHarvester`...

For choices, there is `radioButtonHorizontal` already: ``` #@ String (choices={"a", "b", "c"}, style="radioButtonHorizontal") myChoice ``` (I would also love to see multiple-select (https://github.com/scijava/scijava-common/issues/260) creating a list of checkboxes in this...

> I think, for starters, maybe it would be good to just implement first the horizontal grouping idea and leave out the tab idea for now. What do you think?...

> But will ParameterGroups work easily in scripts ? It would require adding a dedicated script directive, next to the (half-finished) `#@script` and `#@import` directives as well as `#@initialize` and...

@ctrueden do we want to be case-insensitive when testing `style` attributes (i.e. allowing all of `message`, `MESSAGE` and `mEsSaGe`)?

Interestingly, the following works (showing a results table with two rows): ``` double[] a = [2, 2] ``` ... while the following: ``` int[] a = [2, 2] ``` throws...

Maybe `String[]` or `List`?

> But it does not work yet. Right, I get: ``` java.lang.NullPointerException at org.scijava.widget.DefaultWidgetModel.getChoices(DefaultWidgetModel.java:234) ``` As far as I get it, `choicesList.get(i)` seems to be `null` here: https://github.com/scijava/scijava-common/blob/f7457f0e0538a9f61b25ffe43988660766e14a7a/src/main/java/org/scijava/widget/DefaultWidgetModel.java#L234 ... so...