openstreetmap / openstreetmap-website

The Rails application that powers OpenStreetMap
https://www.openstreetmap.org/
GNU General Public License v2.0
2.21k stars 918 forks source link

Restore loading spinner in richtext previews #5311

Closed AntonKhorev closed 1 week ago

AntonKhorev commented 2 weeks ago

Richtext preview loading animation was added in https://github.com/openstreetmap/openstreetmap-website/commit/eb789dbf757e760e8348e0aa75343eb53996b7e3 but later probably got broken. Its loading css class was removed in https://github.com/openstreetmap/openstreetmap-website/commit/df1ec6b6809b533829fc2c171699540bbb92a85b but javascript still adds/removes it to no effect.

This PR adds a Bootstrap spinner to the preview pane.

image

tomhughes commented 1 week ago

Looks good to me, thanks.