remove the activate action, make the focus action "activate" but not raise or unshade

5 files changed, 8 insertions, 16 deletions

diff --git a/Makefile.am b/Makefile.am
index 61bc0b3d..a1bc2645 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -156,13 +156,13 @@ openbox_openbox_SOURCES = \
 	gettext.h \
 	openbox/actions/all.c \
 	openbox/actions/all.h \
-	openbox/actions/activate.c \
 	openbox/actions/breakchroot.c \
 	openbox/actions/close.c \
 	openbox/actions/cyclewindows.c \
 	openbox/actions/debug.c \
 	openbox/actions/execute.c \
 	openbox/actions/exit.c \
+	openbox/actions/focus.c \
 	openbox/actions/move.c \
 	openbox/actions/reconfigure.c \
 	openbox/actions/restart.c \
diff --git a/openbox/actions/all.c b/openbox/actions/all.c
index a65d76b5..71e439e6 100644
--- a/openbox/actions/all.c
+++ b/openbox/actions/all.c
@@ -10,8 +10,8 @@ void action_all_startup()
     action_exit_startup();
     action_restart_startup();
     action_cyclewindows_startup();
-    action_activate_startup();
     action_breakchroot_startup();
     action_close_startup();
     action_move_startup();
+    action_focus_startup();
 }
diff --git a/openbox/actions/all.h b/openbox/actions/all.h
index fa252492..2f2cf18a 100644
--- a/openbox/actions/all.h
+++ b/openbox/actions/all.h
@@ -11,9 +11,9 @@ void action_reconfigure_startup();
 void action_exit_startup();
 void action_restart_startup();
 void action_cyclewindows_startup();
-void action_activate_startup();
 void action_breakchroot_startup();
 void action_close_startup();
 void action_move_startup();
+void action_focus_startup();
 
 #endif
diff --git a/openbox/actions/activate.c b/openbox/actions/focus.c
index c3f3f98e..9be6741c 100644
--- a/openbox/actions/activate.c
+++ b/openbox/actions/focus.c
@@ -5,17 +5,15 @@
 
 typedef struct {
     gboolean here;
-    gboolean raise;
-    gboolean unshade;
 } Options;
 
 static gpointer setup_func(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node);
 static void     free_func(gpointer options);
 static gboolean run_func(ObActionsData *data, gpointer options);
 
-void action_activate_startup()
+void action_focus_startup()
 {
-    actions_register("Activate",
+    actions_register("Focus",
                      setup_func,
                      free_func,
                      run_func,
@@ -28,15 +26,9 @@ static gpointer setup_func(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node)
     Options *o;
 
     o = g_new0(Options, 1);
-    o->raise = TRUE;
-    o->unshade = TRUE;
 
     if ((n = parse_find_node("here", node)))
         o->here = parse_bool(doc, n);
-    if ((n = parse_find_node("raise", node)))
-        o->raise = parse_bool(doc, n);
-    if ((n = parse_find_node("unshade", node)))
-        o->unshade = parse_bool(doc, n);
     return o;
 }
 
@@ -57,7 +49,7 @@ static gboolean run_func(ObActionsData *data, gpointer options)
             data->context != OB_FRAME_CONTEXT_CLIENT ||
             data->context != OB_FRAME_CONTEXT_FRAME)
         {
-            client_activate(data->client, o->here, o->raise, o->unshade, TRUE);
+            client_activate(data->client, o->here, FALSE, FALSE, TRUE);
         }
     } else {
         /* focus action on something other than a client, make keybindings
diff --git a/openbox/focus_cycle.c b/openbox/focus_cycle.c
index 22398851..0396dd86 100644
--- a/openbox/focus_cycle.c
+++ b/openbox/focus_cycle.c
@@ -137,7 +137,7 @@ ObClient* focus_cycle(gboolean forward, gboolean all_desktops,
                                            focus_cycle_all_desktops,
                                            focus_cycle_dock_windows,
                                            focus_cycle_desktop_windows);
-                return NULL;
+                return focus_cycle_target;
             } else if (ft != focus_cycle_target) {
                 focus_cycle_target = ft;
                 done = TRUE;
@@ -316,7 +316,7 @@ ObClient* focus_directional_cycle(ObDirection dir, gboolean dock_windows,
                                       focus_cycle_all_desktops,
                                       focus_cycle_dock_windows,
                                       focus_cycle_desktop_windows);
-    return NULL;
+    return focus_cycle_target;
 
 done_cycle:
     if (done && !cancel) ret = focus_cycle_target;