Skip to content

Make Tiny Notes Easier to Move #8386

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
regulus79 merged 5 commits into from
May 15, 2026
+19 −6
Merged

Make Tiny Notes Easier to Move #8386

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0b313b7
Initial fix
regulus79 May 13, 2026
a2ac34a
Increase fraction
regulus79 May 13, 2026
26a6ad5
Don't round mouse pos to tick when checking if within grip range
regulus79 May 13, 2026
031982f
Fix Formatting
regulus79 May 14, 2026
49c6026
Pass the Note object instead of length
regulus79 May 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions include/PianoRoll.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -371,6 +371,13 @@ protected slots:
static const std::vector<float> m_zoomLevels;
static const std::vector<float> m_zoomYLevels;

/**
* @returns the width of a note's resize area in pixels.
* @note When zoomed in, this is a constant pixel value, but when zoomed far out,
* it is a fraction of the note width, to make it easier to move short notes.
*/
int resizeGripWidth(const Note& note) const;

MidiClip* m_midiClip;
NoteVector m_ghostNotes;

Expand Down
18 changes: 12 additions & 6 deletions src/gui/editors/PianoRoll.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,10 @@ const int PR_TOP_MARGIN = 18;
const int PR_RIGHT_MARGIN = SCROLLBAR_SIZE;


// width of area used for resizing (the grip at the end of a note)
const int RESIZE_AREA_WIDTH = 9;
//! Width of area used for resizing (the grip at the end of a note)
constexpr int RESIZE_GRIP_WIDTH = 9;
//! The maximum fraction of the note width that the resize grip is allowed to take up
constexpr float RESIZE_GRIP_MAX_WIDTH_FRACTION = 0.25f;

// width of line for setting volume/panning of note
const int NOTE_EDIT_LINE_WIDTH = 3;
Expand Down Expand Up @@ -1922,8 +1924,8 @@ void PianoRoll::mousePressEvent(QMouseEvent * me )
}

// clicked at the "tail" of the note?
if( pos_ticks * m_ppb / TimePos::ticksPerBar() >
m_currentNote->endPos() * m_ppb / TimePos::ticksPerBar() - RESIZE_AREA_WIDTH
if (x + m_currentPosition * m_ppb / TimePos::ticksPerBar() >
m_currentNote->endPos() * m_ppb / TimePos::ticksPerBar() - resizeGripWidth(*m_currentNote)
&& m_currentNote->length() > 0 )
{
m_midiClip->addJournalCheckPoint();
Expand Down Expand Up @@ -2719,8 +2721,7 @@ void PianoRoll::mouseMoveEvent( QMouseEvent * me )
int noteRightX = ( note->pos() + note->length() -
m_currentPosition) * m_ppb/TimePos::ticksPerBar();
// cursor at the "tail" of the note?
bool atTail = note->length() > 0 && x > noteRightX -
RESIZE_AREA_WIDTH;
bool atTail = note->length() > 0 && x > noteRightX - resizeGripWidth(*note);
Qt::CursorShape cursorShape = atTail ? Qt::SizeHorCursor :
Qt::SizeAllCursor;
setCursor( cursorShape );
Expand Down Expand Up @@ -3290,6 +3291,11 @@ void PianoRoll::dragNotes(int x, int y, bool alt, bool shift, bool ctrl)
}


int PianoRoll::resizeGripWidth(const Note& note) const
{
const int noteWidth = note.length() * m_ppb / TimePos::ticksPerBar();
return std::min(RESIZE_GRIP_WIDTH, static_cast<int>(std::round(RESIZE_GRIP_MAX_WIDTH_FRACTION * noteWidth)));
}


void PianoRoll::paintEvent(QPaintEvent * pe )
Expand Down
Loading