-
Notifications
You must be signed in to change notification settings - Fork 661
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
TeachingTip throws exception when calling isOpen #349
Comments
Thanks for the report! The missing symbols is something that was broken for this build which I've corrected for future builds, but I did go back and try to re-publish symbols for this prerelease. Hopefully that will be working by the end of today. As for the crash you're seeing, @StephenLPeters can you please take a look when you're back tomorrow? |
@duke7553 look like symbols are published now, could you try resolving symbols and getting a crash stack now? Thanks! |
@jevansaks I just tried to resolve the symbols, and everything went fine until I received an error related to base.h not being found. |
Could you copy/paste the text from the Callstack window? Some of the files (like base.h) are from generated files and therefore aren't possible to source index since they don't live in GitHub. |
This is everything I found in the call stack window
|
Thanks! We'll investigate. |
@duke7553, I know in the earlier version of the control there is a bug that causes the tip to crash if it does not contain fields for an icon and hero (formerly "bleeding") content. It is something that should be fixed in the updated version. But to help you where you're at, I've included the code from my working branch of the XAML Sample Gallery below. Does adding the IconSource and BleedingImageContent fix it for you?
If not, it will require @StephenLPeters to help us look into it. |
@duke7553 I've done some digging and found this issue, it is a bug in teaching tip and we are very grateful for your help finding it. If you need a work around right now you can either set an Icon on your teaching tip, or, because of a very strange quirk in this bug, run your app without a debugger attached. This is actually why this bug escaped us, the crash will only occur while a debugger is attached and the tip has no icon, all of our tests run without a debugger by default and our manual verification included an icon. I'll be submitting a fix for this shortly. Thanks again! |
@StephenLPeters I tried the workaround to add an IconSource and start without a debugger, but one of two things happen:
Not sure if your PR fixes this, but I thought I should report it. |
Thanks for the extra details, we'll make sure we try both of those scenarios ASAP. |
@StephenLPeters Once you've verified that this is fixed, please close. |
🎉This issue was addressed in #364, which has now been successfully released as Handy links: |
@StephenLPeters The TeachingTip is now able to display when IsOpen is called from non-markup code. I'll close this issue. |
Describe the bug
Defining a TeachingTip in XAML markup and setting the
isOpen
property will cause the app to crash after throwing aMicrosoft C++ exception: winrt::hresult_error at memory location 0x0000001B01A7DF90
. Since this is definitely a low level error, it requires certain debug symbols for the Microsoft.UI.Xaml.dll module that are not present on my machine, even after loading all symbols.Steps to reproduce the bug
Steps to reproduce the behavior:
isOpen
property from the XAML or C# source to trueExpected behavior
The TeachingTip should appear, and the application should not crash, nor throw any exceptions related to this control.
Screenshots
Version Info
Please Note: Both target SDK versions tested (1809 and Insider build 18327) seemed to demonstrate this bug.
NuGet package version:
Additional context
Creating a reproduction project with nothing except a button and the teaching tip attached to it had the same problem. Furthermore, I understand that this is a pre-release control, so I don't expect it to be working at all. I just wanted to ensure this bug is evaluated before the control is released to stable.
The text was updated successfully, but these errors were encountered: