* settings-dialogs/xfwm4-settings.c: Only display shortcut conflict

	  dialog if the action of the new shortcut differs from the one
	  that's being edited.

(Old svn revision: 29423)

ChangeLog

+2009-02-01	jannis
+
+	* settings-dialogs/xfwm4-settings.c: Only display shortcut conflict
+	  dialog if the action of the new shortcut differs from the one
+	  that's being edited.
+
 2009-01-29	olivier
 	* src/placement.c: Apparently someone is confusing East and West :)

settings-dialogs/xfwm4-settings.c

@@ -1915,14 +1915,17 @@ xfwm_settings_validate_shortcut (XfceShortcutDialog  *dialog,
 
   if (G_UNLIKELY (other_shortcut != NULL))
     {
-      response = xfce_shortcut_conflict_dialog (xfce_shortcuts_provider_get_name (settings->priv->provider),
-                                                xfce_shortcuts_provider_get_name (other_provider),
-                                                shortcut,
-                                                xfce_shortcut_dialog_get_action_name (dialog),
-                                                xfwm_settings_shortcut_feature_name (other_shortcut->command),
-                                                FALSE);
-
-      accepted = response == GTK_RESPONSE_ACCEPT;
+      if (G_LIKELY (!g_str_equal (xfce_shortcut_dialog_get_action (dialog), other_shortcut->command)))
+        {
+          response = xfce_shortcut_conflict_dialog (xfce_shortcuts_provider_get_name (settings->priv->provider),
+                                                    xfce_shortcuts_provider_get_name (other_provider),
+                                                    shortcut,
+                                                    xfce_shortcut_dialog_get_action_name (dialog),
+                                                    xfwm_settings_shortcut_feature_name (other_shortcut->command),
+                                                    FALSE);
+
+          accepted = response == GTK_RESPONSE_ACCEPT;
+        }
 
       xfce_shortcut_free (other_shortcut);
       g_object_unref (other_provider);