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

Fixes #513 - AdvancedCollectionView not working with NativeAoT #535

Merged
merged 7 commits into from
Oct 30, 2024

Conversation

michael-hawker
Copy link
Member

@michael-hawker michael-hawker commented Oct 15, 2024

Fixes #513

Similar fix to #514 along with a bit of extra clean-up to simplify samples.

PR Type

What kind of change does this PR introduce?

Mainly using x:Bind instead of Binding to make sample work with AoT.

What is the current behavior?

Doesn't display information in list.

What is the new behavior?

Information displayed.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • Tested code with current supported SDKs
  • New component
    • Documentation has been added
    • Sample in sample app has been added
    • Analyzers are passing for documentation and samples
    • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
  • Tests for the changes have been added (if applicable)
  • Header has been added to all new source files
  • Contains NO breaking changes

Other information

Tested in Debug/Release

…which was making for odd namespace collision (less confusing/mixed this way)

The last commit was similar to what was needed to fix #514
@michael-hawker michael-hawker added documentation Improvements or additions to documentation enhancement New feature or request sample app 🖼️ regression What was working is now broke labels Oct 15, 2024
Copy link
Member Author

@michael-hawker michael-hawker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding suggestions from @Jay-o-Way

components/Triggers/src/CompareStateTrigger.cs Outdated Show resolved Hide resolved
components/Triggers/src/IsEqualStateTrigger.cs Outdated Show resolved Hide resolved
Copy link
Member

@Arlodotexe Arlodotexe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, looks good now!

image

@Arlodotexe Arlodotexe merged commit 894dc33 into main Oct 30, 2024
24 checks passed
@Arlodotexe Arlodotexe deleted the llama/acv-fix513 branch October 30, 2024 19:32
@Arlodotexe Arlodotexe changed the title Fixes #513 - AdvancedCollectionView not working with AoT Fixes #513 - AdvancedCollectionView not working with NativeAoT Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request regression What was working is now broke sample app 🖼️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AdvancedCollectionView items are empty in gallery sample under Windows App SDK
3 participants