[gtk+/composite-templates-new: 14/32] GtkScaleButton: Define children with a GtkBuilder template
- From: Tristan Van Berkom <tvb src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/composite-templates-new: 14/32] GtkScaleButton: Define children with a GtkBuilder template
- Date: Mon, 8 Apr 2013 08:21:52 +0000 (UTC)
commit 58d7b59aef57768f8b39a9c93184b852915b6e45
Author: Tristan Van Berkom <tristanvb openismus com>
Date: Sat Mar 23 20:17:15 2013 +0900
GtkScaleButton: Define children with a GtkBuilder template
gtk/Makefile.am | 3 +-
gtk/gtk.gresource.xml | 1 +
gtk/gtkscalebutton.c | 170 +++++++++++++++++++-----------------------------
gtk/gtkscalebutton.ui | 101 +++++++++++++++++++++++++++++
gtk/tests/templates.c | 12 ++++
po/POTFILES.in | 1 +
6 files changed, 184 insertions(+), 104 deletions(-)
---
diff --git a/gtk/Makefile.am b/gtk/Makefile.am
index c2d4cb2..1e375af 100644
--- a/gtk/Makefile.am
+++ b/gtk/Makefile.am
@@ -1101,7 +1101,8 @@ COMPOSITE_TEMPLATES = \
gtkdialog.ui \
gtkinfobar.ui \
gtklockbutton.ui \
- gtkmessagedialog.ui
+ gtkmessagedialog.ui \
+ gtkscalebutton.ui
#
# rules to generate built sources
diff --git a/gtk/gtk.gresource.xml b/gtk/gtk.gresource.xml
index fc936e8..856b450 100644
--- a/gtk/gtk.gresource.xml
+++ b/gtk/gtk.gresource.xml
@@ -17,5 +17,6 @@
<file compressed="true">gtkinfobar.ui</file>
<file compressed="true">gtklockbutton.ui</file>
<file compressed="true">gtkmessagedialog.ui</file>
+ <file compressed="true">gtkscalebutton.ui</file>
</gresource>
</gresources>
diff --git a/gtk/gtkscalebutton.c b/gtk/gtkscalebutton.c
index f10bb03..945e8ef 100644
--- a/gtk/gtkscalebutton.c
+++ b/gtk/gtkscalebutton.c
@@ -167,15 +167,27 @@ static void cb_scale_grab_notify (GtkWidget *widget,
static void gtk_scale_button_update_icon (GtkScaleButton *button);
static void gtk_scale_button_scale_value_changed(GtkRange *range);
-/* see below for scale definitions */
-static GtkWidget *gtk_scale_button_scale_new (GtkScaleButton *button);
-
G_DEFINE_TYPE_WITH_CODE (GtkScaleButton, gtk_scale_button, GTK_TYPE_BUTTON,
G_IMPLEMENT_INTERFACE (GTK_TYPE_ORIENTABLE,
NULL))
static guint signals[LAST_SIGNAL] = { 0, };
+/*
+ * ScaleButtonScale forward declarations
+ */
+#define GTK_TYPE_SCALE_BUTTON_SCALE (_gtk_scale_button_scale_get_type ())
+#define GTK_SCALE_BUTTON_SCALE(obj) (G_TYPE_CHECK_INSTANCE_CAST ((obj), GTK_TYPE_SCALE_BUTTON_SCALE,
GtkScaleButtonScale))
+#define GTK_IS_SCALE_BUTTON_SCALE(obj) (G_TYPE_CHECK_INSTANCE_TYPE ((obj), GTK_TYPE_SCALE_BUTTON_SCALE))
+GType _gtk_scale_button_scale_get_type (void);
+
+typedef struct _GtkScaleButtonScale
+{
+ GtkScale parent_instance;
+ GtkScaleButton *button;
+} GtkScaleButtonScale;
+
+
static void
gtk_scale_button_class_init (GtkScaleButtonClass *klass)
{
@@ -345,6 +357,27 @@ gtk_scale_button_class_init (GtkScaleButtonClass *klass)
gtk_binding_entry_add_signal (binding_set, GDK_KEY_Escape, 0,
"popdown", 0);
+ /* Bind class to template
+ */
+ gtk_widget_class_set_template_from_resource (widget_class,
+ "/org/gtk/libgtk/gtkscalebutton.ui");
+
+ gtk_widget_class_bind_child_internal (widget_class, GtkScaleButtonPrivate, plus_button);
+ gtk_widget_class_bind_child_internal (widget_class, GtkScaleButtonPrivate, minus_button);
+ gtk_widget_class_bind_child (widget_class, GtkScaleButtonPrivate, dock);
+ gtk_widget_class_bind_child (widget_class, GtkScaleButtonPrivate, box);
+ gtk_widget_class_bind_child (widget_class, GtkScaleButtonPrivate, scale);
+ gtk_widget_class_bind_child (widget_class, GtkScaleButtonPrivate, image);
+ gtk_widget_class_bind_child (widget_class, GtkScaleButtonPrivate, adjustment);
+
+ gtk_widget_class_bind_callback (widget_class, cb_dock_button_press);
+ gtk_widget_class_bind_callback (widget_class, cb_dock_key_release);
+ gtk_widget_class_bind_callback (widget_class, cb_dock_grab_notify);
+ gtk_widget_class_bind_callback (widget_class, cb_dock_grab_broken_event);
+ gtk_widget_class_bind_callback (widget_class, cb_button_press);
+ gtk_widget_class_bind_callback (widget_class, cb_button_release);
+ gtk_widget_class_bind_callback (widget_class, cb_scale_grab_notify);
+
gtk_widget_class_set_accessible_type (widget_class, GTK_TYPE_SCALE_BUTTON_ACCESSIBLE);
}
@@ -352,7 +385,6 @@ static void
gtk_scale_button_init (GtkScaleButton *button)
{
GtkScaleButtonPrivate *priv;
- GtkWidget *frame;
button->priv = priv = G_TYPE_INSTANCE_GET_PRIVATE (button,
GTK_TYPE_SCALE_BUTTON,
@@ -363,60 +395,16 @@ gtk_scale_button_init (GtkScaleButton *button)
priv->click_timeout = CLICK_TIMEOUT;
priv->orientation = GTK_ORIENTATION_VERTICAL;
- gtk_button_set_relief (GTK_BUTTON (button), GTK_RELIEF_NONE);
- gtk_button_set_focus_on_click (GTK_BUTTON (button), FALSE);
-
- /* image */
- priv->image = gtk_image_new ();
- gtk_container_add (GTK_CONTAINER (button), priv->image);
- gtk_widget_show_all (priv->image);
-
- /* window */
- priv->dock = gtk_window_new (GTK_WINDOW_POPUP);
- gtk_widget_set_name (priv->dock, "gtk-scalebutton-popup-window");
- g_signal_connect (priv->dock, "button-press-event",
- G_CALLBACK (cb_dock_button_press), button);
- g_signal_connect (priv->dock, "key-release-event",
- G_CALLBACK (cb_dock_key_release), button);
- g_signal_connect (priv->dock, "grab-notify",
- G_CALLBACK (cb_dock_grab_notify), button);
- g_signal_connect (priv->dock, "grab-broken-event",
- G_CALLBACK (cb_dock_grab_broken_event), button);
- gtk_window_set_decorated (GTK_WINDOW (priv->dock), FALSE);
-
- /* frame */
- frame = gtk_frame_new (NULL);
- gtk_frame_set_shadow_type (GTK_FRAME (frame), GTK_SHADOW_OUT);
- gtk_container_add (GTK_CONTAINER (priv->dock), frame);
-
- /* box for scale and +/- buttons */
- priv->box = gtk_box_new (GTK_ORIENTATION_VERTICAL, 0);
- gtk_container_add (GTK_CONTAINER (frame), priv->box);
-
- /* + */
- priv->plus_button = gtk_button_new_with_label ("+");
- gtk_button_set_relief (GTK_BUTTON (priv->plus_button), GTK_RELIEF_NONE);
- g_signal_connect (priv->plus_button, "button-press-event",
- G_CALLBACK (cb_button_press), button);
- g_signal_connect (priv->plus_button, "button-release-event",
- G_CALLBACK (cb_button_release), button);
- gtk_box_pack_start (GTK_BOX (priv->box), priv->plus_button, FALSE, FALSE, 0);
-
- /* - */
- priv->minus_button = gtk_button_new_with_label ("-");
- gtk_button_set_relief (GTK_BUTTON (priv->minus_button), GTK_RELIEF_NONE);
- g_signal_connect (priv->minus_button, "button-press-event",
- G_CALLBACK (cb_button_press), button);
- g_signal_connect (priv->minus_button, "button-release-event",
- G_CALLBACK (cb_button_release), button);
- gtk_box_pack_end (GTK_BOX (priv->box), priv->minus_button, FALSE, FALSE, 0);
-
- priv->adjustment = gtk_adjustment_new (0.0, 0.0, 100.0, 2, 20, 0);
- g_object_ref_sink (priv->adjustment);
-
- /* the scale */
- priv->scale = gtk_scale_button_scale_new (button);
- gtk_container_add (GTK_CONTAINER (priv->box), priv->scale);
+ g_type_ensure (GTK_TYPE_SCALE_BUTTON_SCALE);
+ gtk_widget_init_template (GTK_WIDGET (button));
+
+ /* Assign GtkScaleButtonScale pointer back to 'button',
+ * since there's no property for that we can't do it in the template
+ */
+ GTK_SCALE_BUTTON_SCALE (priv->scale)->button = button;
+
+ /* Need a local reference to the adjustment */
+ g_object_ref (priv->adjustment);
gtk_widget_add_events (GTK_WIDGET (button), GDK_SCROLL_MASK);
}
@@ -1402,17 +1390,6 @@ cb_scale_grab_notify (GtkWidget *widget,
/*
* Scale stuff.
*/
-
-#define GTK_TYPE_SCALE_BUTTON_SCALE (_gtk_scale_button_scale_get_type ())
-#define GTK_SCALE_BUTTON_SCALE(obj) (G_TYPE_CHECK_INSTANCE_CAST ((obj), GTK_TYPE_SCALE_BUTTON_SCALE,
GtkScaleButtonScale))
-#define GTK_IS_SCALE_BUTTON_SCALE(obj) (G_TYPE_CHECK_INSTANCE_TYPE ((obj), GTK_TYPE_SCALE_BUTTON_SCALE))
-
-typedef struct _GtkScaleButtonScale
-{
- GtkScale parent_instance;
- GtkScaleButton *button;
-} GtkScaleButtonScale;
-
typedef struct _GtkScaleButtonScaleClass
{
GtkScaleClass parent_class;
@@ -1423,8 +1400,6 @@ static gboolean gtk_scale_button_scale_press (GtkWidget *widget,
static gboolean gtk_scale_button_scale_release (GtkWidget *widget,
GdkEventButton *event);
-GType _gtk_scale_button_scale_get_type (void);
-
G_DEFINE_TYPE (GtkScaleButtonScale, _gtk_scale_button_scale, GTK_TYPE_SCALE)
static void
@@ -1443,42 +1418,19 @@ _gtk_scale_button_scale_init (GtkScaleButtonScale *scale)
{
}
-static GtkWidget *
-gtk_scale_button_scale_new (GtkScaleButton *button)
-{
- GtkScaleButtonPrivate *priv = button->priv;
- GtkScaleButtonScale *scale;
-
- scale = g_object_new (GTK_TYPE_SCALE_BUTTON_SCALE,
- "orientation", priv->orientation,
- "adjustment", priv->adjustment,
- "draw-value", FALSE,
- NULL);
-
- scale->button = button;
-
- g_signal_connect (scale, "grab-notify",
- G_CALLBACK (cb_scale_grab_notify), button);
-
- if (priv->orientation == GTK_ORIENTATION_VERTICAL)
- {
- gtk_widget_set_size_request (GTK_WIDGET (scale), -1, SCALE_SIZE);
- gtk_range_set_inverted (GTK_RANGE (scale), TRUE);
- }
- else
- {
- gtk_widget_set_size_request (GTK_WIDGET (scale), SCALE_SIZE, -1);
- gtk_range_set_inverted (GTK_RANGE (scale), FALSE);
- }
-
- return GTK_WIDGET (scale);
-}
-
static gboolean
gtk_scale_button_scale_press (GtkWidget *widget,
GdkEventButton *event)
{
- GtkScaleButtonPrivate *priv = GTK_SCALE_BUTTON_SCALE (widget)->button->priv;
+ GtkScaleButtonPrivate *priv;
+
+ /* Avoid a crash when using a GtkScaleButtonScale without a GtkScaleButton,
+ * this can happen while editing gtkscalebutton.ui in Glade
+ */
+ if (!GTK_SCALE_BUTTON_SCALE (widget)->button)
+ return GTK_WIDGET_CLASS (_gtk_scale_button_scale_parent_class)->button_release_event (widget, event);
+
+ priv = GTK_SCALE_BUTTON_SCALE (widget)->button->priv;
/* the scale will grab input; if we have input grabbed, all goes
* horribly wrong, so let's not do that.
@@ -1495,6 +1447,12 @@ gtk_scale_button_scale_release (GtkWidget *widget,
GtkScaleButton *button = GTK_SCALE_BUTTON_SCALE (widget)->button;
gboolean res;
+ /* Avoid a crash when using a GtkScaleButtonScale without a GtkScaleButton,
+ * this can happen while editing gtkscalebutton.ui in Glade
+ */
+ if (!button)
+ return GTK_WIDGET_CLASS (_gtk_scale_button_scale_parent_class)->button_release_event (widget, event);
+
if (button->priv->timeout)
{
/* if we did a quick click, leave the window open; else, hide it */
@@ -1604,6 +1562,12 @@ gtk_scale_button_scale_value_changed (GtkRange *range)
GtkScaleButton *button = GTK_SCALE_BUTTON_SCALE (range)->button;
gdouble value;
+ /* Avoid a crash when using a GtkScaleButtonScale without a GtkScaleButton,
+ * this can happen while editing gtkscalebutton.ui in Glade
+ */
+ if (!button)
+ return;
+
value = gtk_range_get_value (range);
gtk_scale_button_update_icon (button);
diff --git a/gtk/gtkscalebutton.ui b/gtk/gtkscalebutton.ui
new file mode 100644
index 0000000..e12b008
--- /dev/null
+++ b/gtk/gtkscalebutton.ui
@@ -0,0 +1,101 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface domain="gtk30">
+ <!-- interface-requires gtk+ 3.10 -->
+ <!-- interface-requires gtkprivate 3.10 -->
+ <template class="GtkScaleButton" parent="GtkButton">
+ <property name="can_focus">True</property>
+ <property name="receives_default">True</property>
+ <property name="relief">none</property>
+ <property name="focus_on_click">False</property>
+ <child>
+ <object class="GtkImage" id="image">
+ <property name="visible">True</property>
+ <property name="can_focus">False</property>
+ <property name="stock">gtk-missing-image</property>
+ </object>
+ </child>
+ </template>
+ <object class="GtkAdjustment" id="adjustment">
+ <property name="upper">100</property>
+ <property name="step_increment">2</property>
+ <property name="page_increment">20</property>
+ </object>
+ <object class="GtkWindow" id="dock">
+ <property name="name">gtk-scalebutton-popup-window</property>
+ <property name="can_focus">False</property>
+ <property name="type">popup</property>
+ <property name="decorated">False</property>
+ <signal name="button-press-event" handler="cb_dock_button_press" swapped="no"/>
+ <signal name="grab-broken-event" handler="cb_dock_grab_broken_event" swapped="no"/>
+ <signal name="grab-notify" handler="cb_dock_grab_notify" swapped="no"/>
+ <signal name="key-release-event" handler="cb_dock_key_release" swapped="no"/>
+ <child>
+ <object class="GtkFrame" id="frame1">
+ <property name="visible">True</property>
+ <property name="can_focus">False</property>
+ <property name="label_xalign">0</property>
+ <property name="shadow_type">out</property>
+ <child>
+ <object class="GtkBox" id="box">
+ <property name="visible">True</property>
+ <property name="can_focus">False</property>
+ <property name="orientation">vertical</property>
+ <child>
+ <object class="GtkButton" id="plus_button">
+ <property name="label">+</property>
+ <property name="visible">True</property>
+ <property name="can_focus">True</property>
+ <property name="receives_default">True</property>
+ <property name="relief">none</property>
+ <signal name="button-press-event" handler="cb_button_press" swapped="no"/>
+ <signal name="button-release-event" handler="cb_button_release" swapped="no"/>
+ </object>
+ <packing>
+ <property name="expand">False</property>
+ <property name="fill">False</property>
+ <property name="position">0</property>
+ </packing>
+ </child>
+ <child>
+ <object class="GtkScaleButtonScale" id="scale">
+ <property name="height_request">100</property>
+ <property name="visible">True</property>
+ <property name="can_focus">True</property>
+ <property name="orientation">vertical</property>
+ <property name="adjustment">adjustment</property>
+ <property name="inverted">True</property>
+ <property name="round_digits">1</property>
+ <property name="draw_value">False</property>
+ <signal name="grab-notify" handler="cb_scale_grab_notify" swapped="no"/>
+ </object>
+ <packing>
+ <property name="expand">False</property>
+ <property name="fill">True</property>
+ <property name="position">1</property>
+ </packing>
+ </child>
+ <child>
+ <object class="GtkButton" id="minus_button">
+ <property name="label">-</property>
+ <property name="visible">True</property>
+ <property name="can_focus">True</property>
+ <property name="receives_default">True</property>
+ <property name="relief">none</property>
+ <signal name="button-press-event" handler="cb_button_press" swapped="no"/>
+ <signal name="button-release-event" handler="cb_button_release" swapped="no"/>
+ </object>
+ <packing>
+ <property name="expand">False</property>
+ <property name="fill">False</property>
+ <property name="position">2</property>
+ </packing>
+ </child>
+ </object>
+ </child>
+ <child type="label_item">
+ <placeholder/>
+ </child>
+ </object>
+ </child>
+ </object>
+</interface>
diff --git a/gtk/tests/templates.c b/gtk/tests/templates.c
index 9faebb9..7cf7c63 100644
--- a/gtk/tests/templates.c
+++ b/gtk/tests/templates.c
@@ -98,6 +98,17 @@ test_assistant_basic (void)
gtk_widget_destroy (widget);
}
+static void
+test_scale_button_basic (void)
+{
+ GtkWidget *widget;
+
+ widget = gtk_scale_button_new (GTK_ICON_SIZE_MENU,
+ 0, 100, 10, NULL);
+ g_assert (GTK_IS_SCALE_BUTTON (widget));
+ gtk_widget_destroy (widget);
+}
+
int
main (int argc, char **argv)
{
@@ -117,6 +128,7 @@ main (int argc, char **argv)
g_test_add_func ("/Template/GtkInfoBar/Basic", test_info_bar_basic);
g_test_add_func ("/Template/GtkLockButton/Basic", test_lock_button_basic);
g_test_add_func ("/Template/GtkAssistant/Basic", test_assistant_basic);
+ g_test_add_func ("/Template/GtkScaleButton/Basic", test_scale_button_basic);
return g_test_run();
}
diff --git a/po/POTFILES.in b/po/POTFILES.in
index 6732b69..1f9b2f8 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -280,3 +280,4 @@ modules/printbackends/test/gtkprintbackendtest.c
[type: gettext/glade]gtk/gtkinfobar.ui
[type: gettext/glade]gtk/gtklockbutton.ui
[type: gettext/glade]gtk/gtkmessagedialog.ui
+[type: gettext/glade]gtk/gtkscalebutton.ui
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
