|  | Commit message (Collapse) | Author | Age | 
|---|
| | |  | 
| | |  | 
| | |  | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| | the artifacts were caused because we tried to set the gamma right after
receiving the event, this resulted in two pending page-flips, which
not always play well together.
This also seems to fix a screen freeze when turning on a monitor that has
gamma.
Additionally the current method won't work once [0] is merged
[0]: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4423 | 
| | 
| 
| 
| 
| | References: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4131
Closes: https://github.com/djpohly/dwl/issues/411 | 
| | 
| 
| 
| | This reverts commit 72e0a560d9836c5e8658003f548203bcd722e565. | 
| | 
| 
| 
| 
| | References: https://www.freedesktop.org/wiki/Specifications/cursor-spec/
References: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4416 | 
| | |  | 
| |\ |  | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | | 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 | 
| |\| |  | 
| | | |  | 
| | | 
| | 
| | 
| | 
| | 
| | | this fixes a segfault when the client emits .set_hints but it's not mapped
Fixes: 72a7d78a1a7926a207539eb50f44b2e1eb089d49 | 
| |\| |  | 
| | | 
| | 
| | 
| | 
| | 
| | | I'm going to make some changes in the config file anyway
This reverts commit e5367753bb90add013ee5d170a110064298ac2c4. | 
| | | |  | 
| | | |  | 
| | | 
| | 
| | 
| | | iirc this is the same behavior of dwm | 
| | | 
| | 
| | 
| | | Closes: https://github.com/djpohly/dwl/issues/487 | 
| | | 
| | 
| | 
| | 
| | 
| | | is true
Bug: https://github.com/djpohly/dwl/issues/487 | 
| |\| |  | 
| | | 
| | 
| | 
| | 
| | | since in the previous commit we may not applying floating in clients
this is to make sure we do | 
| | | 
| | 
| | 
| | 
| | 
| | | there is still a bug, but for now this prevents a segfault
Bug: https://github.com/djpohly/dwl/issues/472 | 
| | | |  | 
| | | |  | 
| | | 
| | 
| | 
| | | we do not use the node | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | | 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 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | | 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 | 
| | | 
| | 
| | 
| | | It creates an infinite commit-resize loop when scale != 1 | 
| | | |  | 
| | | |  | 
| | | 
| | 
| | 
| | | References: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4106 | 
| | | 
| | 
| | 
| | 
| | 
| | | Now scene will handle sending the preferred buffer scale
References: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4269 | 
| |\| |  | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | | ΔSLOC: -4
Fixes: https://github.com/djpohly/dwl/issues/456
Fixes: https://github.com/djpohly/dwl/issues/459 | 
| | | 
| | 
| | 
| | 
| | | This appears to have been here for the side effect of updating the drag
icon's position. | 
| | | |  | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | | If there is no current drag icon, this node will be empty, but we now
have `drag_icon != NULL` as an invariant.  This allows us to eliminate a
conditional, since there's no harm in moving an empty node's coordinates
around with the pointer.
ΔSLOC: -1 | 
| | | 
| | 
| | 
| | | ΔSLOC: +1 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | | Merge our signal handlers into a single function and let Wayland deal
with all the struct sigaction stuff.
ΔSLOC: -3 | 
| | | 
| | 
| | 
| | | References: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4220 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | | This parallels the LISTEN macro for statically allocated listeners, and
it allows us to remove almost all of the global wl_listener
declarations.
This also fixes a bug with the axisnotify listener, which was declared
with a compound literal.  At block scope, these have automatic storage
duration [1], so the listener was no longer valid after setup()
returned.  (The option to declare it static explicitly was standardized
in C23, if that ever gains suckless traction.)
ΔSLOC: -27
[1]: https://en.cppreference.com/w/c/language/compound_literal#Explanation | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | | There were still a couple of places where we could use this handy
multiple assignment to save a line.
ΔSLOC = -3 | 
| |\| |  | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | | If we treat the drag icon as distinct from other layers (it doesn't have
contents that are interactive, focusable, etc.), then we can iterate
over layers meaningfully with a simple for loop.
ΔSLOC: -8 | 
| | | 
| | 
| | 
| | 
| | | It added complexity, especially with the differences in behavior between
handled and ignored signals across an exec(). | 
| | | |  | 
| | | 
| | 
| | 
| | | Ignored handlers are not reset by exec() calls | 
| | | |  | 
| | | 
| | 
| | 
| | 
| | | not doing it, may freeze the output or do not actually change the gamma until
creating a client |