On Thu, Nov 02, 2017 at 10:29:15AM -0700, Omar Sandoval wrote:
> On Thu, Nov 02, 2017 at 06:19:44PM +0100, Hiltjo Posthuma wrote:
> > On Thu, Nov 02, 2017 at 02:10:35AM -0700, Omar Sandoval wrote:
> > > This is useful for, e.g., compton.
> > > ---
> > > dmenu.c | 8 ++++++++
> > > 1 file changed, 8 insertions(+)
> > >
> > > diff --git a/dmenu.c b/dmenu.c
> > > index d605ab4..2a16ad8 100644
> > > --- a/dmenu.c
> > > +++ b/dmenu.c
> > > @@ -545,6 +545,7 @@ setup(void)
> > > XIM xim;
> > > Window w, dw, *dws;
> > > XWindowAttributes wa;
> > > + XClassHint *ch;
> > > #ifdef XINERAMA
> > > XineramaScreenInfo *info;
> > > Window pw;
> > > @@ -613,6 +614,13 @@ setup(void)
> > > win = XCreateWindow(dpy, parentwin, x, y, mw, mh, 0,
> > > CopyFromParent, CopyFromParent, CopyFromParent,
> > > CWOverrideRedirect | CWBackPixel | CWEventMask,
> > > &swa);
> > > + ch = XAllocClassHint();
> > > + if (ch) {
> > > + ch->res_name = "dmenu";
> > > + ch->res_class = "dmenu";
> > > + XSetClassHint(dpy, win, ch);
> > > + XFree(ch);
> > > + }
> > >
> > > /* open input methods */
> > > xim = XOpenIM(dpy, NULL, NULL, NULL);
> > > --
> > > 2.15.0
> > >
> > >
> >
> > Meh, can't you just use the process name?
>
> No, because dmenu doesn't set _NET_WM_PID either, so there's no way to
> map the window to a process :) I don't think compton and other similar
> stuff supports keying on process name, anyways.
>
Then it should be changed/fixed in (your local) compton. Then all applications
can benefit and all the applications don't need to be patched.
There are too many _NET_WM_CRAP already.
--
Kind regards,
Hiltjo
