Skip to content

Sketch origin marker: distinct display, properties UI, and active-sketch behavior#178

Merged
trailcode merged 5 commits into
mainfrom
Trailcode/sketch_origin
Jul 7, 2026
Merged

Sketch origin marker: distinct display, properties UI, and active-sketch behavior#178
trailcode merged 5 commits into
mainfrom
Trailcode/sketch_origin

Conversation

@trailcode

@trailcode trailcode commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Closes #177

Summary

  • Draw the sketch Origin as a + inside a circle (configurable color in Settings → Sketch → Origin marker color); user-placed Add node markers stay red + without a circle.
  • Show the origin annotation on the active sketch only, with a per-sketch Show origin marker toggle in Sketch properties.
  • Add Position X/Y sliders (view-relative range, underlay-center pattern) and a Set floating input beside each slider (distance-edit behavior: Enter applies, Esc dismisses).
  • [P] in the Sketch List makes that sketch current; while Sketch properties is open, the dialog tracks the current sketch.
  • Origin edits do not push undo snapshots; show_origin_marker is persisted in sketch JSON.

Test plan

  • Create two sketches; confirm origin circle appears only on the active sketch
  • Toggle Show origin marker off/on in Sketch properties
  • Change Origin marker color in Settings and confirm marker updates
  • Drag X/Y sliders and use Set + Enter to type coordinates
  • Click [P] on a non-current sketch — it becomes active and properties open for it
  • With properties open, switch sketch via radio — properties panel updates
  • Confirm origin still snaps when marker is hidden
  • Save/reload project — origin position and show_origin_marker round-trip
  • Run EzyCad_tests --gtest_filter=Sketch_test.*Origin*

@cursor

cursor Bot commented Jul 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@cursor

cursor Bot commented Jul 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@trailcode trailcode merged commit ccbb32f into main Jul 7, 2026
3 checks passed
@trailcode trailcode deleted the Trailcode/sketch_origin branch July 7, 2026 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sketch origin marker: distinct display, properties UI, and active-sketch behavior

1 participant