Age | Commit message (Collapse) | Author |
|
|
|
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
|
|
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
|
|
Merge our signal handlers into a single function and let Wayland deal
with all the struct sigaction stuff.
ΔSLOC: -3
|
|
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
|
|
not doing it, may freeze the output or do not actually change the gamma until
creating a client
|
|
|
|
This reverts commit 65f68e76437275a0c1c25279af646f726e213b6d.
|
|
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
|
|
use ext-idle-notify-v1 instead
|