]> Pileus Git - ~andy/gtk/blobdiff - gtk/gtkspinbutton.c
Deprecate widget flag: GTK_WIDGET_MAPPED
[~andy/gtk] / gtk / gtkspinbutton.c
index 3eca302a8dcc3e63688fcdf22fc3b76742c6f7be..315adc4e4840aadfbcb1c036b8ba526d2aeab13a 100644 (file)
  * GTK+ at ftp://ftp.gtk.org/pub/gtk/. 
  */
 
+#include "config.h"
 #include <stdio.h>
 #include <stdlib.h>
 #include <math.h>
 #include <string.h>
 #include <locale.h>
 #include "gdk/gdkkeysyms.h"
+#include "gtkbindings.h"
 #include "gtkspinbutton.h"
+#include "gtkentryprivate.h"
 #include "gtkmain.h"
-#include "gtksignal.h"
+#include "gtkmarshalers.h"
 #include "gtksettings.h"
+#include "gtkprivate.h"
 #include "gtkintl.h"
+#include "gtkalias.h"
 
-#define MIN_SPIN_BUTTON_WIDTH              30
-#define ARROW_SIZE                         11
-#define SPIN_BUTTON_INITIAL_TIMER_DELAY    200
-#define SPIN_BUTTON_TIMER_DELAY            20
-#define MAX_TIMER_CALLS                    5
-#define EPSILON                            1e-5
-#define        MAX_DIGITS                         20
+#define MIN_SPIN_BUTTON_WIDTH 30
+#define MAX_TIMER_CALLS       5
+#define EPSILON               1e-10
+#define        MAX_DIGITS            20
+#define MIN_ARROW_WIDTH       6
 
 enum {
   PROP_0,
@@ -65,12 +68,14 @@ enum
   INPUT,
   OUTPUT,
   VALUE_CHANGED,
+  CHANGE_VALUE,
+  WRAPPED,
   LAST_SIGNAL
 };
 
-static void gtk_spin_button_class_init     (GtkSpinButtonClass *klass);
-static void gtk_spin_button_init           (GtkSpinButton      *spin_button);
+static void gtk_spin_button_editable_init  (GtkEditableClass   *iface);
 static void gtk_spin_button_finalize       (GObject            *object);
+static void gtk_spin_button_destroy        (GtkObject          *object);
 static void gtk_spin_button_set_property   (GObject         *object,
                                            guint            prop_id,
                                            const GValue    *value,
@@ -101,79 +106,76 @@ static gint gtk_spin_button_leave_notify   (GtkWidget          *widget,
                                            GdkEventCrossing   *event);
 static gint gtk_spin_button_focus_out      (GtkWidget          *widget,
                                            GdkEventFocus      *event);
+static void gtk_spin_button_grab_notify    (GtkWidget          *widget,
+                                           gboolean            was_grabbed);
+static void gtk_spin_button_state_changed  (GtkWidget          *widget,
+                                           GtkStateType        previous_state);
+static void gtk_spin_button_style_set      (GtkWidget          *widget,
+                                            GtkStyle           *previous_style);
 static void gtk_spin_button_draw_arrow     (GtkSpinButton      *spin_button, 
-                                           guint               arrow);
-static gint gtk_spin_button_timer          (GtkSpinButton      *spin_button);
+                                           GdkRectangle       *area,
+                                           GtkArrowType        arrow_type);
+static gboolean gtk_spin_button_timer          (GtkSpinButton      *spin_button);
+static void gtk_spin_button_stop_spinning  (GtkSpinButton      *spin);
 static void gtk_spin_button_value_changed  (GtkAdjustment      *adjustment,
                                            GtkSpinButton      *spin_button); 
-static gint gtk_spin_button_key_press      (GtkWidget          *widget,
-                                           GdkEventKey        *event);
 static gint gtk_spin_button_key_release    (GtkWidget          *widget,
                                            GdkEventKey        *event);
 static gint gtk_spin_button_scroll         (GtkWidget          *widget,
                                            GdkEventScroll     *event);
 static void gtk_spin_button_activate       (GtkEntry           *entry);
+static void gtk_spin_button_get_text_area_size (GtkEntry *entry,
+                                               gint     *x,
+                                               gint     *y,
+                                               gint     *width,
+                                               gint     *height);
 static void gtk_spin_button_snap           (GtkSpinButton      *spin_button,
                                            gdouble             val);
-static void gtk_spin_button_insert_text    (GtkEntry           *entry,
+static void gtk_spin_button_insert_text    (GtkEditable        *editable,
                                            const gchar        *new_text,
                                            gint                new_text_length,
                                            gint               *position);
 static void gtk_spin_button_real_spin      (GtkSpinButton      *spin_button,
                                            gdouble             step);
+static void gtk_spin_button_real_change_value (GtkSpinButton   *spin,
+                                              GtkScrollType    scroll);
+
 static gint gtk_spin_button_default_input  (GtkSpinButton      *spin_button,
                                            gdouble            *new_val);
 static gint gtk_spin_button_default_output (GtkSpinButton      *spin_button);
-static gint spin_button_get_shadow_type    (GtkSpinButton      *spin_button);
 
+static gint spin_button_get_arrow_size     (GtkSpinButton      *spin_button);
+static gint spin_button_get_shadow_type    (GtkSpinButton      *spin_button);
 
-static GtkEntryClass *parent_class = NULL;
 static guint spinbutton_signals[LAST_SIGNAL] = {0};
 
+#define NO_ARROW 2
 
-GtkType
-gtk_spin_button_get_type (void)
-{
-  static guint spin_button_type = 0;
+G_DEFINE_TYPE_WITH_CODE (GtkSpinButton, gtk_spin_button, GTK_TYPE_ENTRY,
+                        G_IMPLEMENT_INTERFACE (GTK_TYPE_EDITABLE,
+                                               gtk_spin_button_editable_init))
 
-  if (!spin_button_type)
-    {
-      static const GtkTypeInfo spin_button_info =
-      {
-       "GtkSpinButton",
-       sizeof (GtkSpinButton),
-       sizeof (GtkSpinButtonClass),
-       (GtkClassInitFunc) gtk_spin_button_class_init,
-       (GtkObjectInitFunc) gtk_spin_button_init,
-       /* reserved_1 */ NULL,
-        /* reserved_2 */ NULL,
-        (GtkClassInitFunc) NULL,
-      };
-
-      spin_button_type = gtk_type_unique (GTK_TYPE_ENTRY, &spin_button_info);
-    }
-  return spin_button_type;
-}
+#define add_spin_binding(binding_set, keyval, mask, scroll)            \
+  gtk_binding_entry_add_signal (binding_set, keyval, mask,             \
+                                "change_value", 1,                     \
+                                GTK_TYPE_SCROLL_TYPE, scroll)
 
 static void
 gtk_spin_button_class_init (GtkSpinButtonClass *class)
 {
   GObjectClass     *gobject_class = G_OBJECT_CLASS (class);
-  GtkObjectClass   *object_class;
-  GtkWidgetClass   *widget_class;
-  GtkEntryClass    *entry_class;
-
-  object_class   = (GtkObjectClass*)   class;
-  widget_class   = (GtkWidgetClass*)   class;
-  entry_class    = (GtkEntryClass*)    class; 
-
-  parent_class = gtk_type_class (GTK_TYPE_ENTRY);
+  GtkObjectClass   *object_class = GTK_OBJECT_CLASS (class);
+  GtkWidgetClass   *widget_class = GTK_WIDGET_CLASS (class);
+  GtkEntryClass    *entry_class = GTK_ENTRY_CLASS (class);
+  GtkBindingSet    *binding_set;
 
   gobject_class->finalize = gtk_spin_button_finalize;
 
   gobject_class->set_property = gtk_spin_button_set_property;
   gobject_class->get_property = gtk_spin_button_get_property;
 
+  object_class->destroy = gtk_spin_button_destroy;
+
   widget_class->map = gtk_spin_button_map;
   widget_class->unmap = gtk_spin_button_unmap;
   widget_class->realize = gtk_spin_button_realize;
@@ -185,119 +187,201 @@ gtk_spin_button_class_init (GtkSpinButtonClass *class)
   widget_class->button_press_event = gtk_spin_button_button_press;
   widget_class->button_release_event = gtk_spin_button_button_release;
   widget_class->motion_notify_event = gtk_spin_button_motion_notify;
-  widget_class->key_press_event = gtk_spin_button_key_press;
   widget_class->key_release_event = gtk_spin_button_key_release;
   widget_class->enter_notify_event = gtk_spin_button_enter_notify;
   widget_class->leave_notify_event = gtk_spin_button_leave_notify;
   widget_class->focus_out_event = gtk_spin_button_focus_out;
+  widget_class->grab_notify = gtk_spin_button_grab_notify;
+  widget_class->state_changed = gtk_spin_button_state_changed;
+  widget_class->style_set = gtk_spin_button_style_set;
 
-  entry_class->insert_text = gtk_spin_button_insert_text;
   entry_class->activate = gtk_spin_button_activate;
+  entry_class->get_text_area_size = gtk_spin_button_get_text_area_size;
 
   class->input = NULL;
   class->output = NULL;
+  class->change_value = gtk_spin_button_real_change_value;
 
   g_object_class_install_property (gobject_class,
                                    PROP_ADJUSTMENT,
                                    g_param_spec_object ("adjustment",
-                                                        _("Adjustment"),
-                                                        _("The adjustment that holds the value of the spinbutton"),
+                                                        P_("Adjustment"),
+                                                        P_("The adjustment that holds the value of the spinbutton"),
                                                         GTK_TYPE_ADJUSTMENT,
-                                                        G_PARAM_READWRITE));
+                                                        GTK_PARAM_READWRITE));
   
   g_object_class_install_property (gobject_class,
                                    PROP_CLIMB_RATE,
-                                   g_param_spec_double ("climb_rate",
-                                                       _("Climb Rate"),
-                                                       _("The acceleration rate when you hold down a button"),
+                                   g_param_spec_double ("climb-rate",
+                                                       P_("Climb Rate"),
+                                                       P_("The acceleration rate when you hold down a button"),
                                                        0.0,
                                                        G_MAXDOUBLE,
                                                        0.0,
-                                                       G_PARAM_READWRITE));  
+                                                       GTK_PARAM_READWRITE));  
   
   g_object_class_install_property (gobject_class,
                                    PROP_DIGITS,
                                    g_param_spec_uint ("digits",
-                                                     _("Digits"),
-                                                     _("The number of decimal places to display"),
+                                                     P_("Digits"),
+                                                     P_("The number of decimal places to display"),
                                                      0,
                                                      MAX_DIGITS,
                                                      0,
-                                                     G_PARAM_READWRITE));
+                                                     GTK_PARAM_READWRITE));
   
   g_object_class_install_property (gobject_class,
                                    PROP_SNAP_TO_TICKS,
-                                   g_param_spec_boolean ("snap_to_ticks",
-                                                        _("Snap to Ticks"),
-                                                        _("Whether erroneous values are automatically changed to a spin button's nearest step increment"),
+                                   g_param_spec_boolean ("snap-to-ticks",
+                                                        P_("Snap to Ticks"),
+                                                        P_("Whether erroneous values are automatically changed to a spin button's nearest step increment"),
                                                         FALSE,
-                                                        G_PARAM_READWRITE));
+                                                        GTK_PARAM_READWRITE));
   
   g_object_class_install_property (gobject_class,
                                    PROP_NUMERIC,
                                    g_param_spec_boolean ("numeric",
-                                                        _("Numeric"),
-                                                        _("Whether non-numeric characters should be ignored"),
+                                                        P_("Numeric"),
+                                                        P_("Whether non-numeric characters should be ignored"),
                                                         FALSE,
-                                                        G_PARAM_READWRITE));
+                                                        GTK_PARAM_READWRITE));
   
   g_object_class_install_property (gobject_class,
                                    PROP_WRAP,
                                    g_param_spec_boolean ("wrap",
-                                                        _("Wrap"),
-                                                        _("Whether a spin button should wrap upon reaching its limits"),
+                                                        P_("Wrap"),
+                                                        P_("Whether a spin button should wrap upon reaching its limits"),
                                                         FALSE,
-                                                        G_PARAM_READWRITE));
+                                                        GTK_PARAM_READWRITE));
   
   g_object_class_install_property (gobject_class,
                                    PROP_UPDATE_POLICY,
-                                   g_param_spec_enum ("update_policy",
-                                                     _("Update Policy"),
-                                                     _("Whether the spin button should update always, or only when the value is legal"),
+                                   g_param_spec_enum ("update-policy",
+                                                     P_("Update Policy"),
+                                                     P_("Whether the spin button should update always, or only when the value is legal"),
                                                      GTK_TYPE_SPIN_BUTTON_UPDATE_POLICY,
                                                      GTK_UPDATE_ALWAYS,
-                                                     G_PARAM_READWRITE));
+                                                     GTK_PARAM_READWRITE));
   
   g_object_class_install_property (gobject_class,
                                    PROP_VALUE,
                                    g_param_spec_double ("value",
-                                                       _("Value"),
-                                                       _("Reads the current value, or sets a new value"),
+                                                       P_("Value"),
+                                                       P_("Reads the current value, or sets a new value"),
                                                        -G_MAXDOUBLE,
                                                        G_MAXDOUBLE,
                                                        0.0,
-                                                       G_PARAM_READWRITE));  
+                                                       GTK_PARAM_READWRITE));  
   
   gtk_widget_class_install_style_property_parser (widget_class,
-                                                 g_param_spec_enum ("shadow_type", "Shadow Type", NULL,
+                                                 g_param_spec_enum ("shadow-type", 
+                                                                    "Shadow Type", 
+                                                                    P_("Style of bevel around the spin button"),
                                                                     GTK_TYPE_SHADOW_TYPE,
                                                                     GTK_SHADOW_IN,
-                                                                    G_PARAM_READABLE),
+                                                                    GTK_PARAM_READABLE),
                                                  gtk_rc_property_parse_enum);
   spinbutton_signals[INPUT] =
