From 925eb1df232c71a744cbbd5183dca72623db04f9 Mon Sep 17 00:00:00 2001 From: Yash Suthar Date: Tue, 20 Jan 2026 00:00:09 +0530 Subject: [PATCH] Sketcher: Switch to perpendicular based instead of "de la Hire" in setting minor axis of ellipse Signed-off-by: Yash Suthar --- src/Mod/Sketcher/Gui/DrawSketchHandlerEllipse.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Mod/Sketcher/Gui/DrawSketchHandlerEllipse.h b/src/Mod/Sketcher/Gui/DrawSketchHandlerEllipse.h index 5d2cac3e4e..714f17f819 100644 --- a/src/Mod/Sketcher/Gui/DrawSketchHandlerEllipse.h +++ b/src/Mod/Sketcher/Gui/DrawSketchHandlerEllipse.h @@ -395,7 +395,7 @@ private: auto lprojy = projy.Length(); // Py = b sin t if (lprojx > firstRadius) { - secondRadius = 0.0; + secondRadius = lprojy; } else { double t = std::acos(lprojx / firstRadius);