mortie / swaylock-effects

Swaylock, with fancy effects
MIT License
698 stars 44 forks source link

Screenshots appear to be broken #85

Open joelazar opened 2 years ago

joelazar commented 2 years ago

Since sway 1.7 (wlroots 0.15) update, it looks like when swaylock-effects screenshot feature is broken, and I only get white image as a lock background. From debug logs I don't see any unusual:

❯ swaylock --screenshots \
            --clock \
            --indicator \
            --indicator-radius 100 \
            --indicator-thickness 7 \
            --effect-blur 7x5 \
            --effect-vignette 0.5:0.5 \
            --ring-color 192330 \
            --key-hl-color 9d79d6 \
            --line-color 000000 \
            --inside-color c94f6d \
            --separator-color 000000 \
            --grace 3 \
            --fade-in 0.5 \
            --effect-greyscale -d
2022-01-26 10:49:08 - [main.c:1780] Parsing CLI Args
2022-01-26 10:49:08 - [main.c:652] output name is eDP-1
2022-01-26 10:49:08 - [main.c:331] Surface for output eDP-1 ready
2022-01-26 10:49:09 - [main.c:614] Loaded screenshot for output eDP-1
tjssoldier commented 2 years ago

I have the same behavior after sway update.

gmontanola commented 2 years ago

I can confirm the issue with Sway 1.7.

matthis-k commented 2 years ago

Can confirm aswell

haxibami commented 2 years ago

I had the same behavior with the stable release, but when I compiled from the latest source (in Arch Linux “swaylock-effects-git” package) it successfully worked.

joelazar commented 2 years ago

I had the same behavior with the stable release, but when I compiled from the latest source (in Arch Linux “swaylock-effects-git” package) it successfully worked.

Thanks, that solved for me as well. It might be a good time then to make a new release of this package. @mortie

misuchiru03 commented 2 years ago

Can verify this finding with Void as well. 1.6.3 screenshots option is available, but broken. Rebuilt with commit a8fc557 and master and screenshots seem to be working fine. I concur this would be a new release would be optimal.

jirutka commented 2 years ago

This is now fixed in jirutka/swaylock-effects releases 1.6.10. See #92.