This is an automated email from the ASF dual-hosted git repository. rusackas pushed a commit to branch timeseries-control-touchup in repository https://gitbox.apache.org/repos/asf/superset.git
commit b5ed074dfe8301bcdffc7d24372b51218190a4df Author: Evan Rusackas <[email protected]> AuthorDate: Thu Feb 9 12:01:54 2023 -0700 chore: moving related stacking and value display controls next to each other. --- .../src/Timeseries/Area/controlPanel.tsx | 34 +++++++++++----------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/Area/controlPanel.tsx b/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/Area/controlPanel.tsx index 2f52bcce59..2732592739 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/Area/controlPanel.tsx +++ b/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/Area/controlPanel.tsx @@ -99,37 +99,37 @@ const config: ControlPanelConfig = { }, }, ], - [showValueControl], [ { - name: 'stack', + name: 'show_extra_controls', config: { - type: 'SelectControl', - label: t('Stacked Style'), + type: 'CheckboxControl', + label: t('Enable Stacking Controls'), renderTrigger: true, - choices: AreaChartExtraControlsOptions, - default: null, - description: t('Stack series on top of each other'), + default: false, + description: t( + 'Enables controls to allow users to select ' + + 'their preferred method for stacking or ' + + 'unstacking series within the chart.', + ), }, }, ], - [onlyTotalControl], [ { - name: 'show_extra_controls', + name: 'stack', config: { - type: 'CheckboxControl', - label: t('Extra Controls'), + type: 'SelectControl', + label: t('Stacked Style'), renderTrigger: true, - default: false, - description: t( - 'Whether to show extra controls or not. Extra controls ' + - 'include things like making mulitBar charts stacked ' + - 'or side by side.', - ), + choices: AreaChartExtraControlsOptions, + default: null, + description: t('Stack series on top of each other'), }, }, ], + [showValueControl], + [onlyTotalControl], [ { name: 'markerEnabled',
