-
Notifications
You must be signed in to change notification settings - Fork 757
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
feat: Skia lifecycle changes #12597
feat: Skia lifecycle changes #12597
Conversation
777c7d6
to
9e5edfb
Compare
|
98da272
to
ab5e943
Compare
ab5e943
to
8492f3a
Compare
src/Uno.UI.Runtime.Skia.Gtk/Extensions/GtkApplicationViewExtension.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.Runtime.Skia.Gtk/Extensions/GtkApplicationViewExtension.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.Runtime.Skia.Gtk/Extensions/GtkApplicationViewExtension.cs
Outdated
Show resolved
Hide resolved
src/Uno.UI.Runtime.Skia.Gtk/Extensions/GtkDisplayInformationExtension.cs
Outdated
Show resolved
Hide resolved
The build 82988 found UI Test snapshots differences: Details
|
96a7a2b
to
3d192bd
Compare
The build 83046 found UI Test snapshots differences: Details
|
If I understand correctly this will allow us to create multiwindow on skia heads? |
@FrozDark Not yet, but the follow up PR might 🙃 |
GitHub Issue (If applicable): closes #
Replaces #12354
PR Type
What kind of change does this PR introduce?
What is the current behavior?
What is the new behavior?
Copilot Summary
🤖 Generated by Copilot at 777c7d6
This pull request improves the support for the Skia backend on GTK and WPF platforms, by refactoring and simplifying some classes, using the new
GtkHost.Current.MainWindow
property, adding a newGtkApplicationViewExtension
class, and registering the GTK-specific extensions. It also adapts theSamplesApp.Skia.WPF
project to use the Skia host and removes some unused files.PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
Internal Issue (If applicable):