[API REVIEW] RT-30171: 3D versions of localToScreen and screenToLocal

[Pavel Safrata]

Hi Richard,

On 2.5.2013 23:51, Richard Bair wrote:
> Hi Pavel,
>
>> // these two already exist:
>> public Point2D localToScreen(double localX, double localY)
>> public Point2D localToScreen(Point2D localPoint)
>> // these two exist but without the 2D suffix, so rename them (they were added in FX8 so no backward incompatibility):
>> public Point2D screenToLocal2D(double screenX, double screenY)
>> public Point2D screenToLocal2D(Point2D screenPoint)
>> // add these four for 3D
>> public Point2D localToScreen(double localX, double localY, double localZ)
>> public Point2D localToScreen(Point3D localPoint)
>> public Point3D screenToLocal3D(double screenX, double screenY)
>> public Point3D screenToLocal3D(Point2D screenPoint)
> Just mulling it over. What about:
>
> // these two already exist:
> public Point2D localToScreen(double localX, double localY)
> public Point2D localToScreen(Point2D localPoint)
> public Point2D screenToLocal(double screenX, double screenY)
> public Point2D screenToLocal(Point2D screenPoint)
> public Point2D localToScreen(double localX, double localY, double localZ)
> public Point2D localToScreen(Point3D localPoint)
> public Point3D screenToLocal3D(double screenX, double screenY)
> public Point3D screenToLocal3D(Point2D screenPoint)
>
> In this case we only append the "3D" to screenToLocal variants that produce Point3D?

I agree.
Pavel