Skip to content

Pipe characters were not escaped properly.#115

Merged
colin-home merged 2 commits into
MicrosoftDocs:masterfrom
pedro-w:patch-1
Oct 17, 2017
Merged

Pipe characters were not escaped properly.#115
colin-home merged 2 commits into
MicrosoftDocs:masterfrom
pedro-w:patch-1

Conversation

@pedro-w
Copy link
Copy Markdown
Contributor

@pedro-w pedro-w commented Oct 16, 2017

Inside backticks, the hex entities were showing up as the literal characters |

This is just a minor typo correction to the table.

pedro-w and others added 2 commits October 16, 2017 21:22
Inside backticks, the hex entities were showing up as the literal characters |
Switch to links and bold instead of unusual code formatting
@msftclas
Copy link
Copy Markdown

msftclas commented Oct 16, 2017

CLA assistant check
All CLA requirements met.

Copy link
Copy Markdown
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Thanks!

@pedro-w
Copy link
Copy Markdown
Contributor Author

pedro-w commented Oct 17, 2017

I should add: I found this typo on your website, https://docs.microsoft.com/en-gb/cpp/cpp/cpp-built-in-operators-precedence-and-associativity
Formatting a pipe character in backticks in a table seems to be a bit of a 'corner case' for markdown, so your website might not convert to HTML the same way that github does it.
An alternative to backtick-backslash-pipe-backtick that works (on github, at least) is <code>&#124;</code>

(edit) sorry, never mind, I see you've done the formatting a different (better!) way

@colin-home colin-home merged commit 2295343 into MicrosoftDocs:master Oct 17, 2017
@pedro-w pedro-w deleted the patch-1 branch October 18, 2017 12:49
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