Skip to content

batches: un-optimize and move loading spinner SVG

Administrator requested to merge kr/fix-loading-svg into main

Created by: courier-new

svgo seems to break the animation on this loading spinner SVG, so I've reverted the optimization and stuck it in an unoptimized folder to keep it separate from the rest and not fail the check.

Before vs. After

https://user-images.githubusercontent.com/8942601/142477638-1293c246-b1f6-42b4-9728-a9a49726c7b4.mov

From a cursory look over svgo's issues, there are many other reports of svgo breaking an animated SVG in some way or another, so I don't think it's worth trying to fix it in this case. The difference the optimization makes is minimal (only 32kb down to 30kb, or 7% reduction), and this SVG is only used within Batch Changes.

cc: @umpox

Merge request reports