[openstreetmap/openstreetmap-website] Css spinner delay (PR #5315)
Anton Khorev
notifications at github.com
Wed Nov 13 12:30:27 UTC 2024
Similarly to #5307, we can avoid scheduling/unscheduling js events to show spinners in the richtext preview #5311 and the sidebar loader:

There's a delay to avoid blinking on fast loads, that delay can be implemented using a css animation.
And we also can stop using `vendor/assets/jquery/jquery.timers.js` which does some ill-advised things like [listening to `unload`](https://developer.mozilla.org/en-US/docs/Web/API/Window/unload_event).
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5315
-- Commit Summary --
* Show spinner using delayed css animation in richtext preview
* Show spinner using delayed css animation in sidebar loader
-- File Changes --
M app/assets/javascripts/index.js (11)
M app/assets/javascripts/richtext.js (9)
M app/assets/stylesheets/common.scss (12)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/5315.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5315.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5315
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5315 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241113/1eb37a7b/attachment.htm>
More information about the rails-dev
mailing list