Change default behavior to lazy loading

executablebooks / sphinx-thebe

A Sphinx extension to convert static code into interactive code cells with Jupyter, Thebe, and Binder.

https://sphinx-thebe.readthedocs.io/en/latest/

MIT License

28 stars 15 forks source link

Change default behavior to lazy loading #36

Closed choldgraf closed 2 years ago

choldgraf commented 3 years ago

Description / Summary

We currently allow for lazy loading of pages (so thebe JS only loads if certain node types are on a page etc) behind a feature flag. This is because we didn't want it to be too disruptive as the kinks were ironed out.

Once #34 is satisfied we should make the default behavior "lazy load".

Value / benefit

This will save a lot of JS time loading for people.

Implementation details

No response

Tasks to complete

[x] #34
[x] Change the default here

choldgraf commented 2 years ago

closed by #41

choldgraf commented 2 years ago

@akhmerov would you be willing to try this out in the contexts you're using this extension (if you are) on the master branch, to make sure we didn't break something by switching to lazy loading?