* Library General Public License for more details.
*
* You should have received a copy of the GNU Library General Public
- * License along with the Gnome Library; see the file COPYING.LIB. If not,
- * write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
+ * License along with this library. If not, see <http://www.gnu.org/licenses/>.
*/
/*
* GTK+ at ftp://ftp.gtk.org/pub/gtk/.
*/
-#include <config.h>
+/**
+ * SECTION:gtkactiongroup
+ * @Short_description: A group of actions
+ * @Title: GtkActionGroup
+ *
+ * Actions are organised into groups. An action group is essentially a
+ * map from names to #GtkAction objects.
+ *
+ * All actions that would make sense to use in a particular context
+ * should be in a single group. Multiple action groups may be used for a
+ * particular user interface. In fact, it is expected that most nontrivial
+ * applications will make use of multiple groups. For example, in an
+ * application that can edit multiple documents, one group holding global
+ * actions (e.g. quit, about, new), and one group per document holding
+ * actions that act on that document (eg. save, cut/copy/paste, etc). Each
+ * window's menus would be constructed from a combination of two action
+ * groups.
+ * </para>
+ * <para id="Action-Accel">
+ * Accelerators are handled by the GTK+ accelerator map. All actions are
+ * assigned an accelerator path (which normally has the form
+ * <literal><Actions>/group-name/action-name</literal>)
+ * and a shortcut is associated with this accelerator path. All menuitems
+ * and toolitems take on this accelerator path. The GTK+ accelerator map
+ * code makes sure that the correct shortcut is displayed next to the menu
+ * item.
+ *
+ * <refsect2 id="GtkActionGroup-BUILDER-UI">
+ * <title>GtkActionGroup as GtkBuildable</title>
+ * <para>
+ * The #GtkActionGroup implementation of the #GtkBuildable interface accepts
+ * #GtkAction objects as <child> elements in UI definitions.
+ *
+ * Note that it is probably more common to define actions and action groups
+ * in the code, since they are directly related to what the code can do.
+ *
+ * The GtkActionGroup implementation of the GtkBuildable interface supports
+ * a custom <accelerator> element, which has attributes named key and
+ * modifiers and allows to specify accelerators. This is similar to the
+ * <accelerator> element of #GtkWidget, the main difference is that
+ * it doesn't allow you to specify a signal.
+ * </para>
+ * <example>
+ * <title>A #GtkDialog UI definition fragment.</title>
+ * <programlisting><![CDATA[
+ * <object class="GtkActionGroup" id="actiongroup">
+ * <child>
+ * <object class="GtkAction" id="About">
+ * <property name="name">About</property>
+ * <property name="stock_id">gtk-about</property>
+ * <signal handler="about_activate" name="activate"/>
+ * </object>
+ * <accelerator key="F1" modifiers="GDK_CONTROL_MASK | GDK_SHIFT_MASK"/>
+ * </child>
+ * </object>
+ * ]]></programlisting>
+ * </example>
+ * </refsect2>
+ */
+
+#include "config.h"
+#include <string.h>
-#include "gtkalias.h"
#include "gtkactiongroup.h"
+#include "gtkbuildable.h"
+#include "gtkiconfactory.h"
+#include "gtkicontheme.h"
#include "gtkstock.h"
#include "gtktoggleaction.h"
#include "gtkradioaction.h"
#include "gtkaccelmap.h"
#include "gtkmarshalers.h"
+#include "gtkbuilderprivate.h"
+#include "gtkprivate.h"
#include "gtkintl.h"
-#define GTK_ACTION_GROUP_GET_PRIVATE(obj) (G_TYPE_INSTANCE_GET_PRIVATE ((obj), GTK_TYPE_ACTION_GROUP, GtkActionGroupPrivate))
struct _GtkActionGroupPrivate
{
GtkTranslateFunc translate_func;
gpointer translate_data;
- GtkDestroyNotify translate_notify;
+ GDestroyNotify translate_notify;
};
enum
static GtkAction *gtk_action_group_real_get_action (GtkActionGroup *self,
const gchar *name);
+/* GtkBuildable */
+static void gtk_action_group_buildable_init (GtkBuildableIface *iface);
+static void gtk_action_group_buildable_add_child (GtkBuildable *buildable,
+ GtkBuilder *builder,
+ GObject *child,
+ const gchar *type);
+static void gtk_action_group_buildable_set_name (GtkBuildable *buildable,
+ const gchar *name);
+static const gchar* gtk_action_group_buildable_get_name (GtkBuildable *buildable);
+static gboolean gtk_action_group_buildable_custom_tag_start (GtkBuildable *buildable,
+ GtkBuilder *builder,
+ GObject *child,
+ const gchar *tagname,
+ GMarkupParser *parser,
+ gpointer *data);
+static void gtk_action_group_buildable_custom_tag_end (GtkBuildable *buildable,
+ GtkBuilder *builder,
+ GObject *child,
+ const gchar *tagname,
+ gpointer *user_data);
GType
gtk_action_group_get_type (void)
if (!type)
{
- static const GTypeInfo type_info =
+ const GTypeInfo type_info =
{
sizeof (GtkActionGroupClass),
NULL, /* base_init */
(GInstanceInitFunc) gtk_action_group_init,
};
- type = g_type_register_static (G_TYPE_OBJECT, "GtkActionGroup",
+ const GInterfaceInfo buildable_info =
+ {
+ (GInterfaceInitFunc) gtk_action_group_buildable_init,
+ NULL,
+ NULL
+ };
+
+ type = g_type_register_static (G_TYPE_OBJECT, I_("GtkActionGroup"),
&type_info, 0);
- }
+ g_type_add_interface_static (type,
+ GTK_TYPE_BUILDABLE,
+ &buildable_info);
+ }
return type;
}
P_("Name"),
P_("A name for the action group."),
NULL,
- G_PARAM_READWRITE |
- G_PARAM_CONSTRUCT_ONLY));
+ GTK_PARAM_READWRITE | G_PARAM_CONSTRUCT_ONLY));
g_object_class_install_property (gobject_class,
PROP_SENSITIVE,
g_param_spec_boolean ("sensitive",
P_("Sensitive"),
P_("Whether the action group is enabled."),
TRUE,
- G_PARAM_READWRITE));
+ GTK_PARAM_READWRITE));
g_object_class_install_property (gobject_class,
PROP_VISIBLE,
g_param_spec_boolean ("visible",
P_("Visible"),
P_("Whether the action group is visible."),
TRUE,
- G_PARAM_READWRITE));
+ GTK_PARAM_READWRITE));
/**
* GtkActionGroup::connect-proxy:
* @action: the action
* @proxy: the proxy
*
- * The connect_proxy signal is emitted after connecting a proxy to
+ * The ::connect-proxy signal is emitted after connecting a proxy to
* an action in the group. Note that the proxy may have been connected
* to a different action before.
*
* Since: 2.4
*/
action_group_signals[CONNECT_PROXY] =
- g_signal_new ("connect_proxy",
+ g_signal_new (I_("connect-proxy"),
G_OBJECT_CLASS_TYPE (klass),
0, 0, NULL, NULL,
_gtk_marshal_VOID__OBJECT_OBJECT,
* @action: the action
* @proxy: the proxy
*
- * The disconnect_proxy signal is emitted after disconnecting a proxy
+ * The ::disconnect-proxy signal is emitted after disconnecting a proxy
* from an action in the group.
*
* #GtkUIManager proxies the signal and provides global notification
* Since: 2.4
*/
action_group_signals[DISCONNECT_PROXY] =
- g_signal_new ("disconnect_proxy",
+ g_signal_new (I_("disconnect-proxy"),
G_OBJECT_CLASS_TYPE (klass),
0, 0, NULL, NULL,
_gtk_marshal_VOID__OBJECT_OBJECT,
* @action_group: the group
* @action: the action
*
- * The pre_activate signal is emitted just before the @action in the
+ * The ::pre-activate signal is emitted just before the @action in the
* @action_group is activated
*
* This is intended for #GtkUIManager to proxy the signal and provide global
* Since: 2.4
*/
action_group_signals[PRE_ACTIVATE] =
- g_signal_new ("pre_activate",
+ g_signal_new (I_("pre-activate"),
G_OBJECT_CLASS_TYPE (klass),
0, 0, NULL, NULL,
_gtk_marshal_VOID__OBJECT,
* @action_group: the group
* @action: the action
*
- * The post_activate signal is emitted just after the @action in the
+ * The ::post-activate signal is emitted just after the @action in the
* @action_group is activated
*
* This is intended for #GtkUIManager to proxy the signal and provide global
* Since: 2.4
*/
action_group_signals[POST_ACTIVATE] =
- g_signal_new ("post_activate",
+ g_signal_new (I_("post-activate"),
G_OBJECT_CLASS_TYPE (klass),
0, 0, NULL, NULL,
_gtk_marshal_VOID__OBJECT,
static void
remove_action (GtkAction *action)
{
- g_object_set (action, "action_group", NULL, NULL);
+ g_object_set (action, I_("action-group"), NULL, NULL);
g_object_unref (action);
}
static void
-gtk_action_group_init (GtkActionGroup *self)
+gtk_action_group_init (GtkActionGroup *action_group)
+{
+ GtkActionGroupPrivate *private;
+
+ action_group->priv = G_TYPE_INSTANCE_GET_PRIVATE (action_group,
+ GTK_TYPE_ACTION_GROUP,
+ GtkActionGroupPrivate);
+ private = action_group->priv;
+
+ private->name = NULL;
+ private->sensitive = TRUE;
+ private->visible = TRUE;
+ private->actions = g_hash_table_new_full (g_str_hash, g_str_equal,
+ NULL,
+ (GDestroyNotify) remove_action);
+ private->translate_func = NULL;
+ private->translate_data = NULL;
+ private->translate_notify = NULL;
+}
+
+static void
+gtk_action_group_buildable_init (GtkBuildableIface *iface)
{
- self->private_data = GTK_ACTION_GROUP_GET_PRIVATE (self);
- self->private_data->name = NULL;
- self->private_data->sensitive = TRUE;
- self->private_data->visible = TRUE;
- self->private_data->actions = g_hash_table_new_full (g_str_hash, g_str_equal,
- (GDestroyNotify) g_free,
- (GDestroyNotify) remove_action);
- self->private_data->translate_func = NULL;
- self->private_data->translate_data = NULL;
- self->private_data->translate_notify = NULL;
+ iface->add_child = gtk_action_group_buildable_add_child;
+ iface->set_name = gtk_action_group_buildable_set_name;
+ iface->get_name = gtk_action_group_buildable_get_name;
+ iface->custom_tag_start = gtk_action_group_buildable_custom_tag_start;
+ iface->custom_tag_end = gtk_action_group_buildable_custom_tag_end;
+}
+
+static void
+gtk_action_group_buildable_add_child (GtkBuildable *buildable,
+ GtkBuilder *builder,
+ GObject *child,
+ const gchar *type)
+{
+ gtk_action_group_add_action_with_accel (GTK_ACTION_GROUP (buildable),
+ GTK_ACTION (child), NULL);
+}
+
+static void
+gtk_action_group_buildable_set_name (GtkBuildable *buildable,
+ const gchar *name)
+{
+ GtkActionGroup *self = GTK_ACTION_GROUP (buildable);
+ GtkActionGroupPrivate *private = self->priv;
+
+ private->name = g_strdup (name);
+}
+
+static const gchar *
+gtk_action_group_buildable_get_name (GtkBuildable *buildable)
+{
+ GtkActionGroup *self = GTK_ACTION_GROUP (buildable);
+ GtkActionGroupPrivate *private = self->priv;
+
+ return private->name;
+}
+
+typedef struct {
+ GObject *child;
+ guint key;
+ GdkModifierType modifiers;
+} AcceleratorParserData;
+
+static void
+accelerator_start_element (GMarkupParseContext *context,
+ const gchar *element_name,
+ const gchar **names,
+ const gchar **values,
+ gpointer user_data,
+ GError **error)
+{
+ gint i;
+ guint key = 0;
+ GdkModifierType modifiers = 0;
+ AcceleratorParserData *parser_data = (AcceleratorParserData*)user_data;
+
+ if (strcmp (element_name, "accelerator") != 0)
+ g_warning ("Unknown <accelerator> tag: %s", element_name);
+
+ for (i = 0; names[i]; i++)
+ {
+ if (strcmp (names[i], "key") == 0)
+ key = gdk_keyval_from_name (values[i]);
+ else if (strcmp (names[i], "modifiers") == 0)
+ {
+ if (!_gtk_builder_flags_from_string (GDK_TYPE_MODIFIER_TYPE,
+ values[i],
+ &modifiers,
+ error))
+ return;
+ }
+ }
+
+ if (key == 0)
+ {
+ g_warning ("<accelerator> requires a key attribute");
+ return;
+ }
+ parser_data->key = key;
+ parser_data->modifiers = modifiers;
+}
+
+static const GMarkupParser accelerator_parser =
+ {
+ accelerator_start_element
+ };
+
+static gboolean
+gtk_action_group_buildable_custom_tag_start (GtkBuildable *buildable,
+ GtkBuilder *builder,
+ GObject *child,
+ const gchar *tagname,
+ GMarkupParser *parser,
+ gpointer *user_data)
+{
+ AcceleratorParserData *parser_data;
+
+ if (child && strcmp (tagname, "accelerator") == 0)
+ {
+ parser_data = g_slice_new0 (AcceleratorParserData);
+ parser_data->child = child;
+ *user_data = parser_data;
+ *parser = accelerator_parser;
+
+ return TRUE;
+ }
+ return FALSE;
+}
+
+static void
+gtk_action_group_buildable_custom_tag_end (GtkBuildable *buildable,
+ GtkBuilder *builder,
+ GObject *child,
+ const gchar *tagname,
+ gpointer *user_data)
+{
+ AcceleratorParserData *data;
+
+ if (strcmp (tagname, "accelerator") == 0)
+ {
+ GtkActionGroup *action_group;
+ GtkActionGroupPrivate *private;
+ GtkAction *action;
+ gchar *accel_path;
+
+ data = (AcceleratorParserData*)user_data;
+ action_group = GTK_ACTION_GROUP (buildable);
+ private = action_group->priv;
+ action = GTK_ACTION (child);
+
+ accel_path = g_strconcat ("<Actions>/",
+ private->name, "/",
+ gtk_action_get_name (action), NULL);
+
+ if (gtk_accel_map_lookup_entry (accel_path, NULL))
+ gtk_accel_map_change_entry (accel_path, data->key, data->modifiers, TRUE);
+ else
+ gtk_accel_map_add_entry (accel_path, data->key, data->modifiers);
+
+ gtk_action_set_accel_path (action, accel_path);
+
+ g_free (accel_path);
+ g_slice_free (AcceleratorParserData, data);
+ }
}
/**
gtk_action_group_new (const gchar *name)
{
GtkActionGroup *self;
+ GtkActionGroupPrivate *private;
self = g_object_new (GTK_TYPE_ACTION_GROUP, NULL);
- self->private_data->name = g_strdup (name);
+ private = self->priv;
+ private->name = g_strdup (name);
return self;
}
gtk_action_group_finalize (GObject *object)
{
GtkActionGroup *self;
+ GtkActionGroupPrivate *private;
self = GTK_ACTION_GROUP (object);
+ private = self->priv;
- g_free (self->private_data->name);
- self->private_data->name = NULL;
+ g_free (private->name);
+ private->name = NULL;
- g_hash_table_destroy (self->private_data->actions);
- self->private_data->actions = NULL;
+ g_hash_table_destroy (private->actions);
+ private->actions = NULL;
- if (self->private_data->translate_notify)
- self->private_data->translate_notify (self->private_data->translate_data);
+ if (private->translate_notify)
+ private->translate_notify (private->translate_data);
- if (parent_class->finalize)
- (* parent_class->finalize) (object);
+ parent_class->finalize (object);
}
static void
GParamSpec *pspec)
{
GtkActionGroup *self;
+ GtkActionGroupPrivate *private;
gchar *tmp;
self = GTK_ACTION_GROUP (object);
+ private = self->priv;
switch (prop_id)
{
case PROP_NAME:
- tmp = self->private_data->name;
- self->private_data->name = g_value_dup_string (value);
+ tmp = private->name;
+ private->name = g_value_dup_string (value);
g_free (tmp);
break;
case PROP_SENSITIVE:
GParamSpec *pspec)
{
GtkActionGroup *self;
+ GtkActionGroupPrivate *private;
self = GTK_ACTION_GROUP (object);
+ private = self->priv;
switch (prop_id)
{
case PROP_NAME:
- g_value_set_string (value, self->private_data->name);
+ g_value_set_string (value, private->name);
break;
case PROP_SENSITIVE:
- g_value_set_boolean (value, self->private_data->sensitive);
+ g_value_set_boolean (value, private->sensitive);
break;
case PROP_VISIBLE:
- g_value_set_boolean (value, self->private_data->visible);
+ g_value_set_boolean (value, private->visible);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
gtk_action_group_real_get_action (GtkActionGroup *self,
const gchar *action_name)
{
- return g_hash_table_lookup (self->private_data->actions, action_name);
+ GtkActionGroupPrivate *private;
+
+ private = self->priv;
+
+ return g_hash_table_lookup (private->actions, action_name);
}
/**
const gchar *
gtk_action_group_get_name (GtkActionGroup *action_group)
{
+ GtkActionGroupPrivate *private;
+
g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), NULL);
- return action_group->private_data->name;
+ private = action_group->priv;
+
+ return private->name;
}
/**
gboolean
gtk_action_group_get_sensitive (GtkActionGroup *action_group)
{
+ GtkActionGroupPrivate *private;
+
g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), FALSE);
- return action_group->private_data->sensitive;
+ private = action_group->priv;
+
+ return private->sensitive;
}
static void
-cb_set_action_sensitivity (const gchar *name, GtkAction *action)
+cb_set_action_sensitivity (const gchar *name,
+ GtkAction *action)
{
- /* Minor optimization, the action_groups state only effects actions that are
- * themselves sensitive */
- if (gtk_action_get_sensitive (action))
- g_object_notify (G_OBJECT (action), "sensitive");
+ /* Minor optimization, the action_groups state only affects actions
+ * that are themselves sensitive */
+ g_object_notify (G_OBJECT (action), "sensitive");
+
}
/**
* Since: 2.4
*/
void
-gtk_action_group_set_sensitive (GtkActionGroup *action_group, gboolean sensitive)
+gtk_action_group_set_sensitive (GtkActionGroup *action_group,
+ gboolean sensitive)
{
+ GtkActionGroupPrivate *private;
+
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
- if (action_group->private_data->sensitive ^ sensitive)
+ private = action_group->priv;
+ sensitive = sensitive != FALSE;
+
+ if (private->sensitive != sensitive)
{
- action_group->private_data->sensitive = sensitive;
- g_hash_table_foreach (action_group->private_data->actions,
+ private->sensitive = sensitive;
+ g_hash_table_foreach (private->actions,
(GHFunc) cb_set_action_sensitivity, NULL);
+
+ g_object_notify (G_OBJECT (action_group), "sensitive");
}
}
gboolean
gtk_action_group_get_visible (GtkActionGroup *action_group)
{
+ GtkActionGroupPrivate *private;
+
g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), FALSE);
- return action_group->private_data->visible;
+ private = action_group->priv;
+
+ return private->visible;
}
static void
-cb_set_action_visiblity (const gchar *name, GtkAction *action)
+cb_set_action_visiblity (const gchar *name,
+ GtkAction *action)
{
- /* Minor optimization, the action_groups state only effects actions that are
- * themselves sensitive */
- if (gtk_action_get_visible (action))
- g_object_notify (G_OBJECT (action), "visible");
+ /* Minor optimization, the action_groups state only affects actions
+ * that are themselves visible */
+ g_object_notify (G_OBJECT (action), "visible");
}
/**
* Since: 2.4
*/
void
-gtk_action_group_set_visible (GtkActionGroup *action_group, gboolean visible)
+gtk_action_group_set_visible (GtkActionGroup *action_group,
+ gboolean visible)
{
+ GtkActionGroupPrivate *private;
+
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
- if (action_group->private_data->visible ^ visible)
+ private = action_group->priv;
+ visible = visible != FALSE;
+
+ if (private->visible != visible)
{
- action_group->private_data->visible = visible;
- g_hash_table_foreach (action_group->private_data->actions,
+ private->visible = visible;
+ g_hash_table_foreach (private->actions,
(GHFunc) cb_set_action_visiblity, NULL);
+
+ g_object_notify (G_OBJECT (action_group), "visible");
}
}
*
* Looks up an action in the action group by name.
*
- * Returns: the action, or %NULL if no action by that name exists
+ * Returns: (transfer none): the action, or %NULL if no action by that name exists
*
* Since: 2.4
*/
g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), NULL);
g_return_val_if_fail (GTK_ACTION_GROUP_GET_CLASS (action_group)->get_action != NULL, NULL);
- return (* GTK_ACTION_GROUP_GET_CLASS (action_group)->get_action)
- (action_group, action_name);
+ return GTK_ACTION_GROUP_GET_CLASS (action_group)->get_action (action_group,
+ action_name);
+}
+
+static gboolean
+check_unique_action (GtkActionGroup *action_group,
+ const gchar *action_name)
+{
+ if (gtk_action_group_get_action (action_group, action_name) != NULL)
+ {
+ GtkActionGroupPrivate *private;
+
+ private = action_group->priv;
+
+ g_warning ("Refusing to add non-unique action '%s' to action group '%s'",
+ action_name,
+ private->name);
+ return FALSE;
+ }
+
+ return TRUE;
}
/**
gtk_action_group_add_action (GtkActionGroup *action_group,
GtkAction *action)
{
+ GtkActionGroupPrivate *private;
+ const gchar *name;
+
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
g_return_if_fail (GTK_IS_ACTION (action));
- g_return_if_fail (gtk_action_get_name (action) != NULL);
- g_hash_table_insert (action_group->private_data->actions,
- g_strdup (gtk_action_get_name (action)),
+ name = gtk_action_get_name (action);
+ g_return_if_fail (name != NULL);
+
+ if (!check_unique_action (action_group, name))
+ return;
+
+ private = action_group->priv;
+
+ g_hash_table_insert (private->actions,
+ (gpointer) name,
g_object_ref (action));
- g_object_set (G_OBJECT (action), "action_group", action_group, NULL);
+ g_object_set (action, I_("action-group"), action_group, NULL);
}
/**
* gtk_action_group_add_action_with_accel:
- * @action_group: the action group
- * @action: the action to add
- * @accelerator: the accelerator for the action, in
- * the format understood by gtk_accelerator_parse(), or "" for no accelerator, or
- * %NULL to use the stock accelerator
+ * @action_group: the action group
+ * @action: the action to add
+ * @accelerator: (allow-none): the accelerator for the action, in
+ * the format understood by gtk_accelerator_parse(), or "" for no accelerator, or
+ * %NULL to use the stock accelerator
*
* Adds an action object to the action group and sets up the accelerator.
*
*/
void
gtk_action_group_add_action_with_accel (GtkActionGroup *action_group,
- GtkAction *action,
- const gchar *accelerator)
+ GtkAction *action,
+ const gchar *accelerator)
{
+ GtkActionGroupPrivate *private;
gchar *accel_path;
guint accel_key = 0;
GdkModifierType accel_mods;
- GtkStockItem stock_item;
- gchar *name;
- gchar *stock_id;
-
- g_object_get (action, "name", &name, "stock_id", &stock_id, NULL);
+ const gchar *name;
+
+ name = gtk_action_get_name (action);
+ if (!check_unique_action (action_group, name))
+ return;
+ private = action_group->priv;
accel_path = g_strconcat ("<Actions>/",
- action_group->private_data->name, "/", name, NULL);
+ private->name, "/", name, NULL);
if (accelerator)
{
accelerator, name);
}
}
- else if (stock_id && gtk_stock_lookup (stock_id, &stock_item))
+ else
{
- accel_key = stock_item.keyval;
- accel_mods = stock_item.modifier;
+ gchar *stock_id;
+ GtkStockItem stock_item;
+
+ g_object_get (action, "stock-id", &stock_id, NULL);
+
+ if (stock_id && gtk_stock_lookup (stock_id, &stock_item))
+ {
+ accel_key = stock_item.keyval;
+ accel_mods = stock_item.modifier;
+ }
+
+ g_free (stock_id);
}
if (accel_key)
gtk_action_group_add_action (action_group, action);
g_free (accel_path);
- g_free (stock_id);
- g_free (name);
}
/**
gtk_action_group_remove_action (GtkActionGroup *action_group,
GtkAction *action)
{
+ GtkActionGroupPrivate *private;
+ const gchar *name;
+
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
g_return_if_fail (GTK_IS_ACTION (action));
- g_return_if_fail (gtk_action_get_name (action) != NULL);
- /* extra protection to make sure action->name is valid */
- g_object_ref (action);
- g_hash_table_remove (action_group->private_data->actions, gtk_action_get_name (action));
- g_object_unref (action);
+ name = gtk_action_get_name (action);
+ g_return_if_fail (name != NULL);
+
+ private = action_group->priv;
+
+ g_hash_table_remove (private->actions, name);
}
static void
*
* Lists the actions in the action group.
*
- * Returns: an allocated list of the action objects in the action group
- *
+ * Returns: (element-type GtkAction) (transfer container): an allocated list of the action objects in the action group
+ *
* Since: 2.4
*/
GList *
gtk_action_group_list_actions (GtkActionGroup *action_group)
{
+ GtkActionGroupPrivate *private;
GList *actions = NULL;
+
g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), NULL);
+
+ private = action_group->priv;
- g_hash_table_foreach (action_group->private_data->actions, add_single_action, &actions);
+ g_hash_table_foreach (private->actions, add_single_action, &actions);
return g_list_reverse (actions);
}
/**
- * gtk_action_group_add_actions:
+ * gtk_action_group_add_actions: (skip)
* @action_group: the action group
- * @entries: an array of action descriptions
+ * @entries: (array length=n_entries): an array of action descriptions
* @n_entries: the number of entries
* @user_data: data to pass to the action callbacks
*
shared_data->ref_count--;
if (shared_data->ref_count == 0)
{
- if (shared_data->destroy)
- (*shared_data->destroy) (shared_data->data);
-
- g_free (shared_data);
+ if (shared_data->destroy)
+ shared_data->destroy (shared_data->data);
+
+ g_slice_free (SharedData, shared_data);
}
}
/**
- * gtk_action_group_add_actions_full:
+ * gtk_action_group_add_actions_full: (skip)
* @action_group: the action group
- * @entries: an array of action descriptions
+ * @entries: (array length=n_entries): an array of action descriptions
* @n_entries: the number of entries
* @user_data: data to pass to the action callbacks
* @destroy: destroy notification callback for @user_data
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
- shared_data = g_new0 (SharedData, 1);
+ shared_data = g_slice_new0 (SharedData);
shared_data->ref_count = 1;
shared_data->data = user_data;
shared_data->destroy = destroy;
const gchar *label;
const gchar *tooltip;
+ if (!check_unique_action (action_group, entries[i].name))
+ continue;
+
label = gtk_action_group_translate_string (action_group, entries[i].label);
tooltip = gtk_action_group_translate_string (action_group, entries[i].tooltip);
action = gtk_action_new (entries[i].name,
label,
tooltip,
- entries[i].stock_id);
+ NULL);
+ if (entries[i].stock_id)
+ {
+ g_object_set (action, "stock-id", entries[i].stock_id, NULL);
+ if (gtk_icon_theme_has_icon (gtk_icon_theme_get_default (),
+ entries[i].stock_id))
+ g_object_set (action, "icon-name", entries[i].stock_id, NULL);
+ }
+
if (entries[i].callback)
{
GClosure *closure;
}
/**
- * gtk_action_group_add_toggle_actions:
+ * gtk_action_group_add_toggle_actions: (skip)
* @action_group: the action group
- * @entries: an array of toggle action descriptions
+ * @entries: (array length=n_entries): an array of toggle action descriptions
* @n_entries: the number of entries
* @user_data: data to pass to the action callbacks
*
/**
- * gtk_action_group_add_toggle_actions_full:
+ * gtk_action_group_add_toggle_actions_full: (skip)
* @action_group: the action group
- * @entries: an array of toggle action descriptions
+ * @entries: (array length=n_entries): an array of toggle action descriptions
* @n_entries: the number of entries
* @user_data: data to pass to the action callbacks
* @destroy: destroy notification callback for @user_data
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
- shared_data = g_new0 (SharedData, 1);
+ shared_data = g_slice_new0 (SharedData);
shared_data->ref_count = 1;
shared_data->data = user_data;
shared_data->destroy = destroy;
const gchar *label;
const gchar *tooltip;
+ if (!check_unique_action (action_group, entries[i].name))
+ continue;
+
label = gtk_action_group_translate_string (action_group, entries[i].label);
tooltip = gtk_action_group_translate_string (action_group, entries[i].tooltip);
action = gtk_toggle_action_new (entries[i].name,
label,
tooltip,
- entries[i].stock_id);
+ NULL);
+
+ if (entries[i].stock_id)
+ {
+ if (gtk_icon_factory_lookup_default (entries[i].stock_id))
+ g_object_set (action, "stock-id", entries[i].stock_id, NULL);
+ else
+ g_object_set (action, "icon-name", entries[i].stock_id, NULL);
+ }
gtk_toggle_action_set_active (action, entries[i].is_active);
g_object_unref (action);
}
- shared_data_unref (shared_data);
+ shared_data_unref (shared_data);
}
/**
- * gtk_action_group_add_radio_actions:
+ * gtk_action_group_add_radio_actions: (skip)
* @action_group: the action group
- * @entries: an array of radio action descriptions
+ * @entries: (array length=n_entries): an array of radio action descriptions
* @n_entries: the number of entries
* @value: the value of the action to activate initially, or -1 if
* no action should be activated
}
/**
- * gtk_action_group_add_radio_actions_full:
+ * gtk_action_group_add_radio_actions_full: (skip)
* @action_group: the action group
- * @entries: an array of radio action descriptions
+ * @entries: (array length=n_entries): an array of radio action descriptions
* @n_entries: the number of entries
* @value: the value of the action to activate initially, or -1 if
* no action should be activated
const gchar *label;
const gchar *tooltip;
+ if (!check_unique_action (action_group, entries[i].name))
+ continue;
+
label = gtk_action_group_translate_string (action_group, entries[i].label);
tooltip = gtk_action_group_translate_string (action_group, entries[i].tooltip);
action = gtk_radio_action_new (entries[i].name,
label,
tooltip,
- entries[i].stock_id,
+ NULL,
entries[i].value);
+ if (entries[i].stock_id)
+ {
+ if (gtk_icon_factory_lookup_default (entries[i].stock_id))
+ g_object_set (action, "stock-id", entries[i].stock_id, NULL);
+ else
+ g_object_set (action, "icon-name", entries[i].stock_id, NULL);
+ }
+
if (i == 0)
first_action = action;
* @action_group: a #GtkActionGroup
* @func: a #GtkTranslateFunc
* @data: data to be passed to @func and @notify
- * @notify: a #GtkDestroyNotify function to be called when @action_group is
+ * @notify: a #GDestroyNotify function to be called when @action_group is
* destroyed and when the translation function is changed again
- *
+ *
* Sets a function to be used for translating the @label and @tooltip of
* #GtkActionGroupEntry<!-- -->s added by gtk_action_group_add_actions().
*
* Since: 2.4
**/
void
-gtk_action_group_set_translate_func (GtkActionGroup *action_group,
- GtkTranslateFunc func,
- gpointer data,
- GtkDestroyNotify notify)
+gtk_action_group_set_translate_func (GtkActionGroup *action_group,
+ GtkTranslateFunc func,
+ gpointer data,
+ GDestroyNotify notify)
{
+ GtkActionGroupPrivate *private;
+
g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
- if (action_group->private_data->translate_notify)
- action_group->private_data->translate_notify (action_group->private_data->translate_data);
+ private = action_group->priv;
+
+ if (private->translate_notify)
+ private->translate_notify (private->translate_data);
- action_group->private_data->translate_func = func;
- action_group->private_data->translate_data = data;
- action_group->private_data->translate_notify = notify;
+ private->translate_func = func;
+ private->translate_data = data;
+ private->translate_notify = notify;
}
static gchar *
dgettext_swapped (const gchar *msgid,
const gchar *domainname)
{
- return dgettext (domainname, msgid);
+ /* Pass through g_dgettext if and only if msgid is nonempty. */
+ if (msgid && *msgid)
+ return (gchar*) g_dgettext (domainname, msgid);
+ else
+ return (gchar*) msgid;
}
/**
* gtk_action_group_set_translation_domain:
* @action_group: a #GtkActionGroup
- * @domain: the translation domain to use for dgettext() calls
+ * @domain: (allow-none): the translation domain to use for g_dgettext()
+ * calls, or %NULL to use the domain set with textdomain()
*
- * Sets the translation domain and uses dgettext() for translating the
+ * Sets the translation domain and uses g_dgettext() for translating the
* @label and @tooltip of #GtkActionEntry<!-- -->s added by
* gtk_action_group_add_actions().
*
*
* Since: 2.6
**/
-G_CONST_RETURN gchar *
+const gchar *
gtk_action_group_translate_string (GtkActionGroup *action_group,
const gchar *string)
{
+ GtkActionGroupPrivate *private;
GtkTranslateFunc translate_func;
gpointer translate_data;
g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), string);
- translate_func = action_group->private_data->translate_func;
- translate_data = action_group->private_data->translate_data;
+ if (string == NULL)
+ return NULL;
+
+ private = action_group->priv;
+
+ translate_func = private->translate_func;
+ translate_data = private->translate_data;
if (translate_func)
return translate_func (string, translate_data);
void
_gtk_action_group_emit_post_activate (GtkActionGroup *action_group,
- GtkAction *action)
+ GtkAction *action)
{
g_signal_emit (action_group, action_group_signals[POST_ACTIVATE], 0, action);
}