aj/OpenNest
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix(ui): avoid double-counting baseline rotation in part label placement

Browse Source 
_labelPoint is computed from BaseDrawing.Program's current geometry,
which already carries BaseDrawing.Program.Rotation (non-zero for
canonical-frame drawings). BasePart.Rotation is cumulative from that
same baseline, so it must be re-applied net of the baseline already
baked into _labelPoint, mirroring CanonicalFrame.RebindToOriginal.
This commit is contained in:
aj
2026-09-22 09:51:15 -04:00
parent 56f39556d1
commit a0417a6077
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
OpenNest/LayoutPart.cs
+7 -1
View File
@@ -179,8 +179,14 @@ namespace OpenNest
LeadInPath = null;
}
// _labelPoint is computed from BaseDrawing.Program's current geometry, which already
// carries BaseDrawing.Program.Rotation (nonzero for canonical-frame drawings, e.g. in
// BestFitViewerForm). BasePart.Rotation is cumulative from that same baseline, so it
// must be re-applied net of the baseline already baked into _labelPoint — otherwise
// the baseline rotation is counted twice. Mirrors CanonicalFrame.RebindToOriginal.
_labelPoint ??= ComputeLabelPoint();
var rotatedLabel = _labelPoint.Value.Rotate(BasePart.Rotation);
var baseRotation = BasePart.BaseDrawing.Program.Rotation;
var rotatedLabel = _labelPoint.Value.Rotate(BasePart.Rotation - baseRotation);
var labelPt = new PointF(
(float)(rotatedLabel.X + BasePart.Location.X),
(float)(rotatedLabel.Y + BasePart.Location.Y)