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

update to d3-sankey 0.7.1 + fix problem with large padding #7

Merged
merged 45 commits into from
Jan 31, 2019
Merged

Conversation

antoinerg
Copy link

No description provided.

tomwanzek and others added 30 commits April 21, 2017 19:04
* Align API Documentation with source code.
 * Add missing methods `size` and `link`
* Remove inapplicable references to accessor functions
* Provide information about `nodes` and `links`
* Clarify how links can be initialized based on numeric index into `nodes` array.
* Incorporate the fix to HTML syntax highlighting proposed by @curran in unmerged PR d3#20
* [Documentation] Clarify wording of convenience method for initializing links. Array of nodes is returned from the nodes accessor functions configured for the generator.
* [Documentation] Default `extent` is `[[0, 0], [1, 1]]` rather than `null` as per source code.
The position outputs of the layout have been changed as follows:

    node.x ↦ node.x0
    node.dx ↦ node.x1 - node.x0
    node.y ↦ node.y0
    node.dy ↦ node.y1 - node.y0
    link.dy ↦ link.width
    link.sy ↦ link.y0 - link.source.y0 - link.width / 2
    link.ty ↦ link.y1 - link.target.y0 - link.width / 2

This is more consistent with layouts in v4, and also simplifies drawing links;
a link’s source position is now [d.source.x1, d.y0] and its target position is
now [d.target.x0, d.y1].
@etpinard
Copy link

From https://github.com/plotly/d3-sankey/tree/new

image

Looking good! 💃

@antoinerg antoinerg merged commit 1edaa26 into master Jan 31, 2019
@antoinerg antoinerg deleted the new branch January 31, 2019 02:56
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.

None yet

7 participants