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

feat(tree): add tree path info to tooltip #14957

merged 2 commits into from May 18, 2021


Copy link

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Add tree path info to the tooltip as Treemap.

Fixed issues

Fixes #8800


Before: What was the problem?


The path of the tooltip cannot be customized

After: How is it fixed in this PR?



  • 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



Merging options

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

Other information

Copy link

echarts-bot bot commented May 16, 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.


interface TreeSeriesCallbackDataParams extends CallbackDataParams {
treePathInfo?: TreePathInfo[]
Copy link

@pissang pissang May 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ignore the fact that the internal method has been named wrapTreePathInfo, is it better to call this property treeAncestors when providing to developers? path can be from an arbitrary node to another arbitrary node.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. Thanks for the suggestion.

@pissang pissang modified the milestones: 5.2.0, 5.1.2 May 18, 2021
@pissang pissang merged commit a055fee into apache:master May 18, 2021
Copy link

echarts-bot bot commented May 18, 2021

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

@stephenLYZ stephenLYZ deleted the tree-pathInfo branch May 18, 2021 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet

Successfully merging this pull request may close these issues.

Tree tooltip 存在的问题。
2 participants