Open 28raining opened 10 months ago
To solve https://github.com/plotly/plotly.js/issues/6706
The issue is about a broken link. I used https://www.drlinkcheck.com/ to find all the broken links in https://plotly.com/javascript/. It only scanned the first 2000 links without paying. This PR fixes multiple issues
'more controls' button doesn't need to exist as all control options are already displayed (there are 5). Subplots has 6 and we see the button
inside a code block throws a console warning about un-escaped HTML - but if we escape it the link won't work. The link is v hidden anyway, imo it can be removed
Fastly's logo is no longer at that URL and I couldn't find where it is
main.css corrected a a typo. There is still an unclosed { (line 9094) but closing it messes up the page CSS... Not obvious how to fix that
{
swapped highlightBlock for highlightElement, to fix console warning. https://highlightjs.readthedocs.io/en/latest/api.html
To solve https://github.com/plotly/plotly.js/issues/6706
The issue is about a broken link. I used https://www.drlinkcheck.com/ to find all the broken links in https://plotly.com/javascript/. It only scanned the first 2000 links without paying. This PR fixes multiple issues
'more controls' button doesn't need to exist as all control options are already displayed (there are 5). Subplots has 6 and we see the button
inside a code block throws a console warning about un-escaped HTML - but if we escape it the link won't work. The link is v hidden anyway, imo it can be removed
Fastly's logo is no longer at that URL and I couldn't find where it is
main.css corrected a a typo. There is still an unclosed
{
(line 9094) but closing it messes up the page CSS... Not obvious how to fix thatswapped highlightBlock for highlightElement, to fix console warning. https://highlightjs.readthedocs.io/en/latest/api.html