Skip to content

Map BarModelEditor (aka "fader" even though it's a different thing) progress directly to cursor#8405

Draft
sqrvrt wants to merge 1 commit into
LMMS:masterfrom
sqrvrt:direct-fader-pos
Draft

Map BarModelEditor (aka "fader" even though it's a different thing) progress directly to cursor#8405
sqrvrt wants to merge 1 commit into
LMMS:masterfrom
sqrvrt:direct-fader-pos

Conversation

@sqrvrt

@sqrvrt sqrvrt commented May 24, 2026

Copy link
Copy Markdown
Contributor
output.mp4

List of sins:

  • BarModelEditor::setPositionAbsolute feels like it shouldn't exist, or at least should be in FloatModelEditorBase
  • BarModelEditor::setPositionAbsolute steals some code from BarModelEditor::paintEvent
  • BarModelEditor::setPositionAbsolute steals everything else from FloatModelEditorBase::setPosition
  • BarModelEditor::mouseMoveEvent is a very close copy of FloatModelEditorBase::mouseMoveEvent, but position isn't relative
  • single click doesn't change the value, only dragging does
  • undocumented

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant