[Github-comments] [geany/geany] Fix horizontal and page scroll on GTK3 (#1843)

Colomban Wendling notifications at xxxxx
Tue Apr 24 22:20:55 UTC 2018


Our custom scroll handler for horizontal (Shift+Scroll) and page (Alt+Scroll) scroll didn't properly check the scroll direction and assume that if it's not down it's up.  This was mostly not a problem because the other types only were left and right scroll events, which are a lot less common.

However, it became a lot more problematic with GTK 3.4 that introduced "smooth scrolling", and thus a new scroll type that can happen for events in any direction.  We then would scroll up (as we assume "not down" is up) regardless of the actual direction of the event.

It's still not clear why we'd get smooth scroll events on X11 as no code I can find asks for it and we generally don't get those, but sometimes a Scintilla widget starts receiving them, leading to the bug. On Wayland on the other hand, Scintilla asks for smooth scroll events, so we need to have a fix for it in any case.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/1843

-- Commit Summary --

  * Fix horizontal and page scroll on GTK3

-- File Changes --

    M src/editor.c (4)

-- Patch Links --

https://github.com/geany/geany/pull/1843.patch
https://github.com/geany/geany/pull/1843.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1843
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20180424/611e54ab/attachment.html>


More information about the Github-comments mailing list