-
Notifications
You must be signed in to change notification settings - Fork 123
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
Updated MAUI bindings to use new Compiled Bindings introduced in .NET 9 #630
base: v.next
Are you sure you want to change the base?
Conversation
prathameshnarkhede
commented
Mar 21, 2025
- This pull request updates the MAUI bindings in Toolkit to leverage the new Compiled Bindings feature introduced in .NET 9.
- The changes include replacing existing bindings with compiled bindings for improved performance and type safety.
@@ -152,7 +152,7 @@ private static void Attachment_Tapped(object? sender, EventArgs e) | |||
return parent as PopupViewer; | |||
} | |||
|
|||
private class AttachmentViewModel : System.ComponentModel.INotifyPropertyChanged | |||
internal class AttachmentViewModel : System.ComponentModel.INotifyPropertyChanged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to make this internally accessible to make use of compiled binding.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a partial review up until overviewmap, since it appears there are general issues with this PR that needs to be addressed and tested.
Removed CodeAnalysis Dynamic Dependencies
This PR is ready with corrected bindings. I made sure that bindings breakpoints are hit with correct datacontext. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done! I am glad to see so many DynamicDependencies gone.