<div dir="ltr"><div><br></div>Hi Yash<br><div class="gmail_extra"><br><div class="gmail_quote">On 15 May 2016 at 17:53, Dave Crossland <span dir="ltr"><<a href="mailto:dave@lab6.com" target="_blank">dave@lab6.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra">On 28 April 2016 at 23:10, Dave Crossland <span dir="ltr"><<a href="mailto:dave@lab6.com" target="_blank">dave@lab6.com</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div>- Small (or large) patches/bug fixes. (These do not need to be directly related to the GSoC project.)<br></div></blockquote></span><span class=""><div><br></div></span><div>(I haven't seen any of those yet, 0/10)</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div>- Participating in code reviews for others. (Even someone who isn't familiar with the project can contribute by pointing out potential inefficiencies, bad error handling, etc.)</div></blockquote><div><br></div></span><div>(I haven't seen any of those either, 0/10)</div></div></div></div></blockquote><div><br></div><div>I think these are still at 0/10 at the deadline - no worries :)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div>- Working with the mentors and other org members on refining their project plan. This might include finalizing deadlines and milestones, adding more detail, figuring out potential issues, etc.</div></blockquote><div><br></div></span><div>We took a first pass at this when we decided to go with Sugar Desktop instead of Sugarizer, but the milestones are still pretty vague and I can easily see the project slipping... 7/10. To max this out, lets take another pass at the end of this week or next weekend :) </div></div></div></div></blockquote><div><br></div><div>You took another pass at this - <a href="https://sugarlabs.github.io/edit-fonts-activity/latest-proposal">https://sugarlabs.github.io/edit-fonts-activity/latest-proposal</a> - and I made some comments - <a href="https://github.com/sugarlabs/edit-fonts-activity/commit/ac6903ed79d9d5bce99d98c6bde63f10fef15a93">https://github.com/sugarlabs/edit-fonts-activity/commit/ac6903ed79d9d5bce99d98c6bde63f10fef15a93</a> - but its not maxed out, and there are some formatting issues, so I think this is 8/10 by the deadline today. To make it 10/10, please follow up on my comments and review the rendering on the site (or your locally served copy) </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div>- Reading (and updating!) documentation they will need to understand to complete their project.</div></blockquote><div><br></div></span><div>You've been doing a lot of reading on the resources I posted, but I'm not totally sure what you read, and if you found any other resources beyond what I listed on the blog to read... 6/10. To totally smash it, I think it would be great if you could go over your browser history and make a blog post noting which resources you read and giving us a few tips on what was most valuable about each one and what you think could be improved about them too :) </div></div></div></div></blockquote><div><br></div><div>Your blog post at <a href="https://sugarlabs.github.io/edit-fonts-activity/my-progress-so-far">https://sugarlabs.github.io/edit-fonts-activity/my-progress-so-far</a> clarified what you've been reading and its a great list :) So I now rate this 9/10, and to make it 10/10 I would have liked to have seen more reflection on which links were the best, which were helpful, and which were interesting but not that relevant.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> - Reporting or replicating bugs.</div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br>I haven't seen any of those either 0/10. On IRC today we discussed a bug with the existing font manager code, so I think it would be great if you could write a issue report in <a href="https://github.com/sugarlabs/edit-fonts-activity/issues" target="_blank">https://github.com/sugarlabs/edit-fonts-activity/issues</a> and then make a pull request to address it this week :) </div></div>
</blockquote></div><br><a href="https://github.com/sugarlabs/edit-fonts-activity/issues/16">https://github.com/sugarlabs/edit-fonts-activity/issues/16</a> is a great bug report - I love the GIF screencast :D But without a PR, its gotta be a 8/10 - to get to 10/10 I'd like to see a PR starting to address this, and that would also bump up the "small patch" item above :)<br clear="all"><div><br></div><div>Overall I'm very happy with your progress in the community bonding stage, and looking forwards to your progress coding this week! Your plan has as the first goal:</div><div><br></div></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div class="gmail_extra"><div><div>1.0 Activate/Deactivate: Build an interactive grid display for showing all fonts in the system, a button to activate/deactivate them, and allow the user to create ‘collections’ within the library (like playlists in a music player)</div></div></div></blockquote><div class="gmail_extra"><div><div><br></div></div><div>And this is indeed the feature with the bug you reported :) Looking forwards to your next blog post and Pull Request about it! :D</div><div><br></div>-- <br><div class="gmail_signature">Cheers<br>Dave</div>
</div></div>