smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-08 07:45:00 +01:00
Code Issues Projects Releases Wiki Activity

do not use DragManager::current_pointer_x() directly during drags

Browse source 
That ignores a _bounding_item and can give incorrect coordindates for the drag when
that is used (e.g. clip editor)
This commit is contained in:
Paul Davis 2025-01-10 11:17:59 -07:00
parent e273ec06bb
commit d0b643b116
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
gtk2_ardour/editor_drag.cc
View file

@ -518,7 +518,7 @@ Drag::motion_handler (GdkEvent* event, bool from_autoscroll)
if (old_move_threshold_passed != _move_threshold_passed) { if (old_move_threshold_passed != _move_threshold_passed) {
/* just changed */ /* just changed */
if (fabs (_drags->current_pointer_y () - _grab_y) > fabs (_drags->current_pointer_x () - _grab_x)) { if (fabs (current_pointer_y () - _grab_y) > fabs (current_pointer_x () - _grab_x)) {
_initially_vertical = true; _initially_vertical = true;
} else { } else {
_initially_vertical = false; _initially_vertical = false;
@ -6182,7 +6182,7 @@ NoteDrag::total_dx (GdkEvent* event) const
} }
/* we need to use absolute positions here to honor the tempo-map */ /* we need to use absolute positions here to honor the tempo-map */
timepos_t const t1 = pixel_duration_to_time (_drags->current_pointer_x ()); timepos_t const t1 = pixel_duration_to_time (current_pointer_x ());
timepos_t const t2 = pixel_duration_to_time (grab_x ()); timepos_t const t2 = pixel_duration_to_time (grab_x ());
/* now calculate proper `b@b` time */ /* now calculate proper `b@b` time */