You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2023/08/17 19:39:45 UTC
[superset] 01/04: 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.
michaelsmolina pushed a commit to branch 3.0
in repository https://gitbox.apache.org/repos/asf/superset.git
commit e20c2967c4d136629913392db947a26584386c31
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)
(cherry picked from commit 2b63577046887f982a8b3a56b335f0c11288f829)
---
.../plugin-chart-echarts/src/Timeseries/transformProps.ts | 15 ++-------------
1 file changed, 2 insertions(+), 13 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 64aafc0237..e46c50331a 100644
--- a/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts
+++ b/superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/transformProps.ts
@@ -20,7 +20,6 @@
import { invert } from 'lodash';
import {
AnnotationLayer,
- AxisType,
CategoricalColorNamespace,
ensureIsArray,
GenericDataType,
@@ -442,23 +441,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 },