LMMS / LMMS/lmms

Fix or remove the "add step" context option on Beat clips

Open Beginner friendly
#8,522 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
10.4k
Forks
1.3k
Avg merge
2d 13h
Merged PRs (30d)
7

Description

### Enhancement Summary

Since #8369 all tracks in a pattern have a clip of the same shared length.

Yet, there are still some ways remaining to change the length of a clip without changing the length of its pattern : Copy+Paste (that should be taken care of by #8512) and the "add steps" option of Beat type clips context menu.

### Implementation Details / Mockup

I see two ways to resolve this issue:

_**1) Fix the option so the length of the pattern is updated**_
This involve adding code which purpose will not be easily understandable and could be the source of potential regressions in future updates.

**_2) Just remove the option_**
This is the simpler way (and imo the better). The context menu is huge and could benefit from some pruning, and there is already dedicated buttons in the pattern editor's ui.

What do you think about this ?

### Please search the issue tracker for existing feature requests before submitting your own.

- [x] I have searched all existing issues and confirmed that this is not a duplicate.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the Beat clip context-menu implementation and the "add steps" action, then compare it with the dedicated pattern editor controls mentioned in the issue. Done means the option is removed or updates the pattern length consistently, without leaving a way to change a clip length independently.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
desktop
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.