Skip to content

Remove POINT from sketcher widget#810

Closed
asmith26 wants to merge 2 commits intojupytercad:mainfrom
asmith26:remove-point-from-sketcher-widget
Closed

Remove POINT from sketcher widget#810
asmith26 wants to merge 2 commits intojupytercad:mainfrom
asmith26:remove-point-from-sketcher-widget

Conversation

@asmith26
Copy link
Contributor

@asmith26 asmith26 commented Nov 9, 2025

Following on comments in #807 I thought it may be helpful to remove the "POINT" tab from the sketcher widget (see the video in #810 (comment), of course no worries if not).

Thanks!

@github-actions
Copy link
Contributor

github-actions bot commented Nov 9, 2025

Preview PR at appsharing.space

@github-actions
Copy link
Contributor

github-actions bot commented Nov 9, 2025

Integration tests report: appsharing.space

@trungleduc
Copy link
Member

Having the point displayed in the Sketcher is useful for referencing the position while you are drawing other elements

@trungleduc trungleduc added the enhancement New Feature or improvement label Nov 9, 2025
@asmith26
Copy link
Contributor Author

asmith26 commented Nov 9, 2025

Having the point displayed in the Sketcher is useful for referencing the position while you are drawing other elements

Thanks for the info, I think the code still does this, e.g.:

I've also compared the behavior visually to the JupyterLite version, and it appears to be the same:

updated-sketcher-widget.mp4

@asmith26
Copy link
Contributor Author

asmith26 commented Nov 10, 2025

Having the point displayed in the Sketcher is useful for referencing the position while you are drawing other elements

Ah sorry I think I've just realised what you mean - not the points that come with the other elements, but allow users to add points for e.g. marking the start/ends/corners etc of objects

This is useful (especially I like the point "snap" behaviour), happy to close this.

Thanks again!

@asmith26 asmith26 closed this Nov 10, 2025
@asmith26 asmith26 deleted the remove-point-from-sketcher-widget branch November 19, 2025 21:34
@asmith26 asmith26 restored the remove-point-from-sketcher-widget branch November 19, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New Feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants