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

stylix: add background image scaling option #372

Merged
merged 2 commits into from
Jun 3, 2024

Conversation

cramt
Copy link
Contributor

@cramt cramt commented May 20, 2024

adds option for which type of scaling the background image should get
currently this only applied sway, but should probably be applied to the other WM/DE also supported in stylix, if someone knows how to configure it for the rest of them i'd appreciate the help
i set the default to be fill, the one the previous option used in sway by stylix, not sure if this should be the right default, as it seems some of the other WM/DEs have different default configs (eg gnome with zoom).

also not sure if "scaling mode" is the right term for this, it was what the sway man pages used.

@trueNAHO
Copy link
Collaborator

Related: #102

Copy link
Owner

@danth danth left a comment

Choose a reason for hiding this comment

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

I think fill is what most people would expect as the default. This list of options seems to be fairly common across different wallpaper programs, too.

This will need to be followed up by setting the option for more programs, and perhaps documenting what each scaling mode does - but it looks good to merge as a starting point.

@danth danth merged commit c760f63 into danth:master Jun 3, 2024
9 checks passed
@danth danth changed the title add background image scaling option stylix: add background image scaling option Jun 3, 2024
@danth danth mentioned this pull request Jun 10, 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.

3 participants