-    gtk_signal_new ("input",
-                   GTK_RUN_LAST,
-                   GTK_CLASS_TYPE (object_class),
-                   GTK_SIGNAL_OFFSET (GtkSpinButtonClass, input),
-                   gtk_marshal_INT__POINTER,
-                   GTK_TYPE_INT, 1, GTK_TYPE_POINTER);
-
+    g_signal_new (I_("input"),
+                 G_TYPE_FROM_CLASS (gobject_class),
+                 G_SIGNAL_RUN_LAST,
+                 G_STRUCT_OFFSET (GtkSpinButtonClass, input),
+                 NULL, NULL,
+                 _gtk_marshal_INT__POINTER,
+                 G_TYPE_INT, 1,
+                 G_TYPE_POINTER);
+
+  /**
+   * GtkSpinButton::output:
+   * @spin_button: the object which received the signal
+   * 
+   * The ::output signal can be used to change to formatting
+   * of the value that is displayed in the spin buttons entry.
+   * |[
+   * /&ast; show leading zeros &ast;/
+   * static gboolean
+   * on_output (GtkSpinButton *spin,
+   *            gpointer       data)
+   * {
+   *    GtkAdjustment *adj;
+   *    gchar *text;
+   *    int value;
+   *    
+   *    adj = gtk_spin_button_get_adjustment (spin);
+   *    value = (int)gtk_adjustment_get_value (adj);
+   *    text = g_strdup_printf ("%02d", value);
+   *    gtk_entry_set_text (GTK_ENTRY (spin), text);
+   *    g_free (text);
+   *    
+   *    return TRUE;
+   * }
+   * ]|
+   *
+   * Returns: %TRUE if the value has been displayed.
+   */
   spinbutton_signals[OUTPUT] =
-    g_signal_new ("output",
-                  G_TYPE_FROM_CLASS(object_class),
-                  G_SIGNAL_RUN_LAST,
-                  G_STRUCT_OFFSET(GtkSpinButtonClass, output),
-                  _gtk_boolean_handled_accumulator, NULL,
-                  gtk_marshal_BOOLEAN__VOID,
-                  G_TYPE_BOOLEAN, 0);
+    g_signal_new (I_("output"),
+                 G_TYPE_FROM_CLASS (gobject_class),
+                 G_SIGNAL_RUN_LAST,
+                 G_STRUCT_OFFSET (GtkSpinButtonClass, output),
+                 _gtk_boolean_handled_accumulator, NULL,
+                 _gtk_marshal_BOOLEAN__VOID,
+                 G_TYPE_BOOLEAN, 0);
 
   spinbutton_signals[VALUE_CHANGED] =
-    gtk_signal_new ("value_changed",
-                   GTK_RUN_LAST,
-                   GTK_CLASS_TYPE (object_class),
-                   GTK_SIGNAL_OFFSET (GtkSpinButtonClass, value_changed),
-                   gtk_marshal_VOID__VOID,
-                   GTK_TYPE_NONE, 0);
+    g_signal_new (I_("value-changed"),
+                 G_TYPE_FROM_CLASS (gobject_class),
+                 G_SIGNAL_RUN_LAST,
+                 G_STRUCT_OFFSET (GtkSpinButtonClass, value_changed),
+                 NULL, NULL,
+                 _gtk_marshal_VOID__VOID,
+                 G_TYPE_NONE, 0);
+
+  /**
+   * GtkSpinButton::wrapped:
+   * @spinbutton: the object which received the signal
+   *
+   * The wrapped signal is emitted right after the spinbutton wraps
+   * from its maximum to minimum value or vice-versa.
+   *
+   * Since: 2.10
+   */
+  spinbutton_signals[WRAPPED] =
+    g_signal_new (I_("wrapped"),
+                 G_TYPE_FROM_CLASS (gobject_class),
+                 G_SIGNAL_RUN_LAST,
+                 G_STRUCT_OFFSET (GtkSpinButtonClass, wrapped),
+                 NULL, NULL,
+                 _gtk_marshal_VOID__VOID,
+                 G_TYPE_NONE, 0);
+
+  /* Action signals */
+  spinbutton_signals[CHANGE_VALUE] =
+    g_signal_new (I_("change-value"),
+                  G_TYPE_FROM_CLASS (gobject_class),
+                  G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+                  G_STRUCT_OFFSET (GtkSpinButtonClass, change_value),
+                  NULL, NULL,
+                  _gtk_marshal_VOID__ENUM,
+                  G_TYPE_NONE, 1,
+                  GTK_TYPE_SCROLL_TYPE);
+  
+  binding_set = gtk_binding_set_by_class (class);
+  
+  add_spin_binding (binding_set, GDK_Up, 0, GTK_SCROLL_STEP_UP);
+  add_spin_binding (binding_set, GDK_KP_Up, 0, GTK_SCROLL_STEP_UP);
+  add_spin_binding (binding_set, GDK_Down, 0, GTK_SCROLL_STEP_DOWN);
+  add_spin_binding (binding_set, GDK_KP_Down, 0, GTK_SCROLL_STEP_DOWN);
+  add_spin_binding (binding_set, GDK_Page_Up, 0, GTK_SCROLL_PAGE_UP);
+  add_spin_binding (binding_set, GDK_Page_Down, 0, GTK_SCROLL_PAGE_DOWN);
+  add_spin_binding (binding_set, GDK_Page_Up, GDK_CONTROL_MASK, GTK_SCROLL_END);
+  add_spin_binding (binding_set, GDK_Page_Down, GDK_CONTROL_MASK, GTK_SCROLL_START);
+}
+
+static void
+gtk_spin_button_editable_init (GtkEditableClass *iface)
+{
+  iface->insert_text = gtk_spin_button_insert_text;
 }
 
 static void
