Skip to content

Fix: Images no longer flash white#216

Merged
Luke-Schreiber merged 6 commits intomainfrom
image-white-flashing-fix
Feb 3, 2026
Merged

Fix: Images no longer flash white#216
Luke-Schreiber merged 6 commits intomainfrom
image-white-flashing-fix

Conversation

@Luke-Schreiber
Copy link

@Luke-Schreiber Luke-Schreiber commented Jan 6, 2026

Does this PR close any open issues?

Closes #212

Give a longer description of what this PR addresses and why it's needed

  • Images no longer flash white after each load

Additions: Removed old code comments + minor refactoring

Provide pictures/videos of the behavior before and after these changes (optional)

@Luke-Schreiber Luke-Schreiber requested a review from Dev-Lan January 6, 2026 03:41
@Luke-Schreiber Luke-Schreiber self-assigned this Jan 6, 2026
@Luke-Schreiber Luke-Schreiber added bug Something isn't working frontend Deals with Vue Frontend labels Jan 6, 2026
@Luke-Schreiber Luke-Schreiber marked this pull request as draft January 6, 2026 16:51
@Luke-Schreiber Luke-Schreiber marked this pull request as ready for review January 6, 2026 17:02
Copy link

@Dev-Lan Dev-Lan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • add visual indication (loading icon/animation) when the different layers are still loading and are not yet in sync
  • Visual indicator is small lives in header, triggered by any layer change, less flickery (maybe debounce, out of sync for 50 milliseconds)

@Luke-Schreiber
Copy link
Author

Working on moving the visual indicator to the header

@Luke-Schreiber
Copy link
Author

Working on moving the visual indicator to the header

done!

@Luke-Schreiber
Copy link
Author

Now I'm making the 'loading...' less flickery

@Luke-Schreiber
Copy link
Author

Removed flickering, just need to resolve conflicts

@Luke-Schreiber
Copy link
Author

Resolved merge conflicts, just reviewing the code to try and remove unneeded complexity

@Luke-Schreiber
Copy link
Author

Resolved merge conflicts, just reviewing the code to try and remove unneeded complexity

Done

@Luke-Schreiber Luke-Schreiber merged commit c02ca09 into main Feb 3, 2026
@Luke-Schreiber Luke-Schreiber deleted the image-white-flashing-fix branch February 3, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend Deals with Vue Frontend

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Images flash white in images view

2 participants