-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Prism.Avalonia Library #3127
base: master
Are you sure you want to change the base?
Prism.Avalonia Library #3127
Conversation
Thanks for the PR. I'll try to review this as soon as I can. |
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.
noticing this is largely just reusing what currently exists in the Prism.Avalonia project. This should start removing duplicated code, similar to what we've done with Prism for Uno.WinUI using linked code from the Prism.Wpf project where possible. The goal should be to have a minimally different API surface and eliminate duplicate code as much as possible.
PrismLibrary_Avalonia.slnf
Outdated
"src\\Containers\\Prism.DryIoc.Shared\\Prism.DryIoc.Shared.shproj", | ||
"src\\Avalonia\\Prism.Avalonia\\Prism.Avalonia.csproj", | ||
"src\\Avalonia\\Prism.DryIoc.Avalonia\\Prism.DryIoc.Avalonia.csproj", | ||
"tests\\Avalonia\\Prism.Avalonia.Tests\\Prism.Avalonia.Tests.csproj", | ||
"tests\\Avalonia\\Prism.DryIoc.Avalonia.Tests\\Prism.DryIoc.Avalonia.Tests.csproj" |
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 would need to be updated
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.
Thank you Dan for the quick response on this one. I was curious about the best strategy to move forward with regarding that. In the first round, I was planning on getting a solid build running and then minimizing if that's okay. The current import is based on tag, v9.0.401-pre
.
An example of super-minor differences is WPF's DependencyObject
vs. Avalonia's AvaloniaObject
.
I'll take a look into the Uno.WinUI implementation, thank you for the tip. Currently, I'm working on the importation of the tests
btw be sure to update the CI Builds. It should be pretty much copy/paste and change the file names... if you need some help on it let me know.
|
Will do. @dansiegel, when you're free, mind if we sync up on Teams? |
It'll have to be next week but shoot me an email with some times that work for you and we can set something up |
Sounds great. Thanks again, Dan. |
You'll want to:
<When Condition="$(IsAvaloniaProject)">
<PropertyGroup>
<DefineConstants>$(DefineConstants);AVALONIA</DefineConstants>
</PropertyGroup>
<ItemGroup>
<Using Include="Avalonia" />
<Using Include="Avalonia.Controls" />
<Using Include="Avalonia.Controls.ApplicationLifetimes" />
<Using Include="Avalonia.Controls.Primitives" />
<Using Include="Avalonia.Interactivity" />
<Using Include="Avalonia.Markup.Xaml"/>
<Using Include="Avalonia.Metadata" />
<Using Include="Avalonia.Threading" />
<Using Include="Avalonia.Styling" />
<Using Include="Avalonia.VisualTree" />
</ItemGroup>
</When>
|
Dan, Thanks for the heads up and suggestions to get this puppy rolling. I'll try to get to it this week. I'm going to upgrade the other repository first to get the GA release of Prism.Avalonia v9.0.537, followed by copying over results to our combined branch. Overall, this should be pretty quick process since the delta between |
@dansiegel thanks again for the sync-up yesterday! Looking forward to the 9.1 release |
…ng` statments to each class
Base sample Prism.Avalonia app
Description of Change
Adds fixture for Prism.Avalonia using Avalonia v11.1.x. Pairing together these two awesome libraries under one house.
API Changes
None
Behavioral Changes
Allows users to utilize the cross-platform framework Avalonia for desktop applications.
PR Checklist