Merge lp:~larsu/ido/remove-slider-hack-953757 into lp:ido/12.10

Proposed by Lars Karlitski
Status: Merged
Merge reported by: Lars Karlitski
Merged at revision: not available
Proposed branch: lp:~larsu/ido/remove-slider-hack-953757
Merge into: lp:ido/12.10
Diff against target: 55 lines (+6/-19) (has conflicts)
2 files modified
src/idorange.c (+2/-0)
src/idoscalemenuitem.c (+4/-19)
Text conflict in src/idoscalemenuitem.c
To merge this branch: bzr merge lp:~larsu/ido/remove-slider-hack-953757
Reviewer Review Type Date Requested Status
jenkins (community) continuous-integration Needs Fixing
Charles Kerr (community) Approve
Review via email: mp+121486@code.launchpad.net

Description of the change

It's an old branch that still works, albeit with a minor merge conflict. I'll merge it manually when someone approves this.

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Looks good. Nice catch on ->constructed

review: Approve
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

FAILED: Autolanding.
No commit message was specified.
http://jenkins.qa.ubuntu.com/job/ido-autolanding/7/

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/idorange.c'
2--- src/idorange.c 2012-05-08 00:18:10 +0000
3+++ src/idorange.c 2012-08-27 19:56:23 +0000
4@@ -156,6 +156,8 @@
5 }
6
7 gtk_range_set_slider_size_fixed (GTK_RANGE (range), TRUE);
8+
9+ G_OBJECT_CLASS (ido_range_parent_class)->constructed (object);
10 }
11
12 static void
13
14=== modified file 'src/idoscalemenuitem.c'
15--- src/idoscalemenuitem.c 2012-05-08 00:18:10 +0000
16+++ src/idoscalemenuitem.c 2012-08-27 19:56:23 +0000
17@@ -205,21 +205,6 @@
18 priv->toggle_size = toggle_size;
19 }
20
21-static gboolean
22-on_scale_button_press_or_release_event (GtkWidget * widget G_GNUC_UNUSED,
23- GdkEventButton * event,
24- gpointer unused G_GNUC_UNUSED)
25-{
26- /* HACK: we want the behaviour you get with the middle button, so we
27- * mangle the event. clicking with other buttons moves the slider in
28- * step increments, clicking with the middle button moves the slider to
29- * the location of the click. */
30- if (event->button == 1)
31- event->button = 2;
32-
33- return FALSE;
34-}
35-
36 static void
37 ido_scale_menu_item_constructed (GObject *object)
38 {
39@@ -238,12 +223,12 @@
40 priv->scale = ido_range_new (adj, range_style);
41 g_object_ref (priv->scale);
42 gtk_scale_set_draw_value (GTK_SCALE (priv->scale), FALSE);
43- g_signal_connect (G_OBJECT (priv->scale), "button-press-event",
44- G_CALLBACK (on_scale_button_press_or_release_event), NULL);
45- g_signal_connect (G_OBJECT (priv->scale), "button-release-event",
46- G_CALLBACK (on_scale_button_press_or_release_event), NULL);
47
48+<<<<<<< TREE
49
50+=======
51+#ifdef USE_GTK3
52+>>>>>>> MERGE-SOURCE
53 hbox = gtk_box_new (GTK_ORIENTATION_HORIZONTAL, 0);
54
55 priv->primary_image = gtk_image_new ();

Subscribers

People subscribed via source and target branches