Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ObjectWidget should have "None" option for non-required inputs #391

Open
imagejan opened this issue Jul 28, 2020 · 6 comments
Open

ObjectWidget should have "None" option for non-required inputs #391

imagejan opened this issue Jul 28, 2020 · 6 comments

Comments

@imagejan
Copy link
Member

When writing something like:

#@ Img input1
#@ Img (required=false) input2

println input2

... we should have a "None" option for input2, otherwise the required=false doesn't make sense as the UI in fact makes it "required". Changes are likely required to go into AbstractInputHarvester, as well as maybe into scijava-ui-swing.

See also ilastik/ilastik4ij#7 (comment).

/cc @k-dominik

@imagesc-bot
Copy link

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

https://forum.image.sc/t/how-to-create-a-plugin-parameter-thar-is-required-false-and-autoffll-false/40950/2

@k-dominik
Copy link

Thank you very much @imagejan for filing this :)

@imagesc-bot
Copy link

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

https://forum.image.sc/t/dynamiccommand-not-displaying-without-error/60767/9

@tischi
Copy link

tischi commented May 16, 2022

@ctrueden @imagejan
Any progress on that front?
I stumbled over the same issue where the UI popped up even though only non-required parameters were missing.
In my case it was calling from IJ-Macro where those non-required parameters were not recorded.

@imagejan
Copy link
Member Author

This issue here is just about ObjectWidget. @tischi I think #317 is more relevant for your issue.

@ctrueden
Copy link
Member

I was going to say the same thing, yeah, #317 is the issue about that. And no, unfortunately no one has worked on adding a None option for non-required multiple choice.

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

No branches or pull requests

5 participants