[GitHub] [echarts] plainheart edited a comment on pull request #16298: feat(pie/sunburst): supports configuring any one border radius.

[GitBox <gi...@apache.org>]

plainheart edited a comment on pull request #16298:
URL: https://github.com/apache/echarts/pull/16298#issuecomment-1006630604


   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...
   
   <img src="https://user-images.githubusercontent.com/26999792/148396957-b8da3393-ac85-4b3c-a2b0-cb819493dc10.png" width="200">
   
   To reproduce:
   
   ```js
   chart.setOption({
     series: [{
       itemStyle: {
         borderRadius: [0, 20, '50%']
       }
     }]
   });
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org