You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by el...@apache.org on 2023/08/18 00:02:29 UTC
[superset] 03/06: fix: Revert "fix(chart): Time Series set showMaxLabel as null for time xAxis (#20627) (#24995)
This is an automated email from the ASF dual-hosted git repository.
elizabeth pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/superset.git
commit 2f19b12a224bbd5e1894ee44075ea5a5955010c6
Author: Elizabeth Thompson <es...@gmail.com>
AuthorDate: Wed Aug 16 09:51:00 2023 -0700
fix: Revert "fix(chart): Time Series set showMaxLabel as null for time xAxis (#20627) (#24995)
---
.../plugin-chart-echarts/src/Timeseries/transformProps.ts | 14 ++------------
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts b/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts
index eadded44a9..4cbcfae230 100644
--- a/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts
+++ b/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts
@@ -339,23 +339,13 @@ export default function transformProps(
rotate: xAxisLabelRotation,
},
minInterval:
- xAxisType === AxisType.time && timeGrainSqla
+ xAxisType === 'time' && timeGrainSqla
? TIMEGRAIN_TO_TIMESTAMP[timeGrainSqla]
: 0,
};
-
- if (xAxisType === AxisType.time) {
- /**
- * Overriding default behavior (false) for time axis regardless of the granilarity.
- * Not including this in the initial declaration above so if echarts changes the default
- * behavior for other axist types we won't unintentionally override it
- */
- xAxis.axisLabel.showMaxLabel = null;
- }
-
let yAxis: any = {
...defaultYAxis,
- type: logAxis ? AxisType.log : AxisType.value,
+ type: logAxis ? 'log' : 'value',
min,
max,
minorTick: { show: true },