13
0

remove copy option for TempoMarkerDrag

This just seems like a dumb UI to implement. Drag-copying tempo markers is not a thing
This commit is contained in:
Paul Davis 2022-05-10 09:33:49 -06:00
parent eff34f2e1b
commit 66a3513dbd
3 changed files with 3 additions and 5 deletions

View File

@ -3527,9 +3527,8 @@ MeterMarkerDrag::aborted (bool moved)
}
TempoMarkerDrag::TempoMarkerDrag (Editor* e, ArdourCanvas::Item* i, bool c)
TempoMarkerDrag::TempoMarkerDrag (Editor* e, ArdourCanvas::Item* i)
: Drag (e, i, Temporal::BeatTime)
, _copy (c)
, _grab_bpm (120.0, 4.0)
, _before_state (0)
{

View File

@ -864,7 +864,7 @@ private:
class TempoMarkerDrag : public Drag
{
public:
TempoMarkerDrag (Editor *, ArdourCanvas::Item *, bool);
TempoMarkerDrag (Editor *, ArdourCanvas::Item *);
void start_grab (GdkEvent *, Gdk::Cursor* c = 0);
void motion (GdkEvent *, bool);
@ -886,7 +886,6 @@ private:
Temporal::TempoPoint const * _real_section;
Temporal::TempoMap::WritableSharedPtr map;
bool _copy;
bool _movable;
Temporal::Tempo _grab_bpm;
Temporal::Beats _grab_qn;

View File

@ -792,7 +792,7 @@ Editor::button_press_handler_1 (ArdourCanvas::Item* item, GdkEvent* event, ItemT
if (ArdourKeyboard::indicates_constraint (event->button.state)) {
_drags->set (new TempoEndDrag (this, item), event);
} else {
_drags->set (new TempoMarkerDrag (this, item, ArdourKeyboard::indicates_copy (event->button.state)), event);
_drags->set (new TempoMarkerDrag (this, item), event);
}
return true;