Gimp/app/widgets/gimpenumaction.c
Ell 114d49510f app: log actions upon activation before emitting "selected" signal
In GimpAction, instead of connecting the action-history log
function to the action's "activate" signal as a user-provided
handler, call it directly from the default handler.

In subclasses of GimpAction, chain to the parent's activate()
function before emitting the "selected" signal, so that we always
log the action in the history before responding to it.

This allows us to avoid the hack in commit
6544ce4301.
2018-07-06 00:08:00 -04:00

185 lines
5.6 KiB
C

/* GIMP - The GNU Image Manipulation Program
* Copyright (C) 1995 Spencer Kimball and Peter Mattis
*
* gimpenumaction.c
* Copyright (C) 2004 Michael Natterer <mitch@gimp.org>
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 3 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*/
#include "config.h"
#include <gtk/gtk.h>
#include "widgets-types.h"
#include "core/gimpmarshal.h"
#include "gimpenumaction.h"
enum
{
SELECTED,
LAST_SIGNAL
};
enum
{
PROP_0,
PROP_VALUE,
PROP_VALUE_VARIABLE
};
static void gimp_enum_action_set_property (GObject *object,
guint prop_id,
const GValue *value,
GParamSpec *pspec);
static void gimp_enum_action_get_property (GObject *object,
guint prop_id,
GValue *value,
GParamSpec *pspec);
static void gimp_enum_action_activate (GtkAction *action);
G_DEFINE_TYPE (GimpEnumAction, gimp_enum_action, GIMP_TYPE_ACTION)
#define parent_class gimp_enum_action_parent_class
static guint action_signals[LAST_SIGNAL] = { 0 };
static void
gimp_enum_action_class_init (GimpEnumActionClass *klass)
{
GObjectClass *object_class = G_OBJECT_CLASS (klass);
GtkActionClass *action_class = GTK_ACTION_CLASS (klass);
object_class->set_property = gimp_enum_action_set_property;
object_class->get_property = gimp_enum_action_get_property;
action_class->activate = gimp_enum_action_activate;
g_object_class_install_property (object_class, PROP_VALUE,
g_param_spec_int ("value",
NULL, NULL,
G_MININT, G_MAXINT, 0,
GIMP_PARAM_READWRITE));
g_object_class_install_property (object_class, PROP_VALUE_VARIABLE,
g_param_spec_boolean ("value-variable",
NULL, NULL,
FALSE,
GIMP_PARAM_READWRITE));
action_signals[SELECTED] =
g_signal_new ("selected",
G_TYPE_FROM_CLASS (klass),
G_SIGNAL_RUN_FIRST,
G_STRUCT_OFFSET (GimpEnumActionClass, selected),
NULL, NULL,
gimp_marshal_VOID__INT,
G_TYPE_NONE, 1,
G_TYPE_INT);
}
static void
gimp_enum_action_init (GimpEnumAction *action)
{
action->value = 0;
action->value_variable = FALSE;
}
static void
gimp_enum_action_get_property (GObject *object,
guint prop_id,
GValue *value,
GParamSpec *pspec)
{
GimpEnumAction *action = GIMP_ENUM_ACTION (object);
switch (prop_id)
{
case PROP_VALUE:
g_value_set_int (value, action->value);
break;
case PROP_VALUE_VARIABLE:
g_value_set_boolean (value, action->value_variable);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
break;
}
}
static void
gimp_enum_action_set_property (GObject *object,
guint prop_id,
const GValue *value,
GParamSpec *pspec)
{
GimpEnumAction *action = GIMP_ENUM_ACTION (object);
switch (prop_id)
{
case PROP_VALUE:
action->value = g_value_get_int (value);
break;
case PROP_VALUE_VARIABLE:
action->value_variable = g_value_get_boolean (value);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
break;
}
}
GimpEnumAction *
gimp_enum_action_new (const gchar *name,
const gchar *label,
const gchar *tooltip,
const gchar *icon_name,
gint value,
gboolean value_variable)
{
return g_object_new (GIMP_TYPE_ENUM_ACTION,
"name", name,
"label", label,
"tooltip", tooltip,
"icon-name", icon_name,
"value", value,
"value-variable", value_variable,
NULL);
}
static void
gimp_enum_action_activate (GtkAction *action)
{
GimpEnumAction *enum_action = GIMP_ENUM_ACTION (action);
GTK_ACTION_CLASS (parent_class)->activate (action);
gimp_enum_action_selected (enum_action, enum_action->value);
}
void
gimp_enum_action_selected (GimpEnumAction *action,
gint value)
{
g_return_if_fail (GIMP_IS_ENUM_ACTION (action));
g_signal_emit (action, action_signals[SELECTED], 0, value);
}