fix bug where original domain would not reset on variable change, breaking reset behaviour

Author: Peter9192

apps/class-solid/src/components/Analysis.tsx

@@ -193,7 +193,7 @@ export function TimeSeriesPlot({ analysis }: { analysis: TimeseriesAnalysis }) {
 
   // Define axis format
   const formatters: Record<string, (value: number) => string> = {
-    t: formatSeconds,
+    t: d3.format(".0f"),
     utcTime: formatUTC,
     default: d3.format(".4"),
   };

apps/class-solid/src/components/plots/Axes.tsx

@@ -13,11 +13,9 @@ type AxisProps = {
 export const AxisBottom = (props: AxisProps) => {
   const [chart, updateChart] = useChartContext();
 
-  // Store original domain on first render
-  props.domain && updateChart("originalDomainX", props.domain());
-
   // Update scale props when domain or type changes
   createEffect(() => {
+    props.domain && updateChart("originalDomainX", props.domain());
     props.domain && updateChart("scalePropsX", { domain: props.domain() });
     props.type && updateChart("scalePropsX", { type: props.type });
   });
@@ -46,11 +44,9 @@ export const AxisBottom = (props: AxisProps) => {
 export const AxisLeft = (props: AxisProps) => {
   const [chart, updateChart] = useChartContext();
 
-  // Store original domain on first render
-  props.domain && updateChart("originalDomainY", props.domain());
-
   // Update scale props when domain or type changes
   createEffect(() => {
+    props.domain && updateChart("originalDomainY", props.domain());
     props.domain && updateChart("scalePropsY", { domain: props.domain() });
     props.type && updateChart("scalePropsY", { type: props.type });
   });