awesome icon indicating copy to clipboard operation
awesome copied to clipboard

Adding zoom-fill

Open Stardust-kyun opened this issue 3 months ago • 4 comments

Added a zoom-fill example to the awful.wallpaper docs and replaced awful.wallpaper default to use zoom-fill in awesomerc.

Stardust-kyun avatar May 07 '24 14:05 Stardust-kyun

I've also noticed that all of the maximized examples are written as "mazimized", with a z instead of an x -- is this intentional?

Stardust-kyun avatar May 07 '24 14:05 Stardust-kyun

as "mazimized",

yeah clearly a typo

(changes in usage of wallpaper module in rc.lua)

i'll leave up the decision to @Elv13 who as far as i remember was designing the wallpaper module. i personally think that result should be achieved by just using (or mb adding new) arguments, not manually pre-cropping the wallpaper straight in the rc.lua

actionless avatar May 08 '24 15:05 actionless

Hi,

Thanks for the extra doc example. Can you have just the example part as part of this PR? I am not really convinced why we would change the default. With this code, it relies on the current screen geometry (which might, sometime, change without this code being re-executed) and uses a rather low level helper function (which will confuse new users). I think those 2 things have side effects which might cause problem in some cases.

Elv13 avatar May 12 '24 08:05 Elv13

Thanks for the reply Elv! Reverted the change to the default. I figured that it would help new users to have it as the default since using zoom-fill for wallpapers is most common, and the default code can be misunderstood as such. However, I agree with your comments about its reliance on screen geometry and gears.surface. Pull should now be ready to go.

Stardust-kyun avatar May 14 '24 22:05 Stardust-kyun