Hamza417 / Peristyle

Simple wallpaper manager app for Android with awesome UI, tags and multiple folder support and a native live wallpaper picker.
Apache License 2.0
248 stars 6 forks source link

wider wallpapers when cropped aren't centralized #48

Closed weiss1597 closed 4 months ago

weiss1597 commented 5 months ago

I'm guessing that when cropping wide wallpapers they should be centralized, and unless it's my impression, the app does seem to do that, sometimes at least, and if it doesn't centralize, it would be a nice addition.

the full wallpaper Screenshot_20240524-052645_Peristyle

and here's how it ends up ResizedImage_2024-05-24_05-34-50_1340

the screenshots were taken on version 1.42, but this same behavior happens on the latest version, just that on the latest version they are sometimes sliding off of the screen as mentioned on #47

Hamza417 commented 5 months ago

I just tested it, cropping is working fine in both app and auto wallpaper. Can you run the app in a different device and preferably a device with vanilla Android?

Hamza417 commented 5 months ago

cropping is working fine in both app and auto wallpaper

Meaning it's cropping the wallpaper from the center as intended. When cropping is disabled ot relies on system wallpaper manager to set the wallpaper as the device intends it to.

weiss1597 commented 5 months ago

I might be able to test it on some other phone, but it may take a while before that.

And also I've found another few wallpapers were it correctly aligns on the center, but they end up zoomed in

left full wallpaper, middle how it should have been, right how it ended up ResizedImage_2024-05-24_07-22-53_2602 centralized but zoomed in, that on auto wallpaper with cropping activated.

Hamza417 commented 5 months ago

There are settings for showing zoomed in wallpapers, or either the device shows zoomed wallpaper by default. This is not an issue with the app.

Untitled

weiss1597 commented 5 months ago

this in fact fix the "issue" mentioned on #49

but the zoom in here is much bigger, I've found out that it does centralize the wallpaper on auto mode but only if the resolution is bigger than the display, and the bigger the wallpaper resolution the more it zoom in

that first one I've showed is smaller than the display