code.less: also set default monospace font for <kbd> and <samp>

[eins78]

Right now the "default monospace" (as set in variables.less) font is only set on code and pre elements.

It should include at least kbd and samp as well (command is obsolete and var doesn't have to be monospace)

Edit: Updated for clarity

[saas786]

@eins78 your pull request is invalid, please review normalize.less and you will see its already set to "monospace font".

Ref: https://github.com/twbs/bootstrap/blob/master/less/normalize.less#L161

After review please close this pull request.

[cvrebert]

@saas786 normalize doesn't use/honor any LESS variables. This does.

[eins78]

Yes, I could have said it more clearly: The configured (as in set as variable in variables.css) monospace font is not used....

[saas786]

@cvrebert normalize doesn't use/honor any LESS variables. This does.

I am aware of this.

@eins78 Yes, I could have said it more clearly: The configured (as in set as variable in variables.css) monospace font is not used....

Yes, from your initial message I thought you just wanted to have monospace font applied, now I understood you want to have customize able variable based monospace fonts applied to < kbd > & < samp >.

[mdo]

Why should we style these elements as we do code snippets? Based on my limited reading I don't see a direct correlation.

[eins78]

This is less about "style" and more about consistent typography.
If there is a variable for the monospace font, to me it seems more correct to apply that to all elements which are using a monospace font (either by browser default and/or normalise.less).
Any other styling, as it is done for code snippets, should of course not be applied to these elements – this is just about the font family.

[mdo]

it seems more correct to apply that to all elements which are using a monospace font (either by browser default and/or normalise.less).

Ah, that last part is the kicker—it's the browser default.

Thanks!