Auto-refresh feature doesn't work with Nucleus template.
jumarko opened this issue · 1 comments
jumarko commented
As discussed on Slack (https://clojurians.slack.com/archives/C0M8UK760/p1644436501870119?thread_ts=1644318541.889299&cid=C0M8UK760)
the ring's auto-refresh functionality doesn't work because nucleaus uses custom window.onload
function.
I fixed this in my clone by commenting out this portion of the Nucleus template: curiousprogrammer-net/curiousprogrammer.blog@fe12141
However, I'm not sure what's the proper way to do this generically so I've just created this issue without submitting a PR.
holyjak commented
IMO the correct fix would be to call also the previous onload:
const oldOnload = window.onload;
window.onload = () => { if (oldOnload) oldOnload(); externalLinks(); }
would you care to send a Pr with that? 🙏