Skip to content

Add apply wallpaper button#18

Merged
bjarneo merged 2 commits into
bjarneo:mainfrom
saeedvaziry:main
Oct 20, 2025
Merged

Add apply wallpaper button#18
bjarneo merged 2 commits into
bjarneo:mainfrom
saeedvaziry:main

Conversation

@saeedvaziry
Copy link
Copy Markdown
Contributor

This PR adds an Apply Wallpaper button next to the Edit button after selecting the Wallpaper.

Background

Sometimes you don't need to generate the whole theme, you just need to change the background image from the wallpaper you find.

This feature enables you to click and apply the wallpaper without changing the theme.

What it does?

Under the hood it symlinks the wallpaper path to ~/.config/omarchy/current/background and then restarts swaybg with the new wallpaper's path.

Screenshot

image

Comment thread src/utils/ConfigWriter.js
Comment on lines +603 to +604
GLib.spawn_command_line_async('pkill -x swaybg');
GLib.spawn_command_line_async(`swaybg -i "${wallpaperPath}" -m fill`);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately omarchy doesn't have a built-in command to use that instead, So this will be a tech-debt maybe until omarchy implements it.

@bjarneo
Copy link
Copy Markdown
Owner

bjarneo commented Oct 19, 2025

Good thinking about this one. I feel the button gets a bit..long? Other than that, it makes sense to me. Really important to nail the UX.

Does only "Apply" make sense?

@saeedvaziry
Copy link
Copy Markdown
Contributor Author

Apply makes more sense yes. Feel free to change or I will do it later a bit

@bjarneo
Copy link
Copy Markdown
Owner

bjarneo commented Oct 20, 2025

Thank you. I'll go through the PR today and get it merged. I might adjust parts of it afterward for the reload bg part, as I need a system for more stuff here. Good starting point.

@bjarneo bjarneo merged commit 237dd41 into bjarneo:main Oct 20, 2025
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