jasonpaige commented on code in PR #18376:
URL: https://github.com/apache/echarts/pull/18376#discussion_r1144829967
##########
src/coord/radar/Radar.ts:
##########
@@ -67,6 +67,7 @@ class Radar implements CoordinateSystem,
CoordinateSystemMaster {
indicatorAxis.name = indicatorModel.get('name');
// Inject model and axis
indicatorAxis.model = indicatorModel;
+ indicatorAxis.inverse = indicatorModel.get('inverse');
Review Comment:
Unfortunately, without this line the test graph does not render as expected.
I think the Axes needs to know it is inversed as well as the data model which
is why this line is there? I assumed this is how it is achieved on the other
charts which support the inverse property as the base Axis class has the
inverse property set.
--
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-unsubscr...@echarts.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@echarts.apache.org
For additional commands, e-mail: commits-h...@echarts.apache.org
