Refactor chart widget

eea · Jan 9, 2020 · b91d53e · b91d53e
1 parent 8ee7886
commit b91d53e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/Widget/ModalEditor.jsx b/src/Widget/ModalEditor.jsx
@@ -96,22 +96,22 @@ class Edit extends Component {
                 selection
                 options={selectProviders}
                 onChange={(ev, { value }) =>
-                  this.props.onChangeValue({ ...this.props.data, url: value })
+                  this.props.onChangeValue({ ...this.props.value, url: value })
                 }
               />
               <LoadablePlotlyEditor
-                data={this.props.data?.data || []}
-                layout={this.props.data?.layout || {}}
+                data={this.props.value?.data || []}
+                layout={this.props.value?.layout || {}}
                 config={config}
-                frames={this.props.data?.frames || []}
+                frames={this.props.value?.frames || []}
                 dataSources={this.props.providerData || dataSources}
                 dataSourceOptions={
                   this.props.dataSourceOptions ||
                   getDataSourceOptions(dataSources)
                 }
                 plotly={plotly}
                 onUpdate={data =>
-                  this.props.onChangeValue(data, this.props.url)
+                  this.props.onChangeValue({ ...this.props.value, data })
                 }
                 useResizeHandler
                 debug

diff --git a/src/Widget/Widget.jsx b/src/Widget/Widget.jsx
@@ -21,7 +21,7 @@ class ModalChartEditor extends Component {
       <Modal open={true} size="fullscreen">
         <Modal.Content scrolling>
           <ChartEditor
-            data={this.props.value}
+            value={this.state.value}
             onChangeValue={value => {
               console.log('Set chart data', value);
               this.setState({ value });