FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add option for multi-camera controls on the top panel to the left of visibility by mattdawkins · Pull Request #1568 · Kitware/dive · GitHub

/ dive Public

Add option for multi-camera controls on the top panel to the left of visibility - #1568

Merged
BryonLewis merged 7 commits into
mainfrom
multi-cam-top-button
Jan 17, 2026
ConversationCommits (7)ChecksFiles changed
Merged

Add option for multi-camera controls on the top panel to the left of visibility#1568
BryonLewis merged 7 commits into
mainfrom
multi-cam-top-button

Conversation

mattdawkins commented Jan 8, 2026 •
edited by BryonLewis
Loading

Copy link
Copy Markdown
Member

Add Multi-Camera Toolbar for Track Editing

Summary

This PR adds a multi-camera toolbar that appears in the top editor menu when a track is selected in multi-camera datasets. The toolbar provides quick access to common multi-camera editing operations without needing to navigate through dropdown menus.

Features

  • Edit on Opposite Camera: Quickly switch to the opposite camera and edit/create a detection for the selected track (keyboard shortcut: e)
  • Link/Unlink Tracks: Link a track to another camera or unlink the current camera from a multi-camera track
  • Delete Detection: Remove a detection from the current camera/frame
  • Delete Track from Camera: Remove the entire track from the current camera while preserving it on other cameras

Changes

  • New Component: MultiCamToolbar.vue - A new toolbar component with multi-camera editing controls
  • EditorMenu Integration: Added multicam-controls-left and multicam-controls-right slots to position the toolbar appropriately
  • Settings Integration: Added toggle in Track Settings Panel and Desktop Settings to show/hide the toolbar
  • Keyboard Shortcuts: Added e key shortcut for quick editing on the opposite camera

Technical Details

  • Toolbar appears only when:
    • Multiple cameras are present
    • A track is selected
    • The setting is enabled (default: enabled)
  • Buttons are context-aware and enable/disable based on track state across cameras
  • Positioned dynamically based on the selected camera (left for primary camera, right for others)

Configuration

Users can toggle the toolbar visibility via:

  • Track Settings Panel → Multi-Camera Settings → "Show Toolbar"
  • Desktop Settings → Annotation Settings → "Show multi-camera toolbar"

BryonLewis commented Jan 17, 2026

Copy link
Copy Markdown
Collaborator

For some reason it had two toolbars in there and only one of them was showing in the Viewer.vue - I fixed that.

Refactored the EditorMenu.vue to use defineComponent and composition API instead of the older Options Api interface.
Moved the Visibility tools into their own AnnotationVisibilityMenu.vue
Updated all menus in the toolbar to be collapsable to their own menu instead of a row of icons. This feature is logged in the user's localStorage so it is persistent.

The chevron to the right of the section can be clicked to collapse/expand the fields

BryonLewis self-requested a review January 17, 2026 18:36
BryonLewis approved these changes Jan 17, 2026
View reviewed changes

BryonLewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

After creating the modifications I'm approving this PR. Mostly it's just a UI change to make editing multicamera items easier. I think the other additions for the toolbar buttons to collapse and expand are better UI elements as we add more functionality.

BryonLewis merged commit a39915b into main Jan 17, 2026
4 checks passed
BryonLewis deleted the multi-cam-top-button branch January 17, 2026 18:38
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

BryonLewis BryonLewis approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL