"Delete Strip and Data" and "Update Scene Frame Range" are misleading #118277

New Issue

Open

opened 2024-02-14 14:55:33 +01:00 by tintwotin · 5 comments

tintwotin commented

2024-02-14 14:55:33 +01:00

System Information
Operating system: Win 11
Graphics card: RTX2060

Blender Version
4.2 Alpha

Short description of error
Delete Strip & Data: actually deletes the strip and the references scene.
Update Scene Frame Range: actually updates the "Hold" ranges of the strip.

Misleading names and tooltips. The "data" wording is very unclear, and might lead users to think that a selection with ex. Movie or Image strips, and selecting this operator will delete the source files too.
If the operator only becomes visible when a Scene strip is the active strip, and it is the referencing scene which is deleted, then this should be made clear in the operator naming and tool tip.
Update Scene Frame Range: actually updates the "Hold" ranges of the strip.

Exact steps for others to reproduce the error
Select a Scene strip and see the operator name and tooltip in the Strip menu:

**System Information** Operating system: Win 11 Graphics card: RTX2060 **Blender Version** 4.2 Alpha **Short description of error** Delete Strip & Data: actually deletes the strip and the references scene. Update Scene Frame Range: actually updates the "Hold" ranges of the strip. Misleading names and tooltips. The "data" wording is very unclear, and might lead users to think that a selection with ex. Movie or Image strips, and selecting this operator will delete the source files too. If the operator only becomes visible when a Scene strip is the active strip, and it is the referencing scene which is deleted, then this should be made clear in the operator naming and tool tip. Update Scene Frame Range: actually updates the "Hold" ranges of the strip. **Exact steps for others to reproduce the error** Select a Scene strip and see the operator name and tooltip in the Strip menu: ![image](/attachments/1de59d55-a607-4cfa-807d-1830babf2a1e)

11 KiB

tintwotin added the

labels 2024-02-14 14:55:33 +01:00

tintwotin added this to the Video Sequencer project 2024-02-14 14:55:36 +01:00

tintwotin changed title from ~~Delete Strip and Data~~ to "Delete Strip and Data" is misleading

2024-02-14 14:56:07 +01:00

Iliya Katushenock removed this from the Video Sequencer project 2024-02-14 14:58:51 +01:00

Iliya Katushenock added the

Video Sequencer

label 2024-02-14 14:58:59 +01:00

Pratik Borhade commented

2024-02-15 10:21:08 +01:00

Member

Hi, thanks for the report. Looks like this deletes the scene linked with the "scene strip": https://projects.blender.org/blender/blender/src/branch/main/source/blender/editors/space_sequencer/sequencer_edit.cc#L1660

What name you would prefer?: Delete strip and scene data

Hi, thanks for the report. Looks like this deletes the scene linked with the "scene strip": https://projects.blender.org/blender/blender/src/branch/main/source/blender/editors/space_sequencer/sequencer_edit.cc#L1660 What name you would prefer?: `Delete strip and scene data`

Pratik Borhade added

and removed

labels 2024-02-15 10:22:08 +01:00

tintwotin commented

2024-02-15 10:30:41 +01:00

Author

I wrote about it in the VSE chat. Maybe it is better to discuss these things there?

I wrote about it in the VSE chat. Maybe it is better to discuss these things there?

Pratik Borhade commented

2024-02-15 11:12:41 +01:00

Member

I see, Sergey suggested "data-block".
cc @iss

Maybe it is better to discuss these things there?

Chat messages are cleared after some days (also things gets easily lost in chat) so best to have some opinions in this thread 🙂

I see, Sergey suggested "data-block". cc @iss - - - > Maybe it is better to discuss these things there? Chat messages are cleared after some days (also things gets easily lost in chat) so best to have some opinions in this thread 🙂

tintwotin commented

2024-02-15 12:06:15 +01:00

Author

I wrote something above Sergey's reply.

The Delete strip and data and the following operator Update Scene Frame Range, should at least be placed inside a Scene Strip submenu(like Move Strip submenu).

Imo, would it be better to as specific as possible with the name and tooltip(most user will not know what a data-block is). Maybe something along these lines?

Scene Strip

Delete (include Scene)
Tooltip: Delete strip and referenced scene.

Refresh Hold Duration
Tooltip: Refresh strip hold duration according to range of referenced scene.

Even better would be to include the Update Scene Frame Range in the Refresh All operator and remove it from the Strip menu. Or making it fully automated, so when a scene range is changed the hold durations of strips referencing those scenes are updated too. Users should not have to manually update these things.

The (include X) syntax is already used in the Select > Box (include Handles).

I wrote something above Sergey's reply. The Delete strip and data and the following operator Update Scene Frame Range, should at least be placed inside a Scene Strip submenu(like Move Strip submenu). Imo, would it be better to as specific as possible with the name and tooltip(most user will not know what a data-block is). Maybe something along these lines? Scene Strip Delete (include Scene) Tooltip: Delete strip and referenced scene. Refresh Hold Duration Tooltip: Refresh strip hold duration according to range of referenced scene. Even better would be to include the Update Scene Frame Range in the Refresh All operator and remove it from the Strip menu. Or making it fully automated, so when a scene range is changed the hold durations of strips referencing those scenes are updated too. Users should not have to manually update these things. The (include X) syntax is already used in the Select > Box (include Handles).

tintwotin commented

2024-02-25 07:52:29 +01:00

Author

Alternative solution, by placing these operators in their right context, they can be implemented in a much more understandable way:

Delete Strip and Data:
(Should be rewritten, so it only deletes data aka. scene)

Refresh Hold Duration:

Alternative solution, by placing these operators in their right context, they can be implemented in a much more understandable way: Delete Strip and Data: (Should be rewritten, so it only deletes data aka. scene) ![image](/attachments/c9063417-e4e1-4c90-abf2-7c95cae04bd9) Refresh Hold Duration: ![image](/attachments/abb41da2-a4c6-4e82-9fdf-185c325e7f38)

267 KiB

306 KiB

tintwotin changed title from ~~"Delete Strip and Data" is misleading~~ to "Delete Strip and Data" and "Update Scene Frame Range" are misleading

2024-02-25 07:53:26 +01:00

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

2 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#118277

No description provided.