Skip to content

feat(paint): add cross-plane sync during painting#767

Merged
floryst merged 6 commits intoKitware:mainfrom
TomTonroe:paint-cross-plane-sync
Aug 20, 2025
Merged

feat(paint): add cross-plane sync during painting#767
floryst merged 6 commits intoKitware:mainfrom
TomTonroe:paint-cross-plane-sync

Conversation

@TomTonroe
Copy link
Copy Markdown
Contributor

  • Adds cross-plane synchronisation and camera centering of non-active views during painting.
  • Toggle added to the settings panel (off by default).

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 31, 2025

Deploy Preview for volview-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 545a304
🔍 Latest deploy log https://app.netlify.com/projects/volview-dev/deploys/68a5703dc074080008c8a802
😎 Deploy Preview https://deploy-preview-767--volview-dev.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 project configuration.

@PaulHax
Copy link
Copy Markdown
Collaborator

PaulHax commented Jul 31, 2025

I like this feature. Kind of want it give it top billing by putting the switch up in the upper left side panel paint controls.

@TomTonroe TomTonroe force-pushed the paint-cross-plane-sync branch from 89172be to 69e7fb2 Compare August 1, 2025 07:56
@TomTonroe
Copy link
Copy Markdown
Contributor Author

I like this feature. Kind of want it give it top billing by putting the switch up in the upper left side panel paint controls.

Thanks! I have moved the toggle to the top left of the paintControls panel.

@PaulHax PaulHax requested a review from floryst August 1, 2025 13:07
@floryst
Copy link
Copy Markdown
Contributor

floryst commented Aug 19, 2025

Thanks for the contribution! A few notes:

  • Running npm run prettify will address the failing checks
  • Toggling Sync Views will cause the existing slice positions to reset, which is not desirable

@TomTonroe TomTonroe force-pushed the paint-cross-plane-sync branch from 8df88a5 to 5047044 Compare August 20, 2025 06:35
@TomTonroe
Copy link
Copy Markdown
Contributor Author

Thanks @floryst! Toggling Sync Views should now preserve the current slice positions.

Copy link
Copy Markdown
Contributor

@floryst floryst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the contribution!

@floryst floryst added this pull request to the merge queue Aug 20, 2025
Merged via the queue into Kitware:main with commit c380f50 Aug 20, 2025
7 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.

3 participants