Skip to content

Either quotation marks or italicization#1751

Merged
iliakan merged 1 commit into
masterfrom
unknown repository
Apr 5, 2020
Merged

Either quotation marks or italicization#1751
iliakan merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Feb 13, 2020

Copy link
Copy Markdown

Highlighting twice is unnecessary.

Highlighting twice is unnecessary.
@iliakan

iliakan commented Apr 2, 2020

Copy link
Copy Markdown
Member

You mean italic is bad there?

@ghost

ghost commented Apr 3, 2020

Copy link
Copy Markdown
Author

@iliakan, the actual problem is that you can't figure out wether it's a cite or the apostrophes suggest some kind irony because you can not really make a web page alive, shortly because it's a personification.

In the first case the apostrophes suffice, for the second I wouldn't even mark it up as it is kind of difficult to mark something up like irony, and if so it would've been sufficient to italicize just "alive" as I suppose you know so I don't think this was your intend. Or am I mistaken?

@iliakan

iliakan commented Apr 4, 2020

Copy link
Copy Markdown
Member

@paroche could you please advise here?

@paroche

paroche commented Apr 5, 2020

Copy link
Copy Markdown
Collaborator

Never caught my attention before (the one or two times I read it), but, now that it's come up, I think it would be a bit better with just the quotes (not apostrophes), without the italicizing.

@ghost

ghost commented Apr 5, 2020

Copy link
Copy Markdown
Author

quotes (not apostrophes)

Oh, you're right.

@iliakan

iliakan commented Apr 5, 2020

Copy link
Copy Markdown
Member

Thanks!

@iliakan
iliakan merged commit c89ddc5 into javascript-tutorial:master Apr 5, 2020
@paroche

paroche commented Apr 5, 2020

Copy link
Copy Markdown
Collaborator

@u-n-known
quotes (not apostrophes)

Oh, you're right.

Well, you got it in the title! :-)

And, of course, in JS itself they are almost interchangeable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants