<div dir="ltr">A few more comments:<div><br></div><div>1) Instead of create a new repository with the sources, fork Michael repository,</div><div>and learn how to send pull requests.</div><div>You can read about that here <a href="http://developer.sugarlabs.org/contributing.md.html#forking">http://developer.sugarlabs.org/contributing.md.html#forking</a></div><div>Then, separate your changes in small patches, by example, one patch by tool you add.</div><div><br></div><div>2) Make commits with good titles and explanations, like this one:</div><div><a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/commit/034706a482deef3db5e8a93b232882af3224016c">https://github.com/sugarlabs/sugar-toolkit-gtk3/commit/034706a482deef3db5e8a93b232882af3224016c</a></div><div><br></div><div>3) I agree with Puneet's comments. Would be good consolidate the different tools in a</div><div>tools selector like in the Sugar Paint activity, and improve the color selector palette to </div><div>enable configure the tool size and shape, and select custom colors. </div><div><br></div><div>The activity is having nice improvements, if you and Michael can coordinate</div><div>to work together will be perfect.</div><div><br></div><div>About the icons <a href="https://thenounproject.com/">https://thenounproject.com/</a> have a lot to use.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jul 18, 2015 at 8:00 AM, Puneet Kaur <span dir="ltr"><<a href="mailto:puneet.gkaur@gmail.com" target="_blank">puneet.gkaur@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi, checked it out. It seems nice. Just a few recommendations and tasks for you :<div><br></div><div>1. Make a github io page so that its easy for people to experiment directly in the browser - like the one Micheal has (<a href="http://mikklfr.github.io/Paint.activity/" target="_blank">http://mikklfr.github.io/Paint.activity/</a>)</div><div>2. The functionality added by the sketcher, airbrush and ribbon tools is nice and enriches the paint app with the diversity to play with different types of strokes. </div><div> I would request you to put all of these under the palette of stroke rather than having separate tool buttons like we already one in the native sugar paint app ( do have a look into that) attaching the samples along.</div><div>3. Better svgs for the icons would make it perfect.</div><div><br></div><div><br></div><div>I loved the new functionality. Eager to see more of that :)</div><div><br></div><div><br></div><div>Regards,</div><div>Puneet</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jul 18, 2015 at 12:05 PM, Sneha Mohanty <span dir="ltr"><<a href="mailto:sm21483@gmail.com" target="_blank">sm21483@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div><br clear="all"></div>Hi Michael, Gonzalo , Manu Sir, Puneet, Lionel and everyone else,<br><br></div>I have made three new additions to Michael's Paint Activity <br><br></div>- sketcher<br><br></div>- Ribbon <br><br></div>-Airbrush<br><br></div>I have done those additions on my github repo here - <a href="https://github.com/SnehaMohanty/MOhayon-Sugar-paint-branch" target="_blank">https://github.com/SnehaMohanty/MOhayon-Sugar-paint-branch</a><br><div><div><div><div><div><div><br><br></div><div>Note: The svg icons as of now, are not well suited for each of the airbrush, ribbon and sketcher , as i haven't made the suitable icons as of yet.<br><br><br></div><div>Screenshots from my additions have been attached with this mail.<br><br><br></div><div>Please take a look and give me a feedback.<br><br></div><div>Regards,<br></div><div>Sneha Mohanty<br></div><div><br><br></div><div><br></div></div></div></div></div></div></div> </blockquote></div><br></div> </div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Gonzalo Odiard<br><br><div>SugarLabs - Software [for | by] children learning <br></div></div></div></div></div> </div>