diff --git a/superset-frontend/plugins/plugin-chart-echarts/src/Funnel/transformProps.ts b/superset-frontend/plugins/plugin-chart-echarts/src/Funnel/transformProps.ts index 87189c355a39..7c096da49dc9 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/src/Funnel/transformProps.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/src/Funnel/transformProps.ts @@ -239,8 +239,6 @@ export default function transformProps( formatter, show: showLabels, color: theme.colorText, - textBorderColor: theme.colorBgBase, - textBorderWidth: 1, }; const legendData = keys.sort((a: string, b: string) => { if (!legendSort) return 0; diff --git a/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts b/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts index b2e36c42bba7..f2215b9ec528 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/test/Funnel/transformProps.test.ts @@ -72,6 +72,15 @@ describe('Funnel transformProps', () => { }), ); }); + + test('does not apply a text border to segment labels', () => { + // A white textBorder washes out the dark text on light-colored segments. + const result = transformProps(chartProps as EchartsFunnelChartProps); + const { label } = (result.echartOptions.series as any)[0]; + expect(label.color).toBe(supersetTheme.colorText); + expect(label.textBorderColor).toBeUndefined(); + expect(label.textBorderWidth).toBeUndefined(); + }); }); describe('formatFunnelLabel', () => {