search

waldyrious / rst-playground

Browser-based reStructuredText playground, built on Pyodide and docutils.
https://waldyrious.github.io/rst-playground
ISC License
1 stars 4 forks source link

Consider using Pyscript #3

Closed waldyrious closed 1 year ago

waldyrious commented 1 year ago

Pyscript provides a <py-script> HTML element that could make the code a little more readable. It relies on Pyodide underneath, so no functionality would be lost.

not-my-profile commented 1 year ago

I wouldn't pull in an additional dependency for something that doesn't benefit users at all.

waldyrious commented 1 year ago

Right. This could be justifiable if the entire code was in the index file (the benefit would be to potential contributors rather than to users), but now that it's been split to separate CSS and JS files (#9), there isn't much going for it indeed, especially when considering the downside of adding another abstraction layer, more complexity to the architecture, and a third dependency (besides Pyodide and docutils). I'll close this.