Microsoft.Toolkit.Parsers Markdown Link dont gets rendered

[Wenish]

I'm submitting a...

Bug report (I searched for similar issues and did not find one)

Current behavior

Im using the UWP Markdown Textblock (https://docs.microsoft.com/en-us/windows/uwpcommunitytoolkit/controls/markdowntextblock)
When im rendering the following Markdown whitout a space in the middle:

[Company Portal] (CompanyPortal://)

It gets displayed as normal Text

Expected behavior

The Markdown parser should render it as Link

Minimal reproduction of the problem with instructions

Add normal UWP Markdowntextblock and user following Markdown whitout a space in the middle:
[Company Portal] (CompanyPortal://)

Environment

Nuget Package(s): 

Package Version(s): 

Windows 10 Build Number:
- [ ] Anniversary Update (14393) 
- [ ] Creators Update (15063)
- [ x] Fall Creators Update (16299)
- [ ] Insider Build (build number: )

App min and target version:
- [ ] Anniversary Update (14393) 
- [ ] Creators Update (15063)
- [x ] Fall Creators Update (16299)
- [ ] Insider Build (xxxxx)

Device form factor:
- [x ] Desktop
- [ ] Mobile
- [ ] Xbox
- [ ] Surface Hub
- [ ] IoT

Visual Studio 
- [ x] 2017 (version: )
- [ ] 2017 Preview (version: )

[avknaidu]

your scheme CompanyPortal is not part of supported scheme's of MarkdownTextBlock Links. Please see below what are all the supported scheme's

https://github.com/Microsoft/WindowsCommunityToolkit/blob/9766eb794deadb56628c258bd87c32ce8d076cc4/Microsoft.Toolkit.Parsers/Markdown/Inlines/HyperlinkInline.cs#L70-L82

[nmetulev]

Should we have a way for developers to inject their custom schema? Thoughts on this

[avknaidu]

Yes. We can add a dp which will apppend the list of custom scheme's. I can take it if you think that is a good sol'n.

[nmetulev]

That would be great, go for it @avknaidu. The DP should support adding multiple custom schemas

[michael-hawker]

@WilliamABradley is there a reason we look for known schemes? Is it to differentiate between an internal vs. external link?

[WilliamABradley]

@michael-hawker Don't know, wasn't my implementation.

[nmetulev]

PR merged

[Wenish]

@avknaidu @nmetulev
What about allow every Link?
there are so many and its not really nice when u have to update the uwp app just to add another string.