feat: Switch to Svg.Skia#21380
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR switches the SVG functionality from using the legacy Svg.Custom and Svg.Model packages to the unified Svg.Skia package, updating to version 3.0.6. This consolidates dependencies and removes external shim code that was previously required to bridge different SVG library versions.
- Replaces multiple SVG package dependencies with single
Svg.Skiapackage - Updates SVG package version from 3.0.0/3.0.2 to 3.0.6
- Removes extensive external shim/bridge code that is no longer needed
Reviewed Changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/Uno.Sdk/packages.json | Updates SvgSkia group version to 3.0.6 |
| src/Directory.Build.targets | Replaces Svg.Custom and Svg.Model with single Svg.Skia reference |
| src/AddIns/Uno.UI.Svg/*.csproj | Updates project files to use Svg.Skia instead of separate packages |
| src/AddIns/Uno.UI.Svg/SvgProvider.cs | Removes unused namespace import |
| src/AddIns/Uno.UI.Svg/External/*.cs | Removes extensive external shim code files |
| build/nuget/Uno.WinUI.Svg.nuspec | Updates NuGet spec dependencies to use Svg.Skia |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
405132d to
a4479f5
Compare
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-21380/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21380/docs/index.html |
|
The build 174769 found UI Test snapshots differences: Details
|
a4479f5 to
e0d6e5d
Compare
e0d6e5d to
d6a2d8e
Compare
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-21380/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21380/docs/index.html |
|
The build 174916 found UI Test snapshots differences: Details
|
chore: Update Svg.Skia version to 3.0.6 following merge of unoplatform/uno#21380
GitHub Issue: closes #21378
PR Type: ✨ Feature
What is the current behavior? 🤔
We were previously using only
Svg.ModelandSvg.Custom, but our own copy of the remainingSvg.Skiatypes.What is the new behavior? 🚀
Using
Svg.SkiadirectlyPR Checklist ✅
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Runresults.Other information ℹ️