In Extend mode is "Snap to Current Frame" not extending #118722

New Issue

Open

opened 2024-02-25 16:53:46 +01:00 by tintwotin · 4 comments

tintwotin commented

2024-02-25 16:53:46 +01:00

System Information
Operating system: Win 11
Graphics card: RTX2060

Blender Version
4.2

Short description of error
In Extend mode is "Snap to Current Frame" not respected.

Exact steps for others to reproduce the error

Select Extend Mode.
Split a strip.
Move play head over the right clip.
Select the right handle of the left strip.
Select Snap to Current Frame.
Notice that the following strips are not moved in spite of the Extend mode.
Expected behavior: the extended strip is extended to the play head, and all following strips moved accordingly.

**System Information** Operating system: Win 11 Graphics card: RTX2060 **Blender Version** 4.2 **Short description of error** In Extend mode is "Snap to Current Frame" not respected. ![snap_expand.gif](/attachments/45364ebb-bb0e-4be7-92ce-1d1dd55c16aa) **Exact steps for others to reproduce the error** 1. Select Extend Mode. 2. Split a strip. 3. Move play head over the right clip. 4. Select the right handle of the left strip. 5. Select Snap to Current Frame. 6. Notice that the following strips are not moved in spite of the Extend mode. Expected behavior: the extended strip is extended to the play head, and all following strips moved accordingly.

snap_expand.gif

160 KiB

tintwotin added the

labels 2024-02-25 16:53:47 +01:00

tintwotin added this to the Video Sequencer project 2024-02-25 16:53:49 +01:00

Iliya Katushenock removed this from the Video Sequencer project 2024-02-25 17:17:01 +01:00

Iliya Katushenock added the

Video Sequencer

labels 2024-02-25 17:17:07 +01:00

tintwotin changed title from ~~In Extend mode is "Snap to Current Frame" not respected~~ to In Extend mode is "Snap to Current Frame" not extending

2024-02-25 22:14:26 +01:00

Pratik Borhade commented

2024-02-26 06:53:27 +01:00

Member

Thanks for the report. Can confirm. Looks like overlap_mode is not considered for this transform operation.

Thanks for the report. Can confirm. Looks like overlap_mode is not considered for this transform operation.

Pratik Borhade added

and removed

labels 2024-02-26 06:54:05 +01:00

Pratik Borhade commented

2024-02-26 07:14:58 +01:00

Member

I think we can call SEQ_transform_handle_overlap() here instead of shuffle.
https://projects.blender.org/blender/blender/src/branch/main/source/blender/editors/space_sequencer/sequencer_edit.cc#L400

I think we can call `SEQ_transform_handle_overlap()` here instead of shuffle. https://projects.blender.org/blender/blender/src/branch/main/source/blender/editors/space_sequencer/sequencer_edit.cc#L400

Pratik Borhade commented

2024-02-26 07:15:58 +01:00

Member

@iss ^

Richard Antalik commented

2024-02-28 20:20:03 +01:00

Member

Technically, we can call SEQ_transform_handle_overlap() in all cases, since shuffle is default. This would require some refactoring of some operators. I was doing this in past, but review stalled. That time I was working full time on VSE, so I would like to resume this effort, but not sure if I will have time for that.

I can do it for this particular operator if that would be helpful though.

Technically, we can call `SEQ_transform_handle_overlap()` in all cases, since shuffle is default. This would require some refactoring of some operators. I was doing this in past, but review stalled. That time I was working full time on VSE, so I would like to resume this effort, but not sure if I will have time for that. I can do it for this particular operator if that would be helpful though.

Sign in to join this conversation.

No Label

Animation & Rigging

Asset Browser Project Overview

Automated Testing

Blender Asset Bundle

Dependency Graph

Development Management

EEVEE & Viewport

Images & Movies

Motion Tracking

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Render Pipeline

Sculpt, Paint & Texture

Video Sequencer

Virtual Reality

Blender 2.8 Project

Milestone 1: Basic, Local Asset Browser

Good First Issue

Animation & Rigging

Development Management

EEVEE & Viewport

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Sculpt, Paint & Texture

Needs Info from Developers

Needs Information from User

No Milestone

No project

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#118722

No description provided.