gtkactiongroup.c 41.6 KB
Newer Older
1
/*
Cody Russell's avatar
Cody Russell committed
2
 * GTK - The GIMP Toolkit
3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30
 * Copyright (C) 1998, 1999 Red Hat, Inc.
 * All rights reserved.
 *
 * This Library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU Library General Public License as
 * published by the Free Software Foundation; either version 2 of the
 * License, or (at your option) any later version.
 *
 * This Library 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
 * 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.
 */

/*
 * Author: James Henstridge <james@daa.com.au>
 *
 * Modified by the GTK+ Team and others 2003.  See the AUTHORS
 * file for a list of people on the GTK+ Team.  See the ChangeLog
 * files for a list of changes.  These files are distributed with
 * GTK+ at ftp://ftp.gtk.org/pub/gtk/. 
 */

31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90
/**
 * 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>&lt;Actions&gt;/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 &lt;child&gt; 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 &lt;accelerator&gt; element, which has attributes named key and
 * modifiers and allows to specify accelerators. This is similar to the
 * &lt;accelerator&gt; 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>
 */

91
#include "config.h"
92
#include <string.h>
93 94

#include "gtkactiongroup.h"
Johan Dahlin's avatar
Johan Dahlin committed
95
#include "gtkbuildable.h"
Matthias Clasen's avatar
Matthias Clasen committed
96
#include "gtkiconfactory.h"
Matthias Clasen's avatar
Matthias Clasen committed
97
#include "gtkicontheme.h"
98
#include "gtkstock.h"
99 100 101
#include "gtktoggleaction.h"
#include "gtkradioaction.h"
#include "gtkaccelmap.h"
102
#include "gtkmarshalers.h"
103
#include "gtkbuilderprivate.h"
104
#include "gtkprivate.h"
105 106 107 108 109
#include "gtkintl.h"


struct _GtkActionGroupPrivate 
{
110
  gchar           *name;
111 112
  gboolean	   sensitive;
  gboolean	   visible;
113 114 115 116
  GHashTable      *actions;

  GtkTranslateFunc translate_func;
  gpointer         translate_data;
117
  GDestroyNotify   translate_notify;
118 119
};

120 121 122 123 124 125 126 127 128
enum 
{
  CONNECT_PROXY,
  DISCONNECT_PROXY,
  PRE_ACTIVATE,
  POST_ACTIVATE,
  LAST_SIGNAL
};

129 130 131
enum 
{
  PROP_0,
132 133 134
  PROP_NAME,
  PROP_SENSITIVE,
  PROP_VISIBLE
135 136
};

137 138 139
static void       gtk_action_group_init            (GtkActionGroup      *self);
static void       gtk_action_group_class_init      (GtkActionGroupClass *class);
static void       gtk_action_group_finalize        (GObject             *object);
140 141 142 143 144 145 146 147
static void       gtk_action_group_set_property    (GObject             *object,
						    guint                prop_id,
						    const GValue        *value,
						    GParamSpec          *pspec);
static void       gtk_action_group_get_property    (GObject             *object,
						    guint                prop_id,
						    GValue              *value,
						    GParamSpec          *pspec);
148 149 150
static GtkAction *gtk_action_group_real_get_action (GtkActionGroup      *self,
						    const gchar         *name);

Johan Dahlin's avatar
Johan Dahlin committed
151 152
/* GtkBuildable */
static void gtk_action_group_buildable_init (GtkBuildableIface *iface);
153 154 155 156
static void gtk_action_group_buildable_add_child (GtkBuildable  *buildable,
						  GtkBuilder    *builder,
						  GObject       *child,
						  const gchar   *type);
Johan Dahlin's avatar
Johan Dahlin committed
157 158 159
static void gtk_action_group_buildable_set_name (GtkBuildable *buildable,
						 const gchar  *name);
static const gchar* gtk_action_group_buildable_get_name (GtkBuildable *buildable);
160 161 162 163 164 165 166 167 168 169 170
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);
171 172 173 174 175 176 177 178

GType
gtk_action_group_get_type (void)
{
  static GType type = 0;

  if (!type)
    {
179
      const GTypeInfo type_info =
180 181 182 183 184 185 186 187 188 189 190 191
      {
        sizeof (GtkActionGroupClass),
	NULL,           /* base_init */
        NULL,           /* base_finalize */
        (GClassInitFunc) gtk_action_group_class_init,
        NULL,           /* class_finalize */
        NULL,           /* class_data */
        sizeof (GtkActionGroup),
        0, /* n_preallocs */
        (GInstanceInitFunc) gtk_action_group_init,
      };

192
      const GInterfaceInfo buildable_info =
Johan Dahlin's avatar
Johan Dahlin committed
193 194 195 196 197 198
      {
	(GInterfaceInitFunc) gtk_action_group_buildable_init,
	NULL,
	NULL
      };

Matthias Clasen's avatar
Matthias Clasen committed
199
      type = g_type_register_static (G_TYPE_OBJECT, I_("GtkActionGroup"),
200 201
				     &type_info, 0);

Johan Dahlin's avatar
Johan Dahlin committed
202 203 204 205
      g_type_add_interface_static (type,
				   GTK_TYPE_BUILDABLE,
				   &buildable_info);
    }
206 207 208 209
  return type;
}

static GObjectClass *parent_class = NULL;
210
static guint         action_group_signals[LAST_SIGNAL] = { 0 };
211 212 213 214 215 216 217 218 219 220

static void
gtk_action_group_class_init (GtkActionGroupClass *klass)
{
  GObjectClass *gobject_class;

  gobject_class = G_OBJECT_CLASS (klass);
  parent_class = g_type_class_peek_parent (klass);

  gobject_class->finalize = gtk_action_group_finalize;
221 222
  gobject_class->set_property = gtk_action_group_set_property;
  gobject_class->get_property = gtk_action_group_get_property;
223 224
  klass->get_action = gtk_action_group_real_get_action;

225 226 227
  g_object_class_install_property (gobject_class,
				   PROP_NAME,
				   g_param_spec_string ("name",
228 229
							P_("Name"),
							P_("A name for the action group."),
230
							NULL,
231
							GTK_PARAM_READWRITE | G_PARAM_CONSTRUCT_ONLY));
232 233 234
  g_object_class_install_property (gobject_class,
				   PROP_SENSITIVE,
				   g_param_spec_boolean ("sensitive",
235 236
							 P_("Sensitive"),
							 P_("Whether the action group is enabled."),
237
							 TRUE,
238
							 GTK_PARAM_READWRITE));
239 240 241
  g_object_class_install_property (gobject_class,
				   PROP_VISIBLE,
				   g_param_spec_boolean ("visible",
242 243
							 P_("Visible"),
							 P_("Whether the action group is visible."),
244
							 TRUE,
245
							 GTK_PARAM_READWRITE));
246 247

  /**
248
   * GtkActionGroup::connect-proxy:
249 250 251 252
   * @action_group: the group
   * @action: the action
   * @proxy: the proxy
   *
253
   * The ::connect-proxy signal is emitted after connecting a proxy to 
254 255 256 257 258 259 260 261 262 263 264 265 266 267
   * an action in the group. Note that the proxy may have been connected 
   * to a different action before.
   *
   * This is intended for simple customizations for which a custom action
   * class would be too clumsy, e.g. showing tooltips for menuitems in the
   * statusbar.
   *
   * #GtkUIManager proxies the signal and provides global notification 
   * just before any action is connected to a proxy, which is probably more
   * convenient to use.
   *
   * Since: 2.4
   */
  action_group_signals[CONNECT_PROXY] =
268
    g_signal_new (I_("connect-proxy"),
269 270 271 272 273 274 275
		  G_OBJECT_CLASS_TYPE (klass),
		  0, 0, NULL, NULL,
		  _gtk_marshal_VOID__OBJECT_OBJECT,
		  G_TYPE_NONE, 2,
		  GTK_TYPE_ACTION, GTK_TYPE_WIDGET);

  /**
276
   * GtkActionGroup::disconnect-proxy:
277 278 279 280
   * @action_group: the group
   * @action: the action
   * @proxy: the proxy
   *
281
   * The ::disconnect-proxy signal is emitted after disconnecting a proxy 
282 283 284 285 286 287 288 289 290
   * from an action in the group. 
   *
   * #GtkUIManager proxies the signal and provides global notification 
   * just before any action is connected to a proxy, which is probably more
   * convenient to use.
   *
   * Since: 2.4
   */
  action_group_signals[DISCONNECT_PROXY] =
291
    g_signal_new (I_("disconnect-proxy"),
292 293 294 295 296 297 298
		  G_OBJECT_CLASS_TYPE (klass),
		  0, 0, NULL, NULL,
		  _gtk_marshal_VOID__OBJECT_OBJECT,
		  G_TYPE_NONE, 2, 
		  GTK_TYPE_ACTION, GTK_TYPE_WIDGET);

  /**
299
   * GtkActionGroup::pre-activate:
300 301 302
   * @action_group: the group
   * @action: the action
   *
303
   * The ::pre-activate signal is emitted just before the @action in the
304 305 306 307 308 309 310 311
   * @action_group is activated
   *
   * This is intended for #GtkUIManager to proxy the signal and provide global
   * notification just before any action is activated.
   *
   * Since: 2.4
   */
  action_group_signals[PRE_ACTIVATE] =
312
    g_signal_new (I_("pre-activate"),
313 314 315 316 317 318 319
		  G_OBJECT_CLASS_TYPE (klass),
		  0, 0, NULL, NULL,
		  _gtk_marshal_VOID__OBJECT,
		  G_TYPE_NONE, 1, 
		  GTK_TYPE_ACTION);

  /**
320
   * GtkActionGroup::post-activate:
321 322 323
   * @action_group: the group
   * @action: the action
   *
324
   * The ::post-activate signal is emitted just after the @action in the
325 326 327 328 329 330 331 332
   * @action_group is activated
   *
   * This is intended for #GtkUIManager to proxy the signal and provide global
   * notification just after any action is activated.
   *
   * Since: 2.4
   */
  action_group_signals[POST_ACTIVATE] =
333
    g_signal_new (I_("post-activate"),
334 335 336 337 338 339
		  G_OBJECT_CLASS_TYPE (klass),
		  0, 0, NULL, NULL,
		  _gtk_marshal_VOID__OBJECT,
		  G_TYPE_NONE, 1, 
		  GTK_TYPE_ACTION);

340 341 342
  g_type_class_add_private (gobject_class, sizeof (GtkActionGroupPrivate));
}

Matthias Clasen's avatar
Matthias Clasen committed
343 344 345 346

static void 
remove_action (GtkAction *action) 
{
347
  g_object_set (action, I_("action-group"), NULL, NULL);
Matthias Clasen's avatar
Matthias Clasen committed
348 349 350
  g_object_unref (action);
}

351
static void
352
gtk_action_group_init (GtkActionGroup *action_group)
353
{
Tim Janik's avatar
Tim Janik committed
354 355
  GtkActionGroupPrivate *private;

356 357 358 359
  action_group->priv = G_TYPE_INSTANCE_GET_PRIVATE (action_group,
                                                    GTK_TYPE_ACTION_GROUP,
                                                    GtkActionGroupPrivate);
  private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
360 361 362 363 364 365 366 367 368 369

  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;
370 371
}

Johan Dahlin's avatar
Johan Dahlin committed
372 373 374
static void
gtk_action_group_buildable_init (GtkBuildableIface *iface)
{
375
  iface->add_child = gtk_action_group_buildable_add_child;
Johan Dahlin's avatar
Johan Dahlin committed
376 377
  iface->set_name = gtk_action_group_buildable_set_name;
  iface->get_name = gtk_action_group_buildable_get_name;
378 379
  iface->custom_tag_start = gtk_action_group_buildable_custom_tag_start;
  iface->custom_tag_end = gtk_action_group_buildable_custom_tag_end;
Johan Dahlin's avatar
Johan Dahlin committed
380 381 382
}

static void
383 384 385 386
gtk_action_group_buildable_add_child (GtkBuildable  *buildable,
				      GtkBuilder    *builder,
				      GObject       *child,
				      const gchar   *type)
Johan Dahlin's avatar
Johan Dahlin committed
387
{
388 389
  gtk_action_group_add_action_with_accel (GTK_ACTION_GROUP (buildable),
					  GTK_ACTION (child), NULL);
Johan Dahlin's avatar
Johan Dahlin committed
390 391 392 393 394 395 396
}

static void
gtk_action_group_buildable_set_name (GtkBuildable *buildable,
				     const gchar  *name)
{
  GtkActionGroup *self = GTK_ACTION_GROUP (buildable);
397
  GtkActionGroupPrivate *private = self->priv;
Tim Janik's avatar
Tim Janik committed
398 399

  private->name = g_strdup (name);
Johan Dahlin's avatar
Johan Dahlin committed
400 401 402 403 404 405
}

static const gchar *
gtk_action_group_buildable_get_name (GtkBuildable *buildable)
{
  GtkActionGroup *self = GTK_ACTION_GROUP (buildable);
406 407
  GtkActionGroupPrivate *private = self->priv;

Tim Janik's avatar
Tim Janik committed
408
  return private->name;
Johan Dahlin's avatar
Johan Dahlin committed
409 410
}

411
typedef struct {
412 413 414
  GObject         *child;
  guint            key;
  GdkModifierType  modifiers;
415 416 417 418 419 420 421 422 423 424 425 426
} 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;
427
  GdkModifierType modifiers = 0;
428 429 430 431 432 433 434 435 436 437 438 439 440 441 442 443 444 445 446 447 448 449 450 451 452
  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;
453
  parser_data->modifiers = modifiers;
454 455 456 457 458 459 460 461 462 463 464 465 466 467 468 469 470 471 472 473 474 475 476 477 478 479 480 481 482 483 484 485 486 487 488 489 490 491 492 493 494
}

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;
Tim Janik's avatar
Tim Janik committed
495
      GtkActionGroupPrivate *private;
496 497 498 499 500
      GtkAction *action;
      gchar *accel_path;
      
      data = (AcceleratorParserData*)user_data;
      action_group = GTK_ACTION_GROUP (buildable);
501
      private = action_group->priv;
502 503 504
      action = GTK_ACTION (child);
	
      accel_path = g_strconcat ("<Actions>/",
Tim Janik's avatar
Tim Janik committed
505
				private->name, "/",
506 507 508 509 510 511 512 513 514 515 516 517 518 519
				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);
    }
}

520 521
/**
 * gtk_action_group_new:
Matthias Clasen's avatar
Matthias Clasen committed
522
 * @name: the name of the action group.
523
 *
Matthias Clasen's avatar
Matthias Clasen committed
524 525 526
 * Creates a new #GtkActionGroup object. The name of the action group
 * is used when associating <link linkend="Action-Accel">keybindings</link> 
 * with the actions.
527 528 529 530 531 532 533 534 535
 *
 * Returns: the new #GtkActionGroup
 *
 * Since: 2.4
 */
GtkActionGroup *
gtk_action_group_new (const gchar *name)
{
  GtkActionGroup *self;
Tim Janik's avatar
Tim Janik committed
536
  GtkActionGroupPrivate *private;
537 538

  self = g_object_new (GTK_TYPE_ACTION_GROUP, NULL);
539
  private = self->priv;
Tim Janik's avatar
Tim Janik committed
540
  private->name = g_strdup (name);
541 542 543 544 545 546 547 548

  return self;
}

static void
gtk_action_group_finalize (GObject *object)
{
  GtkActionGroup *self;
Tim Janik's avatar
Tim Janik committed
549
  GtkActionGroupPrivate *private;
550 551

  self = GTK_ACTION_GROUP (object);
552
  private = self->priv;
553

Tim Janik's avatar
Tim Janik committed
554 555
  g_free (private->name);
  private->name = NULL;
556

Tim Janik's avatar
Tim Janik committed
557 558
  g_hash_table_destroy (private->actions);
  private->actions = NULL;
559

Tim Janik's avatar
Tim Janik committed
560 561
  if (private->translate_notify)
    private->translate_notify (private->translate_data);
562

563
  parent_class->finalize (object);
564 565
}

566 567 568 569 570 571 572
static void
gtk_action_group_set_property (GObject         *object,
			       guint            prop_id,
			       const GValue    *value,
			       GParamSpec      *pspec)
{
  GtkActionGroup *self;
Tim Janik's avatar
Tim Janik committed
573
  GtkActionGroupPrivate *private;
574 575 576
  gchar *tmp;
  
  self = GTK_ACTION_GROUP (object);
577
  private = self->priv;
578 579 580 581

  switch (prop_id)
    {
    case PROP_NAME:
Tim Janik's avatar
Tim Janik committed
582 583
      tmp = private->name;
      private->name = g_value_dup_string (value);
584 585
      g_free (tmp);
      break;
586 587 588 589 590 591
    case PROP_SENSITIVE:
      gtk_action_group_set_sensitive (self, g_value_get_boolean (value));
      break;
    case PROP_VISIBLE:
      gtk_action_group_set_visible (self, g_value_get_boolean (value));
      break;
592 593 594 595 596 597 598 599 600 601 602 603 604
    default:
      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
      break;
    }
}

static void
gtk_action_group_get_property (GObject    *object,
			       guint       prop_id,
			       GValue     *value,
			       GParamSpec *pspec)
{
  GtkActionGroup *self;
Tim Janik's avatar
Tim Janik committed
605
  GtkActionGroupPrivate *private;
606 607
  
  self = GTK_ACTION_GROUP (object);
608
  private = self->priv;
609 610 611 612

  switch (prop_id)
    {
    case PROP_NAME:
Tim Janik's avatar
Tim Janik committed
613
      g_value_set_string (value, private->name);
614
      break;
615
    case PROP_SENSITIVE:
Tim Janik's avatar
Tim Janik committed
616
      g_value_set_boolean (value, private->sensitive);
617 618
      break;
    case PROP_VISIBLE:
Tim Janik's avatar
Tim Janik committed
619
      g_value_set_boolean (value, private->visible);
620
      break;
621 622 623 624 625 626
    default:
      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
      break;
    }
}

627 628 629 630
static GtkAction *
gtk_action_group_real_get_action (GtkActionGroup *self,
				  const gchar    *action_name)
{
Tim Janik's avatar
Tim Janik committed
631 632
  GtkActionGroupPrivate *private;

633
  private = self->priv;
Tim Janik's avatar
Tim Janik committed
634 635

  return g_hash_table_lookup (private->actions, action_name);
636 637 638 639 640 641 642 643 644 645 646 647
}

/**
 * gtk_action_group_get_name:
 * @action_group: the action group
 *
 * Gets the name of the action group.
 *
 * Returns: the name of the action group.
 * 
 * Since: 2.4
 */
648
G_CONST_RETURN gchar *
649 650
gtk_action_group_get_name (GtkActionGroup *action_group)
{
Tim Janik's avatar
Tim Janik committed
651 652
  GtkActionGroupPrivate *private;

653 654
  g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), NULL);

655
  private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
656 657

  return private->name;
658 659
}

660 661 662 663 664 665 666 667 668 669 670 671 672 673 674 675
/**
 * gtk_action_group_get_sensitive:
 * @action_group: the action group
 *
 * Returns %TRUE if the group is sensitive.  The constituent actions
 * can only be logically sensitive (see gtk_action_is_sensitive()) if
 * they are sensitive (see gtk_action_get_sensitive()) and their group
 * is sensitive.
 * 
 * Return value: %TRUE if the group is sensitive.
 *
 * Since: 2.4
 */
gboolean
gtk_action_group_get_sensitive (GtkActionGroup *action_group)
{
Tim Janik's avatar
Tim Janik committed
676 677
  GtkActionGroupPrivate *private;

678 679
  g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), FALSE);

680
  private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
681 682

  return private->sensitive;
683 684 685
}

static void
686 687
cb_set_action_sensitivity (const gchar *name, 
			   GtkAction   *action)
688
{
689 690
  /* Minor optimization, the action_groups state only affects actions 
   * that are themselves sensitive */
691 692
  g_object_notify (G_OBJECT (action), "sensitive");

693 694 695 696 697 698 699 700 701 702 703 704
}

/**
 * gtk_action_group_set_sensitive:
 * @action_group: the action group
 * @sensitive: new sensitivity
 *
 * Changes the sensitivity of @action_group
 * 
 * Since: 2.4
 */
void
705 706
gtk_action_group_set_sensitive (GtkActionGroup *action_group, 
				gboolean        sensitive)
707
{
Tim Janik's avatar
Tim Janik committed
708 709
  GtkActionGroupPrivate *private;

710 711
  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));

712
  private = action_group->priv;
713 714
  sensitive = sensitive != FALSE;

Tim Janik's avatar
Tim Janik committed
715
  if (private->sensitive != sensitive)
716
    {
Tim Janik's avatar
Tim Janik committed
717 718
      private->sensitive = sensitive;
      g_hash_table_foreach (private->actions, 
719
			    (GHFunc) cb_set_action_sensitivity, NULL);
720 721

      g_object_notify (G_OBJECT (action_group), "sensitive");
722 723 724 725 726 727 728 729 730 731 732 733
    }
}

/**
 * gtk_action_group_get_visible:
 * @action_group: the action group
 *
 * Returns %TRUE if the group is visible.  The constituent actions
 * can only be logically visible (see gtk_action_is_visible()) if
 * they are visible (see gtk_action_get_visible()) and their group
 * is visible.
 * 
734
 * Return value: %TRUE if the group is visible.
735 736 737 738 739 740
 * 
 * Since: 2.4
 */
gboolean
gtk_action_group_get_visible (GtkActionGroup *action_group)
{
Tim Janik's avatar
Tim Janik committed
741 742
  GtkActionGroupPrivate *private;

743 744
  g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), FALSE);

745
  private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
746 747

  return private->visible;
748 749 750
}

static void
751 752
cb_set_action_visiblity (const gchar *name, 
			 GtkAction   *action)
753
{
754 755
  /* Minor optimization, the action_groups state only affects actions 
   * that are themselves visible */
756
  g_object_notify (G_OBJECT (action), "visible");
757 758 759 760 761 762 763 764 765 766 767 768
}

/**
 * gtk_action_group_set_visible:
 * @action_group: the action group
 * @visible: new visiblity
 *
 * Changes the visible of @action_group.
 * 
 * Since: 2.4
 */
void
769 770
gtk_action_group_set_visible (GtkActionGroup *action_group, 
			      gboolean        visible)
771
{
Tim Janik's avatar
Tim Janik committed
772 773
  GtkActionGroupPrivate *private;

774 775
  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));

776
  private = action_group->priv;
777 778
  visible = visible != FALSE;

Tim Janik's avatar
Tim Janik committed
779
  if (private->visible != visible)
780
    {
Tim Janik's avatar
Tim Janik committed
781 782
      private->visible = visible;
      g_hash_table_foreach (private->actions, 
783
			    (GHFunc) cb_set_action_visiblity, NULL);
784 785

      g_object_notify (G_OBJECT (action_group), "visible");
786 787 788
    }
}

789 790 791 792 793 794 795
/**
 * gtk_action_group_get_action:
 * @action_group: the action group
 * @action_name: the name of the action
 *
 * Looks up an action in the action group by name.
 *
796
 * Returns: (transfer none): the action, or %NULL if no action by that name exists
797 798 799 800 801 802 803 804 805 806
 *
 * Since: 2.4
 */
GtkAction *
gtk_action_group_get_action (GtkActionGroup *action_group,
			     const gchar    *action_name)
{
  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);

807 808
  return GTK_ACTION_GROUP_GET_CLASS (action_group)->get_action (action_group,
                                                                action_name);
809 810
}

811 812 813 814 815 816
static gboolean
check_unique_action (GtkActionGroup *action_group,
	             const gchar    *action_name)
{
  if (gtk_action_group_get_action (action_group, action_name) != NULL)
    {
Tim Janik's avatar
Tim Janik committed
817 818
      GtkActionGroupPrivate *private;

819
      private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
820

821 822
      g_warning ("Refusing to add non-unique action '%s' to action group '%s'",
	 	 action_name,
Tim Janik's avatar
Tim Janik committed
823
		 private->name);
824 825 826 827 828 829
      return FALSE;
    }

  return TRUE;
}

830 831 832 833 834
/**
 * gtk_action_group_add_action:
 * @action_group: the action group
 * @action: an action
 *
835 836 837 838 839 840
 * Adds an action object to the action group. Note that this function
 * does not set up the accel path of the action, which can lead to problems
 * if a user tries to modify the accelerator of a menuitem associated with
 * the action. Therefore you must either set the accel path yourself with
 * gtk_action_set_accel_path(), or use 
 * <literal>gtk_action_group_add_action_with_accel (..., NULL)</literal>.
841 842 843 844 845 846 847
 *
 * Since: 2.4
 */
void
gtk_action_group_add_action (GtkActionGroup *action_group,
			     GtkAction      *action)
{
Tim Janik's avatar
Tim Janik committed
848
  GtkActionGroupPrivate *private;
849 850
  const gchar *name;

851 852
  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
  g_return_if_fail (GTK_IS_ACTION (action));
853 854 855

  name = gtk_action_get_name (action);
  g_return_if_fail (name != NULL);
856
  
857
  if (!check_unique_action (action_group, name))
858
    return;
859

860
  private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
861 862

  g_hash_table_insert (private->actions, 
863
		       (gpointer) name,
864
                       g_object_ref (action));
865
  g_object_set (action, I_("action-group"), action_group, NULL);
866 867
}

868 869
/**
 * gtk_action_group_add_action_with_accel:
870 871 872 873 874
 * @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
875 876 877
 *
 * Adds an action object to the action group and sets up the accelerator.
 *
878
 * If @accelerator is %NULL, attempts to use the accelerator associated 
Matthias Clasen's avatar
Matthias Clasen committed
879
 * with the stock_id of the action. 
880
 *
881 882 883
 * Accel paths are set to
 * <literal>&lt;Actions&gt;/<replaceable>group-name</replaceable>/<replaceable>action-name</replaceable></literal>.
 *
884 885 886 887
 * Since: 2.4
 */
void
gtk_action_group_add_action_with_accel (GtkActionGroup *action_group,
888 889
					GtkAction      *action,
					const gchar    *accelerator)
890
{
Tim Janik's avatar
Tim Janik committed
891
  GtkActionGroupPrivate *private;
892 893 894
  gchar *accel_path;
  guint  accel_key = 0;
  GdkModifierType accel_mods;
895
  const gchar *name;
896

897 898
  name = gtk_action_get_name (action);
  if (!check_unique_action (action_group, name))
899
    return;
900

901
  private = action_group->priv;
902
  accel_path = g_strconcat ("<Actions>/",
Tim Janik's avatar
Tim Janik committed
903
			    private->name, "/", name, NULL);
904 905

  if (accelerator)
906
    {
Matthias Clasen's avatar
Matthias Clasen committed
907 908 909 910 911 912 913 914 915
      if (accelerator[0] == 0) 
	accel_key = 0;
      else
	{
	  gtk_accelerator_parse (accelerator, &accel_key, &accel_mods);
	  if (accel_key == 0)
	    g_warning ("Unable to parse accelerator '%s' for action '%s'",
		       accelerator, name);
	}
916
    }
917
  else 
918
    {
919 920 921 922 923 924 925 926 927 928 929 930
      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);
931 932 933
    }

  if (accel_key)
Matthias Clasen's avatar
Matthias Clasen committed
934
    gtk_accel_map_add_entry (accel_path, accel_key, accel_mods);
935 936 937

  gtk_action_set_accel_path (action, accel_path);
  gtk_action_group_add_action (action_group, action);
938 939

  g_free (accel_path);
940 941
}

942
/**
Matthias Clasen's avatar
Matthias Clasen committed
943
 * gtk_action_group_remove_action:
944 945 946 947 948 949 950 951 952 953 954
 * @action_group: the action group
 * @action: an action
 *
 * Removes an action object from the action group.
 *
 * Since: 2.4
 */
void
gtk_action_group_remove_action (GtkActionGroup *action_group,
				GtkAction      *action)
{
Tim Janik's avatar
Tim Janik committed
955
  GtkActionGroupPrivate *private;
956 957
  const gchar *name;

958 959 960
  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
  g_return_if_fail (GTK_IS_ACTION (action));

961 962 963
  name = gtk_action_get_name (action);
  g_return_if_fail (name != NULL);

964
  private = action_group->priv;
Tim Janik's avatar
Tim Janik committed
965 966

  g_hash_table_remove (private->actions, name);
967 968 969 970 971 972 973 974 975 976 977 978 979 980 981 982 983 984
}

static void
add_single_action (gpointer key, 
		   gpointer value, 
		   gpointer user_data)
{
  GList **list = user_data;

  *list = g_list_prepend (*list, value);
}

/**
 * gtk_action_group_list_actions:
 * @action_group: the action group
 *
 * Lists the actions in the action group.
 *
985 986
 * Returns: (element-type GtkAction) (transfer container): an allocated list of the action objects in the action group
 *
987 988 989 990 991
 * Since: 2.4
 */
GList *
gtk_action_group_list_actions (GtkActionGroup *action_group)
{
Tim Janik's avatar
Tim Janik committed
992
  GtkActionGroupPrivate *private;
993
  GList *actions = NULL;
Tim Janik's avatar
Tim Janik committed
994

995
  g_return_val_if_fail (GTK_IS_ACTION_GROUP (action_group), NULL);
Tim Janik's avatar
Tim Janik committed
996

997
  private = action_group->priv;
998
  
Tim Janik's avatar
Tim Janik committed
999
  g_hash_table_foreach (private->actions, add_single_action, &actions);
1000 1001 1002 1003 1004 1005

  return g_list_reverse (actions);
}


/**
1006
 * gtk_action_group_add_actions: (skip)
1007
 * @action_group: the action group
1008
 * @entries: (array length=n_entries): an array of action descriptions
1009
 * @n_entries: the number of entries
1010
 * @user_data: data to pass to the action callbacks
1011
 *
1012 1013
 * This is a convenience function to create a number of actions and add them 
 * to the action group.
Matthias Clasen's avatar
Matthias Clasen committed
1014 1015 1016 1017
 *
 * The "activate" signals of the actions are connected to the callbacks and 
 * their accel paths are set to 
 * <literal>&lt;Actions&gt;/<replaceable>group-name</replaceable>/<replaceable>action-name</replaceable></literal>.  
1018 1019 1020 1021
 * 
 * Since: 2.4
 */
void
1022 1023 1024 1025
gtk_action_group_add_actions (GtkActionGroup       *action_group,
			      const GtkActionEntry *entries,
			      guint                 n_entries,
			      gpointer              user_data)
1026 1027 1028 1029 1030 1031
{
  gtk_action_group_add_actions_full (action_group, 
				     entries, n_entries, 
				     user_data, NULL);
}

1032 1033 1034 1035 1036 1037 1038 1039 1040 1041 1042 1043 1044 1045 1046 1047
typedef struct _SharedData  SharedData;

struct _SharedData {
  guint          ref_count;
  gpointer       data;
  GDestroyNotify destroy;
};

static void
shared_data_unref (gpointer data)
{
  SharedData *shared_data = (SharedData *)data;

  shared_data->ref_count--;
  if (shared_data->ref_count == 0)
    {
1048 1049 1050
      if (shared_data->destroy)
	shared_data->destroy (shared_data->data);

1051
      g_slice_free (SharedData, shared_data);
1052 1053 1054
    }
}

1055 1056

