search

MicrosoftDocs / winrt-api

WinRT reference content for developing Microsoft Universal Windows Platform (UWP) apps
Creative Commons Attribution 4.0 International
231 stars 500 forks source link

Escape broken links for contoso.com #2417

Closed Saisang closed 1 year ago

Saisang commented 1 year ago

https://dev.azure.com/msft-skilling/Content/_workitems/edit/173987

prmerger-automator[bot] commented 1 year ago

@Saisang : Thanks for your contribution! The author(s) have been notified to review your proposed change.

stevewhims commented 1 year ago

I recommend visiting the published webpage for each of these hits for contoso, and confirming that there really is a hyperlink before considering making any changes.

Of the many changes in this PR, the only ones that are necessary are where the content contains www.contoso.com (without the backticks). Once you've put that string in backticks, you can remove any other markdown formatting, such as italics (single asterisk).

So most of the changes here are unnecessary, and some of them are undesirable--for example, putting backticks INSIDE quotation marks.

If you have only "contoso.com" (without quotes) in the content, then that doesn't render as a hyperlink (so there can't be a broken link). So no need to escape ANY case like that.

If we could have a new PR with ONLY the necessary changes, that'd be great.

Thanks! -Steve

Court72 commented 1 year ago

Thanks, @stevewhims!