Skip to content

treemap: use treeAncestors in callback params#14976

Merged
pissang merged 1 commit intomasterfrom
pissang-patch-1
Jul 18, 2021
Merged

treemap: use treeAncestors in callback params#14976
pissang merged 1 commit intomasterfrom
pissang-patch-1

Conversation

@pissang
Copy link
Contributor

@pissang pissang commented May 18, 2021

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Use more specific treeAncestors to replace the original treePathInfo. path can be used to represent from one arbitrary node to another arbitrary node.

Related PR: #14957

Misc

  • The API has been changed (apache/echarts-doc#xxx).
  • This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

NA.

Others

Merging options

  • Please squash the commits into a single one when merge.

Other information

@echarts-bot
Copy link

echarts-bot bot commented May 18, 2021

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

The pull request is marked to be PR: author is committer because you are a committer of this project.

Document changes are required in this PR. Please also make a PR to apache/echarts-doc for document changes and update the issue id in the PR description. When the doc PR is merged, the maintainers will remove the PR: awaiting doc label.

@echarts-bot echarts-bot bot added PR: author is committer PR: awaiting doc Document changes is required for this PR. PR: awaiting review labels May 18, 2021
@pissang pissang added this to the 5.2.0 milestone May 18, 2021
@pissang pissang merged commit 0db3feb into master Jul 18, 2021
@pissang pissang deleted the pissang-patch-1 branch July 18, 2021 08:16
@echarts-bot
Copy link

echarts-bot bot commented Jul 18, 2021

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

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

Labels

PR: author is committer PR: awaiting doc Document changes is required for this PR. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants