@tldraw/editor 3.14.0-canary.e0ab6f4c80f9 → 3.14.0-canary.f6a0206007b3

package/dist-esm/version.mjs.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../src/version.ts"],
-  "sourcesContent": ["// This file is automatically generated by internal/scripts/refresh-assets.ts.\n// Do not edit manually. Or do, I'm a comment, not a cop.\n\nexport const version = '3.14.0-canary.e0ab6f4c80f9'\nexport const publishDates = {\n\tmajor: '2024-09-13T14:36:29.063Z',\n\tminor: '2025-05-23T10:08:18.976Z',\n\tpatch: '2025-05-23T10:08:18.976Z',\n}\n"],
+  "sourcesContent": ["// This file is automatically generated by internal/scripts/refresh-assets.ts.\n// Do not edit manually. Or do, I'm a comment, not a cop.\n\nexport const version = '3.14.0-canary.f6a0206007b3'\nexport const publishDates = {\n\tmajor: '2024-09-13T14:36:29.063Z',\n\tminor: '2025-06-03T09:18:36.909Z',\n\tpatch: '2025-06-03T09:18:36.909Z',\n}\n"],
   "mappings": "AAGO,MAAM,UAAU;AAChB,MAAM,eAAe;AAAA,EAC3B,OAAO;AAAA,EACP,OAAO;AAAA,EACP,OAAO;AACR;",
   "names": []
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tldraw/editor",
   "description": "A tiny little drawing app (editor).",
-  "version": "3.14.0-canary.e0ab6f4c80f9",
+  "version": "3.14.0-canary.f6a0206007b3",
   "author": {
     "name": "tldraw Inc.",
     "email": "hello@tldraw.com"
@@ -48,12 +48,12 @@
     "@tiptap/core": "^2.9.1",
     "@tiptap/pm": "^2.9.1",
     "@tiptap/react": "^2.9.1",
-    "@tldraw/state": "3.14.0-canary.e0ab6f4c80f9",
-    "@tldraw/state-react": "3.14.0-canary.e0ab6f4c80f9",
-    "@tldraw/store": "3.14.0-canary.e0ab6f4c80f9",
-    "@tldraw/tlschema": "3.14.0-canary.e0ab6f4c80f9",
-    "@tldraw/utils": "3.14.0-canary.e0ab6f4c80f9",
-    "@tldraw/validate": "3.14.0-canary.e0ab6f4c80f9",
+    "@tldraw/state": "3.14.0-canary.f6a0206007b3",
+    "@tldraw/state-react": "3.14.0-canary.f6a0206007b3",
+    "@tldraw/store": "3.14.0-canary.f6a0206007b3",
+    "@tldraw/tlschema": "3.14.0-canary.f6a0206007b3",
+    "@tldraw/utils": "3.14.0-canary.f6a0206007b3",
+    "@tldraw/validate": "3.14.0-canary.f6a0206007b3",
     "@types/core-js": "^2.5.8",
     "@use-gesture/react": "^10.3.1",
     "classnames": "^2.5.1",

package/src/index.ts

@@ -4,6 +4,7 @@ import 'core-js/stable/array/flat-map.js'
 import 'core-js/stable/array/flat.js'
 import 'core-js/stable/string/at.js'
 import 'core-js/stable/string/replace-all.js'
+export { areShapesContentEqual } from './lib/utils/areShapesContentEqual'
 
 // eslint-disable-next-line local/no-export-star
 export * from '@tldraw/state'
@@ -185,7 +186,10 @@ export {
 	type TLShapeUtilConstructor,
 } from './lib/editor/shapes/ShapeUtil'
 export { GroupShapeUtil } from './lib/editor/shapes/group/GroupShapeUtil'
-export { getPerfectDashProps } from './lib/editor/shapes/shared/getPerfectDashProps'
+export {
+	getPerfectDashProps,
+	type PerfectDashTerminal,
+} from './lib/editor/shapes/shared/getPerfectDashProps'
 export { resizeBox, type ResizeBoxOptions } from './lib/editor/shapes/shared/resizeBox'
 export { resizeScaled } from './lib/editor/shapes/shared/resizeScaled'
 export { BaseBoxShapeTool } from './lib/editor/tools/BaseBoxShapeTool/BaseBoxShapeTool'

package/src/lib/editor/Editor.ts

@@ -506,6 +506,8 @@ export class Editor extends EventEmitter<TLEventMap> {
 				shape: {
 					afterChange: (shapeBefore, shapeAfter) => {
 						for (const binding of this.getBindingsInvolvingShape(shapeAfter)) {
+							if (areShapesContentEqual(shapeBefore, shapeAfter)) continue
+
 							invalidBindingTypes.add(binding.type)
 							if (binding.fromId === shapeAfter.id) {
 								this.getBindingUtil(binding).onAfterChangeFromShape?.({
@@ -5796,8 +5798,7 @@ export class Editor extends EventEmitter<TLEventMap> {
 		parent: TLParentId | TLPage | TLShape,
 		visitor: (id: TLShapeId) => void | false
 	): this {
-		const parentId = typeof parent === 'string' ? parent : parent.id
-		const children = this.getSortedChildIdsForParent(parentId)
+		const children = this.getSortedChildIdsForParent(parent)
 		for (const id of children) {
 			if (visitor(id) === false) continue
 			this.visitDescendants(id, visitor)

package/src/lib/editor/derivations/notVisibleShapes.ts

@@ -1,49 +1,48 @@
 import { computed, isUninitialized } from '@tldraw/state'
 import { TLShapeId } from '@tldraw/tlschema'
-import { Box } from '../../primitives/Box'
 import { Editor } from '../Editor'
 
-function isShapeNotVisible(editor: Editor, id: TLShapeId, viewportPageBounds: Box): boolean {
-	const maskedPageBounds = editor.getShapeMaskedPageBounds(id)
-	// if the shape is fully outside of its parent's clipping bounds...
-	if (maskedPageBounds === undefined) return true
-
-	// if the shape is fully outside of the viewport page bounds...
-	return !viewportPageBounds.includes(maskedPageBounds)
+function fromScratch(editor: Editor): Set<TLShapeId> {
+	const shapesIds = editor.getCurrentPageShapeIds()
+	const viewportPageBounds = editor.getViewportPageBounds()
+	const notVisibleShapes = new Set<TLShapeId>()
+	shapesIds.forEach((id) => {
+		// If the shape is fully outside of the viewport page bounds, add it to the set.
+		// We'll ignore masks here, since they're more expensive to compute and the overhead is not worth it.
+		const pageBounds = editor.getShapePageBounds(id)
+		if (pageBounds === undefined || !viewportPageBounds.includes(pageBounds)) {
+			notVisibleShapes.add(id)
+		}
+	})
+	return notVisibleShapes
 }
 
 /**
  * Incremental derivation of not visible shapes.
- * Non visible shapes are shapes outside of the viewport page bounds and shapes outside of parent's clipping bounds.
+ * Non visible shapes are shapes outside of the viewport page bounds.
  *
  * @param editor - Instance of the tldraw Editor.
  * @returns Incremental derivation of non visible shapes.
  */
-export const notVisibleShapes = (editor: Editor) => {
-	function fromScratch(editor: Editor): Set<TLShapeId> {
-		const shapes = editor.getCurrentPageShapeIds()
-		const viewportPageBounds = editor.getViewportPageBounds()
-		const notVisibleShapes = new Set<TLShapeId>()
-		shapes.forEach((id) => {
-			if (isShapeNotVisible(editor, id, viewportPageBounds)) {
-				notVisibleShapes.add(id)
-			}
-		})
-		return notVisibleShapes
-	}
-	return computed<Set<TLShapeId>>('notVisibleShapes', (prevValue) => {
+export function notVisibleShapes(editor: Editor) {
+	return computed<Set<TLShapeId>>('notVisibleShapes', function updateNotVisibleShapes(prevValue) {
+		const nextValue = fromScratch(editor)
+
 		if (isUninitialized(prevValue)) {
-			return fromScratch(editor)
+			return nextValue
 		}
 
-		const nextValue = fromScratch(editor)
-
+		// If there are more or less shapes, we know there's a change
 		if (prevValue.size !== nextValue.size) return nextValue
+
+		// If any of the old shapes are not in the new set, we know there's a change
 		for (const prev of prevValue) {
 			if (!nextValue.has(prev)) {
 				return nextValue
 			}
 		}
+
+		// If we've made it here, we know that the set is the same
 		return prevValue
 	})
 }

package/src/lib/editor/managers/FocusManager.ts

@@ -58,6 +58,8 @@ export class FocusManager {
 
 	private handleKeyDown(keyEvent: KeyboardEvent) {
 		const container = this.editor.getContainer()
+		if (this.editor.isIn('select.editing_shape')) return
+		if (document.activeElement === container && this.editor.getSelectedShapeIds().length > 0) return
 		if (['Tab', 'ArrowUp', 'ArrowDown'].includes(keyEvent.key)) {
 			container.classList.remove('tl-container__no-focus-ring')
 		}

package/src/lib/editor/shapes/ShapeUtil.ts

@@ -240,6 +240,15 @@ export abstract class ShapeUtil<Shape extends TLUnknownShape = TLUnknownShape> {
 		return true
 	}
 
+	/**
+	 * When the shape is resized, whether the shape's children should also be resized.
+	 *
+	 * @public
+	 */
+	canResizeChildren(_shape: Shape): boolean {
+		return true
+	}
+
 	/**
 	 * Whether the shape can be edited in read-only mode.
 	 *

package/src/lib/editor/shapes/group/GroupShapeUtil.tsx

@@ -20,6 +20,14 @@ export class GroupShapeUtil extends ShapeUtil<TLGroupShape> {
 		return false
 	}
 
+	canResize() {
+		return false
+	}
+
+	canResizeChildren() {
+		return true
+	}
+
 	getDefaultProps(): TLGroupShape['props'] {
 		return {}
 	}

package/src/lib/editor/shapes/shared/getPerfectDashProps.ts

@@ -1,5 +1,8 @@
 import { TLDefaultDashStyle } from '@tldraw/tlschema'
 
+/** @public */
+export type PerfectDashTerminal = 'skip' | 'outset' | 'none'
+
 /** @public */
 export function getPerfectDashProps(
 	totalLength: number,
@@ -7,8 +10,8 @@ export function getPerfectDashProps(
 	opts: {
 		style?: TLDefaultDashStyle
 		snap?: number
-		end?: 'skip' | 'outset' | 'none'
-		start?: 'skip' | 'outset' | 'none'
+		end?: PerfectDashTerminal
+		start?: PerfectDashTerminal
 		lengthRatio?: number
 		closed?: boolean
 		forceSolid?: boolean

package/src/lib/primitives/Box.ts

@@ -591,6 +591,14 @@ export class Box {
 		return b.x === a.x && b.y === a.y && b.w === a.w && b.h === a.h
 	}
 
+	prettyMuchEquals(other: Box | BoxModel) {
+		return this.clone().toFixed().equals(Box.From(other).toFixed())
+	}
+
+	static PrettyMuchEquals(a: Box | BoxModel, b: Box | BoxModel) {
+		return b.x === a.x && b.y === a.y && b.w === a.w && b.h === a.h
+	}
+
 	zeroFix() {
 		this.w = Math.max(1, this.w)
 		this.h = Math.max(1, this.h)

package/src/lib/primitives/Vec.test.ts

@@ -144,8 +144,8 @@ describe('Vec.Uni', () => {
 		expect(Vec.Uni(new Vec(10, 10))).toMatchObject(new Vec(0.7071067811865475, 0.7071067811865475))
 	})
 
-	it('Divide-by-zero spits out NaN (at the moment)', () => {
-		expect(Vec.Uni(new Vec(0, 0))).toMatchObject(new Vec(NaN, NaN))
+	it('Divide-by-zero spits out 0', () => {
+		expect(Vec.Uni(new Vec(0, 0))).toMatchObject(new Vec(0, 0))
 	})
 })
 

package/src/lib/primitives/Vec.ts

@@ -1,6 +1,6 @@
 import { VecModel } from '@tldraw/tlschema'
 import { EASINGS } from './easings'
-import { toFixed } from './utils'
+import { clamp, toFixed } from './utils'
 
 /** @public */
 export type VecLike = Vec | VecModel
@@ -189,11 +189,15 @@ export class Vec {
 	}
 
 	uni() {
-		return Vec.Uni(this)
+		const l = this.len()
+		if (l === 0) return this
+		this.x /= l
+		this.y /= l
+		return this
 	}
 
 	tan(V: VecLike): Vec {
-		return Vec.Tan(this, V)
+		return this.sub(V).uni()
 	}
 
 	dist(V: VecLike): number {
@@ -236,15 +240,15 @@ export class Vec {
 		return Vec.EqualsXY(this, x, y)
 	}
 
+	/** @deprecated use `uni` instead */
 	norm() {
-		const l = this.len()
-		this.x = l === 0 ? 0 : this.x / l
-		this.y = l === 0 ? 0 : this.y / l
-		return this
+		return this.uni()
 	}
 
 	toFixed() {
-		return Vec.ToFixed(this)
+		this.x = toFixed(this.x)
+		this.y = toFixed(this.y)
+		return this
 	}
 
 	toString() {
@@ -375,7 +379,8 @@ export class Vec {
 	 * Get the unit vector of A.
 	 */
 	static Uni(A: VecLike) {
-		return Vec.Div(A, Vec.Len(A))
+		const l = Vec.Len(A)
+		return new Vec(l === 0 ? 0 : A.x / l, l === 0 ? 0 : A.y / l)
 	}
 
 	static Tan(A: VecLike, B: VecLike): Vec {
@@ -487,7 +492,7 @@ export class Vec {
 			(Math.pow(A.x, 2) + Math.pow(A.y, 2)) * (Math.pow(B.x, 2) + Math.pow(B.y, 2))
 		)
 		const sign = A.x * B.y - A.y * B.x < 0 ? -1 : 1
-		const angle = sign * Math.acos(p / n)
+		const angle = sign * Math.acos(clamp(p / n, -1, 1))
 
 		return angle
 	}

package/src/lib/primitives/geometry/Arc2d.ts

@@ -6,16 +6,15 @@ import { getVerticesCountForLength } from './geometry-constants'
 
 /** @public */
 export class Arc2d extends Geometry2d {
-	_center: Vec
-	radius: number
-	start: Vec
-	end: Vec
-	largeArcFlag: number
-	sweepFlag: number
-
-	measure: number
-	angleStart: number
-	angleEnd: number
+	private _center: Vec
+	private _radius: number
+	private _start: Vec
+	private _end: Vec
+	private _largeArcFlag: number
+	private _sweepFlag: number
+	private _measure: number
+	private _angleStart: number
+	private _angleEnd: number
 
 	constructor(
 		config: Omit<Geometry2dOptions, 'isFilled' | 'isClosed'> & {
@@ -31,21 +30,29 @@ export class Arc2d extends Geometry2d {
 		if (start.equals(end)) throw Error(`Arc must have different start and end points.`)
 
 		// ensure that the start and end are clockwise
-		this.angleStart = Vec.Angle(center, start)
-		this.angleEnd = Vec.Angle(center, end)
-		this.radius = Vec.Dist(center, start)
-		this.measure = getArcMeasure(this.angleStart, this.angleEnd, sweepFlag, largeArcFlag)
+		this._angleStart = Vec.Angle(center, start)
+		this._angleEnd = Vec.Angle(center, end)
+		this._radius = Vec.Dist(center, start)
+		this._measure = getArcMeasure(this._angleStart, this._angleEnd, sweepFlag, largeArcFlag)
 
-		this.start = start
-		this.end = end
+		this._start = start
+		this._end = end
 
-		this.sweepFlag = sweepFlag
-		this.largeArcFlag = largeArcFlag
+		this._sweepFlag = sweepFlag
+		this._largeArcFlag = largeArcFlag
 		this._center = center
 	}
 
 	nearestPoint(point: VecLike): Vec {
-		const { _center, measure, radius, angleEnd, angleStart, start: A, end: B } = this
+		const {
+			_center,
+			_measure: measure,
+			_radius: radius,
+			_angleEnd: angleEnd,
+			_angleStart: angleStart,
+			_start: A,
+			_end: B,
+		} = this
 		const t = getPointInArcT(measure, angleStart, angleEnd, _center.angle(point))
 		if (t <= 0) return A
 		if (t >= 1) return B
@@ -68,7 +75,13 @@ export class Arc2d extends Geometry2d {
 	}
 
 	hitTestLineSegment(A: VecLike, B: VecLike): boolean {
-		const { _center, radius, measure, angleStart, angleEnd } = this
+		const {
+			_center,
+			_radius: radius,
+			_measure: measure,
+			_angleStart: angleStart,
+			_angleEnd: angleEnd,
+		} = this
 		const intersection = intersectLineSegmentCircle(A, B, _center, radius)
 		if (intersection === null) return false
 
@@ -79,7 +92,7 @@ export class Arc2d extends Geometry2d {
 	}
 
 	getVertices(): Vec[] {
-		const { _center, measure, length, radius, angleStart } = this
+		const { _center, _measure: measure, length, _radius: radius, _angleStart: angleStart } = this
 		const vertices: Vec[] = []
 		for (let i = 0, n = getVerticesCountForLength(Math.abs(length)); i < n + 1; i++) {
 			const t = (i / n) * measure
@@ -90,11 +103,17 @@ export class Arc2d extends Geometry2d {
 	}
 
 	getSvgPathData(first = true) {
-		const { start, end, radius, largeArcFlag, sweepFlag } = this
+		const {
+			_start: start,
+			_end: end,
+			_radius: radius,
+			_largeArcFlag: largeArcFlag,
+			_sweepFlag: sweepFlag,
+		} = this
 		return `${first ? `M${start.toFixed()}` : ``} A${radius} ${radius} 0 ${largeArcFlag} ${sweepFlag} ${end.toFixed()}`
 	}
 
 	override getLength() {
-		return Math.abs(this.measure * this.radius)
+		return Math.abs(this._measure * this._radius)
 	}
 }

package/src/lib/primitives/geometry/Circle2d.ts

@@ -7,10 +7,10 @@ import { getVerticesCountForLength } from './geometry-constants'
 
 /** @public */
 export class Circle2d extends Geometry2d {
-	_center: Vec
-	radius: number
-	x: number
-	y: number
+	private _center: Vec
+	private _radius: number
+	private _x: number
+	private _y: number
 
 	constructor(
 		public config: Omit<Geometry2dOptions, 'isClosed'> & {
@@ -22,18 +22,18 @@ export class Circle2d extends Geometry2d {
 	) {
 		super({ isClosed: true, ...config })
 		const { x = 0, y = 0, radius } = config
-		this.x = x
-		this.y = y
+		this._x = x
+		this._y = y
 		this._center = new Vec(radius + x, radius + y)
-		this.radius = radius
+		this._radius = radius
 	}
 
 	getBounds() {
-		return new Box(this.x, this.y, this.radius * 2, this.radius * 2)
+		return new Box(this._x, this._y, this._radius * 2, this._radius * 2)
 	}
 
 	getVertices(): Vec[] {
-		const { _center, radius } = this
+		const { _center, _radius: radius } = this
 		const perimeter = PI2 * radius
 		const vertices: Vec[] = []
 		for (let i = 0, n = getVerticesCountForLength(perimeter); i < n; i++) {
@@ -44,18 +44,18 @@ export class Circle2d extends Geometry2d {
 	}
 
 	nearestPoint(point: VecLike): Vec {
-		const { _center, radius } = this
+		const { _center, _radius: radius } = this
 		if (_center.equals(point)) return Vec.AddXY(_center, radius, 0)
 		return Vec.Sub(point, _center).uni().mul(radius).add(_center)
 	}
 
 	hitTestLineSegment(A: VecLike, B: VecLike, distance = 0): boolean {
-		const { _center, radius } = this
+		const { _center, _radius: radius } = this
 		return intersectLineSegmentCircle(A, B, _center, radius + distance) !== null
 	}
 
 	getSvgPathData(): string {
-		const { _center, radius } = this
+		const { _center, _radius: radius } = this
 		return `M${_center.x + radius},${_center.y} a${radius},${radius} 0 1,0 ${radius * 2},0a${radius},${radius} 0 1,0 -${radius * 2},0`
 	}
 }

package/src/lib/primitives/geometry/CubicBezier2d.test.ts

@@ -0,0 +1,5 @@
+describe('CubicBezier2d', () => {
+	it('should be tested', () => {
+		expect(true).toBe(true)
+	})
+})

package/src/lib/primitives/geometry/CubicBezier2d.ts

@@ -4,10 +4,10 @@ import { Polyline2d } from './Polyline2d'
 
 /** @public */
 export class CubicBezier2d extends Polyline2d {
-	a: Vec
-	b: Vec
-	c: Vec
-	d: Vec
+	private _a: Vec
+	private _b: Vec
+	private _c: Vec
+	private _d: Vec
 
 	constructor(
 		config: Omit<Geometry2dOptions, 'isFilled' | 'isClosed'> & {
@@ -20,15 +20,15 @@ export class CubicBezier2d extends Polyline2d {
 		const { start: a, cp1: b, cp2: c, end: d } = config
 		super({ ...config, points: [a, d] })
 
-		this.a = a
-		this.b = b
-		this.c = c
-		this.d = d
+		this._a = a
+		this._b = b
+		this._c = c
+		this._d = d
 	}
 
 	override getVertices() {
 		const vertices = [] as Vec[]
-		const { a, b, c, d } = this
+		const { _a: a, _b: b, _c: c, _d: d } = this
 		// we'll always use ten vertices for each bezier curve
 		for (let i = 0, n = 10; i <= n; i++) {
 			const t = i / n
@@ -48,10 +48,6 @@ export class CubicBezier2d extends Polyline2d {
 		return vertices
 	}
 
-	midPoint() {
-		return CubicBezier2d.GetAtT(this, 0.5)
-	}
-
 	nearestPoint(A: VecLike): Vec {
 		let nearest: Vec | undefined
 		let dist = Infinity
@@ -71,12 +67,12 @@ export class CubicBezier2d extends Polyline2d {
 	}
 
 	getSvgPathData(first = true) {
-		const { a, b, c, d } = this
+		const { _a: a, _b: b, _c: c, _d: d } = this
 		return `${first ? `M ${a.toFixed()} ` : ``} C${b.toFixed()} ${c.toFixed()} ${d.toFixed()}`
 	}
 
 	static GetAtT(segment: CubicBezier2d, t: number) {
-		const { a, b, c, d } = segment
+		const { _a: a, _b: b, _c: c, _d: d } = segment
 		return new Vec(
 			(1 - t) * (1 - t) * (1 - t) * a.x +
 				3 * ((1 - t) * (1 - t)) * t * b.x +
@@ -89,9 +85,9 @@ export class CubicBezier2d extends Polyline2d {
 		)
 	}
 
-	override getLength(filters?: Geometry2dFilters, precision = 32) {
+	override getLength(_filters?: Geometry2dFilters, precision = 32) {
 		let n1: Vec,
-			p1 = this.a,
+			p1 = this._a,
 			length = 0
 		for (let i = 1; i <= precision; i++) {
 			n1 = CubicBezier2d.GetAtT(this, i / precision)

package/src/lib/primitives/geometry/CubicSpline2d.ts

@@ -4,22 +4,22 @@ import { Geometry2d, Geometry2dOptions } from './Geometry2d'
 
 /** @public */
 export class CubicSpline2d extends Geometry2d {
-	points: Vec[]
+	private _points: Vec[]
 
 	constructor(config: Omit<Geometry2dOptions, 'isClosed' | 'isFilled'> & { points: Vec[] }) {
 		super({ ...config, isClosed: false, isFilled: false })
 		const { points } = config
 
-		this.points = points
+		this._points = points
 	}
 
-	_segments?: CubicBezier2d[]
+	private _segments?: CubicBezier2d[]
 
 	// eslint-disable-next-line no-restricted-syntax
 	get segments() {
 		if (!this._segments) {
 			this._segments = []
-			const { points } = this
+			const { _points: points } = this
 
 			const len = points.length
 			const last = len - 2
@@ -54,7 +54,7 @@ export class CubicSpline2d extends Geometry2d {
 		const vertices = this.segments.reduce((acc, segment) => {
 			return acc.concat(segment.vertices)
 		}, [] as Vec[])
-		vertices.push(this.points[this.points.length - 1])
+		vertices.push(this._points[this._points.length - 1])
 		return vertices
 	}
 

package/src/lib/primitives/geometry/Edge2d.ts

@@ -1,41 +1,36 @@
-import { linesIntersect } from '../intersect'
 import { Vec, VecLike } from '../Vec'
 import { Geometry2d } from './Geometry2d'
 
 /** @public */
 export class Edge2d extends Geometry2d {
-	start: Vec
-	end: Vec
-	d: Vec
-	u: Vec
-	ul: number
+	private _start: Vec
+	private _end: Vec
+	private _d: Vec
+	private _u: Vec
+	private _ul: number
 
 	constructor(config: { start: Vec; end: Vec }) {
 		super({ ...config, isClosed: false, isFilled: false })
 		const { start, end } = config
 
-		this.start = start
-		this.end = end
+		this._start = start
+		this._end = end
 
-		this.d = start.clone().sub(end) // the delta from start to end
-		this.u = this.d.clone().uni() // the unit vector of the edge
-		this.ul = this.u.len() // the length of the unit vector
+		this._d = start.clone().sub(end) // the delta from start to end
+		this._u = this._d.clone().uni() // the unit vector of the edge
+		this._ul = this._u.len() // the length of the unit vector
 	}
 
 	override getLength() {
-		return this.d.len()
-	}
-
-	midPoint(): Vec {
-		return this.start.lrp(this.end, 0.5)
+		return this._d.len()
 	}
 
 	override getVertices(): Vec[] {
-		return [this.start, this.end]
+		return [this._start, this._end]
 	}
 
 	override nearestPoint(point: VecLike): Vec {
-		const { start, end, d, u, ul: l } = this
+		const { _start: start, _end: end, _d: d, _u: u, _ul: l } = this
 		if (d.len() === 0) return start // start and end are the same
 		if (l === 0) return start // no length in the unit vector
 		const k = Vec.Sub(point, start).dpr(u) / l
@@ -48,14 +43,8 @@ export class Edge2d extends Geometry2d {
 		return new Vec(cx, cy)
 	}
 
-	override hitTestLineSegment(A: VecLike, B: VecLike, distance = 0): boolean {
-		return (
-			linesIntersect(A, B, this.start, this.end) || this.distanceToLineSegment(A, B) <= distance
-		)
-	}
-
 	getSvgPathData(first = true) {
-		const { start, end } = this
+		const { _start: start, _end: end } = this
 		return `${first ? `M${start.toFixed()}` : ``} L${end.toFixed()}`
 	}
 }