Skip to content

Pipe characters were not escaped properly. - #115

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

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

Conversation

@pedro-w

Copy link
Copy Markdown
Contributor

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

This is just a minor typo correction to the table.

Peter Hull (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

Microsoft Contribution License Agreements (msftclas) commented Oct 16, 2017

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved. Thanks!

@pedro-w

Peter Hull (pedro-w) commented Oct 17, 2017

Copy link
Copy Markdown
Contributor Author

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 Robertson (colin-home) merged commit 2295343 into MicrosoftDocs:master Oct 17, 2017
@pedro-w
Peter Hull (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