Skip to content

MarkdownTextBlock's links will cause app freeze if background acrylic is applied to Page #4385

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.

Sign up for GitHub

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

Jump to bottom
Closed
4 of 17 tasks
lee011 opened this issue Nov 18, 2021 · 24 comments
Closed
4 of 17 tasks

MarkdownTextBlock's links will cause app freeze if background acrylic is applied to Page #4385

lee011 opened this issue Nov 18, 2021 · 24 comments
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior need more info 📌 needs author feedback 📝 no-recent-activity 📉 Open Issues that require attention

Comments

@lee011
Copy link

lee011 commented Nov 18, 2021

Describe the bug

When hovering on a link in MarkdownTextBlock of a Page with background acrylic for some time, the app will freeze.

  • Is this bug a regression in the toolkit? If so, what toolkit version did you last see it work: I don't know.

Steps to Reproduce

  • Can this be reproduced in the Sample App? (Either in a sample as-is or with new XAML pasted in the editor.) If so, please provide custom XAML or steps to reproduce. If not, let us know why it can't be reproduced (e.g. more complex setup, environment, dependencies, etc...)
    Unable to reproduce in Sample App.

Steps to reproduce the behavior:

  1. Set a Page with Background="{ThemeResource AcrylicBackgroundFillColorDefaultBrush}"
  2. Set a MarkdownTextBlock with markdown text with links (like [This is a link](https://www.microsoft.com/))
  3. Build and run the app.
  4. Hover on the link and wait for tooltip of the link appears.
  5. The app freezes.

An app to reproduce this issue:
App1.zip

Expected behavior

The app should not freeze when hovering on a link.

Screenshots

App1.2021-11-18.19-09-15.mp4

Environment

NuGet Package(s): Microsoft.Toolkit.Uwp.UI.Controls

Package Version(s): 7.1.1

Windows 11 Build Number:

  • 22000

App min and target version:

  • Fall Creators Update (16299)
  • April 2018 Update (17134)
  • October 2018 Update (17763)
  • May 2019 Update (18362)
  • May 2020 Update (19041)
  • Windows 11 (22000)
  • Insider Build ({build_number})

Device form factor:

  • Desktop
  • Xbox
  • Surface Hub
  • IoT

Visual Studio version:

  • 2017 (15.{minor_version})
  • 2019 (16.{minor_version})
  • 2022 (17.0.0)

Additional context
@lee011 lee011 added the bug 🐛 An unexpected issue that highlights incorrect behavior label Nov 18, 2021
@ghost ghost added the needs triage 🔍 label Nov 18, 2021
@ghost
Copy link

ghost commented Nov 18, 2021

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

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

@XAML-Knight
Copy link
Contributor

I wasn't able to repro the app freeze, but when interacting with this project's files within Visual Studio 2022 (like trying to expand MainPage.xaml within the Solution Explorer window) caused VS22 to close unexpectedly and without warning four separate times.

@XAML-Knight
Copy link
Contributor

@lee011

  1. Do you by chance have the Visual Studio Designer open (such as, for MainPage.xaml) in the IDE while this is happening?
  2. Could you update your App.xaml to this newer style for XAML controls, and see if the app still crashes?:
    <Application x:Class="App1.App" xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:controls="using:Microsoft.UI.Xaml.Controls" xmlns:local="using:App1"> <Application.Resources> <controls:XamlControlsResources> <controls:XamlControlsResources.MergedDictionaries> <!--Other app resources here--> </controls:XamlControlsResources.MergedDictionaries> </controls:XamlControlsResources> </Application.Resources> </Application>

@XAML-Knight
Copy link
Contributor

I wasn't able to repro the app freeze, but when interacting with this project's files within Visual Studio 2022 (like trying to expand MainPage.xaml within the Solution Explorer window) caused VS22 to close unexpectedly and without warning four separate times.

Confirmed that VS appears stable as long as I don't have the VS Designer open (see ref #4279 and #4310).

@lee011
Copy link
Author

lee011 commented Nov 23, 2021

@lee011

  1. Do you by chance have the Visual Studio Designer open (such as, for MainPage.xaml) in the IDE while this is happening?
  2. Could you update your App.xaml to this newer style for XAML controls, and see if the app still crashes?:
    <Application x:Class="App1.App" xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:controls="using:Microsoft.UI.Xaml.Controls" xmlns:local="using:App1"> <Application.Resources> <controls:XamlControlsResources> <controls:XamlControlsResources.MergedDictionaries> <!--Other app resources here--> </controls:XamlControlsResources.MergedDictionaries> </controls:XamlControlsResources> </Application.Resources> </Application>
  1. This happens regardless of whether Visual Studio Designer is opened or not.
  2. I have tried to modify App.xaml as above, but still have problem.

@Arlodotexe Arlodotexe mentioned this issue Dec 4, 2021
Open
@ghost ghost added the needs attention 👋 label Dec 8, 2021
@ghost
Copy link

ghost commented Dec 8, 2021

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

1 similar comment
@ghost
Copy link

ghost commented Dec 23, 2021

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@Arlodotexe
Copy link
Member

I'm also unable to reproduce this bug with the provided sample / repro steps.

@ghost
Copy link

ghost commented Jan 12, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

12 similar comments
@ghost
Copy link

ghost commented Jan 28, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Feb 12, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Feb 27, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Mar 14, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Mar 29, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Apr 13, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Apr 28, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented May 13, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented May 28, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Jun 12, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Jun 27, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@ghost
Copy link

ghost commented Jul 12, 2022

This issue has been marked as "needs attention 👋" due to no activity for 15 days. Please triage the issue so the fix can be established.

@LalithaNadimpalli
Copy link
Contributor

I am unable to reproduce the bug with the provided sample. It would be helpful if you could provide more info on this issue

@ghost ghost added the no-recent-activity 📉 Open Issues that require attention label Aug 6, 2022
@ghost
Copy link

ghost commented Aug 6, 2022

lee011 This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 30 days of this comment.

@ghost
Copy link

ghost commented Sep 5, 2022

lee011 This issue has been automatically closed because it was marked as requiring author feedback but has not had any activity for over 45 days.

@ghost ghost closed this as completed Sep 5, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Nov 5, 2022
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior need more info 📌 needs author feedback 📝 no-recent-activity 📉 Open Issues that require attention
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Arlodotexe @lee011 @LalithaNadimpalli @XAML-Knight