From 1a5eb8b87990921a4d3155c1cb8452cd412e855b Mon Sep 17 00:00:00 2001 From: Arthur Williams Date: Mon, 22 Jun 2020 22:52:54 -0700 Subject: [PATCH] Added ICCCM WM_HINTS When the window is mapped, some ICCCM WM_HINTS are set. The input field is set to true and state is set to NormalState. To quote the spec, "The input field is used to communicate to the window manager the input focus model used by the client" and "[c]lients with the Passive and Locally Active models should set the input flag to True". sxiv falls under the Passive Input model, since it expects keyboard input, but only listens for key events on its single, top-level window instead of subordinate windows (Locally Active) or the root window (Globally Active). From the end users prospective, all EWMH/ICCCM compliant WMs (especially the minimalistic ones) will allow the user to focus sxiv, which will allow sxiv to receive key events. If the input field is not set, WMs are allowed to assume that sxiv doesn't require focus. --- window.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/window.c b/window.c index 6f9a3906..eb534a70 100644 --- a/window.c +++ b/window.c @@ -154,6 +154,7 @@ void win_open(win_t *win) Pixmap none; int gmask; XSizeHints sizehints; + XWMHints hints; e = &win->env; parent = options->embed != 0 ? options->embed : RootWindow(e->dpy, e->scr); @@ -252,6 +253,11 @@ void win_open(win_t *win) sizehints.y = win->y; XSetWMNormalHints(win->env.dpy, win->xwin, &sizehints); + hints.flags = InputHint | StateHint; + hints.input = 1; + hints.initial_state = NormalState; + XSetWMHints(win->env.dpy, win->xwin, &hints); + win->h -= win->bar.h; win->buf.w = e->scrw;