API sites: increase line-height

[JoseTomasTocino]

The new redesign is great, but I find the line-height to be too small. Text is really hard to follow, specially in big screens (27" and up).

[ajpiano]

I am a bit confused by this one - currently on the site, the line height in that area is not 18px, it's already 1.5em (adding a new rule like .longdesc p { line-height: 1.5em; } confirms this). Bumping it up to 1.7em, as @kleinmaetschke did in #200, seems to me to be too much

I think one of the issues here is that code elements can bleed into each other, e.g.,
,

but I just fixed that in 89168fd.

I am not sure we actually need any action here, unless we still think 1.5em is not enough. What do you think @JoseTomasTocino @kleinmaetschke?

[kleinmaetschke]

I suppose its a matter of preference. I'm on a high resolution screen and more line-height, the better. But it's not entirely unmanageable without the change. Especially now that the code elements no longer bleed.

[ajpiano]

Also worth noting is that it's not actuall 1.5em now, it's 15px/22.5px as specified on the body, so maybe the real discussion we should be having is #174. I guess my point is that if there's an issue with the line-height, we should probably address it more globally than just in the longdesc on the api sites, because that's just using the default. Presumably we'd want the same readability/style on blog posts, articles, etc.

[kleinmaetschke]

Soon as I'm able, I'll update with your padding change and check it out. It may well have been simply the code elements visually destroying the line-height, though technically it was the same. Because I don't recall the general blog posts or long paragraphs of text elsewhere being especially crowded.

Though, @JoseTomasTocino may have a different answer.

[JoseTomasTocino]

I'm OK the way it is right now. 15px/22.5px is in fact 1.5em of line-height, so I think it's alright by now. You can close this if feel like it.

[ajpiano]

Great. Thanks all. Let's still think about #174 though