Merge lp:~matttbe/gtk/949414 into lp:~ubuntu-desktop/gtk/ubuntugtk3

Proposed by Matthieu Baerts
Status: Merged
Merged at revision: 165
Proposed branch: lp:~matttbe/gtk/949414
Merge into: lp:~ubuntu-desktop/gtk/ubuntugtk3
Diff against target: 82 lines (+62/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/git_touchpad_like_mouse.patch (+53/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~matttbe/gtk/949414
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+98353@code.launchpad.net

Description of the change

Hello,

This branch includes a patch (from git) to fix this bug LP: #949414 .
I don't know where I've to push this branch because it seems lp:ubuntu/gtk+3.0 is not up to date :-/
But I'm going to propose a debdiff on the bug report too.

Thank you,

Matt

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks Matthieu, not sure what your "where to push" is about, the merge request location and diff seems fine, the packaging vcs is debian dir only which is easier to handle than full source outdated imports ;-)

review: Approve
Revision history for this message
Matthieu Baerts (matttbe) wrote :

Thank you!

> the packaging vcs is debian dir only which is easier to handle than full source outdated imports ;-)
Yes, I understand :)

> not sure what your "where to push" is about, the merge request location and diff seems fine
It's just because I used to modify the content of a branch like "lp:ubuntu/PROGRAM" and I didn't know if I had to change the reviewer (but I guess most people from ubuntu-sponsors team can't push on this branch: lp:~ubuntu-desktop/gtk/ubuntugtk3 ) or to push my modification on a BZR branch linked to "ubuntu/gtk+3-0".

Revision history for this message
Sebastien Bacher (seb128) wrote :

> but I guess most people from ubuntu-sponsors team can't push on this branch: lp:~ubuntu-desktop/gtk/ubuntugtk3

they can, the dev teams is a member of the desktop team to let anyone who has upload rights be able to commit as well ;-)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-03-16 16:00:24 +0000
3+++ debian/changelog 2012-03-20 07:56:21 +0000
4@@ -1,3 +1,11 @@
5+gtk+3.0 (3.3.18-0ubuntu4) precise; urgency=low
6+
7+ * debian/patches/git_touchpad_like_mouse.patch:
8+ - git patch, a touchpad device should behave like a mouse and not like a
9+ touchscreen (lp: #949414)
10+
11+ -- Matthieu Baerts (matttbe) <matttbe@gmail.com> Tue, 20 Mar 2012 07:35:57 +0100
12+
13 gtk+3.0 (3.3.18-0ubuntu3) precise; urgency=low
14
15 * debian/patches/043_ubuntu_menu_proxy.patch:
16
17=== added file 'debian/patches/git_touchpad_like_mouse.patch'
18--- debian/patches/git_touchpad_like_mouse.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/git_touchpad_like_mouse.patch 2012-03-20 07:56:21 +0000
20@@ -0,0 +1,53 @@
21+From bd55519f7e6bb5da2fdf6f5e99bd79213059f81b Mon Sep 17 00:00:00 2001
22+From: Benjamin Otte <otte@redhat.com>
23+Date: Tue, 20 Mar 2012 01:04:22 +0000
24+Subject: gdk: A TOUCHPAD device behaves like a mouse
25+
26+and not like a TOUCHSCREEN. So treat it like that.
27+
28+https://bugzilla.gnome.org/show_bug.cgi?id=672009
29+---
30+diff --git a/gdk/gdkwindow.c b/gdk/gdkwindow.c
31+index 72f0a18..4ae08ca 100644
32+--- a/gdk/gdkwindow.c
33++++ b/gdk/gdkwindow.c
34+@@ -9271,7 +9271,6 @@ proxy_pointer_event (GdkDisplay *display,
35+
36+ if (pointer_info->need_touch_press_enter &&
37+ gdk_device_get_source (pointer_info->last_slave) != GDK_SOURCE_TOUCHSCREEN &&
38+- gdk_device_get_source (pointer_info->last_slave) != GDK_SOURCE_TOUCHPAD &&
39+ (source_event->type != GDK_TOUCH_UPDATE ||
40+ _gdk_event_get_pointer_emulated (source_event)))
41+ {
42+@@ -9682,8 +9681,7 @@ proxy_button_event (GdkEvent *source_event,
43+ * which synthesized a leave notify event, so synthesize another enter
44+ * notify to tell the pointer is on the window.
45+ */
46+- if (gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN ||
47+- gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHPAD)
48++ if (gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN)
49+ mode = GDK_CROSSING_TOUCH_BEGIN;
50+ else
51+ mode = GDK_CROSSING_DEVICE_SWITCH;
52+@@ -9743,8 +9741,7 @@ proxy_button_event (GdkEvent *source_event,
53+ (type == GDK_TOUCH_END &&
54+ _gdk_event_get_pointer_emulated (source_event))) &&
55+ pointer_window == pointer_info->window_under_pointer &&
56+- (gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN ||
57+- gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHPAD))
58++ gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN)
59+ {
60+ /* Synthesize a leave notify event
61+ * whenever a touch device is released
62+@@ -9780,8 +9777,7 @@ proxy_button_event (GdkEvent *source_event,
63+ if ((type == GDK_TOUCH_END &&
64+ _gdk_event_get_pointer_emulated (source_event)) &&
65+ pointer_window == pointer_info->window_under_pointer &&
66+- (gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN ||
67+- gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHPAD))
68++ gdk_device_get_source (source_device) == GDK_SOURCE_TOUCHSCREEN)
69+ {
70+ /* Synthesize a leave notify event
71+ * whenever a touch device is released
72+--
73+cgit v0.9.0.2
74
75=== modified file 'debian/patches/series'
76--- debian/patches/series 2012-03-09 17:06:39 +0000
77+++ debian/patches/series 2012-03-20 07:56:21 +0000
78@@ -14,3 +14,4 @@
79 dont_rename_annotation.patch
80 dont_show_numlock_warning.patch
81 git_scroll_reset.patch
82+git_touchpad_like_mouse.patch

Subscribers

People subscribed via source and target branches