-
Notifications
You must be signed in to change notification settings - Fork 1.4k
ListvewExtensionsPage Command prop not found #4039
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
Comments
Hello RosarioPulella, 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 🙌 |
@deanchalk and @Sergio0694, maybe you can us investigate whats going on here. |
Mh, this is odd, the |
I should have mentioned, Local build, Debug|x64 |
This seems similar to the issue we were seeing before with Menu as well. Which all work fine in Release mode I believe in the store app. So this may be an SDK issue? We probably need to construct a small repo with XamlReader in an app to open an issue on WinUI? |
I have the same issue. These are the errors i get:
and
|
Wondering if this is related or the same as microsoft/microsoft-ui-xaml#4614 as we did take the dependency on WinUI 2.5 at that time. We probably need a small sample with just our Toolkit @RosarioPulella once you're done with the UI Tests, want to see if you can test this out in a file new UWP app and see what happens? Then if we have a repro showing its at least related to WinUI, we can attach to the above issue and raise with the WinUI team for 2.6. |
I managed to trigger (x64, Debug) this in my own app after making changes in the configuration manager and updating a few Uno packages. I have a hunch that this may be related to VS 16.10 in some way. If I can confirm it, it'll be the third tooling issue broken by 16.10 today alone 😅 Edit: I'm getting a different error message now related to Command being missing from InvokeActionCommand. Could be related, but this shows on 16.9 as well 🤷♂️ |
There is definitely something strange happening. I'm also missing the file "Microsoft.UI.Xaml.Markup.winmd" in the generated output of the failing application. |
FYI @ranjeshj @StephenLPeters - does this seem like the linked WinUI bug or something else you may be tracking? |
hmm, I wonder if @alwu-msft might know what this is. |
On the surface they do appear similar, although the fact that it only appears to be broken in Debug builds is a wrinkle. What happens if you add (@RealTommyKlein as FYI) |
Pretty sure this is the linked bug I opened against WinUI 2, microsoft/microsoft-ui-xaml#4614 (or at least the error message is idewntical) - @michael-hawker and I had an email thread about it titled "Issue with ICommand Binding in UWP Debug Mode" and the error message/repro is identical to the issue I'd investigated. |
Thanks @RealTommyKlein for the reminder! I had forgotten about that thread from before I was on a vacation in April. Yeah this still seems related then, we had just opened this issue on GitHub later for our own tracking. I'm not sure what impact of your proposed workaround would have on the Toolkit (and its binary size), do you think a fix will be coming in WinUI 2.6 (as we do plan to update the dependency for our next revision)? Would it be helpful for us to confirm or try anything else on our side (like setup an isolated project which shows adding WinUI 2.5 causes the issue)? FYI @alwu-msft we did confirm that we already should have |
I don't think it's fixed for WinUI 2.6, but @ranjeshj can confirm. For putting the workaround in the toolkit (and its samples), the size impact would be minimal and would allow the WCT sample apps to run, but it could still be hit by other external apps which consume WCT and use |
@michael-hawker I would like to try this proposed workaround, but I could not find any mention of one anywhere. Can you elaborate so I can try it out locally? |
…4061) We missed updating a template somewhere, so fatal errors aren't displaying in the Sample App. Ideally this message never appears, this is being surfaced from our issues with `ICommand` in the ListView samples. #4039 ## PR Type What kind of change does this PR introduce? <!-- Please uncomment one or more that apply to this PR. --> <!-- - Bugfix --> <!-- - Feature --> <!-- - Code style update (formatting) --> <!-- - Refactoring (no functional changes, no api changes) --> <!-- - Build or CI related changes --> <!-- - Documentation content changes --> - Sample app changes <!-- - Other... Please describe: --> ## What is the current behavior? Error box appears blank. ## What is the new behavior? Error message shows exception message. ## PR Checklist Please check if your PR fulfills the following requirements: - [ ] Tested code with current [supported SDKs](../readme.md#supported) - [ ] Pull Request has been submitted to the documentation repository [instructions](..\contributing.md#docs). Link: <!-- docs PR link --> - [ ] Sample in sample app has been added / updated (for bug fixes / features) - [ ] Icon has been created (if new sample) following the [Thumbnail Style Guide and templates](https://github.com/windows-toolkit/WindowsCommunityToolkit-design-assets) - [ ] New major technical changes in the toolkit have or will be added to the [Wiki](https://github.com/windows-toolkit/WindowsCommunityToolkit/wiki) e.g. build changes, source generators, testing infrastructure, sample creation changes, etc... - [ ] Tests for the changes have been added (for bug fixes / features) (if applicable) - [ ] Header has been added to all new source files (run *build/UpdateHeaders.bat*) - [ ] Contains **NO** breaking changes <!-- If this PR contains a breaking change, please describe the impact and migration path for existing applications below. Please note that breaking changes are likely to be rejected within minor release cycles or held until major versions. --> ## Other information
Hi guys, me again.
When i analyze the Microsoft.Xaml.Behaviors package, it uses
Could this be the issue why we get all these ICommand/Command errors? |
This was fixed as part of #4182 with the fix from WinUI 2.6.2 |
Uh oh!
There was an error while loading. Please reload this page.
Describe the bug
A clear and concise description of what the bug is.
After building the sample app locally (Debug|x64) when opening the
ListViewExtensions
sample the sample app crashes with the errorSteps to Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Sample opens
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: