Fix skin-canvas visibility

[catfromplan9]

skin-canvas is being hidden due to being part of noscript-hidden class. Added JS to remove skin-canvas from noscript-hidden class, because if SkinURL is set the skin-canvas will be drawn. When JS is disabled, this will not be executed and the canvas will remain hidden.

[evan-goode]

This is already done in a script in layout.tmpl?

[catfromplan9]

This is already done in a script in layout.tmpl?

Doesn't seem to be working for me

[evan-goode]

Can you reproduce it on a fresh install (the OCI container might be useful)? Here is where it's done:

drasl/view/layout.tmpl

Line 29 in dd346df

for (const el of document.querySelectorAll(".noscript-hidden")) {

[catfromplan9]

Can you reproduce it on a fresh install (the OCI container might be useful)? Here is where it's done:

drasl/view/layout.tmpl

Line 29 in dd346df

for (const el of document.querySelectorAll(".noscript-hidden")) {

The problem is that this relies upon .App.Config.EnableBackgroundEffect being set to true, you've put it into the same if statement as that

[catfromplan9]

Oh, I guess the version I am running predates this commit: 03ee17c

[catfromplan9]

I am running the tagged 3.0.0 release, which was tagged a week before that commit fixed it. I'm waiting for a new tagged release to fix it.

[evan-goode]

I realize I didn't push the v3.1.0 tag, so I just pushed v3.1.1.