Sequencer: Meta Strip bugs/unfinished implementation #74130

New Issue

tintwotin · 2020-02-23T09:15:25+01:00

tintwotin commented

2020-02-23 09:15:25 +01:00

System Information
Operating system: Windows-10-10.0.17763-SP0 64 Bits
Graphics card: Intel(R) UHD Graphics 600 Intel 4.5.0 - Build 26.20.100.7262

Blender Version
Broken: version: 2.83 (sub 4), branch: master (modified), commit date: 2020-02-22 16:08, hash: c26f470cfe
Worked: (optional)

Short description of error
There are a couple of bugs/unfinished implementation of the Meta Strips in the Sequencer:

When toggle to open the Meta strip, no active strip is selected and it breaks the sidebar panel:
There is no Meta Strip options in the sidebar(like this):

There is not indication of being "inside" a Meta strip, other than a very subtle drawing of the Meta strip range. Maybe this range should have a color so it is clearly distinct:

There is no Back to Previous button:

AFAIK there is no "inside meta" flag which can be used in space_sequencer.py to show/not show such a button.

**System Information** Operating system: Windows-10-10.0.17763-SP0 64 Bits Graphics card: Intel(R) UHD Graphics 600 Intel 4.5.0 - Build 26.20.100.7262 **Blender Version** Broken: version: 2.83 (sub 4), branch: master (modified), commit date: 2020-02-22 16:08, hash: `c26f470cfe` Worked: (optional) **Short description of error** There are a couple of bugs/unfinished implementation of the Meta Strips in the Sequencer: - When toggle to open the Meta strip, no active strip is selected and it breaks the sidebar panel: ![image.png](https://archive.blender.org/developer/F8364028/image.png) - There is no Meta Strip options in the sidebar(like this): ![image.png](https://archive.blender.org/developer/F8364051/image.png) There is not indication of being "inside" a Meta strip, other than a very subtle drawing of the Meta strip range. Maybe this range should have a color so it is clearly distinct: ![image.png](https://archive.blender.org/developer/F8364062/image.png) There is no Back to Previous button: ![image.png](https://archive.blender.org/developer/F8364075/image.png) AFAIK there is no "inside meta" flag which can be used in space_sequencer.py to show/not show such a button.

tintwotin commented

2020-02-23 09:15:25 +01:00

Added subscriber: @tintwotin

William Reynish commented

2020-02-23 13:05:00 +01:00

Added subscriber: @WilliamReynish

William Reynish commented

2020-02-23 13:05:00 +01:00

Changed status from 'Needs Triage' to: 'Needs Developer To Reproduce'

William Reynish commented

2020-02-23 13:05:00 +01:00

When toggle to open the Meta strip, no active strip is selected and it breaks the sidebar panel:

I cannot reproduce this. Here, when I open a Meta strip, it looks just like normal when there is no active strip (no strip properties are shown)

There is no Meta Strip options in the sidebar

This I don't understand. Yes there is no such panel, but why would there be? Doesn't look like a bug.

There is not indication of being "inside" a Meta strip, other than a very subtle drawing of the Meta strip range.
There is no Back to Previous button:

The UI could show this more clearly, but it's not a bug and doesn't belong in the bug tracker.

>When toggle to open the Meta strip, no active strip is selected and it breaks the sidebar panel: I cannot reproduce this. Here, when I open a Meta strip, it looks just like normal when there is no active strip (no strip properties are shown) >There is no Meta Strip options in the sidebar This I don't understand. Yes there is no such panel, but why would there be? Doesn't look like a bug. >There is not indication of being "inside" a Meta strip, other than a very subtle drawing of the Meta strip range. >There is no Back to Previous button: The UI could show this more clearly, but it's not a bug and doesn't belong in the bug tracker.

tintwotin commented

2020-02-23 14:51:06 +01:00

The UI could show this more clearly, but it's not a bug and doesn't belong in the bug tracker.

This valid issue was raised on RCS almost 4 years ago, so apparently RCS isn't the right place for it either: https://blender.community/c/rightclickselect/Qcbbbc/

> The UI could show this more clearly, but it's not a bug and doesn't belong in the bug tracker. This valid issue was raised on RCS almost 4 years ago, so apparently RCS isn't the right place for it either: https://blender.community/c/rightclickselect/Qcbbbc/

William Reynish commented

2020-02-23 15:09:35 +01:00

At that time there may not have been a context menu where you can find the edit meta strip operator. Now there is.

Richard Antalik commented

2020-02-24 18:40:31 +01:00

Added subscriber: @iss

Richard Antalik commented

2020-02-24 18:40:31 +01:00

Changed status from 'Needs Developer To Reproduce' to: 'Archived'

Richard Antalik closed this issue

2020-02-24 18:40:31 +01:00

Richard Antalik self-assigned this 2020-02-24 18:40:31 +01:00

Richard Antalik commented

2020-02-24 18:40:32 +01:00

There are a couple of bugs/unfinished implementation of the Meta Strips in the Sequencer:

Always report one bug per report.

When toggle to open the Meta strip, no active strip is selected and it breaks the sidebar panel:

Not really a bug - it works as expected.
You may say, that some strip should be active, not must be active, so it's feature request.

There is no Meta Strip options in the sidebar(like this):
There is not indication of being "inside" a Meta strip, other than a very subtle drawing of the Meta strip range. Maybe this range should have a color so it is clearly distinct:

Same thing, a feature request.

AFAIK there is no "inside meta" flag which can be used in space_sequencer.py to show/not show such a button.

Technically you could do is_top_view = (C.sequences- [x] == C.scene.sequence_editor.sequences_all- [x]), but it's hacky

I am not saying, this can not be done better, but it's not something that restricts any functionality

>There are a couple of bugs/unfinished implementation of the Meta Strips in the Sequencer: Always report one bug per report. >When toggle to open the Meta strip, no active strip is selected and it breaks the sidebar panel: Not really a bug - it works as expected. You may say, that some strip should be active, not must be active, so it's feature request. >There is no Meta Strip options in the sidebar(like this): >There is not indication of being "inside" a Meta strip, other than a very subtle drawing of the Meta strip range. Maybe this range should have a color so it is clearly distinct: Same thing, a feature request. >AFAIK there is no "inside meta" flag which can be used in space_sequencer.py to show/not show such a button. Technically you could do `is_top_view = (C.sequences- [x] == C.scene.sequence_editor.sequences_all- [x])`, but it's hacky I am not saying, this can not be done better, but it's not something that restricts any functionality

tintwotin commented

2020-02-24 18:50:41 +01:00

This report was linked to a patch, which cleaned up the polls giving the impression of a broken sidebar. It also exposed the mentioned meta strip functions. In the VSE chat I posted a suggestion on how to make the Meta Strip range more clear and with your line of code the "Return" button could be implemented. So all of these things could be solved, but I'll leave it up to you guys.
The patch: https://developer.blender.org/D6913

This report was linked to a patch, which cleaned up the polls giving the impression of a broken sidebar. It also exposed the mentioned meta strip functions. In the VSE chat I posted a suggestion on how to make the Meta Strip range more clear and with your line of code the "Return" button could be implemented. So all of these things could be solved, but I'll leave it up to you guys. The patch: https://developer.blender.org/D6913

Richard Antalik commented

2020-02-24 19:29:17 +01:00

@tintwotin No need for report if you are not fixing bug, just improving interface

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

Sequencer: Meta Strip bugs/unfinished implementation #74130