From 93b6fac46888df89bbed86daf097360fa03d9ebb Mon Sep 17 00:00:00 2001 From: Sebastian Reinhard Date: Sun, 6 Dec 2020 10:13:19 +0100 Subject: nimdoc: Initialize theme switch and pragma dots on DOMContentLoaded (#16247) The default HTML template for nimdoc currently initializes the dark mode switch and pragma dots when the onload event is fired. But since the onload event does not fire until all external resources (images, stylesheets, fonts, etc.) have been loaded, the light theme is shown for a brief moment before the document is fully loaded, and it switches to the dark theme. This is quite jarring, especially on slower internet connections. So let's instead initialize these things on the DOMContentLoaded event, which is fired right after the document has been parsed and the initial DOM structure is ready. This means that we now display the dark mode immediately, without having to wait for external resources to load first. For reference, see: - https://developer.mozilla.org/en-US/docs/Web/API/Window/load_event - https://developer.mozilla.org/en-US/docs/Web/API/Window/DOMContentLoaded_event I have updated the snapshot tests in `nimdoc/`, and done some manual testing of both the theme switch and the pragma dots, to confirm that this does not break anything. Please let me know if I've missed anything. --- nimdoc/rst2html/expected/rst_examples.html | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'nimdoc/rst2html') diff --git a/nimdoc/rst2html/expected/rst_examples.html b/nimdoc/rst2html/expected/rst_examples.html index 8a08e8df5..0b9207688 100644 --- a/nimdoc/rst2html/expected/rst_examples.html +++ b/nimdoc/rst2html/expected/rst_examples.html @@ -56,10 +56,12 @@ function main() { } } } + +window.addEventListener('DOMContentLoaded', main); - +

Not a Nim Manual

-- cgit 1.4.1-2-gfad0