-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: draft autowidget #54
Conversation
Not sure how far we can get with this approach or a similar one, but to avoid having to create lots of very similar widgets for different workflow parameters it would be nice to automate what we can |
Can you clarify your concern? I didn't think there would be too many different widgets. |
There's some bookkeeping involved in making a widget that creates a composite value from a number of underlying basic widgets. It's good if we can avoid repeating it unnecessarily. In particular, the There are other ways to achieve that, not sure if this is the best, but I found it to work quite well so far. |
Not necessarily too many, I agree. I thought there would be more than there seems to be. It might be easier to just keep the widget implementations separate |
If I understood correctly during the standup we will not pursue this for now, may be revisited later? |
Related to #28
Experiment with
AutoWidget
helper to make writing widgets for parameters a bit easier.The
AutoWidget
class creates a composite widget from a constructor and a range of argument widgets.The resulting widget is a simple form that produces a value of the constructor type.