PR #648 fixed a lot of the issues with the ImageViewer widget, but there is still one major remaining problem.
The auto-hide/show of the overlay UI (buttons + metadata) does not always work consistently.
- A lot of the time it doesn't show the UI when I move the mouse (after it was previously hidden).
- Sometimes it only hides the UI after I click/tap to manually toggle it, instead of hiding it automatically after the timeout has elapsed.
- It doesn't auto-hide at all after the user first opens the ImageViewer (for the first time). I tried to fix the behavior a bit by starting the timer in
show_loaded() and show_loading(), but it still wasn't consistent. Unfortunately i don't have more time to spend on it.
I think the overall logic for determining whether the UI overlays needs to be shown or hidden can be simplified in general.
Originally posted by @kevinaboos in #648 (review)