createkeyboard: do not segfault if xkb_keymap_new_from_names returns NULL

Passing NULL to wlr_keyboard_set_keymap results in a segfault.
Example:

  Thread 1 "dwl" received signal SIGSEGV, Segmentation fault.
  0x00007ffff7e49b64 in xkb_keymap_ref () from /usr/lib/libxkbcommon.so.0
  (gdb) bt
  #0  0x00007ffff7e49b64 in xkb_keymap_ref () at /usr/lib/libxkbcommon.so.0
  #1  0x00007ffff7f06389 in wlr_keyboard_set_keymap () at /usr/lib/libwlroots.so.11
  #2  0x000055555555bc54 in createkeyboard ()
  #3  0x000055555555c283 in inputdevice ()
  #4  0x00007ffff7e8101e in wl_signal_emit_mutable () at /usr/lib/libwayland-server.so.0
  #5  0x00007ffff7e8101e in wl_signal_emit_mutable () at /usr/lib/libwayland-server.so.0
  #6  0x00007ffff7edb52c in  () at /usr/lib/libwlroots.so.11
  #7  0x00007ffff7ee44b6 in  () at /usr/lib/libwlroots.so.11
  #8  0x000055555555fe66 in main ()

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.