check client is mapped before setting border color

For some reason brave configured for as a wayland client triggers this code on startup and segfaults.

Checking if the client is mapped fixes this, like with the previous fix for urgent border colour.

References: 887fde65a3010905aa10f373cfcfe540cfc1781e
Fixes: 72a7d78a1a7926a207539eb50f44b2e1eb089d49

use <xcb/xcb.h> instead of <X11/Xlib.h>

only set border color for urgent *and* mapped X11 clients

this fixes a segfault when the client emits .set_hints but it's not mapped

Fixes: 72a7d78a1a7926a207539eb50f44b2e1eb089d49

Revert "just add define"

I'm going to make some changes in the config file anyway

This reverts commit e5367753bb90add013ee5d170a110064298ac2c4.

add [-d] flag to enable debug logging

simplify setting the border color of clients

make the borders red borders for urgent clients

iirc this is the same behavior of dwm

Check if c is fullscreen before reparenting it to LyrFloating

Closes: https://github.com/djpohly/dwl/issues/487

make sure that fullscreen clients are on the correct screen when isfloating is true

Bug: https://github.com/djpohly/dwl/issues/487

call setfloating in setmon

since in the previous commit we may not applying floating in clients
this is to make sure we do

return early if the client doesn't have monitor in setfloating

there is still a bug, but for now this prevents a segfault

Bug: https://github.com/djpohly/dwl/issues/472

just add define

tagcount should have been a #define

return nothing in xytonode()

we do not use the node

Style: use early-return to clarify code

Use an early return to avoid indenting the main logic instead of
wrapping the tail of a function in an if statement.

No functional change, except for a handful of places where printstatus()
was being called spuriously (tag, toggletag, toggleview).

ΔSLOC: 0

No need to send surface.leave/enter events

The scene graph implementation sends these for us, and it does so more
accurately than our overly-simplified approach.  Layer shell surfaces
don't appear to receive these events at all, according to my
WAYLAND_DEBUG experiments with bemenu and dtao.

ΔSLOC: -4

don't resize clients on commit

It creates an infinite commit-resize loop when scale != 1

do not use wl_event_loop for signal handling

ΔSLOC: -4

Fixes: https://github.com/djpohly/dwl/issues/456
Fixes: https://github.com/djpohly/dwl/issues/459

Remove now-unneeded call to motionnotify

This appears to have been here for the side effect of updating the drag
icon's position.

Remove unused icon variable