diff options
| author | Ruben Gonzalez <gonzaru@sdf.org> | 2026-03-13 15:23:24 +0200 |
|---|---|---|
| committer | Hiltjo Posthuma <hiltjo@codemadness.org> | 2026-03-13 18:27:18 +0100 |
| commit | 44dbc6809d05b8f2addc483f882e670db0b6b8e9 (patch) | |
| tree | 00807887438324092f0ebd3bf3c44f1a5d58e7e1 | |
| parent | 2bb919e6342ae04242e3af6d5921e550d3e0a619 (diff) | |
The status bar in drawbar() calculates the text width as TEXTW(stext)
- lrpad + 2. However, the click detection in buttonpress() used
TEXTW(stext) without adjusting for that padding.
This created an "extra" clickable area of some pixels to the left of
the status text that would incorrectly trigger ClkStatusText actions
instead of ClkWinTitle.
Steps to reproduce:
1. Set a status text: xsetroot -name "HELLO"
2. Move the mouse to the empty space with some pixels close to the
left of the word "HELLO" but in the title area.
3. Middle-click (or any binding for ClkStatusText).
4. You can see that the status bar action is triggered (default a
terminal spawns), even though you clicked in the window title area.
This fix ensures that the clickable area matches the visual text.
| -rw-r--r-- | dwm.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -440,7 +440,7 @@ buttonpress(XEvent *e) arg.ui = 1 << i; } else if (ev->x < x + TEXTW(selmon->ltsymbol)) click = ClkLtSymbol; - else if (ev->x > selmon->ww - (int)TEXTW(stext)) + else if (ev->x > selmon->ww - (int)TEXTW(stext) + lrpad - 2) click = ClkStatusText; else click = ClkWinTitle; |
