Feat: blurred wallpaper variant and save-existing-theme flow - #125
Open
tristonarmstrong wants to merge 17 commits into
Open
Feat: blurred wallpaper variant and save-existing-theme flow#125tristonarmstrong wants to merge 17 commits into
tristonarmstrong wants to merge 17 commits into
Conversation
Author
|
damn out of date already.. i worked on this last night lmfao will sync |
Author
|
theres lots of formatting changes.. ill work on undoing those - my b |
Apply sends a backend-generated blurred JPEG instead of the original while extraction keeps sampling the unblurred source. Includes the regenerated Wails bindings (reformatted by the v2.15 CLI; also carries the ThemeFolderExists and originalWallpaperPath fields used by the following commits).
This reverts commit 8405404.
tristonarmstrong
force-pushed
the
feature/wallpaper-blur
branch
from
August 28, 2026 22:13
eea7d95 to
3c00c9d
Compare
tristonarmstrong
marked this pull request as ready for review
August 29, 2026 00:12
Author
|
Please review, have your AI review. Let me know what you think, if its something you think is worth including.. I absolutely LOVE it but i can understand if its not something youere into Feel free to commit straight to this if you have changes you wanna make or just let me know and i can do it |
Collaborator
|
I misunderstood your post on X. Will have a look at this as soon as I have time! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is meant as a seed planter — to start the conversation. I also looked at the existing blur functionality, and although it does blur, it's... not great... and doesn't really do what I wanted + its kinda hidden away. This is a single button, front and center, and offers a gorgeous background blur for those aesthetics nuts out there.
What
BlurWallpaper— the blurred JPEG is applied as the desktop background while the original stays untouched for palette extractionOriginalWallpaperPaththrough the full theme pipeline so both the blurred and unblurred copies land inbackgrounds/(desktop cycler can switch between them)omarchy-theme-setto prevent it cycling to a stock imageThemeFolderExiststo let the frontend offer updating an existing saved theme in placeWhy
Wallpapers with high-frequency detail (text, sharp edges, busy patterns) clash with overlaid UI elements. A blurred variant gives a clean backdrop while preserving palette extraction from the sharp source.
Testing
TestBlurWallpaperCreatesVariant,TestBlurWallpaperRejectsNonImage,TestBlurWallpaperRejectsMissingFileTestPrepareThemeDirCopiesWallpaperVariants(blurred + original both copied)TestThemeFolderExists(omarchy + saved dirs, case normalization, invalid names)make testandnpm run checkpass