Add go-to / copy current video position to the player slider#11625
Add go-to / copy current video position to the player slider#11625muaz978 wants to merge 1 commit into
Conversation
|
Hey, Muaz. The This is how I see it on most platforms, but it's up to @nikolajolsson to decide. Another option is to add a NumericUpDown with the video position at the bottom in the toolbar, but maybe it wouldn't fit or would look too crowded. |
|
Hey dear, I can handle both, but let's see what @nikolajolsson say, and then I will act accordingly. |
|
We can take a look after v5.0.0 :) |
|
On your mark. |
Right-clicking the video position slider now offers "Go to video position" (the existing ShowGoToVideoPosition dialog) and "Copy" (copies the current position as a timecode, applying the configured video offset). This restores the SE4 "Video position" control's ability to jump to and copy the current position, which had no UI entry point in SE5 - only a shortcut for go-to. - New CopyVideoPosition command in MainViewModel. - VideoPlayerControl exposes GoToPositionRequested/CopyPositionRequested events via a context menu on the position slider; InitVideoPlayer wires them to the view-model commands. Reuses existing language strings (no new tags).
muaz978
force-pushed
the
feature/video-position-goto-copy
branch
from
eac5c3c to
26a17b1
Compare
|
Rebased this branch onto the latest main. This is purely a sync, no functional changes: it just confirms this stays conflict-free and cleanly mergeable against current main. |
|
We already have a "Goto video position" shortcut |
3 participants
Adds go-to / copy of the current video position, addressing the follow-up on #11598 (@OmrSi: "a way to go to a video position and copy the current video position", which SE4 had on the Video position control).
Right-click the video position slider:
New
CopyVideoPositioncommand;VideoPlayerControlraisesGoToPositionRequested/CopyPositionRequestedfrom a context menu on the slider, wired to the view-model inInitVideoPlayer. Reuses existing language strings (no new tags). Builds clean (0/0).Couldn't GUI-test here; a Windows/macOS test build can be provided to confirm the menu and copy if useful.