Skip to content

Conversation

@JayGeorge
Copy link

This closes #13436 by adding the checkerboard to the asset modal on hover.

A few considerations:

  • I believe the checkerboard was designed to not be present so that you can see a clean view of the image
  • This can cause problems with certain images, as reported in [6.x] Checkerboard pattern for png's #13436
  • Therefore, the checkerboard now shows on hover, similar to the thumbnail. This is likely how you'll see it when you first open the modal, since the hover target is so large—I think this works well. It's more like: if you move your mouse away, you see a clean image without a background.

FWIW, I tried playing with hover transitions here, but they don't work well when the background is initially transparent, like it is here, because the checkerboard effect is layered. I think this is fine—it can just be utilitarian, without a transition.

…lly this is the best of both world's so you can still move your mouse out of the way to see what it looks like on a clean background.
@JayGeorge JayGeorge linked an issue Jan 8, 2026 that may be closed by this pull request
@JayGeorge JayGeorge marked this pull request as draft January 8, 2026 14:12
…sparency is supported. This is less jarring than a hover effect with a large target area
@JayGeorge
Copy link
Author

I changed the approach here show a transparency control instead, only when images support transparency.
The hover approach felt janky on such a big hover target area.

On reflection, I think not showing the checkerboard should be the default. It's pretty rare that you don't simply want to see a clean image. Again—I hope this is the best of both worlds.

@JayGeorge JayGeorge marked this pull request as ready for review January 8, 2026 14:48
@jasonvarga jasonvarga merged commit 4f7977d into master Jan 8, 2026
11 checks passed
@jasonvarga jasonvarga deleted the asset-modal-with-checkerboard-pattern branch January 8, 2026 14:59
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.

[6.x] Checkerboard pattern for png's

3 participants