diff --git a/openbox/action.c b/openbox/action.c
index 50b1e545..a9c7737a 100644
--- a/openbox/action.c
+++ b/openbox/action.c
@@ -750,6 +750,6 @@ void action_cycle_windows(union ActionData *data)
     
     c = focus_cycle(data->cycle.forward, data->cycle.linear, data->cycle.final,
                     data->cycle.cancel);
-    popup_cycle(c, !c && (data->cycle.final||data->cycle.cancel));
+    popup_cycle(c, !c || data->cycle.final || data->cycle.cancel);
 }