search

openskope / skopeui

SKOPE user interface and visualization app
https://www.openskope.org
MIT License
0 stars 4 forks source link

clean up TimeSeriesPlot temporal range handling #106

Closed alee closed 3 years ago

alee commented 3 years ago

The form inputs for the temporal range is currently one-way bound with the dataset store's temporalRange property but we need it to be two-way to support #104

another thing to consider is to split temporalRange in the store into two variables, temporalRangeGte and temporalRangeLte instead of an array

Currently loading a request.json works for everything except updating the formTemporalRange in TimeSeriesPlot

alee commented 3 years ago

fixed in ba81bdd88d429358fb94330969d84f4f26766ee4

still need feedback from the science team on whether the new UX works for them (clicking into a disabled element to edit it)