danyspin97 / wpaperd

Modern wallpaper daemon for Wayland
GNU General Public License v3.0
329 stars 25 forks source link

Can't go to previous wallpaper if sorting is set to random #60

Closed Rolv-Apneseth closed 6 months ago

Rolv-Apneseth commented 7 months ago

Hi again.

I saw the AUR package had been disowned by the previous maintainer so I decided to adopt it. I went to try out wpaperd again (great progress btw, it's very clean now), but found that the previous option doesn't work when I have sorting set to random. Both ascending and descending work, so it's just with random. There's no error either, it just silently does nothing.

I'm the one who originally added that previous wallpaper functionality, but I see you've made some nice improvements! If you want I could look into what's causing the issue, just wanted to confirm if other people are having it too or it's something weird with my setup.

This is my current config:

 [default]
path = "/home/rolv/repos/Wallpapers/main/"
duration = "30m"
sorting = "random"
transition_time = 400
queue_size = 20
danyspin97 commented 7 months ago

Hi @Rolv-Apneseth, I am happy to hear from you again. Although this is a bug, and I am not sure that I should say I am happy xD

I saw the AUR package had been disowned by the previous maintainer so I decided to adopt it.

Thank you for maintaining the AUR package and for the compliments :)

If you want I could look into what's causing the issue, just wanted to confirm if other people are having it too or it's something weird with my setup.

I can confirm this happens on my setup as well, unfortunately. I am attending a conference all week, I probably won't have a lot of time to look into this, so please go ahead!

Rolv-Apneseth commented 7 months ago

Alright I'll look into when I have some time, thanks for confirming

danyspin97 commented 6 months ago

Fixed by #61 .