search

PreTeXtBook / pretext

PreTeXt: an authoring and publishing system for scholarly documents
https://pretextbook.org
Other
254 stars 202 forks source link

WeBWorK: remove MathView css #2125

Closed Alex-Jordan closed 2 months ago

Alex-Jordan commented 2 months ago

As a comment suggests (a comment now being deleted) it is unclear why this CSS file was loaded this way. Maybe there was a temporary, experimental period where we loaded live WW problems by inserting them into the DOM without using an iframe. Then I could see this being used. But the old 2.16- style of live rendering, and also the settled 2.17+ style of a static version that transitions to the live problem both use iframes, and the proper CSS for that tool should be handled correctly now by pretext-webwork.js.

So these lines can go. Note that on a 2.18 WW server, the file here exists but is in a different location. So this is actually causing console errors (404) at sites using a 2.18 server.

rbeezer commented 2 months ago

Thanks for keeping this neat and tidy. I added a follow-on commit to rename the relevant template to make its purpose clearer.