fix: restore hover effect on pie charts when using Lumo theme classes #6850
Conversation
|
|
||
| States states = new States(); | ||
| states.setHover(new Hover(false)); | ||
| getPlotOptions().getPie().setStates(states); |
There was a problem hiding this comment.
This was added initially with vaadin/vaadin-charts-flow#393. From the Hover JavaDoc I can not quite understand what the enabled state is supposed to do, but apparently disabling it doesn't do anything sensible as it then marks all chart slices as inactive on hover. Removing this configuration seems to make the chart behave like any other without a theme.
There was a problem hiding this comment.
And this was there since Charts for FW 7(? maybe).
From what I understand from the hover configuration, disabling it would still allow to hover the legend items and see the correspondent slice in the chart highlighted. But, by default, the legends in a pie chart show in the pie itself, so it should require some extra configuration to show the legend in a separate box, which is not expected.
This is how it would look like (you can check here):
pie-chart.mp4
|
Fixes vaadin/web-components#8196