-
Notifications
You must be signed in to change notification settings - Fork 60
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
Merge polygons #617
Merge polygons #617
Conversation
✅ Deploy Preview for volview-dev ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Functionality-wise, seems good so far. Is there existing software that merges polygons, and what is the desired behavior post-merge? |
HistomicsUI large image viewer. <- Behavior is different though as Histomics has a named annotation that can be made up of multiple vector shapes. Drawn vector shapes are automaticly merged.
Currently polygon store is deleting selected polygons and creating a new polygon. Is that what you mean? |
I'm referring to the connectivity visually looking like a line that has to be teased apart to reveal the connection point. I'm just curious if there are other merging strategies that other software uses. Other than that, LGTM |
New additions:
Good to go by me. |
Adds pan slice views with middle mouse button besides shift + left mouse button. This is because polygon tool consumes mouse events even with shift pressed.
@PaulHax is this ready? I see some new changes since the last approval. |
Yes, its ready. The new changes
|
Right click to merge selected polygons.
Adds a new deprendency
polybool
=/closes #614