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

More control over wallpaper [2/2] #154

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TpmKranz
Copy link

The old implementation allowed only to set one of the two backgrounds at one time, but not both at once.

The Bash part of this is backwards compatible with the old WallpaperAPI implementation, but trying -l home would paradoxically set the lockscreen wallpaper. Cf. termux/termux-api#526 for the Java part.

@TpmKranz TpmKranz changed the title Allow setting both wallpapers at once [2/2] More control over wallpaper [2/2] Jun 25, 2022
@TpmKranz
Copy link
Author

The old implementation sets the leftmost part of the image as the initially visible area, making it necessary to crop images before handing them over to termux-wallpaper, when that functionality is already provided by the setBitmap method.

The rectangle specification was inspired by ImageMagick, but cut down (and in the case of relative offsets changed) a bit for the specific use case of setting wallpapers.

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.

1 participant