Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/playground: pressing run or share on lynx gives a "bad html" error #52585

Closed
mindfarm opened this issue Apr 27, 2022 · 4 comments
Closed

x/playground: pressing run or share on lynx gives a "bad html" error #52585

mindfarm opened this issue Apr 27, 2022 · 4 comments

Comments

@mindfarm
Copy link

This was reported and confirmed on IRC

13:46 < Andrew> While visiting and putting code (playing)
                with play.golang.org, lynx says bad html:
                no form action defined, while pressing run
                or share
13:53 < b0nn> Yeah - I get it too; you can report it as an
              issue on the go github page I think
13:57 < Andrew> problem is (1) submitting issues/prs on
                github requires nonfree javascript and a
                modern web browser and (2) I don
13:57 < Andrew> 't have one... 

The problem is that accessing the play.golang.org website using lynx (command line browser) produces warnings and errors

@dieselburner
Copy link

My vision is that whoever nowadays uses lynx is hardcore enough to fix the code (or implement new) by them own.

Sorry, but for developers it's a waste of time - this browser is used by 0.5% of population. Oh wait, article was written in 1997.

@mvdan
Copy link
Member

mvdan commented Apr 27, 2022

If the browser is right that the HTML is incorrect, I think it's fair to try to fix bugs like that when they are reported. I don't expect the developers to test on every browser under the sun continuously, but if valid bugs are reported, I don't see why they can't be fixed if someone sends a patch.

@mvdan mvdan changed the title Website x/playground: pressing run or share on lynx gives a "bad html" error Apr 27, 2022
@gopherbot gopherbot added this to the Unreleased milestone Apr 27, 2022
@seankhliao
Copy link
Member

The playground API accepts json, not a html form. I also don't see how the run results would be streamed to a browser not running js without significant rework of both front and backend.

@mindfarm
Copy link
Author

I've tried with a number of other command line browsers, but wasn't able to get the Run button to activate
Tried

I think it's right to close this as javascript does indeed seem to be the stumbling block (I tried to build spidermonkey for elinks so that I could get javascript running in a CLI browser, but to no avail [Run-time dependency tre found: NO] and it's not that important to me)

@golang golang locked and limited conversation to collaborators Apr 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants