Skip to content

Commit

Permalink
SketchOnFace UI (#1664)
Browse files Browse the repository at this point in the history
* always enter edit mode

* initial blocking of extra code-mirror updates

* dry out code

* rejig selections

* A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)

* clean up

* stream clean up

* update export

* sketch mode can be entered and exited for extrude faces

But has bugs

* startSketchOn working in some cases, editsketch animation working but not orientation of instersection plane etc

* Revert "A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)"

This reverts commit 406fca4.

* A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)

* remove comment

* add sketch on face e2e test

* tweenCamToNegYAxis should respect reduced motion

* initial sketch on face working with test

* remove temporary toolbar button and xState flow

* un-used vars

* snapshot test tweak

* A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)

* type tidy up

* Revert "A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)"

This reverts commit c39b8eb.

* Revert "A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)"

This reverts commit fecf6f4.

* A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)

* rename

* sketch on sketch on sketch

* A snapshot a day keeps the bugs away! πŸ“·πŸ› (OS: ubuntu)

* typo

* startSketchOn Endcaps

end works, start is weird still

* clear selections for entity_ids that are not recognised

* fix sketch on end cap of second order extrustion

* tiny clean up

* fix sketch on close segment/face

* clean up 'lastCodeMirrorSelectionUpdatedFromScene'

* add code mode test for sketchOnExtrudedFace

* make end cap selection more robust

* update js artifacts for extrudes

* update kcl docs

* clean up

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
  • Loading branch information
Irev-Dev and github-actions[bot] committed Mar 21, 2024
1 parent e773e93 commit 0e916cf
Show file tree
Hide file tree
Showing 63 changed files with 1,854 additions and 714 deletions.
2 changes: 2 additions & 0 deletions docs/kcl/angleToMatchLengthX.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,8 @@ const part001 = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/angleToMatchLengthY.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,8 @@ const part001 = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/angledLine.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -244,6 +246,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/angledLineOfXLength.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ startSketchOn('XZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -244,6 +246,8 @@ startSketchOn('XZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/angledLineOfYLength.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,8 @@ startSketchOn('YZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -245,6 +247,8 @@ startSketchOn('YZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/angledLineThatIntersects.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,8 @@ const part001 = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -250,6 +252,8 @@ const part001 = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/angledLineToX.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -244,6 +246,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/angledLineToY.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -243,6 +245,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/arc.md
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,8 @@ startSketchOn('-YZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -254,6 +256,8 @@ startSketchOn('-YZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/bezierCurve.md
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -247,6 +249,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
6 changes: 6 additions & 0 deletions docs/kcl/circle.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,8 @@ const rectangle = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -135,6 +137,8 @@ const rectangle = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -302,6 +306,8 @@ const rectangle = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/close.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@ startSketchOn('YZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -240,6 +242,8 @@ startSketchOn('YZ')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/extrude.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
6 changes: 6 additions & 0 deletions docs/kcl/hole.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,8 @@ const square = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -236,6 +238,8 @@ const square = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -403,6 +407,8 @@ const square = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/lastSegX.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@ startSketchOn("YZ")
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/lastSegY.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@ startSketchOn("YZ")
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/line.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@ startSketchOn('-XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -234,6 +236,8 @@ startSketchOn('-XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/lineTo.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@ const part = rectShape([0, 0], 20, 20)
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -240,6 +242,8 @@ const part = rectShape([0, 0], 20, 20)
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/patternCircular2d.md
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,8 @@ const part = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/patternLinear2d.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,8 @@ const part = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/segAng.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,8 @@ const part001 = startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/segEndX.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ startSketchOn("YZ")
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/segEndY.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ startSketchOn("YZ")
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/segLen.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ startSketchOn("YZ")
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
4 changes: 4 additions & 0 deletions docs/kcl/startProfileAt.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down Expand Up @@ -128,6 +130,8 @@ startSketchOn('XY')
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/startSketchAt.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,8 @@ startSketchAt([0, 0])
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
2 changes: 2 additions & 0 deletions docs/kcl/startSketchOn.md
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,8 @@ string
},
} |
{
// the face id the sketch is on
faceId: uuid,
// The id of the face.
id: uuid,
// The original sketch group id of the object we are sketching on.
Expand Down
Loading

0 comments on commit 0e916cf

Please sign in to comment.