On 6/9/20 7:58 AM, Dave Page wrote: > So here's the updated patch following some discussion with Magnus: > > - Make the copyScript function more generic so it can be used elsewhere. > - Don't include comments and blank lines in the copied text. > - Move the css into main.css. > - Add the actual install step to the Debian/Ubuntu instructions. > - Improve naming of vars, classes, ids etc. > - Ensure the copyScript function works fine with < > and so-on in the > script.
That certainly changed a lot overnight ;)
LGTM. I don't have any additional comments on it, so I will give a +1.
Thanks. Any thoughts on the server vs. client thing?