<div dir="ltr">Hi Ale and All,<div><br></div><div>First I want to apologize that I could not take a look at some PR's recently for new styles, but Anita and Ale are doing great job giving some time to look at them. So thanks to both!</div><div><br></div><div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">Unfortunately the implementation is rather different.<br>But of course I suggested Akbar to try to re-use any existing code from other parts of QGIS and I'm pretty sure he did</blockquote><div>Yes, the implementation is quite different . IIRC I tried to put plugin as a resource and it worked fine after creating its handler. But it was just a 'dirty curious' work. I need to take some time to think thoroughly the best way to have them both in one big architecture. </div></div><div><br></div><div>I agree with Nyall too on both points. As I mentioned in the final report, there are some room for improvements, one of them is to make a good set of unittests (I am ashamed of myself as some people called me as the unittest guy and yet I abandoned it on this plugin :)). And it's also not replacing Style Manager.</div><div><br></div><div>Since I couldn't really give some love for this plugin right now (probably until this summer), if anyone wants to maintain it, please let me know and I will register you as collaborator. Ale and Anita are already contributors too to the repository. And if there is a chance for me to work on this again for GSoC 2017, I am eager to apply once more. But this is something that I need to talk to my supervisor and depending on this semester result. Or whether I need to take 'official' internship. So maybe in February I will know exactly what to do for this summer. But for sure, I will keep my eyes on this. So keep discussing :)</div><div><br></div><div>Cheers </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 6, 2017 at 10:48 PM, Nyall Dawson <span dir="ltr"><<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 7 January 2017 at 02:05, Alessandro Pasotti <<a href="mailto:apasotti@gmail.com">apasotti@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I think the original idea for the plugin was eventually having it into core<br>
> as a Python core plugin.<br>
><br>
> Any objections if we start the process to make it a core plugin for 3.x ?<br>
><br>
> It will require some testing and minor tweaks of course.<br>
<br>
</span>Obviously, it'll also need porting to PyQGIS 3 before it can be<br>
merged, and I'd want to make sure it's got sufficient unit tests in<br>
place to not add extra burden while the QGIS 3.x API<br>
breaks/refactorings are being performed.<br>
<span class=""><br>
> IMHO if it will be merged into core, it should replace style manager,<br>
> Processing sharing code and Plugin Manager.<br>
<br>
</span>I don't think this replaces style manager though. Style manager has<br>
also undergone significant work in 3.0 and is now really polished and<br>
smooth, so I don't want to see it go. I'd rather python hooks were<br>
added to style manager to allow the resource sharing plugin to<br>
integrate into the existing style manager dialog instead.<br>
<span class="HOEnZb"><font color="#888888"><br>
Nyall<br>
</font></span><span class="im HOEnZb"><br>
<br>
><br>
><br>
> --<br>
> Alessandro Pasotti<br>
> w3:   <a href="http://www.itopen.it" rel="noreferrer" target="_blank">www.itopen.it</a><br>
><br>
</span><div class="HOEnZb"><div class="h5">> ______________________________<wbr>_________________<br>
> Qgis-developer mailing list<br>
> <a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
> List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
> Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
______________________________<wbr>_________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><p><b><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">-------------------</span></b></p><b>Akbar Gumbira </b></div><div><b style="font-size:12.8px"><a href="http://www.akbargumbira.com" target="_blank">www.akbargumbira.com</a></b></div></div></div></div></div>
</div>