Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Apply wallpaper to all desktops including the current one #50

Closed
wants to merge 1 commit into from

Conversation

ThisFungus
Copy link

This change ensures that the Set-DesktopWallpaper command correctly applies the wallpaper to all virtual desktops, including the current one, by removing the incorrect index reference from the $params.imagePaths array.

This change ensures that the Set-DesktopWallpaper command correctly applies the wallpaper to all virtual desktops, including the current one, by removing the incorrect index reference from the $params.imagePaths array.
@t1m0thyj
Copy link
Owner

@ThisFungus Could you provide an example of the error you encountered with the old script and an explanation of how this change fixes it? As far as I can tell the -Path parameter must be a string, and since params.imagePaths is an array we need to pass only the first item.

@ThisFungus
Copy link
Author

@t1m0thyj Weird, after updating to OS Build 22631.3737 (may or may not be relevant for this case), I can confirm that the line:
Set-DesktopWallpaper -Desktop $_.Number -Path $params.imagePaths[0]
is now working correctly, and I am no longer able to replicate the bug I previously encountered. As a result, I'm closing this pull request.

@ThisFungus ThisFungus closed this Jun 23, 2024
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.

2 participants