echarts
echarts copied to clipboard
feat(sankey): `sankey` support `edgeLabel`
Brief Information
This pull request is in the type of:
- [ ] bug fixing
- [x] new feature
- [ ] others
What does this PR do?
Fixed issues
- Close #7233
- Close #16600
- Close #12855
Details
Before: What was the problem?
Sankey
don't have label on edges

After: How is it fixed in this PR?

Misc
- [x] 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
test/sankey-depth.html
Others
Merging options
- [x] Please squash the commits into a single one when merging.
Other information
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.
@pissang It's there anything else you need to leave a review for this PR
Any more suggestion? @pissang @plainheart
To reviewers: If this PR is going to be described in the changelog in the future release, please make sure this PR has one of the following labels: PR: doc ready
, PR: awaiting doc
, PR: doc unchanged
This message is shown because the PR description doesn't contain the document related template.
@pissang Any suggestion for this feature?
Waiting for this feature.
Sorry for missing this PR. LGTM now
Congratulations! Your PR has been merged. Thanks for your contribution! 👍
Has this feature release? I set "edgeLabel" in sankey is not work
@Tiramisupxl Not yet. This feature is included in v5.4.1, which is under the voting process.