WallYou icon indicating copy to clipboard operation
WallYou copied to clipboard

Small bug reports (3 listed)

Open Zeeex opened this issue 1 year ago • 2 comments

Steps to reproduce

First and foremost, this app is amazing and beautiful. Using it is simply delightful. 💜

Didn't want to split bugs found into multiple tickets, because they are probably easy to fix.

  1. Occasionally, a lowercase "timeout" toast emerges. It's uncertain whether this is related to the service provider "Wallhaven" or the in-app functionality. Would be nice to get a more verbose message so we know what happened.

  2. When you rotate your phone with auto rotation on, scroll position resets to 0, resulting in the loss of your current position.

  3. When in landscape mode, images get cropped even if they are in portrait mode. Height should be increased to accommodate their original (near original) height.

Expected behavior

  1. Include detailed message on what error occurred, and perhaps provide guidance on the necessary steps to take.
  2. Scroll position should be maintained when rotating your phone.
  3. Images should be full visible (or at least give us an option to toggle the full / cropped view).

Actual behavior

  1. Sometimes toast displays with no explanation.
  2. Scroll position resets to 0 on rotating phone.
  3. Images height in landscape mode is cropped.

Wall You version

5.0

Android version

Android 9

Other details

No response

Acknowledgements

  • [X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
  • [X] I have written a short but informative title.
  • [X] I will fill out all of the requested information in this form.

Zeeex avatar Aug 16 '23 06:08 Zeeex

  1. means that wallhaven doesn't respond - I don't think we should add extra custom error messages for any possible error (network, timeout, dns, ...)
  2. is certainly UX unfriendly, thus that's reasonable to address.
  3. has been adressed in #115.

Bnyro avatar Aug 19 '23 09:08 Bnyro

  1. 👍
  2. Maybe some message like "Wallhaven - timeout", so we know it's the service / network issue, and not in-app problem. But I'm ok with not changing it either.
  3. 😊

Zeeex avatar Aug 19 '23 15:08 Zeeex