@@ -306,10 +390,8 @@ gtk_spin_button_set_property (GObject      *object,
                              const GValue *value,
                              GParamSpec   *pspec)
 {
-  GtkSpinButton *spin_button;
+  GtkSpinButton *spin_button = GTK_SPIN_BUTTON (object);
 
-  spin_button = GTK_SPIN_BUTTON (object);
-  
   switch (prop_id)
     {
       GtkAdjustment *adjustment;
@@ -348,6 +430,7 @@ gtk_spin_button_set_property (GObject      *object,
       gtk_spin_button_set_value (spin_button, g_value_get_double (value));
       break;
     default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
     }
 }
@@ -358,14 +441,12 @@ gtk_spin_button_get_property (GObject      *object,
                              GValue       *value,
                              GParamSpec   *pspec)
 {
-  GtkSpinButton *spin_button;
+  GtkSpinButton *spin_button = GTK_SPIN_BUTTON (object);
 
-  spin_button = GTK_SPIN_BUTTON (object);
-  
   switch (prop_id)
     {
     case PROP_ADJUSTMENT:
-      g_value_set_object (value, G_OBJECT (spin_button->adjustment));
+      g_value_set_object (value, spin_button->adjustment);
       break;
     case PROP_CLIMB_RATE:
       g_value_set_double (value, spin_button->climb_rate);
@@ -400,12 +481,11 @@ gtk_spin_button_init (GtkSpinButton *spin_button)
   spin_button->adjustment = NULL;
   spin_button->panel = NULL;
   spin_button->timer = 0;
-  spin_button->ev_time = 0;
   spin_button->climb_rate = 0.0;
   spin_button->timer_step = 0.0;
   spin_button->update_policy = GTK_UPDATE_ALWAYS;
-  spin_button->in_child = 2;
-  spin_button->click_child = 2;
+  spin_button->in_child = NO_ARROW;
+  spin_button->click_child = NO_ARROW;
   spin_button->button = 0;
   spin_button->need_timer = FALSE;
   spin_button->timer_calls = 0;
@@ -413,6 +493,7 @@ gtk_spin_button_init (GtkSpinButton *spin_button)
   spin_button->numeric = FALSE;
   spin_button->wrap = FALSE;
   spin_button->snap_to_ticks = FALSE;
+
   gtk_spin_button_set_adjustment (spin_button,
          (GtkAdjustment*) gtk_adjustment_new (0, 0, 0, 0, 0, 0));
 }
@@ -420,22 +501,25 @@ gtk_spin_button_init (GtkSpinButton *spin_button)
 static void
 gtk_spin_button_finalize (GObject *object)
 {
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (object));
+  gtk_spin_button_set_adjustment (GTK_SPIN_BUTTON (object), NULL);
+  
+  G_OBJECT_CLASS (gtk_spin_button_parent_class)->finalize (object);
+}
 
-  gtk_object_unref (GTK_OBJECT (GTK_SPIN_BUTTON (object)->adjustment));
+static void
+gtk_spin_button_destroy (GtkObject *object)
+{
+  gtk_spin_button_stop_spinning (GTK_SPIN_BUTTON (object));
   
-  G_OBJECT_CLASS (parent_class)->finalize (object);
+  GTK_OBJECT_CLASS (gtk_spin_button_parent_class)->destroy (object);
 }
 
 static void
 gtk_spin_button_map (GtkWidget *widget)
 {
-  g_return_if_fail (widget != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (widget));
-
-  if (GTK_WIDGET_REALIZED (widget) && !GTK_WIDGET_MAPPED (widget))
+  if (GTK_WIDGET_REALIZED (widget) && !gtk_widget_get_mapped (widget))
     {
-      GTK_WIDGET_CLASS (parent_class)->map (widget);
+      GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->map (widget);
       gdk_window_show (GTK_SPIN_BUTTON (widget)->panel);
     }
 }
@@ -443,38 +527,30 @@ gtk_spin_button_map (GtkWidget *widget)
 static void
 gtk_spin_button_unmap (GtkWidget *widget)
 {
-  g_return_if_fail (widget != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (widget));
-
-  if (GTK_WIDGET_MAPPED (widget))
+  if (gtk_widget_get_mapped (widget))
     {
+      gtk_spin_button_stop_spinning (GTK_SPIN_BUTTON (widget));
+
       gdk_window_hide (GTK_SPIN_BUTTON (widget)->panel);
-      GTK_WIDGET_CLASS (parent_class)->unmap (widget);
+      GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->unmap (widget);
     }
 }
 
 static void
 gtk_spin_button_realize (GtkWidget *widget)
 {
-  GtkSpinButton *spin_button;
+  GtkSpinButton *spin_button = GTK_SPIN_BUTTON (widget);
   GdkWindowAttr attributes;
   gint attributes_mask;
-  guint real_width;
-  gint return_val;
+  gboolean return_val;
+  gint arrow_size;
 
-  g_return_if_fail (widget != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (widget));
-  
-  spin_button = GTK_SPIN_BUTTON (widget);
+  arrow_size = spin_button_get_arrow_size (spin_button);
 
-  real_width = widget->allocation.width;
-  widget->allocation.width -= ARROW_SIZE + 2 * widget->style->xthickness;
   gtk_widget_set_events (widget, gtk_widget_get_events (widget) |
                         GDK_KEY_RELEASE_MASK);
-  GTK_WIDGET_CLASS (parent_class)->realize (widget);
+  GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->realize (widget);
 
-  widget->allocation.width = real_width;
-  
   attributes.window_type = GDK_WINDOW_CHILD;
   attributes.wclass = GDK_INPUT_OUTPUT;
   attributes.visual = gtk_widget_get_visual (widget);
@@ -486,37 +562,33 @@ gtk_spin_button_realize (GtkWidget *widget)
 
   attributes_mask = GDK_WA_X | GDK_WA_Y | GDK_WA_VISUAL | GDK_WA_COLORMAP;
 
-  attributes.x = (widget->allocation.x + widget->allocation.width - ARROW_SIZE -
+  attributes.x = (widget->allocation.width - arrow_size -
                  2 * widget->style->xthickness);
-  attributes.y = widget->allocation.y + (widget->allocation.height -
+  attributes.y = (widget->allocation.height -
                                         widget->requisition.height) / 2;
-  attributes.width = ARROW_SIZE + 2 * widget->style->xthickness;
+  attributes.width = arrow_size + 2 * widget->style->xthickness;
   attributes.height = widget->requisition.height;
   
-  spin_button->panel = gdk_window_new (gtk_widget_get_parent_window (widget)
+  spin_button->panel = gdk_window_new (widget->window
                                       &attributes, attributes_mask);
   gdk_window_set_user_data (spin_button->panel, widget);
 
   gtk_style_set_background (widget->style, spin_button->panel, GTK_STATE_NORMAL);
 
   return_val = FALSE;
-  gtk_signal_emit (GTK_OBJECT (spin_button), spinbutton_signals[OUTPUT],
-                  &return_val);
+  g_signal_emit (spin_button, spinbutton_signals[OUTPUT], 0, &return_val);
   if (return_val == FALSE)
     gtk_spin_button_default_output (spin_button);
+
+  gtk_widget_queue_resize (GTK_WIDGET (spin_button));
 }
 
 static void
 gtk_spin_button_unrealize (GtkWidget *widget)
 {
-  GtkSpinButton *spin;
-
-  g_return_if_fail (widget != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (widget));
-
-  spin = GTK_SPIN_BUTTON (widget);
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
-  GTK_WIDGET_CLASS (parent_class)->unrealize (widget);
+  GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->unrealize (widget);
 
   if (spin->panel)
     {
@@ -553,223 +625,273 @@ static void
 gtk_spin_button_size_request (GtkWidget      *widget,
                              GtkRequisition *requisition)
 {
-  GtkEntry *entry;
-  GtkSpinButton *spin_button;
-  
-  g_return_if_fail (widget != NULL);
-  g_return_if_fail (requisition != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (widget));
+  GtkSpinButton *spin_button = GTK_SPIN_BUTTON (widget);
+  GtkEntry *entry = GTK_ENTRY (widget);
+  gint arrow_size;
 
-  entry = GTK_ENTRY (widget);
-  spin_button = GTK_SPIN_BUTTON (widget);
-  
-  GTK_WIDGET_CLASS (parent_class)->size_request (widget, requisition);
+  arrow_size = spin_button_get_arrow_size (spin_button);
+
+  GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->size_request (widget, requisition);
 
   if (entry->width_chars < 0)
     {
       PangoContext *context;
-      PangoFontMetrics metrics;
+      PangoFontMetrics *metrics;
       gint width;
       gint w;
-      int string_len;
+      gint string_len;
+      gint max_string_len;
+      gint digit_width;
+      gboolean interior_focus;
+      gint focus_width;
+      gint xborder, yborder;
+      GtkBorder inner_border;
+
+      gtk_widget_style_get (widget,
+                           "interior-focus", &interior_focus,
+                           "focus-line-width", &focus_width,
+                           NULL);
 
       context = gtk_widget_get_pango_context (widget);
-      pango_context_get_metrics (context,
-                                widget->style->font_desc,
-                                pango_context_get_language (context),
-                                &metrics);
+      metrics = pango_context_get_metrics (context,
+                                          widget->style->font_desc,
+                                          pango_context_get_language (context));
+
+      digit_width = pango_font_metrics_get_approximate_digit_width (metrics);
+      digit_width = PANGO_SCALE *
+        ((digit_width + PANGO_SCALE - 1) / PANGO_SCALE);
+
+      pango_font_metrics_unref (metrics);
       
       /* Get max of MIN_SPIN_BUTTON_WIDTH, size of upper, size of lower */
       
       width = MIN_SPIN_BUTTON_WIDTH;
+      max_string_len = MAX (10, compute_double_length (1e9 * spin_button->adjustment->step_increment,
+                                                       spin_button->digits));
 
       string_len = compute_double_length (spin_button->adjustment->upper,
                                           spin_button->digits);
-      w = MIN (string_len, 10) * PANGO_PIXELS (metrics.approximate_digit_width);
+      w = PANGO_PIXELS (MIN (string_len, max_string_len) * digit_width);
       width = MAX (width, w);
       string_len = compute_double_length (spin_button->adjustment->lower,
-                                         spin_button->adjustment->step_increment);
-      w = MIN (string_len, 10) * PANGO_PIXELS (metrics.approximate_digit_width);
+                                         spin_button->digits);
+      w = PANGO_PIXELS (MIN (string_len, max_string_len) * digit_width);
       width = MAX (width, w);
       
-      requisition->width = width + ARROW_SIZE + 2 * widget->style->xthickness;
+      _gtk_entry_get_borders (entry, &xborder, &yborder);
+      _gtk_entry_effective_inner_border (entry, &inner_border);
+
+      requisition->width = width + xborder * 2 + inner_border.left + inner_border.right;
     }
-  else
-    requisition->width += ARROW_SIZE + 2 * widget->style->xthickness;
+
+  requisition->width += arrow_size + 2 * widget->style->xthickness;
 }
 
 static void
 gtk_spin_button_size_allocate (GtkWidget     *widget,
                               GtkAllocation *allocation)
 {
-  GtkAllocation child_allocation;
-
-  g_return_if_fail (widget != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (widget));
-  g_return_if_fail (allocation != NULL);
-
-  child_allocation = *allocation;
-  if (child_allocation.width > ARROW_SIZE + 2 * widget->style->xthickness)
-    child_allocation.width -= ARROW_SIZE + 2 * widget->style->xthickness;
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
+  GtkAllocation panel_allocation;
+  gint arrow_size;
+  gint panel_width;
 
+  arrow_size = spin_button_get_arrow_size (spin);
+  panel_width = arrow_size + 2 * widget->style->xthickness;
+  
+  widget->allocation = *allocation;
+  
   if (gtk_widget_get_direction (widget) == GTK_TEXT_DIR_RTL)
-    child_allocation.x += ARROW_SIZE + 2 * widget->style->xthickness;
+    panel_allocation.x = 0;
+  else
+    panel_allocation.x = allocation->width - panel_width;
 
-  GTK_WIDGET_CLASS (parent_class)->size_allocate (widget, &child_allocation);
+  panel_allocation.width = panel_width;
+  panel_allocation.height = MIN (widget->requisition.height, allocation->height);
 
-  widget->allocation = *allocation;
+  panel_allocation.y = 0;
+
+  GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->size_allocate (widget, allocation);
 
   if (GTK_WIDGET_REALIZED (widget))
     {
-      child_allocation.width = ARROW_SIZE + 2 * widget->style->xthickness;
-      child_allocation.height = widget->requisition.height;
-
-      if (gtk_widget_get_direction (widget) == GTK_TEXT_DIR_LTR)
-       child_allocation.x = (allocation->x + allocation->width - ARROW_SIZE - 
-                             2 * widget->style->xthickness);
-      else
-       child_allocation.x = allocation->x;      
-
-      child_allocation.y = allocation->y + (allocation->height - widget->requisition.height) / 2;
-
       gdk_window_move_resize (GTK_SPIN_BUTTON (widget)->panel, 
-                             child_allocation.x,
-                             child_allocation.y,
-                             child_allocation.width,
-                             child_allocation.height); 
+                             panel_allocation.x,
+                             panel_allocation.y,
+                             panel_allocation.width,
+                             panel_allocation.height); 
     }
+
+  gtk_widget_queue_draw (GTK_WIDGET (spin));
 }
 
 static gint
 gtk_spin_button_expose (GtkWidget      *widget,
                        GdkEventExpose *event)
 {
-  GtkSpinButton *spin;
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
+
+  if (gtk_widget_is_drawable (widget))
+    {
+      if (event->window == spin->panel)
+       {
+         GtkShadowType shadow_type;
 
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
+         shadow_type = spin_button_get_shadow_type (spin);
 
-  spin = GTK_SPIN_BUTTON (widget);
+         if (shadow_type != GTK_SHADOW_NONE)
+           {
+             gint width, height;
 
-  if (GTK_WIDGET_DRAWABLE (widget))
-    {
-      GtkShadowType shadow_type;
+             gdk_drawable_get_size (spin->panel, &width, &height);
 
-      /* FIXME this seems like really broken code -
-       * why aren't we looking at event->window
-       * and acting accordingly?
-       */
+             gtk_paint_box (widget->style, spin->panel,
+                            GTK_STATE_NORMAL, shadow_type,
+                            &event->area, widget, "spinbutton",
+                            0, 0, width, height);
+           }
 
-      shadow_type = spin_button_get_shadow_type (spin);
-      if (shadow_type != GTK_SHADOW_NONE)
-       gtk_paint_box (widget->style, spin->panel,
-                      GTK_STATE_NORMAL, shadow_type,
-                      &event->area, widget, "spinbutton",
-                      0, 0, 
-                      ARROW_SIZE + 2 * widget->style->xthickness,
-                      widget->requisition.height); 
+         gtk_spin_button_draw_arrow (spin, &event->area, GTK_ARROW_UP);
+         gtk_spin_button_draw_arrow (spin, &event->area, GTK_ARROW_DOWN);
+       }
       else
-        {
-           gdk_window_set_back_pixmap (spin->panel, NULL, TRUE);
-           gdk_window_clear_area (spin->panel,
-                                   event->area.x, event->area.y,
-                                   event->area.width, event->area.height);
-        }
-       gtk_spin_button_draw_arrow (spin, GTK_ARROW_UP);
-       gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
-
-       GTK_WIDGET_CLASS (parent_class)->expose_event (widget, event);
+       GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->expose_event (widget, event);
     }
+  
+  return FALSE;
+}
 
+static gboolean
+spin_button_at_limit (GtkSpinButton *spin_button,
+                     GtkArrowType   arrow)
+{
+  GtkArrowType effective_arrow;
+
+  if (spin_button->wrap)
+    return FALSE;
+
+  if (spin_button->adjustment->step_increment > 0)
+    effective_arrow = arrow;
+  else
+    effective_arrow = arrow == GTK_ARROW_UP ? GTK_ARROW_DOWN : GTK_ARROW_UP; 
+  
+  if (effective_arrow == GTK_ARROW_UP &&
+      (spin_button->adjustment->upper - spin_button->adjustment->value <= EPSILON))
+    return TRUE;
+  
+  if (effective_arrow == GTK_ARROW_DOWN &&
+      (spin_button->adjustment->value - spin_button->adjustment->lower <= EPSILON))
+    return TRUE;
+  
   return FALSE;
 }
 
 static void
 gtk_spin_button_draw_arrow (GtkSpinButton *spin_button, 
-                           guint          arrow)
+                           GdkRectangle  *area,
+                           GtkArrowType   arrow_type)
 {
-  GtkShadowType spin_shadow_type;
   GtkStateType state_type;
   GtkShadowType shadow_type;
   GtkWidget *widget;
   gint x;
   gint y;
+  gint height;
+  gint width;
+  gint h, w;
+
+  g_return_if_fail (arrow_type == GTK_ARROW_UP || arrow_type == GTK_ARROW_DOWN);
 
-  g_return_if_fail (spin_button != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (spin_button));
-  
   widget = GTK_WIDGET (spin_button);
-  spin_shadow_type = spin_button_get_shadow_type (spin_button);
 
-  if (GTK_WIDGET_DRAWABLE (spin_button))
+  if (gtk_widget_is_drawable (widget))
     {
-      if (!spin_button->wrap &&
-         (((arrow == GTK_ARROW_UP &&
-         (spin_button->adjustment->upper - spin_button->adjustment->value
-          <= EPSILON))) ||
-         ((arrow == GTK_ARROW_DOWN &&
-         (spin_button->adjustment->value - spin_button->adjustment->lower
-          <= EPSILON)))))
+      width = spin_button_get_arrow_size (spin_button) + 2 * widget->style->xthickness;
+
+      if (arrow_type == GTK_ARROW_UP)
        {
-         shadow_type = GTK_SHADOW_ETCHED_IN;
-         state_type = GTK_STATE_NORMAL;
+         x = 0;
+         y = 0;
+
+         height = widget->requisition.height / 2;
        }
       else
        {
-         if (spin_button->in_child == arrow)
-           {
-             if (spin_button->click_child == arrow)
-               state_type = GTK_STATE_ACTIVE;
-             else
-               state_type = GTK_STATE_PRELIGHT;
-           }
-         else
-           state_type = GTK_STATE_NORMAL;
-         
-         if (spin_button->click_child == arrow)
-           shadow_type = GTK_SHADOW_IN;
-         else
-           shadow_type = GTK_SHADOW_OUT;
+         x = 0;
+         y = widget->requisition.height / 2;
+
+         height = (widget->requisition.height + 1) / 2;
        }
-      if (arrow == GTK_ARROW_UP)
+
+      if (spin_button_at_limit (spin_button, arrow_type))
        {
-         if (spin_shadow_type != GTK_SHADOW_NONE)
-           {
-             x = widget->style->xthickness;
-             y = widget->style->ythickness;
-           }
-         else
-           {
-             x = widget->style->xthickness - 1;
-             y = widget->style->ythickness - 1;
-           }
-         gtk_paint_arrow (widget->style, spin_button->panel,
-                          state_type, shadow_type, 
-                          NULL, widget, "spinbutton",
-                          arrow, TRUE, 
-                          x, y, ARROW_SIZE, widget->requisition.height / 2 
-                          - widget->style->ythickness);
+         shadow_type = GTK_SHADOW_OUT;
+         state_type = GTK_STATE_INSENSITIVE;
        }
       else
        {
-         if (spin_shadow_type != GTK_SHADOW_NONE)
+         if (spin_button->click_child == arrow_type)
            {
-             x = widget->style->xthickness;
-             y = widget->requisition.height / 2;
+             state_type = GTK_STATE_ACTIVE;
+             shadow_type = GTK_SHADOW_IN;
            }
          else
            {
-             x = widget->style->xthickness - 1;
-             y = widget->requisition.height / 2 + 1;
+             if (spin_button->in_child == arrow_type &&
+                 spin_button->click_child == NO_ARROW)
+               {
+                 state_type = GTK_STATE_PRELIGHT;
+               }
+             else
+               {
+                 state_type = GTK_WIDGET_STATE (widget);
+               }
+             
+             shadow_type = GTK_SHADOW_OUT;
            }
-         gtk_paint_arrow (widget->style, spin_button->panel,
-                          state_type, shadow_type, 
-                          NULL, widget, "spinbutton",
-                          arrow, TRUE, 
-                          x, y, ARROW_SIZE, widget->requisition.height / 2 
-                          - widget->style->ythickness);
        }
+      
+      gtk_paint_box (widget->style, spin_button->panel,
+                    state_type, shadow_type,
+                    area, widget,
+                    (arrow_type == GTK_ARROW_UP)? "spinbutton_up" : "spinbutton_down",
+                    x, y, width, height);
+
+      height = widget->requisition.height;
+
+      if (arrow_type == GTK_ARROW_DOWN)
+       {
+         y = height / 2;
+         height = height - y - 2;
+       }
+      else
+       {
+         y = 2;
+         height = height / 2 - 2;
+       }
+
+      width -= 3;
+
+      if (widget && gtk_widget_get_direction (widget) == GTK_TEXT_DIR_RTL)
+       x = 2;
+      else
+       x = 1;
+
+      w = width / 2;
+      w -= w % 2 - 1; /* force odd */
+      h = (w + 1) / 2;
+      
+      x += (width - w) / 2;
+      y += (height - h) / 2;
+      
+      height = h;
+      width = w;
+
+      gtk_paint_arrow (widget->style, spin_button->panel,
+                      state_type, shadow_type, 
+                      area, widget, "spinbutton",
+                      arrow_type, TRUE, 
+                      x, y, width, height);
     }
 }
 
@@ -777,13 +899,7 @@ static gint
 gtk_spin_button_enter_notify (GtkWidget        *widget,
                              GdkEventCrossing *event)
 {
-  GtkSpinButton *spin;
-
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-
-  spin = GTK_SPIN_BUTTON (widget);
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
   if (event->window == spin->panel)
     {
@@ -793,18 +909,16 @@ gtk_spin_button_enter_notify (GtkWidget        *widget,
       gdk_window_get_pointer (spin->panel, &x, &y, NULL);
 
       if (y <= widget->requisition.height / 2)
-       {
-         spin->in_child = GTK_ARROW_UP;
-         if (spin->click_child == 2) 
-           gtk_spin_button_draw_arrow (spin, GTK_ARROW_UP);
-       }
+       spin->in_child = GTK_ARROW_UP;
       else
-       {
-         spin->in_child = GTK_ARROW_DOWN;
-         if (spin->click_child == 2) 
-           gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
-       }
+       spin->in_child = GTK_ARROW_DOWN;
+
+      gtk_widget_queue_draw (GTK_WIDGET (spin));
     }
+  if (GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->enter_notify_event)
+    return GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->enter_notify_event (widget, event);
+
   return FALSE;
 }
 
@@ -812,27 +926,14 @@ static gint
 gtk_spin_button_leave_notify (GtkWidget        *widget,
                              GdkEventCrossing *event)
 {
-  GtkSpinButton *spin;
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-
-  spin = GTK_SPIN_BUTTON (widget);
+  spin->in_child = NO_ARROW;
+  gtk_widget_queue_draw (GTK_WIDGET (spin));
+  if (GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->leave_notify_event)
+    return GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->leave_notify_event (widget, event);
 
-  if (event->window == spin->panel && spin->click_child == 2)
-    {
-      if (spin->in_child == GTK_ARROW_UP) 
-       {
-         spin->in_child = 2;
-         gtk_spin_button_draw_arrow (spin, GTK_ARROW_UP);
-       }
-      else
-       {
-         spin->in_child = 2;
-         gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
-       }
-    }
   return FALSE;
 }
 
@@ -840,37 +941,66 @@ static gint
 gtk_spin_button_focus_out (GtkWidget     *widget,
                           GdkEventFocus *event)
 {
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-
   if (GTK_ENTRY (widget)->editable)
     gtk_spin_button_update (GTK_SPIN_BUTTON (widget));
 
-  return GTK_WIDGET_CLASS (parent_class)->focus_out_event (widget, event);
+  return GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->focus_out_event (widget, event);
+}
+
+static void
+gtk_spin_button_grab_notify (GtkWidget *widget,
+                            gboolean   was_grabbed)
+{
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
+
+  if (!was_grabbed)
+    {
+      gtk_spin_button_stop_spinning (spin);
+      gtk_widget_queue_draw (GTK_WIDGET (spin));
+    }
+}
+
+static void
+gtk_spin_button_state_changed (GtkWidget    *widget,
+                              GtkStateType  previous_state)
+{
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
+
+  if (!gtk_widget_is_sensitive (widget))
+    {
+      gtk_spin_button_stop_spinning (spin);    
+      gtk_widget_queue_draw (GTK_WIDGET (spin));
+    }
 }
 
+static void
+gtk_spin_button_style_set (GtkWidget *widget,
+                          GtkStyle  *previous_style)
+{
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
+
+  if (previous_style && GTK_WIDGET_REALIZED (widget))
+    gtk_style_set_background (widget->style, spin->panel, GTK_STATE_NORMAL);
+
+  GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->style_set (widget, previous_style);
+}
+
+
 static gint
 gtk_spin_button_scroll (GtkWidget      *widget,
                        GdkEventScroll *event)
 {
-  GtkSpinButton *spin;
-
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-
-  spin = GTK_SPIN_BUTTON (widget);
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
   if (event->direction == GDK_SCROLL_UP)
     {
-      if (!GTK_WIDGET_HAS_FOCUS (widget))
+      if (!gtk_widget_has_focus (widget))
        gtk_widget_grab_focus (widget);
       gtk_spin_button_real_spin (spin, spin->adjustment->step_increment);
     }
   else if (event->direction == GDK_SCROLL_DOWN)
     {
-      if (!GTK_WIDGET_HAS_FOCUS (widget))
+      if (!gtk_widget_has_focus (widget))
        gtk_widget_grab_focus (widget);
       gtk_spin_button_real_spin (spin, -spin->adjustment->step_increment); 
     }
@@ -880,25 +1010,65 @@ gtk_spin_button_scroll (GtkWidget      *widget,
   return TRUE;
 }
 
+static void
+gtk_spin_button_stop_spinning (GtkSpinButton *spin)
+{
+  if (spin->timer)
+    {
+      g_source_remove (spin->timer);
+      spin->timer = 0;
+      spin->timer_calls = 0;
+      spin->need_timer = FALSE;
+    }
+
+  spin->button = 0;
+  spin->timer = 0;
+  spin->timer_step = spin->adjustment->step_increment;
+  spin->timer_calls = 0;
+
+  spin->click_child = NO_ARROW;
+  spin->button = 0;
+}
+
+static void
+start_spinning (GtkSpinButton *spin,
+               GtkArrowType   click_child,
+               gdouble        step)
+{
+  g_return_if_fail (click_child == GTK_ARROW_UP || click_child == GTK_ARROW_DOWN);
+  
+  spin->click_child = click_child;
+  
+  if (!spin->timer)
+    {
+      GtkSettings *settings = gtk_widget_get_settings (GTK_WIDGET (spin));
+      guint        timeout;
+
+      g_object_get (settings, "gtk-timeout-initial", &timeout, NULL);
+
+      spin->timer_step = step;
+      spin->need_timer = TRUE;
+      spin->timer = gdk_threads_add_timeout (timeout,
+                                  (GSourceFunc) gtk_spin_button_timer,
+                                  (gpointer) spin);
+    }
+  gtk_spin_button_real_spin (spin, click_child == GTK_ARROW_UP ? step : -step);
+
+  gtk_widget_queue_draw (GTK_WIDGET (spin));
+}
+
 static gint
 gtk_spin_button_button_press (GtkWidget      *widget,
                              GdkEventButton *event)
 {
-  GtkSpinButton *spin;
-
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-
-  spin = GTK_SPIN_BUTTON (widget);
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
   if (!spin->button)
     {
       if (event->window == spin->panel)
        {
-         if (!GTK_WIDGET_HAS_FOCUS (widget))
+         if (!gtk_widget_has_focus (widget))
            gtk_widget_grab_focus (widget);
-         gtk_grab_add (widget);
          spin->button = event->button;
          
          if (GTK_ENTRY (widget)->editable)
@@ -906,70 +1076,26 @@ gtk_spin_button_button_press (GtkWidget      *widget,
          
          if (event->y <= widget->requisition.height / 2)
            {
-             spin->click_child = GTK_ARROW_UP;
              if (event->button == 1)
-               {
-                gtk_spin_button_real_spin (spin, 
-                                           spin->adjustment->step_increment);
-                 if (!spin->timer)
-                   {
-                     spin->timer_step = spin->adjustment->step_increment;
-                     spin->need_timer = TRUE;
-                     spin->timer = gtk_timeout_add 
-                       (SPIN_BUTTON_INITIAL_TIMER_DELAY, 
-                        (GtkFunction) gtk_spin_button_timer, (gpointer) spin);
-                   }
-               }
+               start_spinning (spin, GTK_ARROW_UP, spin->adjustment->step_increment);
              else if (event->button == 2)
-               {
-                gtk_spin_button_real_spin (spin, 
-                                           spin->adjustment->page_increment);
-                 if (!spin->timer) 
-                   {
-                     spin->timer_step = spin->adjustment->page_increment;
-                     spin->need_timer = TRUE;
-                     spin->timer = gtk_timeout_add 
-                       (SPIN_BUTTON_INITIAL_TIMER_DELAY, 
-                        (GtkFunction) gtk_spin_button_timer, (gpointer) spin);
-                   }
-               }
-             gtk_spin_button_draw_arrow (spin, GTK_ARROW_UP);
+               start_spinning (spin, GTK_ARROW_UP, spin->adjustment->page_increment);
+             else
+               spin->click_child = GTK_ARROW_UP;
            }
          else 
            {
-             spin->click_child = GTK_ARROW_DOWN;
              if (event->button == 1)
-               {
-                 gtk_spin_button_real_spin (spin,
-                                            -spin->adjustment->step_increment);
-                 if (!spin->timer)
-                   {
-                     spin->timer_step = spin->adjustment->step_increment;
-                     spin->need_timer = TRUE;
-                     spin->timer = gtk_timeout_add 
-                       (SPIN_BUTTON_INITIAL_TIMER_DELAY, 
-                        (GtkFunction) gtk_spin_button_timer, (gpointer) spin);
-                   }
-               }      
+               start_spinning (spin, GTK_ARROW_DOWN, spin->adjustment->step_increment);
              else if (event->button == 2)
-               {
-                 gtk_spin_button_real_spin (spin,
-                                            -spin->adjustment->page_increment);
-                 if (!spin->timer) 
-                   {
-                     spin->timer_step = spin->adjustment->page_increment;
-                     spin->need_timer = TRUE;
-                     spin->timer = gtk_timeout_add 
-                       (SPIN_BUTTON_INITIAL_TIMER_DELAY, 
-                        (GtkFunction) gtk_spin_button_timer, (gpointer) spin);
-                   }
-               }
-             gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
+               start_spinning (spin, GTK_ARROW_DOWN, spin->adjustment->page_increment);
+             else
+               spin->click_child = GTK_ARROW_DOWN;
            }
          return TRUE;
        }
       else
-       return GTK_WIDGET_CLASS (parent_class)->button_press_event (widget, event);
+       return GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->button_press_event (widget, event);
     }
   return FALSE;
 }
@@ -978,33 +1104,24 @@ static gint
 gtk_spin_button_button_release (GtkWidget      *widget,
                                GdkEventButton *event)
 {
-  GtkSpinButton *spin;
-
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
+  gint arrow_size;
 
-  spin = GTK_SPIN_BUTTON (widget);
+  arrow_size = spin_button_get_arrow_size (spin);
 
   if (event->button == spin->button)
     {
-      guint click_child;
+      int click_child = spin->click_child;
 
-      if (spin->timer)
-       {
-         gtk_timeout_remove (spin->timer);
-         spin->timer = 0;
-         spin->timer_calls = 0;
-         spin->need_timer = FALSE;
-       }
+      gtk_spin_button_stop_spinning (spin);
 
       if (event->button == 3)
        {
          if (event->y >= 0 && event->x >= 0 && 
              event->y <= widget->requisition.height &&
-             event->x <= ARROW_SIZE + 2 * widget->style->xthickness)
+             event->x <= arrow_size + 2 * widget->style->xthickness)
            {
-             if (spin->click_child == GTK_ARROW_UP &&
+             if (click_child == GTK_ARROW_UP &&
                  event->y <= widget->requisition.height / 2)
                {
                  gdouble diff;
@@ -1013,7 +1130,7 @@ gtk_spin_button_button_release (GtkWidget      *widget,
                  if (diff > EPSILON)
                    gtk_spin_button_real_spin (spin, diff);
                }
-             else if (spin->click_child == GTK_ARROW_DOWN &&
+             else if (click_child == GTK_ARROW_DOWN &&
                       event->y > widget->requisition.height / 2)
                {
                  gdouble diff;
@@ -1024,60 +1141,46 @@ gtk_spin_button_button_release (GtkWidget      *widget,
                }
            }
        }                 
-      gtk_grab_remove (widget);
-      click_child = spin->click_child;
-      spin->click_child = 2;
-      spin->button = 0;
-      gtk_spin_button_draw_arrow (spin, click_child);
+      gtk_widget_queue_draw (GTK_WIDGET (spin));
+
       return TRUE;
     }
   else
-    return GTK_WIDGET_CLASS (parent_class)->button_release_event (widget, event);
-
-  return FALSE;
+    return GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->button_release_event (widget, event);
 }
 
 static gint
 gtk_spin_button_motion_notify (GtkWidget      *widget,
                               GdkEventMotion *event)
 {
-  GtkSpinButton *spin;
-
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
-  spin = GTK_SPIN_BUTTON (widget);
-  
   if (spin->button)
     return FALSE;
 
   if (event->window == spin->panel)
     {
-      gint y;
-
-      y = event->y;
-      if (event->is_hint)
-       gdk_window_get_pointer (spin->panel, NULL, &y, NULL);
+      gint y = event->y;
 
+      gdk_event_request_motions (event);
+  
       if (y <= widget->requisition.height / 2 && 
          spin->in_child == GTK_ARROW_DOWN)
        {
          spin->in_child = GTK_ARROW_UP;
-         gtk_spin_button_draw_arrow (spin, GTK_ARROW_UP);
-         gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
+         gtk_widget_queue_draw (GTK_WIDGET (spin));
        }
       else if (y > widget->requisition.height / 2 && 
          spin->in_child == GTK_ARROW_UP)
        {
          spin->in_child = GTK_ARROW_DOWN;
-         gtk_spin_button_draw_arrow (spin, GTK_ARROW_UP);
-         gtk_spin_button_draw_arrow (spin, GTK_ARROW_DOWN);
+         gtk_widget_queue_draw (GTK_WIDGET (spin));
        }
+      
       return FALSE;
     }
          
-  return GTK_WIDGET_CLASS (parent_class)->motion_notify_event (widget, event);
+  return GTK_WIDGET_CLASS (gtk_spin_button_parent_class)->motion_notify_event (widget, event);
 }
 
 static gint
@@ -1085,8 +1188,6 @@ gtk_spin_button_timer (GtkSpinButton *spin_button)
 {
   gboolean retval = FALSE;
   
-  GDK_THREADS_ENTER ();
-
   if (spin_button->timer)
     {
       if (spin_button->click_child == GTK_ARROW_UP)
@@ -1096,10 +1197,15 @@ gtk_spin_button_timer (GtkSpinButton *spin_button)
 
       if (spin_button->need_timer)
        {
+          GtkSettings *settings = gtk_widget_get_settings (GTK_WIDGET (spin_button));
+          guint        timeout;
+
+          g_object_get (settings, "gtk-timeout-repeat", &timeout, NULL);
+
          spin_button->need_timer = FALSE;
-         spin_button->timer = gtk_timeout_add 
-           (SPIN_BUTTON_TIMER_DELAY, (GtkFunction) gtk_spin_button_timer, 
-            (gpointer) spin_button);
+         spin_button->timer = gdk_threads_add_timeout (timeout,
+                                             (GSourceFunc) gtk_spin_button_timer, 
+                                             (gpointer) spin_button);
        }
       else 
        {
@@ -1118,8 +1224,6 @@ gtk_spin_button_timer (GtkSpinButton *spin_button)
        }
     }
 
-  GDK_THREADS_LEAVE ();
-
   return retval;
 }
 
@@ -1127,164 +1231,128 @@ static void
 gtk_spin_button_value_changed (GtkAdjustment *adjustment,
                               GtkSpinButton *spin_button)
 {
-  gint return_val;
+  gboolean return_val;
 
-  g_return_if_fail (adjustment != NULL);
   g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
 
   return_val = FALSE;
-  gtk_signal_emit (GTK_OBJECT (spin_button), spinbutton_signals[OUTPUT],
-                  &return_val);
+  g_signal_emit (spin_button, spinbutton_signals[OUTPUT], 0, &return_val);
   if (return_val == FALSE)
     gtk_spin_button_default_output (spin_button);
 
-  gtk_signal_emit (GTK_OBJECT (spin_button), 
-                  spinbutton_signals[VALUE_CHANGED]);
+  g_signal_emit (spin_button, spinbutton_signals[VALUE_CHANGED], 0);
 
-  gtk_spin_button_draw_arrow (spin_button, GTK_ARROW_UP);
-  gtk_spin_button_draw_arrow (spin_button, GTK_ARROW_DOWN);
+  gtk_widget_queue_draw (GTK_WIDGET (spin_button));
   
   g_object_notify (G_OBJECT (spin_button), "value");
 }
 
-static gint
-gtk_spin_button_key_press (GtkWidget     *widget,
-                          GdkEventKey   *event)
+static void
+gtk_spin_button_real_change_value (GtkSpinButton *spin,
+                                  GtkScrollType  scroll)
 {
-  GtkSpinButton *spin;
-  gint key;
-  gboolean key_repeat = FALSE;
-  gboolean retval = FALSE;
-  
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
-  g_return_val_if_fail (event != NULL, FALSE);
-  
-  spin = GTK_SPIN_BUTTON (widget);
-  key = event->keyval;
-
-  key_repeat = (event->time == spin->ev_time);
-
-  if (GTK_ENTRY (widget)->editable)
+  gdouble old_value;
+
+  /* When the key binding is activated, there may be an outstanding
+   * value, so we first have to commit what is currently written in
+   * the spin buttons text entry. See #106574
+   */
+  gtk_spin_button_update (spin);
+
+  old_value = spin->adjustment->value;
+
+  /* We don't test whether the entry is editable, since
+   * this key binding conceptually corresponds to changing
+   * the value with the buttons using the mouse, which
+   * we allow for non-editable spin buttons.
+   */
+  switch (scroll)
     {
-      switch (key)
-        {
-        case GDK_KP_Up:
-        case GDK_Up:
-
-          if (GTK_WIDGET_HAS_FOCUS (widget))
-            {
-              gtk_signal_emit_stop_by_name (GTK_OBJECT (widget), 
-                                            "key_press_event");
-              if (!key_repeat)
-                spin->timer_step = spin->adjustment->step_increment;
-
-              gtk_spin_button_real_spin (spin, spin->timer_step);
-
-              if (key_repeat)
-                {
-                  if (spin->climb_rate > 0.0 && spin->timer_step
-                      < spin->adjustment->page_increment)
-                    {
-                      if (spin->timer_calls < MAX_TIMER_CALLS)
-                        spin->timer_calls++;
-                      else 
-                        {
-                          spin->timer_calls = 0;
-                          spin->timer_step += spin->climb_rate;
-                        }
-                    }
-                }
-              retval = TRUE;
-            }
-          break;
-
-        case GDK_KP_Down:
-        case GDK_Down:
-
-          if (GTK_WIDGET_HAS_FOCUS (widget))
-            {
-              gtk_signal_emit_stop_by_name (GTK_OBJECT (widget), 
-                                            "key_press_event");
-              if (!key_repeat)
-                spin->timer_step = spin->adjustment->step_increment;
-
-              gtk_spin_button_real_spin (spin, -spin->timer_step);
-
-              if (key_repeat)
-                {
-                  if (spin->climb_rate > 0.0 && spin->timer_step
-                      < spin->adjustment->page_increment)
-                    {
-                      if (spin->timer_calls < MAX_TIMER_CALLS)
-                        spin->timer_calls++;
-                      else 
-                        {
-                          spin->timer_calls = 0;
-                          spin->timer_step += spin->climb_rate;
-                        }
-                    }
-                }
-              retval = TRUE;
-            }
-          break;
-
-        case GDK_KP_Page_Up:
-        case GDK_Page_Up:
-
-          if (event->state & GDK_CONTROL_MASK)
-            {
-              gdouble diff = spin->adjustment->upper - spin->adjustment->value;
-              if (diff > EPSILON)
-                gtk_spin_button_real_spin (spin, diff);
-            }
-          else
-            gtk_spin_button_real_spin (spin, spin->adjustment->page_increment);
-
-          retval = TRUE;
-          break;
-          
-        case GDK_KP_Page_Down:
-        case GDK_Page_Down:
-
-          if (event->state & GDK_CONTROL_MASK)
-            {
-              gdouble diff = spin->adjustment->value - spin->adjustment->lower;
-              if (diff > EPSILON)
-                gtk_spin_button_real_spin (spin, -diff);
-            }
-          else
-            gtk_spin_button_real_spin (spin, -spin->adjustment->page_increment);
-
-          retval = TRUE;
-          break;
-
-        default:
-          break;
-        }
+    case GTK_SCROLL_STEP_BACKWARD:
+    case GTK_SCROLL_STEP_DOWN:
+    case GTK_SCROLL_STEP_LEFT:
+      gtk_spin_button_real_spin (spin, -spin->timer_step);
+      
+      if (spin->climb_rate > 0.0 && spin->timer_step
+         < spin->adjustment->page_increment)
+       {
+         if (spin->timer_calls < MAX_TIMER_CALLS)
+           spin->timer_calls++;
+         else 
+           {
+             spin->timer_calls = 0;
+             spin->timer_step += spin->climb_rate;
+           }
+       }
+      break;
+      
+    case GTK_SCROLL_STEP_FORWARD:
+    case GTK_SCROLL_STEP_UP:
+    case GTK_SCROLL_STEP_RIGHT:
+      gtk_spin_button_real_spin (spin, spin->timer_step);
+      
+      if (spin->climb_rate > 0.0 && spin->timer_step
+         < spin->adjustment->page_increment)
+       {
+         if (spin->timer_calls < MAX_TIMER_CALLS)
+           spin->timer_calls++;
+         else 
+           {
+             spin->timer_calls = 0;
+             spin->timer_step += spin->climb_rate;
+           }
+       }
+      break;
+      
+    case GTK_SCROLL_PAGE_BACKWARD:
+    case GTK_SCROLL_PAGE_DOWN:
+    case GTK_SCROLL_PAGE_LEFT:
+      gtk_spin_button_real_spin (spin, -spin->adjustment->page_increment);
+      break;
+      
+    case GTK_SCROLL_PAGE_FORWARD:
+    case GTK_SCROLL_PAGE_UP:
+    case GTK_SCROLL_PAGE_RIGHT:
+      gtk_spin_button_real_spin (spin, spin->adjustment->page_increment);
+      break;
+      
+    case GTK_SCROLL_START:
+      {
+       gdouble diff = spin->adjustment->value - spin->adjustment->lower;
+       if (diff > EPSILON)
+         gtk_spin_button_real_spin (spin, -diff);
+       break;
+      }
+      
+    case GTK_SCROLL_END:
+      {
+       gdouble diff = spin->adjustment->upper - spin->adjustment->value;
+       if (diff > EPSILON)
+         gtk_spin_button_real_spin (spin, diff);
+       break;
+      }
+      
+    default:
+      g_warning ("Invalid scroll type %d for GtkSpinButton::change-value", scroll);
+      break;
     }
+  
+  gtk_spin_button_update (spin);
 
-  if (retval)
-    {
-      gtk_spin_button_update (spin);
-      return TRUE;
-    }
-  else
-    return GTK_WIDGET_CLASS (parent_class)->key_press_event (widget, event);
+  if (spin->adjustment->value == old_value)
+    gtk_widget_error_bell (GTK_WIDGET (spin));
 }
 
 static gint
 gtk_spin_button_key_release (GtkWidget   *widget,
                             GdkEventKey *event)
 {
-  GtkSpinButton *spin;
+  GtkSpinButton *spin = GTK_SPIN_BUTTON (widget);
 
-  g_return_val_if_fail (widget != NULL, FALSE);
-  g_return_val_if_fail (GTK_IS_SPIN_BUTTON (widget), FALSE);
+  /* We only get a release at the end of a key repeat run, so reset the timer_step */
+  spin->timer_step = spin->adjustment->step_increment;
+  spin->timer_calls = 0;
   
-  spin = GTK_SPIN_BUTTON (widget);
-  
-  spin->ev_time = event->time;
   return TRUE;
 }
 
@@ -1294,24 +1362,18 @@ gtk_spin_button_snap (GtkSpinButton *spin_button,
 {
   gdouble inc;
   gdouble tmp;
-  
+
   inc = spin_button->adjustment->step_increment;
+  if (inc == 0)
+    return;
+  
   tmp = (val - spin_button->adjustment->lower) / inc;
   if (tmp - floor (tmp) < ceil (tmp) - tmp)
     val = spin_button->adjustment->lower + floor (tmp) * inc;
   else
     val = spin_button->adjustment->lower + ceil (tmp) * inc;
 
-  if (fabs (val - spin_button->adjustment->value) > EPSILON)
-    gtk_adjustment_set_value (spin_button->adjustment, val);
-  else
-    {
-      gint return_val = FALSE;
-      gtk_signal_emit (GTK_OBJECT (spin_button), spinbutton_signals[OUTPUT],
-                      &return_val);
-      if (return_val == FALSE)
-       gtk_spin_button_default_output (spin_button);
-    }
+  gtk_spin_button_set_value (spin_button, val);
 }
 
 static void
@@ -1319,16 +1381,42 @@ gtk_spin_button_activate (GtkEntry *entry)
 {
   if (entry->editable)
     gtk_spin_button_update (GTK_SPIN_BUTTON (entry));
+
+  /* Chain up so that entry->activates_default is honored */
+  GTK_ENTRY_CLASS (gtk_spin_button_parent_class)->activate (entry);
+}
+
+static void
+gtk_spin_button_get_text_area_size (GtkEntry *entry,
+                                   gint     *x,
+                                   gint     *y,
+                                   gint     *width,
+                                   gint     *height)
+{
+  gint arrow_size;
+  gint panel_width;
+
+  GTK_ENTRY_CLASS (gtk_spin_button_parent_class)->get_text_area_size (entry, x, y, width, height);
+
+  arrow_size = spin_button_get_arrow_size (GTK_SPIN_BUTTON (entry));
+  panel_width = arrow_size + 2 * GTK_WIDGET (entry)->style->xthickness;
+
+  if (width)
+    *width -= panel_width;
+
+  if (gtk_widget_get_direction (GTK_WIDGET (entry)) == GTK_TEXT_DIR_RTL && x)
+    *x += panel_width;
 }
 
 static void
-gtk_spin_button_insert_text (GtkEntry    *entry,
+gtk_spin_button_insert_text (GtkEditable *editable,
                             const gchar *new_text,
                             gint         new_text_length,
                             gint        *position)
 {
-  GtkEditable *editable = GTK_EDITABLE (entry);
+  GtkEntry *entry = GTK_ENTRY (editable);
   GtkSpinButton *spin = GTK_SPIN_BUTTON (editable);
+  GtkEditableClass *parent_editable_iface = g_type_interface_peek (gtk_spin_button_parent_class, GTK_TYPE_EDITABLE);
  
   if (spin->numeric)
     {
@@ -1339,8 +1427,10 @@ gtk_spin_button_insert_text (GtkEntry    *entry,
       GdkWChar pos_sign;
       GdkWChar neg_sign;
       gint entry_length;
+      const gchar *entry_text;
 
-      entry_length = entry->text_length;
+      entry_length = gtk_entry_get_text_length (entry);
+      entry_text = gtk_entry_get_text (entry);
 
       lc = localeconv ();
 
@@ -1354,9 +1444,29 @@ gtk_spin_button_insert_text (GtkEntry    *entry,
       else 
        pos_sign = '+';
 
+#ifdef G_OS_WIN32
+      /* Workaround for bug caused by some Windows application messing
+       * up the positive sign of the current locale, more specifically
+       * HKEY_CURRENT_USER\Control Panel\International\sPositiveSign.
+       * See bug #330743 and for instance
+       * http://www.msnewsgroups.net/group/microsoft.public.dotnet.languages.csharp/topic36024.aspx
+       *
+       * I don't know if the positive sign always gets bogusly set to
+       * a digit when the above Registry value is corrupted as
+       * described. (In my test case, it got set to "8", and in the
+       * bug report above it presumably was set ot "0".) Probably it
+       * might get set to almost anything? So how to distinguish a
+       * bogus value from some correct one for some locale? That is
+       * probably hard, but at least we should filter out the
+       * digits...
+       */
+      if (pos_sign >= '0' && pos_sign <= '9')
+       pos_sign = '+';
+#endif
+
       for (sign=0, i=0; i<entry_length; i++)
-       if ((entry->text[i] == neg_sign) ||
-           (entry->text[i] == pos_sign))
+       if ((entry_text[i] == neg_sign) ||
+           (entry_text[i] == pos_sign))
          {
            sign = 1;
            break;
@@ -1366,7 +1476,7 @@ gtk_spin_button_insert_text (GtkEntry    *entry,
        return;
 
       for (dotpos=-1, i=0; i<entry_length; i++)
-       if (entry->text[i] == *(lc->decimal_point))
+       if (entry_text[i] == *(lc->decimal_point))
          {
            dotpos = i;
            break;
@@ -1398,8 +1508,8 @@ gtk_spin_button_insert_text (GtkEntry    *entry,
        }
     }
 
-  GTK_ENTRY_CLASS (parent_class)->insert_text (entry, new_text,
-                                              new_text_length, position);
+  parent_editable_iface->insert_text (editable, new_text,
+                                     new_text_length, position);
 }
 
 static void
@@ -1408,9 +1518,7 @@ gtk_spin_button_real_spin (GtkSpinButton *spin_button,
 {
   GtkAdjustment *adj;
   gdouble new_value = 0.0;
-
-  g_return_if_fail (spin_button != NULL);
-  g_return_if_fail (GTK_IS_SPIN_BUTTON (spin_button));
+  gboolean wrapped = FALSE;
   
   adj = spin_button->adjustment;
 
@@ -1421,7 +1529,10 @@ gtk_spin_button_real_spin (GtkSpinButton *spin_button,
       if (spin_button->wrap)
        {
          if (fabs (adj->value - adj->upper) < EPSILON)
-           new_value = adj->lower;
+           {
+             new_value = adj->lower;
+             wrapped = TRUE;
+           }
          else if (new_value > adj->upper)
            new_value = adj->upper;
        }
@@ -1433,7 +1544,10 @@ gtk_spin_button_real_spin (GtkSpinButton *spin_button,
       if (spin_button->wrap)
        {
          if (fabs (adj->value - adj->lower) < EPSILON)
-           new_value = adj->upper;
+           {
+             new_value = adj->upper;
+             wrapped = TRUE;
+           }
          else if (new_value < adj->lower)
            new_value = adj->lower;
        }
@@ -1443,6 +1557,11 @@ gtk_spin_button_real_spin (GtkSpinButton *spin_button,
 
   if (fabs (new_value - adj->value) > EPSILON)
     gtk_adjustment_set_value (adj, new_value);
+
+  if (wrapped)
+    g_signal_emit (spin_button, spinbutton_signals[WRAPPED], 0);
+
+  gtk_widget_queue_draw (GTK_WIDGET (spin_button));
 }
 
 static gint
@@ -1451,7 +1570,7 @@ gtk_spin_button_default_input (GtkSpinButton *spin_button,
 {
   gchar *err = NULL;
 
-  *new_val = strtod (gtk_entry_get_text (GTK_ENTRY (spin_button)), &err);
+  *new_val = g_strtod (gtk_entry_get_text (GTK_ENTRY (spin_button)), &err);
   if (*err)
     return GTK_INPUT_ERROR;
   else
@@ -1477,13 +1596,22 @@ gtk_spin_button_default_output (GtkSpinButton *spin_button)
  ***********************************************************/
 
 
+/**
+ * gtk_spin_button_configure:
+ * @spin_button: a #GtkSpinButton
+ * @adjustment: (allow-none):  a #GtkAdjustment.
+ * @climb_rate: the new climb rate.
+ * @digits: the number of decimal places to display in the spin button.
+ *
+ * Changes the properties of an existing spin button. The adjustment, climb rate,
+ * and number of decimal places are all changed accordingly, after this function call.
+ */
 void
 gtk_spin_button_configure (GtkSpinButton  *spin_button,
                           GtkAdjustment  *adjustment,
                           gdouble         climb_rate,
                           guint           digits)
 {
-  g_return_if_fail (spin_button != NULL);
   g_return_if_fail (GTK_IS_SPIN_BUTTON (spin_button));
 
   if (adjustment)
@@ -1491,6 +1619,7 @@ gtk_spin_button_configure (GtkSpinButton  *spin_button,
   else
     adjustment = spin_button->adjustment;
 
+  g_object_freeze_notify (G_OBJECT (spin_button));
   if (spin_button->digits != digits) 
     {
       spin_button->digits = digits;
@@ -1500,8 +1629,9 @@ gtk_spin_button_configure (GtkSpinButton  *spin_button,
   if (spin_button->climb_rate != climb_rate)
     {
       spin_button->climb_rate = climb_rate;
-      g_object_notify (G_OBJECT (spin_button), "climb_rate");
+      g_object_notify (G_OBJECT (spin_button), "climb-rate");
     }
+  g_object_thaw_notify (G_OBJECT (spin_button));
 
   gtk_adjustment_value_changed (adjustment);
 }
@@ -1516,7 +1646,7 @@ gtk_spin_button_new (GtkAdjustment *adjustment,
   if (adjustment)
     g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), NULL);
 
-  spin = gtk_type_new (GTK_TYPE_SPIN_BUTTON);
+  spin = g_object_new (GTK_TYPE_SPIN_BUTTON, NULL);
 
   gtk_spin_button_configure (spin, adjustment, climb_rate, digits);
 
@@ -1533,9 +1663,13 @@ gtk_spin_button_new (GtkAdjustment *adjustment,
  * #GtkSpinButton without manually creating an adjustment. The value is 
  * initially set to the minimum value and a page increment of 10 * @step
  * is the default. The precision of the spin button is equivalent to the 
- * precision of @step.
+ * precision of @step. 
+ * 
+ * Note that the way in which the precision is derived works best if @step 
+ * is a power of ten. If the resulting precision is not suitable for your 
+ * needs, use gtk_spin_button_set_digits() to correct it.
  * 
- * Return value: the newly instantiated spin button
+ * Return value: The new spin button as a #GtkWidget.
  **/
 GtkWidget *
 gtk_spin_button_new_with_range (gdouble min,
@@ -1546,12 +1680,12 @@ gtk_spin_button_new_with_range (gdouble min,
   GtkSpinButton *spin;
   gint digits;
 
-  g_return_val_if_fail (min < max, NULL);
+  g_return_val_if_fail (min <= max, NULL);
   g_return_val_if_fail (step != 0.0, NULL);
 
-  spin = gtk_type_new (GTK_TYPE_SPIN_BUTTON);
+  spin = g_object_new (GTK_TYPE_SPIN_BUTTON, NULL);
 
-  adj = gtk_adjustment_new (min, min, max, step, 10 * step, step);
+  adj = gtk_adjustment_new (min, min, max, step, 10 * step, 0);
 
   if (fabs (step) >= 1.0 || step == 0.0)
     digits = 0;
@@ -1568,6 +1702,13 @@ gtk_spin_button_new_with_range (gdouble min,
   return GTK_WIDGET (spin);
 }
 
+static void
+warn_nonzero_page_size (GtkAdjustment *adjustment)
+{
+  if (gtk_adjustment_get_page_size (adjustment) != 0.0)
+    g_warning ("GtkSpinButton: setting an adjustment with non-zero page size is deprecated");
+}
+
 /* Callback used when the spin button's adjustment changes.  We need to redraw
  * the arrows when the adjustment's range changes, and reevaluate our size request.
  */
@@ -1578,6 +1719,8 @@ adjustment_changed_cb (GtkAdjustment *adjustment, gpointer data)
 
   spin_button = GTK_SPIN_BUTTON (data);
 
+  spin_button->timer_step = spin_button->adjustment->step_increment;
+  warn_nonzero_page_size (adjustment);
   gtk_widget_queue_resize (GTK_WIDGET (spin_button));
 }
 
@@ -1598,21 +1741,26 @@ gtk_spin_button_set_adjustment (GtkSpinButton *spin_button,
     {
       if (spin_button->adjustment)
         {
-          gtk_signal_disconnect_by_data (GTK_OBJECT (spin_button->adjustment),
-                                         (gpointer) spin_button);
-          gtk_object_unref (GTK_OBJECT (spin_button->adjustment));
+         g_signal_handlers_disconnect_by_func (spin_button->adjustment,
+                                               gtk_spin_button_value_changed,
+                                               spin_button);
+         g_signal_handlers_disconnect_by_func (spin_button->adjustment,
+                                               adjustment_changed_cb,
+                                               spin_button);
+         g_object_unref (spin_button->adjustment);
         }
       spin_button->adjustment = adjustment;
       if (adjustment)
         {
-          gtk_object_ref (GTK_OBJECT (adjustment));
-         gtk_object_sink (GTK_OBJECT (adjustment));
-          gtk_signal_connect (GTK_OBJECT (adjustment), "value_changed",
-                             (GtkSignalFunc) gtk_spin_button_value_changed,
-                             (gpointer) spin_button);
-         gtk_signal_connect (GTK_OBJECT (adjustment), "changed",
-                             (GtkSignalFunc) adjustment_changed_cb,
-                             (gpointer) spin_button);
+         g_object_ref_sink (adjustment);
+         g_signal_connect (adjustment, "value-changed",
+                           G_CALLBACK (gtk_spin_button_value_changed),
+                           spin_button);
+         g_signal_connect (adjustment, "changed",
+                           G_CALLBACK (adjustment_changed_cb),
+                           spin_button);
+         spin_button->timer_step = spin_button->adjustment->step_increment;
+          warn_nonzero_page_size (adjustment);
         }
 
       gtk_widget_queue_resize (GTK_WIDGET (spin_button));
@@ -1623,7 +1771,7 @@ gtk_spin_button_set_adjustment (GtkSpinButton *spin_button,
 
 /**
  * gtk_spin_button_get_adjustment:
- * @spin_button: 
+ * @spin_button: a #GtkSpinButton
  * 
  * Get the adjustment associated with a #GtkSpinButton
  * 
@@ -1640,7 +1788,7 @@ gtk_spin_button_get_adjustment (GtkSpinButton *spin_button)
 /**
  * gtk_spin_button_set_digits:
  * @spin_button: a #GtkSpinButton
- * @digits: the number of digits to be displayed for the spin button's value
+ * @digits: the number of digits after the decimal point to be displayed for the spin button's value
  * 
  * Set the precision to be displayed by @spin_button. Up to 20 digit precision
  * is allowed.
@@ -1700,9 +1848,9 @@ gtk_spin_button_set_increments (GtkSpinButton *spin_button,
 
 /**
  * gtk_spin_button_get_increments:
- * @psin_button: a #GtkSpinButton
- * @step: location to store step increment, or %NULL
- * @page: location to store page increment, or %NULL
+ * @spin_button: a #GtkSpinButton
+ * @step: (allow-none): location to store step increment, or %NULL
+ * @page: (allow-none): location to store page increment, or %NULL
  *
  * Gets the current step and page the increments used by @spin_button. See
  * gtk_spin_button_set_increments().
@@ -1753,8 +1901,8 @@ gtk_spin_button_set_range (GtkSpinButton *spin_button,
 /**
  * gtk_spin_button_get_range:
  * @spin_button: a #GtkSpinButton
- * @min: location to store minimum allowed value, or %NULL
- * @max: location to store maximum allowed value, or %NULL
+ * @min: (allow-none): location to store minimum allowed value, or %NULL
+ * @max: (allow-none): location to store maximum allowed value, or %NULL
  *
  * Gets the range allowed for @spin_button. See
  * gtk_spin_button_set_range().
@@ -1828,8 +1976,7 @@ gtk_spin_button_set_value (GtkSpinButton *spin_button,
   else
     {
       gint return_val = FALSE;
-      gtk_signal_emit (GTK_OBJECT (spin_button), spinbutton_signals[OUTPUT],
-                      &return_val);
+      g_signal_emit (spin_button, spinbutton_signals[OUTPUT], 0, &return_val);
       if (return_val == FALSE)
        gtk_spin_button_default_output (spin_button);
     }
@@ -1852,7 +1999,7 @@ gtk_spin_button_set_update_policy (GtkSpinButton             *spin_button,
   if (spin_button->update_policy != policy)
     {
       spin_button->update_policy = policy;
-      g_object_notify (G_OBJECT (spin_button), "update_policy");
+      g_object_notify (G_OBJECT (spin_button), "update-policy");
     }
 }
 
@@ -1887,9 +2034,13 @@ gtk_spin_button_set_numeric (GtkSpinButton  *spin_button,
 {
   g_return_if_fail (GTK_IS_SPIN_BUTTON (spin_button));
 
-  spin_button->numeric = (numeric != 0);
+  numeric = numeric != FALSE;
 
-  g_object_notify (G_OBJECT (spin_button), "numeric");
+  if (spin_button->numeric != numeric)
+    {
+       spin_button->numeric = numeric;
+       g_object_notify (G_OBJECT (spin_button), "numeric");
+    }
 }
 
 /**
@@ -1923,9 +2074,14 @@ gtk_spin_button_set_wrap (GtkSpinButton  *spin_button,
 {
   g_return_if_fail (GTK_IS_SPIN_BUTTON (spin_button));
 
-  spin_button->wrap = (wrap != 0);
+  wrap = wrap != FALSE; 
+
+  if (spin_button->wrap != wrap)
+    {
+       spin_button->wrap = (wrap != 0);
   
-  g_object_notify (G_OBJECT (spin_button), "wrap");
+       g_object_notify (G_OBJECT (spin_button), "wrap");
+    }
 }
 
 /**
@@ -1946,6 +2102,17 @@ gtk_spin_button_get_wrap (GtkSpinButton *spin_button)
   return spin_button->wrap;
 }
 
+static gint
+spin_button_get_arrow_size (GtkSpinButton *spin_button)
+{
+  gint size = pango_font_description_get_size (GTK_WIDGET (spin_button)->style->font_desc);
+  gint arrow_size;
+
+  arrow_size = MAX (PANGO_PIXELS (size), MIN_ARROW_WIDTH);
+
+  return arrow_size - arrow_size % 2; /* force even */
+}
+
 /**
  * spin_button_get_shadow_type:
  * @spin_button: a #GtkSpinButton 
@@ -1960,7 +2127,7 @@ spin_button_get_shadow_type (GtkSpinButton *spin_button)
 {
   GtkShadowType rc_shadow_type;
 
-  gtk_widget_style_get (GTK_WIDGET (spin_button), "shadow_type", &rc_shadow_type, NULL);
+  gtk_widget_style_get (GTK_WIDGET (spin_button), "shadow-type", &rc_shadow_type, NULL);
 
   return rc_shadow_type;
 }
@@ -1989,7 +2156,7 @@ gtk_spin_button_set_snap_to_ticks (GtkSpinButton *spin_button,
       if (new_val && GTK_ENTRY (spin_button)->editable)
        gtk_spin_button_update (spin_button);
       
-      g_object_notify (G_OBJECT (spin_button), "snap_to_ticks");
+      g_object_notify (G_OBJECT (spin_button), "snap-to-ticks");
     }
 }
 
@@ -2104,8 +2271,7 @@ gtk_spin_button_update (GtkSpinButton *spin_button)
   g_return_if_fail (GTK_IS_SPIN_BUTTON (spin_button));
 
   return_val = FALSE;
-  gtk_signal_emit (GTK_OBJECT (spin_button), spinbutton_signals[INPUT],
-                  &val, &return_val);
+  g_signal_emit (spin_button, spinbutton_signals[INPUT], 0, &val, &return_val);
   if (return_val == FALSE)
     {
       return_val = gtk_spin_button_default_input (spin_button, &val);
@@ -2114,6 +2280,8 @@ gtk_spin_button_update (GtkSpinButton *spin_button)
   else if (return_val == GTK_INPUT_ERROR)
     error = 1;
 
+  gtk_widget_queue_draw (GTK_WIDGET (spin_button));
+
   if (spin_button->update_policy == GTK_UPDATE_ALWAYS)
     {
       if (val < spin_button->adjustment->lower)
@@ -2133,17 +2301,8 @@ gtk_spin_button_update (GtkSpinButton *spin_button)
   if (spin_button->snap_to_ticks)
     gtk_spin_button_snap (spin_button, val);
   else
-    {
-      if (fabs (val - spin_button->adjustment->value) > EPSILON)
-       gtk_adjustment_set_value (spin_button->adjustment, val);
-      else
-       {
-         return_val = FALSE;
-         gtk_signal_emit (GTK_OBJECT (spin_button), spinbutton_signals[OUTPUT],
-                          &return_val);
-         if (return_val == FALSE)
-           gtk_spin_button_default_output (spin_button);
-       }
-    }
+    gtk_spin_button_set_value (spin_button, val);
 }
 
+#define __GTK_SPIN_BUTTON_C__
+#include "gtkaliasdef.c"