/**
1057
 * gtk_action_group_add_actions_full: (skip)
1058
 * @action_group: the action group
1059
 * @entries: (array length=n_entries): an array of action descriptions
1060 1061 1062 1063 1064 1065 1066 1067 1068 1069
 * @n_entries: the number of entries
 * @user_data: data to pass to the action callbacks
 * @destroy: destroy notification callback for @user_data
 *
 * This variant of gtk_action_group_add_actions() adds a #GDestroyNotify
 * callback for @user_data. 
 * 
 * Since: 2.4
 */
void
1070 1071 1072 1073 1074
gtk_action_group_add_actions_full (GtkActionGroup       *action_group,
				   const GtkActionEntry *entries,
				   guint                 n_entries,
				   gpointer              user_data,
				   GDestroyNotify        destroy)
1075
{
1076 1077 1078 1079

  /* Keep this in sync with the other 
   * gtk_action_group_add_..._actions_full() functions.
   */
1080
  guint i;
1081
  SharedData *shared_data;
1082

1083
  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));
1084

1085
  shared_data = g_slice_new0 (SharedData);
1086 1087 1088 1089
  shared_data->ref_count = 1;
  shared_data->data = user_data;
  shared_data->destroy = destroy;

1090 1091 1092
  for (i = 0; i < n_entries; i++)
    {
      GtkAction *action;
1093 1094
      const gchar *label;
      const gchar *tooltip;
1095

1096 1097 1098
      if (!check_unique_action (action_group, entries[i].name))
        continue;

1099 1100
      label = gtk_action_group_translate_string (action_group, entries[i].label);
      tooltip = gtk_action_group_translate_string (action_group, entries[i].tooltip);
1101

1102 1103 1104
      action = gtk_action_new (entries[i].name,
			       label,
			       tooltip,
1105
			       NULL);
1106

1107 1108
      if (entries[i].stock_id) 
	{
1109 1110 1111
	  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))
1112 1113 1114
	    g_object_set (action, "icon-name", entries[i].stock_id, NULL);
	}
	  
1115
      if (entries[i].callback)
1116 1117 1118 1119 1120 1121 1122
	{
	  GClosure *closure;

	  closure = g_cclosure_new (entries[i].callback, user_data, NULL);
	  g_closure_add_finalize_notifier (closure, shared_data, 
					   (GClosureNotify)shared_data_unref);
	  shared_data->ref_count++;
1123

1124 1125 1126
	  g_signal_connect_closure (action, "activate", closure, FALSE);
	}
	  
1127 1128 1129
      gtk_action_group_add_action_with_accel (action_group, 
					      action,
					      entries[i].accelerator);
1130 1131
      g_object_unref (action);
    }
1132 1133

  shared_data_unref (shared_data);
1134 1135 1136
}

/**
1137
 * gtk_action_group_add_toggle_actions: (skip)
1138
 * @action_group: the action group
1139
 * @entries: (array length=n_entries): an array of toggle action descriptions
1140 1141 1142 1143 1144 1145 1146 1147 1148 1149 1150 1151 1152
 * @n_entries: the number of entries
 * @user_data: data to pass to the action callbacks
 *
 * This is a convenience function to create a number of toggle actions and add them 
 * to the action group.
 *
 * The "activate" signals of the actions are connected to the callbacks and 
 * their accel paths are set to 
 * <literal>&lt;Actions&gt;/<replaceable>group-name</replaceable>/<replaceable>action-name</replaceable></literal>.  
 * 
 * Since: 2.4
 */
void
1153 1154 1155 1156
gtk_action_group_add_toggle_actions (GtkActionGroup             *action_group,
				     const GtkToggleActionEntry *entries,
				     guint                       n_entries,
				     gpointer                    user_data)
1157 1158 1159 1160 1161 1162 1163 1164
{
  gtk_action_group_add_toggle_actions_full (action_group, 
					    entries, n_entries, 
					    user_data, NULL);
}


/**
1165
 * gtk_action_group_add_toggle_actions_full: (skip)
1166
 * @action_group: the action group
1167
 * @entries: (array length=n_entries): an array of toggle action descriptions
1168 1169 1170 1171 1172 1173 1174 1175 1176 1177
 * @n_entries: the number of entries
 * @user_data: data to pass to the action callbacks
 * @destroy: destroy notification callback for @user_data
 *
 * This variant of gtk_action_group_add_toggle_actions() adds a 
 * #GDestroyNotify callback for @user_data. 
 * 
 * Since: 2.4
 */
void
1178 1179 1180 1181 1182
gtk_action_group_add_toggle_actions_full (GtkActionGroup             *action_group,
					  const GtkToggleActionEntry *entries,
					  guint                       n_entries,
					  gpointer                    user_data,
					  GDestroyNotify              destroy)
1183
{
1184 1185 1186
  /* Keep this in sync with the other 
   * gtk_action_group_add_..._actions_full() functions.
   */
1187
  guint i;
1188
  SharedData *shared_data;
1189 1190 1191

  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));

1192
  shared_data = g_slice_new0 (SharedData);
1193 1194 1195 1196
  shared_data->ref_count = 1;
  shared_data->data = user_data;
  shared_data->destroy = destroy;

1197 1198
  for (i = 0; i < n_entries; i++)
    {
1199
      GtkToggleAction *action;
1200 1201
      const gchar *label;
      const gchar *tooltip;
1202

1203 1204 1205
      if (!check_unique_action (action_group, entries[i].name))
        continue;

1206 1207
      label = gtk_action_group_translate_string (action_group, entries[i].label);
      tooltip = gtk_action_group_translate_string (action_group, entries[i].tooltip);
1208

1209 1210 1211
      action = gtk_toggle_action_new (entries[i].name,
				      label,
				      tooltip,
1212 1213 1214 1215
				      NULL);

      if (entries[i].stock_id) 
	{
1216
	  if (gtk_icon_factory_lookup_default (entries[i].stock_id))
1217 1218 1219 1220
	    g_object_set (action, "stock-id", entries[i].stock_id, NULL);
	  else
	    g_object_set (action, "icon-name", entries[i].stock_id, NULL);
	}
1221

1222
      gtk_toggle_action_set_active (action, entries[i].is_active);
1223

1224
      if (entries[i].callback)
1225 1226 1227 1228 1229 1230 1231
	{
	  GClosure *closure;

	  closure = g_cclosure_new (entries[i].callback, user_data, NULL);
	  g_closure_add_finalize_notifier (closure, shared_data, 
					   (GClosureNotify)shared_data_unref);
	  shared_data->ref_count++;
1232

1233 1234 1235
	  g_signal_connect_closure (action, "activate", closure, FALSE);
	}
	  
1236
      gtk_action_group_add_action_with_accel (action_group, 
1237
					      GTK_ACTION (action),
1238
					      entries[i].accelerator);
1239 1240
      g_object_unref (action);
    }
1241

1242
  shared_data_unref (shared_data);
1243 1244 1245
}

/**
1246
 * gtk_action_group_add_radio_actions: (skip)
1247
 * @action_group: the action group
1248
 * @entries: (array length=n_entries): an array of radio action descriptions
1249
 * @n_entries: the number of entries
1250 1251
 * @value: the value of the action to activate initially, or -1 if
 *   no action should be activated
1252 1253 1254
 * @on_change: the callback to connect to the changed signal
 * @user_data: data to pass to the action callbacks
 * 
Matthias Clasen's avatar
Matthias Clasen committed
1255 1256 1257
 * This is a convenience routine to create a group of radio actions and
 * add them to the action group. 
 *
1258 1259
 * The "changed" signal of the first radio action is connected to the 
 * @on_change callback and the accel paths of the actions are set to 
Matthias Clasen's avatar
Matthias Clasen committed
1260
 * <literal>&lt;Actions&gt;/<replaceable>group-name</replaceable>/<replaceable>action-name</replaceable></literal>.  
1261 1262 1263 1264
 * 
 * Since: 2.4
 **/
void            
1265 1266 1267 1268 1269 1270
gtk_action_group_add_radio_actions (GtkActionGroup            *action_group,
				    const GtkRadioActionEntry *entries,
				    guint                      n_entries,
				    gint                       value,
				    GCallback                  on_change,
				    gpointer                   user_data)
1271 1272 1273
{
  gtk_action_group_add_radio_actions_full (action_group, 
					   entries, n_entries, 
1274
					   value,
1275 1276 1277
					   on_change, user_data, NULL);
}

Matthias Clasen's avatar
Matthias Clasen committed
1278
/**
1279
 * gtk_action_group_add_radio_actions_full: (skip)
Matthias Clasen's avatar
Matthias Clasen committed
1280
 * @action_group: the action group
1281
 * @entries: (array length=n_entries): an array of radio action descriptions
Matthias Clasen's avatar
Matthias Clasen committed
1282
 * @n_entries: the number of entries
1283 1284
 * @value: the value of the action to activate initially, or -1 if
 *   no action should be activated
Matthias Clasen's avatar
Matthias Clasen committed
1285 1286 1287 1288 1289 1290 1291 1292 1293
 * @on_change: the callback to connect to the changed signal
 * @user_data: data to pass to the action callbacks
 * @destroy: destroy notification callback for @user_data
 *
 * This variant of gtk_action_group_add_radio_actions() adds a 
 * #GDestroyNotify callback for @user_data. 
 * 
 * Since: 2.4
 **/
1294
void            
1295 1296 1297 1298 1299 1300 1301
gtk_action_group_add_radio_actions_full (GtkActionGroup            *action_group,
					 const GtkRadioActionEntry *entries,
					 guint                      n_entries,
					 gint                       value,
					 GCallback                  on_change,
					 gpointer                   user_data,
					 GDestroyNotify             destroy)
1302
{
1303 1304 1305
  /* Keep this in sync with the other 
   * gtk_action_group_add_..._actions_full() functions.
   */
1306
  guint i;
1307
  GSList *group = NULL;
1308
  GtkRadioAction *first_action = NULL;
1309 1310 1311 1312 1313

  g_return_if_fail (GTK_IS_ACTION_GROUP (action_group));

  for (i = 0; i < n_entries; i++)
    {
1314
      GtkRadioAction *action;
1315 1316
      const gchar *label;
      const gchar *tooltip; 
1317

1318 1319 1320
      if (!check_unique_action (action_group, entries[i].name))
        continue;

1321 1322
      label = gtk_action_group_translate_string (action_group, entries[i].label);
      tooltip = gtk_action_group_translate_string (action_group, entries[i].tooltip);
1323

1324 1325 1326
      action = gtk_radio_action_new (entries[i].name,
				     label,
				     tooltip,
1327
				     NULL