-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Spec missusing the samp tag? #1364
Comments
Interesting find! I think the program in question is the console, not ssh itself. This appears to be a sample of a console session, not a sample of ssh output. It might be worth adding a note, or even a third example, clarifying this. E.g. modify the existing example to say "... a block of sample output from a console session", then adding a third example saying something like
What do you think? |
That looks great and i agree that the program is the console, not the commands itself. The more interesting use case for the <pre>
<code class="language-javascript">console.log(2.3 + 2.4)</code>
<samp>4.699999999999999</samp>
</pre> |
Yeah, that makes sense to me. Are you interested in working on a pull request to work on some mashup of my suggestion and your example? |
I'm going to try, but my english kinda sucks and writing specs is even harder. Give me a few minutes for the PR :) |
I hope this looks good. |
https://html.spec.whatwg.org/multipage/semantics.html#the-samp-element
I'm curious here. Samp tags represent output from another program. But the ssh command is not output actually, it's input.
So i would only wrap the above as this:
Why? Because as i said, the commands like ssh are actually input and they are generated by the human. Those bits are not output from computer.
The same as if you run this command:
That is javascript that prints "foo" in screen. foo is the output and should be wrapped inside samp tags. But the above bits are input so i would use a
<code>
tag for it, and use something like language-shell, or bash, or whatever (i'm not sure about the correct "language" of command execution/interpreters).But definitely not output, and i think abusing the output tag
<samp>
to show input it's a bad decission and should be modified in the spec.The text was updated successfully, but these errors were encountered: