huge PR - skiasharp support

[gentledepp]

Hi guys!

First of all thank you for this svg library - it is just great!

A while ago, I forked this library with the goal to get it to run on android.
Meanwhile, we are using SkiaSharp for rendering.
This allows us to render on Android, iOS, Mac, Windows (and UWP) so basically everything except Linux.
Also, Skia supports hardware-rendering (OpenGL and Angle to translate the OpenGL calls to DirectX).

However, we did not just rip all the "System.Drawing" stuff out and replaced it, but refactored it into a factory with our own interfaces, so that you can (literally) stuff in any other render library you like.

My question is: Are you basically interested in merging this in?
As we changed a lot (e.g. removed all TypeDescriptor stuff as this is not supported on UWP) so merging will be quite tedious.
On the other hand, I see a lot of benefits for you guys too.

In that case, please reach out to me.
Our repository is: https://github.com/gentledepp/SVG
The latest branch with plattform support is https://github.com/gentledepp/SVG/tree/uwp_ios

Cheers!

[tebjan]

hello, this sounds totally interesting and actually we are also planning to use SkiaSharp internally in other projects and have looked into it quite recently. so that would fit quite well.

i'll discuss it with my colleagues and get back to you.

[gentledepp]

Great - the build is actually already working and our tests run on ios, android, uwp and windows.
Maybe we could arrange a skype chat sometime. (I am based in Austria, ... also sollten wir kein Zeitzonenproblem haben ;-))

[tebjan]

we had a short chat about the merge and came to the conclusion that it would be better for us to schedule the merge bit further. even as far as july/august since we are preparing for a big event.
the merge could take a week or more of work. also a GDI implementation or at least forms adapter is needed. so it would be good to do it after our event.
and having a skype chat to plan and discuss the merge would be great!

[gentledepp]

sure - I am not in a hurry at all.
My skype nickname is the same as my github name.
Feel free to drop me a line ;-)

[tebjan]

hello @gentledepp how is the status of the fork on your end? do you still maintain it?
maybe its more easy to release a final version of this master branch and then completely switch to yours. some PRs might get lost, but overall your branch looks like the way to go.

[gentledepp]

Hi!
Yes, it is still maintained.
The library is quite stable and we use it in our app in production for some time now. Everything is working nicely.
Please just look at the skiasharp branch!

There are some things on our list, but I don't know when we'll find the time for them:

1. We still do not support all SVG features, just as you do.
   Additionally, I have not found time yet to split the failing unit tests into "failing because of an unknown error" and "failing, and we know this feature is not yet supported". I had the idea to add like a "blacklist" or "white list" of test names, but never found the time for it. Maybe you would like to look into it?

2. At the time being, the port builds on top of a PCL core library - so no .NET standard yet.
   As we have other things to do, there are no plans for porting to .NET standard at the moment. Also, we are using the bait and switch trick with PCL so we can use features in that library that are actually not supported. I have no idea how that trick can be translated to .NET standard yet. However, it might be as simple after all.

3. As you may have much more insights on this topic than we (because, after all, you originally ported it to Github, right), you may want to become a contributor?

[tebjan]

great, thanks for the update!

[gentledepp]

What are your plans now?
We are happy to help lifting this codebase to a new level 👍

[mrbean-bremen]

@tebjan - can you elaborate about your plans? If you plan to switch to / integrate with the skiasharp branch, this may have change the priorities of the other issues, or make them obsolete.