Merge lp:~gerardo-santana/gnome-control-center/gnome-control-center into lp:gnome-control-center

Proposed by Gerardo Santana
Status: Rejected
Rejected by: William Grant
Proposed branch: lp:~gerardo-santana/gnome-control-center/gnome-control-center
Merge into: lp:gnome-control-center
Diff against target: 86 lines (+51/-0)
2 files modified
panels/mouse/gnome-mouse-properties.c (+7/-0)
panels/mouse/gnome-mouse-properties.ui (+44/-0)
To merge this branch: bzr merge lp:~gerardo-santana/gnome-control-center/gnome-control-center
Reviewer Review Type Date Requested Status
Sebastien Bacher (community) Disapprove
Bastien Nocera Pending
VCS imports Pending
Review via email: mp+109335@code.launchpad.net

Description of the change

adds a checkbox for "Enable natural scrolling" under "Enable horizontal scrolling" in Touchpad tab

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

Thank you for taking the time to report this bug and helping to make Ubuntu better. The issue you are reporting is an upstream one and it would be nice if somebody having it could send the bug to the developers of the software by following the instructions at https://wiki.ubuntu.com/Bugs/Upstream/GNOME. If you have done so, please tell us the number of the upstream bug (or the link), so we can add a bugwatch that will inform us about its status. Thanks in advance.

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

the current version has a checkbox for natural scrolling, the patch would need to be redone to add horizontal scrolling if that's still something wanted.

Closing since it was never forwarded upstream as asked in the previous comment

review: Disapprove

Unmerged revisions

11812. By Gerardo Santana

Enables natural scrolling setting

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'panels/mouse/gnome-mouse-properties.c'
2--- panels/mouse/gnome-mouse-properties.c 2012-03-13 18:12:47 +0000
3+++ panels/mouse/gnome-mouse-properties.c 2012-06-08 11:54:44 +0000
4@@ -174,6 +174,8 @@
5
6 gtk_widget_set_sensitive (WID ("horiz_scroll_toggle"),
7 !gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (WID ("scroll_disabled_radio"))));
8+ gtk_widget_set_sensitive (WID ("natural_scroll_toggle"),
9+ !gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (WID ("scroll_disabled_radio"))));
10 }
11
12 static void
13@@ -187,6 +189,8 @@
14
15 gtk_widget_set_sensitive (WID ("horiz_scroll_toggle"),
16 !gtk_toggle_button_get_active (disabled));
17+ gtk_widget_set_sensitive (WID ("natural_scroll_toggle"),
18+ !gtk_toggle_button_get_active (disabled));
19
20 if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (WID ("scroll_edge_radio"))))
21 method = GSD_TOUCHPAD_SCROLL_METHOD_EDGE_SCROLLING;
22@@ -304,6 +308,9 @@
23 g_settings_bind (touchpad_settings, "horiz-scroll-enabled",
24 WID ("horiz_scroll_toggle"), "active",
25 G_SETTINGS_BIND_DEFAULT);
26+ g_settings_bind (touchpad_settings, "natural-scroll-enabled",
27+ WID ("natural_scroll_toggle"), "active",
28+ G_SETTINGS_BIND_DEFAULT);
29 g_settings_bind (touchpad_settings, "motion-acceleration",
30 gtk_range_get_adjustment (GTK_RANGE (WID ("touchpad_accel_scale"))), "value",
31 G_SETTINGS_BIND_DEFAULT);
32
33=== modified file 'panels/mouse/gnome-mouse-properties.ui'
34--- panels/mouse/gnome-mouse-properties.ui 2012-04-22 11:57:19 +0000
35+++ panels/mouse/gnome-mouse-properties.ui 2012-06-08 11:54:44 +0000
36@@ -1087,6 +1087,50 @@
37 <property name="position">2</property>
38 </packing>
39 </child>
40+ <child>
41+ <object class="GtkHBox" id="hbox17">
42+ <property name="visible">True</property>
43+ <property name="can_focus">False</property>
44+ <property name="events">GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK</property>
45+ <child>
46+ <object class="GtkLabel" id="label37">
47+ <property name="visible">True</property>
48+ <property name="can_focus">False</property>
49+ <property name="events">GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK</property>
50+ <property name="xalign">0</property>
51+ <property name="yalign">0</property>
52+ <property name="label"> </property>
53+ </object>
54+ <packing>
55+ <property name="expand">False</property>
56+ <property name="fill">False</property>
57+ <property name="position">0</property>
58+ </packing>
59+ </child>
60+ <child>
61+ <object class="GtkCheckButton" id="natural_scroll_toggle">
62+ <property name="label" translatable="yes">Enable _natural scrolling</property>
63+ <property name="visible">True</property>
64+ <property name="can_focus">True</property>
65+ <property name="receives_default">False</property>
66+ <property name="events">GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK</property>
67+ <property name="use_action_appearance">False</property>
68+ <property name="use_underline">True</property>
69+ <property name="draw_indicator">True</property>
70+ </object>
71+ <packing>
72+ <property name="expand">False</property>
73+ <property name="fill">False</property>
74+ <property name="position">1</property>
75+ </packing>
76+ </child>
77+ </object>
78+ <packing>
79+ <property name="expand">False</property>
80+ <property name="fill">False</property>
81+ <property name="position">2</property>
82+ </packing>
83+ </child>
84 </object>
85 <packing>
86 <property name="expand">False</property>

Subscribers

People subscribed via source and target branches

to all changes: