Skip to content

Remove unnecessary ObeliskWidget

Jean-Paul Calderone requested to merge remove-unnecessary-obelisk-widget into main

This removes all uses of ObeliskWidget from the frontend and replaces it with more specific constraints.

ObeliskWidget is a succinct constraint but it almost always includes more constraints than required. This reduces reusability of the constrained function since it means it can be called from fewer contexts.

The new constraints should be the minimal requirements for the current implementations. If we make the functions do new things then they may need to declare new constraints - or consider doing things differently.

There is one behavioral change here is to add a use of rangeInput which is what prompted the realization that we shouldn't just request ObeliskWidget everywhere.

Merge request reports