Closed alebastr closed 8 months ago
Attention: 188 lines
in your changes are missing coverage. Please review.
Comparison is base (
d314806
) 66.17% compared to head (728b302
) 65.24%.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Tested the branch and at least the warning mentioned in #1214 seems to be gone. But didn't had the impression it was working. But I need to test that further.
Tested the branch and at least the warning mentioned in #1214 seems to be gone. But didn't had the impression it was working. But I need to test that further.
Can you set a very short (like 10s short) idle_threshold, start dunst as WAYLAND_DEBUG=client dunst -verbosity debug
and collect the output?
I've been testing various corner-cases with Sway 1.9 snapshot and everything has been working so far. labwc shouldn't have any significant differences in behavior with Sway.
All good. Can confirm this PR does fix #1214 I hadn't noticed that I removed the setting from the config (and the default seems to be to not set a time)? Typical user error :D Thank you for the work on the wayland related side of things
Thanks for this much needed wayland work! With a quick look, the X11 impl seems to be fine as well, but I'm not too familiar with the xlib either. If someone is able to test it on X11, that would be great.
I've taken a look at all the code. I have some questions and minor things, but over all it looks good.
The wl.c file is getting a bit big and could use some refactoring to separate files. I'm not saying it should happen in this PR, but this PR does make it a little bit bigger as well.
Thanks for all the fixes and wayland features! Now dunst is up to date with the protocols again
I'll go and merge this PR. If you want to work on refactoring the wayaland code a bit, that would be cool. But don't feel obliged :)
The wl.c file is getting a bit big and could use some refactoring to separate files. I'm not saying it should happen in this PR, but this PR does make it a little bit bigger as well.
I'll take a look. An obvious idea is to split seat-related code to src/wayland/seat.c
, but there's a couple of things to address before I'll be able to do that.
A bunch of fixes for Wayland backend:
ext-idle-notify-v1
, required by wlroots >= 0.17cursor-shape-v1
for compositor-rendered cursors to reduce headache with themes and scalingglobal.monitor = <name>
(eDP-1, etc...)Tested on Fedora 39 and FreeBSD 14. Haven't tried with older systems, but everything version-dependent should be properly guarded. Cursor-shape and ext-idle-notify could be tested with a compositor already ported to the new wlroots (Sway master or v1.9 branches).
I'm 100% unsure about the X11 impl for monitor names. It's only been tested with XWayland, and frankly I'm not familiar with the Xlib.
Fixes: #1132 Likely also addresses #1214