Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nodes in Sankey chart not connected properly when height is set too short and minLinkWidth is used #19110

Closed
siropkin opened this issue Jun 7, 2023 · 1 comment · Fixed by #19496
Assignees

Comments

@siropkin
Copy link

siropkin commented Jun 7, 2023

Expected behaviour

The Sankey chart should display all nodes and maintain proper connections between "to" nodes, regardless of the specified height and the use of minLinkWidth parameter.

Actual behaviour

When the height of the Sankey chart is set too short, and the minLinkWidth parameter is used, the connections between "to" nodes become disorganized and messy. Some connections may appear broken or not properly aligned.
image

But if I will removeminLinkWidth parameter or will increase chart height all nodes will work fine:
image

Live demo with steps to reproduce

Please refer to the following Codepen for a live demo showcasing the bug: Bug Demo

Steps to reproduce:

  1. Open the provided Codepen link.
  2. Observe the default Sankey chart.
  3. Modify the height parameter to a shorter value (e.g., decrease the value of chart: { height: ... }).
  4. Set the minLinkWidth parameter to a non-zero value (e.g., modify the data array and add minLinkWidth property to some links).
  5. Notice that the connections between "to" nodes are not properly aligned and may appear broken or disconnected.

Product version

Highcharts JS v11.1.0 (2023-06-05)

Affected browser(s)

Chrome Version 114.0.5735.110 (Official Build) (64-bit)

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Jun 7, 2023
@bm64
Copy link
Member

bm64 commented Jun 8, 2023

Hello @siropkin and thanks for reporting the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

4 participants