Skip to content

Commit b91188f

Browse files
markov00CAWilson94
authored andcommitted
[TSVB] fix point visibility regression (elastic#202358)
## Summary fix elastic#200255 The regression was caused by a [breaking change](elastic/elastic-charts#2525) introduced by elastic-charts around the `area.points.visible` style, that passed from a `boolean` to a union of `'always' | 'never' | 'auto'`;
1 parent 8c0c0d8 commit b91188f

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

src/plugins/vis_types/timeseries/public/application/visualizations/views/timeseries/decorators/__snapshots__/area_decorator.test.js.snap

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/plugins/vis_types/timeseries/public/application/visualizations/views/timeseries/utils/__snapshots__/series_styles.test.js.snap

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/plugins/vis_types/timeseries/public/application/visualizations/views/timeseries/utils/series_styles.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export const getAreaStyles = ({ points, lines, color }) => ({
2727
radius: points.radius || 0.5,
2828
stroke: color || DEFAULT_COLOR,
2929
strokeWidth: points.lineWidth || 5,
30-
visible: points.lineWidth > 0 && Boolean(points.show),
30+
visible: points.lineWidth > 0 && Boolean(points.show) ? 'always' : 'never',
3131
},
3232
},
3333
curve: lines.steps ? CurveType.CURVE_STEP_AFTER : CurveType.LINEAR,

0 commit comments

Comments
 (0)