From 43db5b55eaae67515ae926c0d195c1ef6aa4b607 Mon Sep 17 00:00:00 2001
From: okan <okan>
Date: Fri, 29 Dec 2017 18:50:43 +0000
Subject: As done for buttonrelease, work specific un-cycling and
 un-highlighting actions into the keyrelease event, only performing what's
 actually needed for each; should result in much fewer events against
 keyreleases. No intended behaviour change.

Additionally, like we do for group membership, grab the keyboard only when
required for cycling.
---
 kbfunc.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

(limited to 'kbfunc.c')

diff --git a/kbfunc.c b/kbfunc.c
index 55e8d67..cf3f7e3 100644
--- a/kbfunc.c
+++ b/kbfunc.c
@@ -398,7 +398,14 @@ kbfunc_client_vtile(void *ctx, struct cargs *cargs)
 void
 kbfunc_client_cycle(void *ctx, struct cargs *cargs)
 {
-	client_cycle(ctx, cargs->flag);
+	struct screen_ctx	*sc = ctx;
+
+	/* For X apps that ignore/steal events. */
+	if (cargs->xev == CWM_XEV_KEY)
+		XGrabKeyboard(X_Dpy, sc->rootwin, True,
+		    GrabModeAsync, GrabModeAsync, CurrentTime);
+
+	client_cycle(sc, cargs->flag);
 }
 
 void
@@ -406,7 +413,7 @@ kbfunc_client_toggle_group(void *ctx, struct cargs *cargs)
 {
 	struct client_ctx	*cc = ctx;
 
-	/* For X apps that steal events. */
+	/* For X apps that ignore/steal events. */
 	if (cargs->xev == CWM_XEV_KEY)
 		XGrabKeyboard(X_Dpy, cc->win, True,
 		    GrabModeAsync, GrabModeAsync, CurrentTime);
-- 
cgit 1.4.1