Skip to content

add Copy Vide Name action to the video list context menu#373

Merged
gbeane merged 2 commits into
mainfrom
feature/add-copy-video-name-to-context-menu
May 4, 2026
Merged

add Copy Vide Name action to the video list context menu#373
gbeane merged 2 commits into
mainfrom
feature/add-copy-video-name-to-context-menu

Conversation

@gbeane
Copy link
Copy Markdown
Collaborator

@gbeane gbeane commented May 3, 2026

This pull request adds a small but useful feature to the video list context menu in the video_list_widget.py file. Now, in addition to "Get Info", users can copy the video name directly from the context menu.

  • UI Enhancement:
    • Added a "Copy Video Name" option to the context menu in the video list widget, allowing users to copy the video name to the clipboard.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a small UI convenience to the main window’s video list by extending the existing right-click context menu so users can copy a video's name directly from the list. It fits into the existing VideoListDockWidget behavior, which already exposes per-video actions like “Get Info”.

Changes:

  • Added a new Copy Video Name action to the video list context menu.
  • Wired the new action to copy the clicked video's canonical name to the system clipboard.
  • Kept the existing Get Info action behavior unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/jabs/ui/main_window/video_list_widget.py Outdated
@keithshep
Copy link
Copy Markdown
Contributor

Nice! I've wanted this a few times.

@gbeane gbeane merged commit a35961d into main May 4, 2026
5 checks passed
@gbeane gbeane deleted the feature/add-copy-video-name-to-context-menu branch May 4, 2026 13:50
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