Comment 2 for bug 1377566

Revision history for this message
Robert Nordan (rpvn) wrote :

HI, it's great to get new contributions! Before we can have a closer look at it, it would be nice if you could prepare a GtiHub pull request as described in this document: https://github.com/PintaProject/Pinta/blob/master/patch-guidelines.md . That will make it a lot easier for us to review the code.

But even without looking at it, I'm thinking that sounds like a pretty substantial rewrite of the ora provider, can we be sure that we don't introduce any issues with ora files, especially backwards compatability with existing Pinta-made ora? (Also, having to rename to odg files afterwards is not ideal.) Because of that I'm a bit sceptical to merging this into the Pinta core, but have no fear, we have an even better solution! This looks like an ideal candidate for a file format add-in!

See this guide: https://github.com/PintaProject/Pinta/wiki/Writing-a-File-Format-Addin

I imagine you probably wouldn't have to do a lot of work to package this as a new addin, and then you could support .odg files directly, as well as add in the metadata dialogue you were talking about. We'd love to see more addins now that we have a functional addin infrastructure!