Arguably, a few typographic improvement.

mrkline / concurrency-primer

The basics of low-level concurrency in a sub-30 minute read

Other

176 stars 7 forks source link

Arguably, a few typographic improvement. #1

Closed TApplencourt closed 6 years ago

TApplencourt commented 7 years ago

Remove the use of italic and small caps on punctuation marks

Anyways, thank you for this great introduction!

mrkline commented 6 years ago

Hey, sorry for the delay, and thanks for taking the time to read it and offer suggestions!

At least in the typeface I'm using (Equity), punctuation set in small caps seems to look a touch better than the "full-sized" versions, especially when set in the middle of body text:

Regarding italicized punctuation, itcomma (The first comma is italicized, and the second is not.)

If there's any common style guidelines that disagree with these decisions, let me know - I'll readily admit that I might be making some wrong choices here.

tavianator commented 6 years ago

@mrkline Most style guides seem to advocate not italicizing the punctuation after an italic word or phrase (e.g. https://english.stackexchange.com/a/90775/62522)

mrkline commented 6 years ago

@tavianator Thanks for linking that! Looking through the project history, I think I ran into that same suggestion in December. See 9d71b403 - let me know if I missed a spot.

@TApplencourt - I'd prefer to continue using the small caps parens. Besides those, are there other changes you'd like to make with this PR?

TApplencourt commented 6 years ago

@mrkline No, and I was merely a suggestion. So I will close this PR . Thanks for this nice paper!