Skip to content

Component | Sankey: Add node collapse feature (1.7) - #656

Draft
lee00678 wants to merge 3 commits into
mainfrom
qian/sankey-poc
Draft

Component | Sankey: Add node collapse feature (1.7)#656
lee00678 wants to merge 3 commits into
mainfrom
qian/sankey-poc

Conversation

@lee00678

Copy link
Copy Markdown
Collaborator

Adding a feature to Sankey to allow pre-collapsed node.
Three new props:

   /** Enable node collapse functionality. When enabled, clicking on nodes will toggle their collapse state. Default: `false` */
  enableNodeCollapse?: boolean;
  /** Node collapse animation duration, ms. Default: `300` */
  collapseAnimationDuration?: number;
  /** Field name in the node data that indicates if a node should be pre-collapsed.
   *  For example, if set to "disabled", nodes with `disabled: true` will start collapsed.
   *  Default: `undefined` */
  disabledField?: string;
Screen.Recording.2025-10-23.at.12.41.10.PM.mov

@suryahanumandla

suryahanumandla commented Nov 6, 2025

Copy link
Copy Markdown
Collaborator

@lee00678 is it expected behavior?

Screen.Recording.2025-11-06.at.8.25.30.AM.mov

Comment thread packages/ts/src/components/sankey/index.ts
@lee00678
lee00678 marked this pull request as ready for review November 13, 2025 23:10
@lee00678 lee00678 changed the title Component | Sankey: Add node collapse feature [1.7 FEATURE DO NOT MERGE, PLEASE REVIEW] Component | Sankey: Add node collapse feature Nov 13, 2025
@lee00678
lee00678 marked this pull request as draft December 4, 2025 20:51
@lee00678

lee00678 commented May 5, 2026

Copy link
Copy Markdown
Collaborator Author

@lee00678 is it expected behavior?

Screen.Recording.2025-11-06.at.8.25.30.AM.mov

Yes, this is. Because when collapse, we disable to and from the node.

@lee00678 lee00678 changed the title Component | Sankey: Add node collapse feature Component | Sankey: Add node collapse feature (1.7) Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants