Transform: Redo doesn't work properly in Video Sequencer time manipulation #71455

New Issue

Alessio Monti di Sopra · 2019-11-09T18:39:29+01:00

2019-11-09 18:39:29 +01:00

Blender Version
Broken: 2.82

Description of error(s)
Redo doesn't work properly in Video Sequencer Move/Extend from Playhead (TIME_EXTEND mode).

Exact steps for others to reproduce the error

Position the current frame in the middle of a selected strip
Press E to activate the operator, with the mouse cursor on either side of the strip, to move one of its side
Try to modify the result with the redo panel -> the strip will shift, as if both handles were being modified

**Blender Version** Broken: 2.82 **Description of error(s)** Redo doesn't work properly in Video Sequencer *Move/Extend from Playhead* (TIME_EXTEND mode). **Exact steps for others to reproduce the error** - Position the current frame in the middle of a selected strip - Press E to activate the operator, with the mouse cursor on either side of the strip, to move one of its side - Try to modify the result with the redo panel -> the strip will shift, as if both handles were being modified

Alessio Monti di Sopra commented

2019-11-09 18:39:29 +01:00

Added subscriber: @a.monti

Germano Cavalcante was assigned by Alessio Monti di Sopra

2019-11-09 18:44:03 +01:00

Alessio Monti di Sopra commented

2019-11-09 18:44:03 +01:00

Here it is, if you prefer I can make separate reports for each operator, but they seemed quite related to me.

Germano Cavalcante commented

2019-11-09 20:19:17 +01:00

Well, it's usually better separated.
But I think that's fine, it looks like it can be solved in a single commit.

Well, it's usually better separated. But I think that's fine, it looks like it can be solved in a single commit.

Germano Cavalcante was unassigned by Dalai Felinto

2019-12-23 13:44:17 +01:00

Dalai Felinto commented

2019-12-23 13:44:17 +01:00

Added subscriber: @mano-wii

Germano Cavalcante self-assigned this 2020-01-25 00:07:47 +01:00

Germano Cavalcante commented

2020-02-14 18:08:35 +01:00

Thansk for the report.
I tried to reproduce each of the problems but I was only able to replicate the "Graph editor Extend (TIME_EXTEND mode)".

So I will remove these other problems from the description, as they are different:

Graph editor Extend (TIME_EXTEND mode):

Select a keyframe or a handle and press E to activate the operator

Try to modify the result with the redo panel -> both x and y value are inverted in the panel, resulting in a mirrored transformation

NLA Extend (TIME_EXTEND mode):

same steps as for the video sequencer case, but the result will also be mirrored

NLA Move :

Select a strip and press G to move it

Try to tweak the result -> the result gets mirrored

DopeSheet Extend (TIME_EXTEND mode):

Select two or more keyframes and put the current frame somewhere in between them

Press E to activate the operator

Tweak the operation -> same result as for the vse case, all the keyframes will move

If you are still able to reproduce any of these problems, please create another report.

Thansk for the report. I tried to reproduce each of the problems but I was only able to replicate the `"Graph editor Extend (TIME_EXTEND mode)"`. So I will remove these other problems from the description, as they are different: > * Graph editor *Extend* (TIME_EXTEND mode): > # Select a keyframe or a handle and press E to activate the operator > # Try to modify the result with the redo panel -> both x and y value are inverted in the panel, resulting in a mirrored transformation > > * NLA *Extend* (TIME_EXTEND mode): > # same steps as for the video sequencer case, but the result will also be mirrored > > * NLA *Move* : > # Select a strip and press G to move it > # Try to tweak the result -> the result gets mirrored > > * DopeSheet *Extend* (TIME_EXTEND mode): > # Select two or more keyframes and put the current frame somewhere in between them > # Press E to activate the operator > # Tweak the operation -> same result as for the vse case, all the keyframes will move If you are still able to reproduce any of these problems, please create another report.

Germano Cavalcante changed title from ~~Transform operator: Redo doesn't work properly in time manipulation modes~~ to Transform: Redo doesn't work properly in Video Sequencer time manipulation

2020-02-14 18:11:28 +01:00

Alessio Monti di Sopra commented

2020-02-15 19:55:15 +01:00

Hi! Yes, I can reproduce all of them.
I'll make 4 separate reports then, or for the Graph editor one it's not needed?

Hi! Yes, I can reproduce all of them. I'll make 4 separate reports then, or for the Graph editor one it's not needed?

Germano Cavalcante commented

2020-02-15 20:06:32 +01:00

In #71455#872439, @a.monti wrote:
Hi! Yes, I can reproduce all of them.
I'll make 4 separate reports then, or for the Graph editor one it's not needed?

If you're sure it's a different bug, I don't see why not.
But please give us more clear instructions on how to reproduce it from scratch.

> In #71455#872439, @a.monti wrote: > Hi! Yes, I can reproduce all of them. > I'll make 4 separate reports then, or for the Graph editor one it's not needed? If you're sure it's a different bug, I don't see why not. But please give us more clear instructions on how to reproduce it from scratch.

Alessio Monti di Sopra commented

2020-02-15 20:55:57 +01:00

In #71455#872447, @mano-wii wrote:
If you're sure it's a different bug, I don't see why not.
But please give us more clear instructions on how to reproduce it from scratch.

I'm not sure of anything 🙂
They for sure seem related to each other, so I guess they could be solved in a similar way, but that transform code was a bit too much for me to look into.
I'll try to make the steps clearer in the new reports.

> In #71455#872447, @mano-wii wrote: > If you're sure it's a different bug, I don't see why not. > But please give us more clear instructions on how to reproduce it from scratch. I'm not sure of anything 🙂 They for sure seem related to each other, so I guess they could be solved in a similar way, but that transform code was a bit too much for me to look into. I'll try to make the steps clearer in the new reports.

blender-admin commented

2020-02-17 17:33:29 +01:00

This issue was referenced by a33b261473

This issue was referenced by a33b261473e95de8d8d438dbb6aa7afe513fb082

Germano Cavalcante commented

2020-02-17 17:33:48 +01:00

Changed status from 'Confirmed' to: 'Resolved'

Germano Cavalcante closed this issue

2020-02-17 17:33:48 +01:00

blender-admin commented

2020-04-27 17:07:07 +02:00

This issue was referenced by a4df7f78a8

This issue was referenced by a4df7f78a8816ca463cb09f23ac64b989cd8ebb0

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Transform: Redo doesn't work properly in Video Sequencer time manipulation #71455

Select a keyframe or a handle and press E to activate the operator

Try to modify the result with the redo panel -> both x and y value are inverted in the panel, resulting in a mirrored transformation

same steps as for the video sequencer case, but the result will also be mirrored

Select a strip and press G to move it

Try to tweak the result -> the result gets mirrored

Select two or more keyframes and put the current frame somewhere in between them

Press E to activate the operator

Tweak the operation -> same result as for the vse case, all the keyframes will move