Consider removing img.onload

[ianb]

Because of our onload handler, you can't see a partially loaded image, or other useful things that <img> tags do. We did this because of the alt text, but if we wanted we could style the alt text how we wanted

I think we should consider making the image visible always, changing the alt text, and maybe putting a loading spinner on top that we hide on load.

[ianb]

Related comment

[ianb]

Here's a codepen that demonstrates a technique without any Javascript. It puts the loading icon under the image, so it automatically gets hidden when the image loads. The alt text is visible in the codepen, but we can change the color if we want. We could also use aria-label (or maybe there's a better aria attribute) if we want to maintain accessibility without using alt.