The <tt> tag is obsolete

[eric-wieser]

At least, according to https://developer.mozilla.org/en/HTML/Element/tt

[bry4n]

You want to replace that with <code> for making texts displayed in monospace text.

[eric-wieser]

<code> feels semantically incorrect - I wouldn't call bash code. Perhaps I'm looking at it wrongly.

At any rate, shouldn't the pre tag be doing the monospacing? See issue #128

[bry4n]

I think tt tag was intended for formatting. See example (jsfiddle.net).

The example showed that results are different from without tt/code and tt/code.

You are right, the pre tag should make text displayed in monospace.

[eric-wieser]

The fact that it was intended for formatting is exactly the reason it became obsolete. Formatting is the job of CSS.

I see no difference in the formatting of those three (the spacing is different between the ---s, but that happens if you put any tag in a pre).

[bry4n]

I agree with you that tt is an obsolete tag.

Since I am not a designer, I think you want to send the CSS fix for formatting and the monospace font for #128.

[eric-wieser]

As I mentioned in #128, I don't understand the directory structure, and can't find any CSS files to edit.

[jasonlong]

I believe the tt tag is output from the asciidoc parser. @schacon / @nickh - what are your thoughts here?

[eric-wieser]

Looking at the code, it appears that the asciidoc parser just uses templates/section_listing.html.erb

[nickh]

My initial goal was to output (nearly) the same HTML that the asciidoc command did, and they indeed still output tt tags. That said, in gitscm-next that's done by our own templates and not our Asciidoc parser, so we do have some freedom to make it look the way we like. I wouldn't want us to drift too far from the official HTML docs, but this seems safe to me.

[peff]

Closing as stale (the template in question isn't used anymore; we just output whatever asciidoctor says).

[eric-wieser]

If these templates are no longer used, should they not be deleted from master?

[peff]

If these templates are no longer used, should they not be deleted from master?

They are, aren't they?

[eric-wieser]

My mistake, confused my fork with upstream