-
Notifications
You must be signed in to change notification settings - Fork 62
IImage.Save (SkiaImage) not implemented #437
Comments
IMO it's not ok to tell devs to migrate from System.Drawing.Common to Microsoft.Maui.Graphics when this lib is clearly not ready for production |
We're just using SkiaSharp directly, as that's the primary rendering engine behind MAUI.Graphics anyway. Our primary UC has us using SkiaSharp in Blazor WASM (with fantastic success), then we'll move onto other platforms. Skia/SkiaSharp supports as many platforms as you would need. If MAUI.Graphics gets its act together and additionally supports Blazor Server (which is required for Blazor Hybrid support) that would give us a compelling reason to move off of SkiaSharp. |
Then what's the purpose of Microsoft.Maui.Graphics, when it's using SkipSharp any way? |
It's an abstraction layer, allowing other rendering implementations to be used when appropriate in the future. It may not make sense for MAUI to require delivery of Google's Skia and MS's SkiaSharp on every platform MAUI wants to support. |
Yes exactly, e.g on windows. GDI+ is fast and solid. It's a mess that there is no GDI+ Implementation yet. (Source code is available but not published as a Nuget) |
Are there plans to publish the GDI package? I'd be more inclined to use and contribute to it if it were published on NuGet with the other packages |
This is a follow-on from #436. Building some code to replace
System.Drawing.Common
image manipulation for a ASP.NET 6 website running on Windows. Tried to take anIImage
over to a byte array via a memory stream as shown here. The exact code doing this work is:The
IImage
was extracted from a byte array usingSkiaImage
as shown in #436. The call to.Save()
ends up with aPlatformNotImplementedException
.The same exception is thrown if the
using
block above is replaced with theAsBytes
extension method:Being that I'm using version 6.0.300 of both the
Microsoft.Maui.Graphics
andMicrosoft.Maui.Graphics.Skia
NuGet packages, I thought MAUI was up to feature complete status. Did this method get overlooked is is Skia not intended to support Windows?The text was updated successfully, but these errors were encountered: