Skip to content

MarkdownTextBlock throw an exception while handling link with space link text #3120

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Tlaster opened this issue Feb 4, 2020 · 8 comments
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior markdown 📑 Issues related to Markdown
Milestone

Comments

@Tlaster
Copy link

Tlaster commented Feb 4, 2020

Describe the bug

Try rendering markdown like this

[ ](https://www.google.com/)

And the MarkdownTextBlock throw an exception

Steps to Reproduce

Steps to reproduce the behavior:

  1. Open Sample App
  2. Open MarkDownTextBlock sample
  3. Add the following markdown to the textbox at the top:
[ ](https://www.google.com/)
  1. Sample App crash

Expected behavior

MarkdownTextBlock render a space

Environment

NuGet Package(s): 
6.0.0
Package Version(s): 
6.0.0

Windows 10 Build Number:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [x] May 2019 Update (18362)
- [ ] Insider Build (build number: )

App min and target version:
- [ ] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [x] October 2018 Update (17763)
- [ ] May 2019 Update (18362)
- [ ] Insider Build (xxxxx)

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

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

@Tlaster Tlaster added the bug 🐛 An unexpected issue that highlights incorrect behavior label Feb 4, 2020
@ghost ghost added the needs triage 🔍 label Feb 4, 2020
@ghost
Copy link

ghost commented Feb 4, 2020

Hello Tlaster, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will investigate the issue and help solve it ASAP. Other community members may also look into the issue and provide feedback 🙌

@Kyaa-dost
Copy link
Contributor

Hi, @Tlaster I am able to reproduce this as well. That's a good catch! Our developers will certainly look into this and get back to you.

@Kyaa-dost
Copy link
Contributor

@Sergio0694 There was a similar issue that occurred in past #2806. Any thoughts on this?

@azchohfi
Copy link
Contributor

It's not the same issue. This one is a StaskOverflowException.
@WilliamABradley, any idea?

@Kyaa-dost
Copy link
Contributor

@Jasonstein I am still able to reproduce this issue. Is this still being worked on your end?

@Kyaa-dost Kyaa-dost modified the milestones: 6.1, 7.0 Jun 22, 2020
@0x7c13
Copy link
Contributor

0x7c13 commented Jun 22, 2020

@Kyaa-dost It is fixed on my side in my app:
image
It is invisible (after rendering empty space) but it will not crash the app.

@Kyaa-dost Kyaa-dost modified the milestones: 7.0, 7.1 Feb 18, 2021
@Kyaa-dost Kyaa-dost modified the milestones: 7.1, future Apr 29, 2021
@tesar-tech
Copy link

I also have this issue. Simple link like this: [somehting](https://github.com) will freeze the app when hovering the link. It does not crash (immediately ), it just use more and more of memory (so I guess it's SO exception)?

@Arlodotexe
Copy link
Member

I also have this issue. Simple link like this: [somehting](https://github.com) will freeze the app when hovering the link. It does not crash (immediately ), it just use more and more of memory (so I guess it's SO exception)?

@tesar-tech Are you referring to #4385?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior markdown 📑 Issues related to Markdown
Projects
None yet
Development

No branches or pull requests

7 participants