Skip to content

fix: enable recursive scanning for custom wallpaper directories - #115

Merged
mmstick merged 2 commits into
pop-os:masterfrom
ZeusCraft10:fix/wallpaper-recursive
Jun 22, 2026
Merged

fix: enable recursive scanning for custom wallpaper directories#115
mmstick merged 2 commits into
pop-os:masterfrom
ZeusCraft10:fix/wallpaper-recursive

Conversation

@ZeusCraft10

@ZeusCraft10 ZeusCraft10 commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

Fixes pop-os/cosmic-epoch#2841. Enables recursive scanning for custom wallpaper directories using WalkDir, consistent with XDG_DATA_DIRS behavior. Added a unit test to verify recursion.

mmstick
mmstick previously approved these changes Jan 21, 2026
@mmstick
mmstick requested a review from a team January 21, 2026 17:52
@garrettjwilke

Copy link
Copy Markdown

@ZeusCraft10 i am going to test/review this...can you fix the conflicts?

@jacobgkau
jacobgkau force-pushed the fix/wallpaper-recursive branch 3 times, most recently from 935c7af to 47bd85e Compare June 16, 2026 21:25
@jacobgkau
jacobgkau force-pushed the fix/wallpaper-recursive branch from 47bd85e to 6e0b9e0 Compare June 16, 2026 21:25

@jacobgkau jacobgkau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recreated pop-os/cosmic-epoch#2841 using a structure like this:

/home/system76/Wallpapers/
/home/system76/Wallpapers/Category-A/
/home/system76/Wallpapers/Category-B/

The wallpapers in the subfolders showed up for individual selection, but the background turned plain grey after turning the slideshow on. After installing this fix, the slideshow works as expected in this environment.

@jacobgkau
jacobgkau requested review from a team and mmstick June 16, 2026 22:56
@mmstick
mmstick merged commit 8dd97b6 into pop-os:master Jun 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wallpaper slideshow does not work with my images

4 participants