#13469 - selecting one item does select several - circle only select …

[Zeljko-Predjeskovic]

…bounding box and select highest layer

[Zeljko-Predjeskovic]

Session summary

1. Circle/ellipse border-only hit testing — SvgCircle and SvgEllipse had no IntersectsWith override, so unfilled circles/ellipses were always selectable everywhere in their bounding box, including dead center. Added polygon-approximated border checks to both (SvgCircle.cs, SvgEllipse.cs), matching the existing pattern used by SvgRectangle/SvgPolygon/SvgPath. Root cause of the "why doesn't my fix work" follow-up: the app's EllipseTool draws SvgEllipse, not SvgCircle — two separate classes — so the ellipse needed its own fix.

2. "Fat finger" stroke tolerance — border-only hit tests for unfilled shapes only counted a hit on the exact mathematical outline, ignoring how wide the stroke actually renders. Added GetStrokeHitTestTolerance() (half the element's StrokeWidth, 0 if no stroke) and threaded it through IsIntersectingWithLine in SvgHitTestingExtensions.cs, wired into SvgRectangle, SvgPolygon, SvgPolyline, SvgPath, SvgLine, SvgCircle, SvgEllipse.

3. Tap-select vs area-select — SelectionTool was selecting every overlapping element on a single tap, not just the top one (area-drag correctly selected multiple). Changed SelectElementsUnderPoint's default maxItems from unlimited to 1 (SelectionTool.cs), relying on HitTest's existing top-to-bottom z-order.

Tests added across SvgHitTests.cs and SelectionToolTests.cs: circle/ellipse no-fill center-miss/border-hit cases, two-layered-rectangle z-order, thick-vs-thin stroke tolerance, no-stroke gets no bonus tolerance, and end-to-end tap-vs-area selection on overlapping rectangles.

[gentledepp]

But this should work similar for all tools we provide:

• stroke/arrow
• rectangle
• ellipse
• ...am I missing some?

[gentledepp]

Scope — the outline / path hit-testing changes look out of scope for this PR.

The stated goal of #13469 is the select-topmost fix, and that part (the maxItems = 1 change in SelectionTool) is correct and well covered by the new tests.

However this PR also introduces a second, independent feature: border/outline hit-testing for unfilled shapes — the new IntersectsWith overrides on SvgCircle/SvgEllipse, the GetStrokeHitTestTolerance / ScaleTolerance plumbing, and the stroke-tolerance wiring across SvgLine / SvgPolygon / SvgPolyline / SvgRectangle / SvgPath.

Please pull the outline/path hit-testing work into its own PR and keep this one limited to the select-topmost fix. See the inline note on SvgCircle.IntersectsWith for the consistency requirement if any of it does stay.

(Verified locally: checked the branch out into a worktree and ran the new suites — Svg.Editor.Core.Tests 4/4 and the new Svg.Tests.Win hit-test cases 18/18 pass.)