feat(pie/sunburst): supports configuring any one border radius.

[plainheart]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

Enhance the border radius for pie series & sunburst series to allow the developer to configure any one border radius instead of a pair.

Details

Before: What was the problem?

After: How is it fixed in this PR?

[5, 15, 0, 20]	[20, 0, 0, 20]

Misc

• The API has been changed (apache/echarts-doc#xxx).
• This PR depends on ZRender changes (feat(sector): enhance sector corner radius to support configuring any one corner. ecomfe/zrender#865).

Related test cases or examples to use the new APIs

Please refer to test/pie-cornerRadius.html.

Others

Merging options

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

Other information

[echarts-bot]

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.

This PR depends on ZRender changes. Please update the ZRender dependency to the latest nightly version including this change, which takes place everyday at 8:00 UTC (16:00 Beijing Time).
You can use npm i zrender@npm:zrender-nightly@dev to update package.json.
If you have any question about this, please leave a comment and we will give you extra help on this.

[Ovilia]

LGTM

[pissang]

@plainheart Do you mind adding user interactions record on the test?

[pissang]

Found some bad cases when clicking random data with inner radius 30%. Seems it happens when sector angle is small.

Chart width: 1000
Chart height: 400

[plainheart]

It seems if the borderRadius doesn't get normalized in ECharts, the percentage value will become NaN during animation. It may be better to normalize this in ECharts instead of zrender... But zrender also needs this normalization...

To reproduce:

chart.setOption({
  series: [{
    itemStyle: {
      borderRadius: [0, 20, '50%']
    }
  }]
});

[pissang]

I think we can do percentage calculation in echarts and only support pixel value in zrender.

[plainheart]

@pissang Yeah. I'm doing so.

[pissang]

@plainheart Please let me know if it's ready for next round review

[echarts-bot]

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