Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OHI-1065] [OHI-1090] fix(probeTool): Implement isPointNearTool so EraserTool can detect probe measurements and delete them, use line width value from config for drawing the probe circle #1721

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

IbrahimCSAE
Copy link
Collaborator

@IbrahimCSAE IbrahimCSAE commented Dec 20, 2024

Context

The measurement eraser tool was not deleting probe measurements due to ProbeTool not having a isPointNearTool implementation. I have implemented the function so measurements can be detected and deleted.

The probe tool was also not using the line width set by users in the annotation configuration, it has now been fixed to use the value provided from the config.

Eraser fix:

CleanShot 2024-12-20 at 01 35 02

Line width fix:

CleanShot 2024-12-20 at 01 36 03@2x

Closes OHI-1065 and OHI-1090
Closes #1304, #1181

Copy link

stackblitz bot commented Dec 20, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@IbrahimCSAE IbrahimCSAE requested a review from sedghi December 20, 2024 06:39
Copy link

netlify bot commented Dec 20, 2024

Deploy Preview for cornerstone-3d-docs ready!

Name Link
🔨 Latest commit 697146d
🔍 Latest deploy log https://app.netlify.com/sites/cornerstone-3d-docs/deploys/676511311f084d0008d9a8c8
😎 Deploy Preview https://deploy-preview-1721--cornerstone-3d-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sedghi sedghi merged commit 90c1a98 into main Dec 20, 2024
26 checks passed
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.

[Bug] A problem in ProbeTool affects the normal use of EraserTool.
2 participants