In quite a few places context is passed between functions unnecessarily, sometimes just to display an error message. This can cause a memory leak, and is just generally messy anyway.
Some pics just dont work as a background, e. g. The black and white line drawing from a couple of weeks ago. It would be nice to have button to ignore the current pic and revert to the previous selection. This sort of happens already with the videos.
Hi!
Today, if we want to open the APOD image of yesterday, we have to open the calendar and select the date of yesterday. And we have to repeat this each time we want to see the previous image.
Of course the calendar is a great idea if we want to jump to a specific day, but I think we should swipe from left or right if we want to browse on a day to day basis.
I hope I've been clear enough as English isn't my native tongue.
Hi,
First thank you very much 😁 for this app.
Then the trouble : when I download an image as wallpaper that appears to be also the lock screen image. Hopefully not the reverse 😉.
Best regards,
Baual
Currently the app will not work if power saving mode is on. Power saving mode can be easily detected, so this should be used to display a warning to the user.
This warning should state the app may not work correctly, and try to help the user add the app to an "excluded apps" list.
When images are set as a wallpaper, some users might prefer always viewing the whole thing. Currently, the image "fits" itself, losing some of the content off-screen.
Current % based approach is counterintuitive, but preference library doesn't seem to support floats. Overriding the UI when the value changes might work.
Currently it's an untestable, randomly sprawling mess. Rewriting it into an MVP / MVVM architecture would mean others can work on it, and tests can begin being added.
A nice improvement would be the ability to filter images according to a colour / brightness criteria. For example, only use images that are more dark than light.
As this app is a greate opensource app, I'd love to see it on F-Droid.
EDIT: Still this app doesn't use only FLOSS libraries, but a fork without all that crap and using something like Android Jetpack Workmanager instead of Firebase Jobdispatcher which got deprecated anyway would be able to be included