Merge lp:~cimi/overlay-scrollbar/fix-951121-with-scroll-wheel into lp:overlay-scrollbar

Proposed by Andrea Cimitan
Status: Merged
Merged at revision: 341
Proposed branch: lp:~cimi/overlay-scrollbar/fix-951121-with-scroll-wheel
Merge into: lp:overlay-scrollbar
Diff against target: 48 lines (+16/-4)
2 files modified
os/os-scrollbar.c (+14/-3)
os/os-thumb.c (+2/-1)
To merge this branch: bzr merge lp:~cimi/overlay-scrollbar/fix-951121-with-scroll-wheel
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
Review via email: mp+98625@code.launchpad.net

Description of the change

A couple of fixes for gtk 3.3.18, and comment the workaround for 951121

To post a comment you must log in.
Revision history for this message
Ted Gould (ted) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'os/os-scrollbar.c'
--- os/os-scrollbar.c 2012-03-13 14:23:10 +0000
+++ os/os-scrollbar.c 2012-03-21 13:16:26 +0000
@@ -2022,9 +2022,12 @@
2022#ifdef USE_GTK32022#ifdef USE_GTK3
2023 /* On touch devices with XI2 and Gtk+ >= 3.3.18,2023 /* On touch devices with XI2 and Gtk+ >= 3.3.18,
2024 * the event enter-notify is not emitted.2024 * the event enter-notify is not emitted.
2025 * Deal with it in motion-notify. */ 2025 * Deal with it in motion-notify. */
2026 if (!(priv->event & OS_EVENT_ENTER_NOTIFY))2026
2027 enter_event (scrollbar);2027 /* Should be fixed with:
2028 * https://bugs.launchpad.net/ubuntu/+source/gtk+3.0/+bug/949414 */
2029 // if (!(priv->event & OS_EVENT_ENTER_NOTIFY))
2030 // enter_event (scrollbar);
2028#endif2031#endif
20292032
2030 if (priv->event & OS_EVENT_BUTTON_PRESS)2033 if (priv->event & OS_EVENT_BUTTON_PRESS)
@@ -2368,6 +2371,14 @@
2368 OsScrollbarPrivate *priv;2371 OsScrollbarPrivate *priv;
2369 gdouble delta;2372 gdouble delta;
23702373
2374#ifdef USE_GTK3
2375 /* Gtk+ 3.3.18 adds a smooth scroll support,
2376 * but at the moment is not ready to be used without various issues.
2377 * Don't use it for thumb scrolling in overlay scrollbar. */
2378 if (event->direction == GDK_SCROLL_SMOOTH)
2379 return FALSE;
2380#endif
2381
2371 scrollbar = OS_SCROLLBAR (user_data);2382 scrollbar = OS_SCROLLBAR (user_data);
2372 priv = scrollbar->priv;2383 priv = scrollbar->priv;
23732384
23742385
=== modified file 'os/os-thumb.c'
--- os/os-thumb.c 2012-03-12 18:45:21 +0000
+++ os/os-thumb.c 2012-03-21 13:16:26 +0000
@@ -209,7 +209,8 @@
209 gtk_widget_add_events (GTK_WIDGET (thumb), GDK_BUTTON_PRESS_MASK |209 gtk_widget_add_events (GTK_WIDGET (thumb), GDK_BUTTON_PRESS_MASK |
210 GDK_BUTTON_RELEASE_MASK |210 GDK_BUTTON_RELEASE_MASK |
211 GDK_POINTER_MOTION_MASK |211 GDK_POINTER_MOTION_MASK |
212 GDK_POINTER_MOTION_HINT_MASK);212 GDK_POINTER_MOTION_HINT_MASK |
213 GDK_SCROLL_MASK);
213214
214 os_thumb_screen_changed (GTK_WIDGET (thumb), NULL);215 os_thumb_screen_changed (GTK_WIDGET (thumb), NULL);
215 os_thumb_composited_changed (GTK_WIDGET (thumb));216 os_thumb_composited_changed (GTK_WIDGET (thumb));

Subscribers

People subscribed via source and target branches