circle object radius is incorrect? about fabric.js

[kangax]

Thanks for kind words and glad you like fabric.

You're right that there's no getRadius() method, although there is "radius" property which you can also access via .get('radius'). The thing about "radius" is that it's unaffected by scaling. So when you scale an object (by dragging its corners), you really change values of its "scaleX"/"scaleY" properties.

To get actual radius you can currently multiply radius by "scaleX" or "scaleY" (doesn't matter which, if object is resized proportionally).

But you're right that getRadius could be useful. We already have getWidth which returns actual width ("width" multiplied by scale), so getRadius would be consistent with that. The only issue is — what if scaleX is not the same as scaleY. Which radius to return then? Only return if scaleX==scaleY? Or have getRadiusX, getRadiusY?

[maxim]

getRadiusX, getRadiusY

[csmaster2005-duplicate]

yes I ran into the same problem of the two differnt radius's depending on how it was scaled - I think getRadiusX and getRadiusY sounds appropriate.

[kangax]

Added in cc0c508