Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

Commit

Permalink
fix(plugin-chart-echarts): fix time-series chart xAxisShowMin(Max)Lab…
Browse files Browse the repository at this point in the history
…el default value (#1161)

* fix(plugin-chart-echarts): delete showMinLabel and showMaxLabel

fix apache/superset#15042

* fix(plugin-chart-echarts):
  • Loading branch information
xiezhongfu committed Jul 5, 2021
1 parent 56a3b01 commit c383774
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 32 deletions.
26 changes: 0 additions & 26 deletions plugins/plugin-chart-echarts/src/Timeseries/controlPanel.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,6 @@ const {
yAxisBounds,
zoomable,
xAxisLabelRotation,
xAxisShowMinLabel,
xAxisShowMaxLabel,
} = DEFAULT_FORM_DATA;
const config: ControlPanelConfig = {
controlPanelSections: [
Expand Down Expand Up @@ -334,30 +332,6 @@ const config: ControlPanelConfig = {
},
},
],
[
{
name: 'xAxisShowMinLabel',
config: {
type: 'CheckboxControl',
label: t('Show Min Label'),
default: xAxisShowMinLabel,
renderTrigger: true,
description: t('Show Min Label'),
},
},
],
[
{
name: 'xAxisShowMaxLabel',
config: {
type: 'CheckboxControl',
label: t('Show Max Label'),
default: xAxisShowMaxLabel,
renderTrigger: true,
description: t('Show Max Label'),
},
},
],
[
{
name: 'xAxisLabelRotation',
Expand Down
4 changes: 0 additions & 4 deletions plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,6 @@ export default function transformProps(chartProps: ChartProps): EchartsProps {
stack,
truncateYAxis,
yAxisFormat,
xAxisShowMinLabel,
xAxisShowMaxLabel,
xAxisTimeFormat,
yAxisBounds,
yAxisTitle,
Expand Down Expand Up @@ -148,8 +146,6 @@ export default function transformProps(chartProps: ChartProps): EchartsProps {
xAxis: {
type: 'time',
axisLabel: {
showMinLabel: xAxisShowMinLabel,
showMaxLabel: xAxisShowMaxLabel,
formatter: xAxisFormatter,
rotate: xAxisLabelRotation,
},
Expand Down
2 changes: 0 additions & 2 deletions plugins/plugin-chart-echarts/src/Timeseries/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ export const DEFAULT_FORM_DATA: EchartsTimeseriesFormData = {
tooltipTimeFormat: 'smart_date',
truncateYAxis: true,
yAxisBounds: [null, null],
xAxisShowMinLabel: false,
xAxisShowMaxLabel: false,
zoomable: false,
richTooltip: true,
xAxisLabelRotation: 0,
Expand Down

1 comment on commit c383774

@vercel
Copy link

@vercel vercel bot commented on c383774 Jul 5, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.