smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2026-01-30 08:53:08 +01:00
Code Issues Projects Releases Wiki Activity

do not double-transform event coords for piano roll headers

Browse source 
the canvas PRH alreadys transforms event coordinates before deliverying the event
to a handler; the non-canvas version always gets item-centric coordinates from
the GUI toolkit
This commit is contained in:
Paul Davis 2025-07-09 10:21:00 -06:00
parent 5801716a78
commit 2b008400c7
1 changed files with 0 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

10
gtk2_ardour/prh_base.cc
View file

@ -132,8 +132,6 @@ bool
PianoRollHeaderBase::scroll_handler (GdkEventScroll* ev)
{
double evy = ev->y;
double ignore;
event_transform (ignore, evy);
int note_range = _adj.get_page_size ();
int note_lower = _adj.get_value ();
@ -524,8 +522,6 @@ PianoRollHeaderBase::motion_handler (GdkEventMotion* ev)
/* event coordinates are in canvas/window space */
double evy = ev->y;
double ignore;
event_transform (ignore, evy);
if (!_scroomer_drag && ev->x < _scroomer_size){
@ -673,8 +669,6 @@ bool
PianoRollHeaderBase::button_press_handler (GdkEventButton* ev)
{
double evy = ev->y;
double ignore;
event_transform (ignore, evy);
/* Convert canvas-coordinates to item coordinates */
@ -747,8 +741,6 @@ bool
PianoRollHeaderBase::button_release_handler (GdkEventButton* ev)
{
double evy = ev->y;
double ignore;
event_transform (ignore, evy);
end_scroomer_drag ();
@ -789,8 +781,6 @@ bool
PianoRollHeaderBase::enter_handler (GdkEventCrossing* ev)
{
double evy = ev->y;
double ignore;
event_transform (ignore, evy);
set_note_highlight (_midi_context.y_to_note (evy));
set_cursor (_midi_context.editing_context().cursors()->selector);