Skip to content

feat(InteractorStyleTrackballActor): add vtkInteractorStyleTrackballActor#3459

Open
daker wants to merge 1 commit intoKitware:masterfrom
daker:feat-interactorstyletrackballactor
Open

feat(InteractorStyleTrackballActor): add vtkInteractorStyleTrackballActor#3459
daker wants to merge 1 commit intoKitware:masterfrom
daker:feat-interactorstyletrackballactor

Conversation

@daker
Copy link
Copy Markdown
Collaborator

@daker daker commented Mar 27, 2026

Context

fix #1838

Results

image

Changes

  • Documentation and TypeScript definitions were updated to match those changes

PR and Code Checklist

  • semantic-release commit messages
  • Run npm run reformat to have correctly formatted code

Testing

  • This change adds or fixes unit tests
  • Tested environment:
    • vtk.js:
    • OS:
    • Browser:

@daker daker marked this pull request as ready for review March 27, 2026 13:55
@daker daker force-pushed the feat-interactorstyletrackballactor branch 2 times, most recently from 50f219a to c6b4f09 Compare March 27, 2026 14:19
Copy link
Copy Markdown
Member

@finetjul finetjul left a comment

Choose a reason for hiding this comment

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

This does not behave like the interactor style trackball actor in VTK C++.
(e.g. pick top left corner, then drag mouse down, then drag mouse right)

In VTK.js, we prefer the more modern approach of "manipulators" (that will eventually land in VTK C++).

@jourdain
Copy link
Copy Markdown
Collaborator

BTW, it has been merged on VTK/C++ https://gitlab.kitware.com/vtk/vtk/-/merge_requests/13039

@daker daker force-pushed the feat-interactorstyletrackballactor branch from c6b4f09 to bdb2a94 Compare March 30, 2026 21:16
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.

InteractorstyleTrackballActor

3 participants