@b4n commented on this pull request.


>               sci_scroll_columns(editor->sci, amount);
                return TRUE;
        }
+       else if (!event->state || (editor_prefs.scrollwheel_zoom_disable && 
(event->state & GDK_CONTROL_MASK)))

Possibly, did you see some? This very function was fine because it only checked 
whether some modifiers were present, not that they were specific -- and the 
ones checked weren't in the special cases for macos -- but I can totally 
imagine we have issues some other places (although I'd think they'd be 
discovered by now -- your case was a lot more subtle to see given that if your 
test failed the behavior was still very similar)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3835#discussion_r1564509430
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3835/review/1999482...@github.com>

Reply via email to