commit a859676ead17017bbe81b4989b2f2e0b00a0b4ba
from: Stein <bakkeby@gmail.com>
via: Hiltjo Posthuma <hiltjo@codemadness.org>
date: Wed Aug 10 13:31:21 2022 UTC

Simplify client y-offset correction

The reasoning behind the original line may be lost to time as
it does not make much sense checking the position on the x-axis
to determine how to position the client on the y-axis.

In the context of multi-monitor setups the monitor y position
(m->my) may be greater than 0 (say 500), in which case the window
could be placed out of view if:
   - the window attributes have a 0 value for the y position and
   - we end up using the y position of bh (e.g. 22)

If the aim is to avoid a new floating client covering the bar then
restricting y position to be at least that of the window area
(m->wy) should cover the two cases of using a top bar and using a
bottom bar.

commit - e0dee911455cee739a5b05a994828f4a37a2764d
commit + a859676ead17017bbe81b4989b2f2e0b00a0b4ba
blob - 967c9e8bf9c294d27fbb7c252c60483d037ec9e8
blob + 87d0adab5920880acb15550c68ae0f24abaff867
--- dwm.c
+++ dwm.c
@@ -1049,9 +1049,7 @@ manage(Window w, XWindowAttributes *wa)
 	if (c->y + HEIGHT(c) > c->mon->my + c->mon->mh)
 		c->y = c->mon->my + c->mon->mh - HEIGHT(c);
 	c->x = MAX(c->x, c->mon->mx);
-	/* only fix client y-offset, if the client center might cover the bar */
-	c->y = MAX(c->y, ((c->mon->by == c->mon->my) && (c->x + (c->w / 2) >= c->mon->wx)
-		&& (c->x + (c->w / 2) < c->mon->wx + c->mon->ww)) ? bh : c->mon->my);
+	c->y = MAX(c->y, c->mon->wy);
 	c->bw = borderpx;
 
 	wc.border_width = c->bw;