feat(Skia): Implement PlaneProjection and Matrix3DProjection#22449
Merged
MartinZikmund merged 18 commits intomasterfrom Feb 24, 2026
Merged
feat(Skia): Implement PlaneProjection and Matrix3DProjection#22449MartinZikmund merged 18 commits intomasterfrom
PlaneProjection and Matrix3DProjection#22449MartinZikmund merged 18 commits intomasterfrom
Conversation
github-actions
bot
added
area/skia ✏️
Contributor
There was a problem hiding this comment.
Pull request overview
This PR implements 3D perspective projection support for Skia targets by adding PlaneProjection and Matrix3DProjection classes. These features allow UIElements to be transformed in 3D space with perspective effects, addressing a long-standing feature request from issue #181.
Changes:
- Implements
Projectionbase class and two concrete implementations:PlaneProjection(for common 3D rotations and offsets) andMatrix3DProjection(for custom 4x4 matrices) - Adds
Projectiondependency property toUIElementfor Skia targets - Implements complete
Matrix3Dstruct with operators, equality, and conversion methods - Integrates projection transforms into the rendering pipeline via
NativeRenderTransformAdapter - Includes comprehensive runtime tests and interactive sample applications
Reviewed changes
Copilot reviewed 16 out of 20 changed files in this pull request and generated 17 comments.
Show a summary per file
| File | Description |
|---|---|
| Projection.cs | Base class for all projection types with owner tracking and change notifications |
| PlaneProjection.cs | Implements perspective transforms with rotation, center of rotation, and offset properties |
| Matrix3DProjection.cs | Allows custom 4x4 transformation matrices |
| Matrix3D.cs | Complete 4x4 matrix struct implementation with multiplication, inversion, and conversion |
| UIElement.cs | Adds Projection property for Skia with event handling and transform updates |
| UIElement.skia.cs | Updates layout arrangement to handle projection calculations |
| NativeRenderTransformAdapter.skia.cs | Integrates projection matrices into the render transform pipeline |
| Generated files | Updates conditional compilation to exclude Skia from NotImplemented attributes |
| Given_PlaneProjection.cs | Comprehensive unit tests for PlaneProjection functionality |
| Given_Matrix3DProjection.cs | Unit tests for Matrix3DProjection |
| Given_Matrix3D.cs | Tests for Matrix3D struct operations |
| PlaneProjection_Basic.xaml | Interactive sample demonstrating all PlaneProjection features |
| Matrix3DProjection_Basic.xaml | Sample showing custom matrix transformations |
| CompositionColorBrush.skia.cs | Enables anti-aliasing (appears unrelated to PR) |
| UITests.Shared.projitems | Adds new sample files to project |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/SamplesApp/UITests.Shared/Windows_UI_Xaml_Media/Projection/PlaneProjection_Basic.xaml
Show resolved
Hide resolved
src/Uno.UI/Generated/3.0.0.0/Microsoft.UI.Xaml.Media/Matrix3DProjection.cs
Show resolved
Hide resolved
src/Uno.UI.RuntimeTests/Tests/Windows_UI_Xaml_Media/Given_PlaneProjection.cs
Outdated
Show resolved
Hide resolved
Contributor
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-22449/docs/index.html |
Contributor
|
The build 192769 found UI Test snapshots differences: Details
|
Contributor
|
The build 192769 found UI Test snapshots differences: Details
|
Contributor
|
The build 192769 found UI Test snapshots differences: Details
|
MartinZikmund
force-pushed
the
dev/mazi/planeprojection
branch
from
9208616 to
5f29a9c
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 16 out of 20 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/Uno.UI.RuntimeTests/Tests/Windows_UI_Xaml_Media/Given_Matrix3D.cs
Outdated
Show resolved
Hide resolved
Contributor
|
The build 195090 found UI Test snapshots differences: Details
|
Contributor
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-22449/docs/index.html |
Contributor
|
The build 195090 found UI Test snapshots differences: Details
|
Contributor
|
The build 195090 found UI Test snapshots differences: Details
|
MartinZikmund
force-pushed
the
dev/mazi/planeprojection
branch
from
ee79dc4 to
cc88eb0
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 16 out of 20 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
The build 196837 found UI Test snapshots differences: Details
|
Contributor
|
The build 196837 found UI Test snapshots differences: Details
|
Contributor
|
The build 196837 found UI Test snapshots differences: Details
|
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 16 out of 20 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22449/wasm-skia-net9/index.html |
Contributor
|
The build 196937 found UI Test snapshots differences: Details
|
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 21 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/Uno.UI.RuntimeTests/Tests/Windows_UI_Xaml_Media/Given_Matrix3DProjection.cs
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 21 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 21 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22449/wasm-skia-net9/index.html |
Xiaoy312
approved these changes
Feb 23, 2026
src/SamplesApp/UITests.Shared/Windows_UI_Xaml_Media/Projection/Matrix3DProjection_Basic.xaml.cs
Show resolved
Hide resolved
Contributor
|
The build 197819 found UI Test snapshots differences: Details
|
Contributor
|
The build 197819 found UI Test snapshots differences: Details
|
Co-authored-by: Xiaotian Gu <[email protected]>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 21 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 21 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22449/wasm-skia-net9/index.html |
Contributor
|
The build 197899 found UI Test snapshots differences: Details
|
Contributor
|
The build 197899 found UI Test snapshots differences: Details
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
GitHub Issue: closes #181
PR Type: ✨ Feature
What is the current behavior? 🤔
Not implemented
What is the new behavior? 🚀
Implemented
PR Checklist ✅
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Runresults.