Skip to content

Replace loading GIFs - #12787

Open
hlfan wants to merge 4 commits into
openstreetmap:developfrom
hlfan:gifless
Open

hlfan wants to merge 4 commits into
openstreetmap:developfrom
hlfan:gifless

Conversation

@hlfan

@hlfan hlfan commented Aug 26, 2026 •

Copy link
Copy Markdown
Contributor

I'd like to transition the loader GIFs away from the GIF format. I'm open to whether SVG is the better choice.

Here you can see the old and new loading backgrounds next to each other:

This is a first stab; for the non-background loaders, some JS needs to be involved too to load the backgrounds in a way to make use of the CSS colour variables.

@matkoniecz

Copy link
Copy Markdown
Collaborator

what is wrong with gifs? Why using JS-animations will be better?

@hlfan

hlfan commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

I wanna reuse the CSS colour definitions, since iD has much infrastructure built around SVGs, I'd hook into that too.

Just to use that, the SVGs have to be inserted directly into the DOM and not via an image tag, nor just as a CSS background.

@tyrasd tyrasd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, this looks good to me.

Just one question: the background-image: url(img/mini-loader.gif); gif could also be converted to an animated svg, while we're at it?

Comment thread dist/img/arrow-icon.png Outdated
@hlfan

hlfan commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor Author

Yeah the mini-loader.gif is also on my radar, but I haven't yet figured out which way to model it. Done.

I think converting the turn restriction gifs into APNGs would fit here as well (I think animated SVGs for that are overkill). But that should probably go into another PR, as the asset sizes and compatibility shouldn't impact this change.

This branch was successfully deployed

1 active deployment
pr-preview-12787 — 54b55389 Deployed Oct 28, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

icon An issue involving icons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants