• Magnuz Binder's avatar
    Change parameter name to be consistent with code. · 6bdec963
    Magnuz Binder authored
    Both radiusSegment and radialSegments are misleading. I would prefer to call it archSegments, both in code and documentation. For consistency and easier understanding, I think the prefix theta should also be replaced by arch here and in some other places, e.g. Ring(Buffer)Geometry. For easier understanding, I also think phiSegments should be replaced with radialSegments there, where it would be correct. But since these are a bit major changes, at least if implemented in the code, I would like the views of those with a better overview and understanding.
    6bdec963
ConeGeometry.html 2.2 KB