Skip to content
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

Microsoft.Toolkit.Uwp.UI.Controls.Core/Resources ResourceMap Not Found #4931

Open
Manohar-Gunturu opened this issue Aug 30, 2023 · 2 comments
Labels
bug 🐛 An unexpected issue that highlights incorrect behavior controls 🎛️

Comments

@Manohar-Gunturu
Copy link

Manohar-Gunturu commented Aug 30, 2023

Line throwing the error 'ResourceMap Not Found':

AutomationProperties.SetName(_dismissButton, "WCT_InAppNotification_DismissButton_AutomationName".GetLocalized("Microsoft.Toolkit.Uwp.UI.Controls.Core/Resources"));

We have defined our own InAppNotificationAlias.Template and this line has been crashing the app on Win10. Is our app supposed to access Microsoft.Toolkit.Uwp.UI.Controls.Core/Resources, or do we need to define the WCT_InAppNotification_DismissButton_AutomationName string in the app's resource file?

@ghost ghost added the needs triage 🔍 label Aug 30, 2023
@ghost
Copy link

ghost commented Aug 30, 2023

Hello Manohar-Gunturu, 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 🙌

@michael-hawker michael-hawker added bug 🐛 An unexpected issue that highlights incorrect behavior controls 🎛️ and removed needs triage 🔍 labels Sep 2, 2023
@michael-hawker
Copy link
Member

This is defined by the Core package itself here:

<data name="WCT_InAppNotification_DismissButton_AutomationName" xml:space="preserve">
<value>Dismiss</value>
<comment>The automation name for the dismiss button of the InAppNotification control.</comment>
</data>

There shouldn't be a reason for it not to be found. Unless if you custom template doesn't have a dismiss button maybe? Otherwise, you should be able to define the full resource string in your app.

In either case, InAppNotification won't ship in the upcoming 8.0 release. It's been replaced with the StackedNotificationsBehavior on top of WinUI's InfoBar control. So, I'd recommend that approach moving forward with the incoming 8.0 release.

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 controls 🎛️
Projects
None yet
Development

No branches or pull requests

2 participants