<br><br>On Sat, Jul 2, 2016 at 6:35 PM, Yash Agarwal <agrwal.ysh94@gmail.com> wrote:<br>
<blockquote type="cite"><p dir="ltr">Work report <br>
I worked on the following issues:<br>
Adding support for ufoz format <br>
Adding git submodules<br>
The editor has been integrated with the activity </p>
<p dir="ltr">I'll begin adding the features for manipulating the bezier curves over the weekend <br>
I'll make the complete week report on Sunday </p>
<p dir="ltr">A question to ponder upon:<br>
Should we make 2 separate activities <br>
A font manager <br>
A font editor<br>
The manager will have root access and be integrated into sugars glucose module (correct me if I'm wrong) so that it's there with the pre installed activities<br>
The editor will be separate and will only deal in ufoz /ufo format </p></blockquote><div>Hi Yash,</div><div><br></div><div>Why do we need root to install fonts?  Can't you just manage the fonts in "~/.fonts" and call the rest of the fonts system fonts.  We really shouldn't make it easy for users to delete system fonts - it will annoy the package manager and it will probably break applications.  We don't want users thinking "I don't use Deja Vu Sans" and then having a very odd looking Sugar because they uninstalled the font we use in the UI.</div><div><br></div><div>Maybe it would be best to add a control panel for fonts?  Activities must have state, and write something to the journal.  Control panels don't have to write a state to the journal.  Control panels are "extensions" in sugar terms, meaning they are loaded from either "/usr/share/sugar/extensions/cpsection/" or "~/.sugar/$SUGAR_PROFILE/extensions".  This means that you could include the font control panel extension with your app, and then install it on your app's first run.</div><div><br></div><div>Thanks,</div><div>Sam</div><br><blockquote type="cite">
<br><div class="gmail_quote"><div dir="ltr">On Thu, 30 Jun 2016 21:28 Dave Crossland, <<a href="mailto:dave@lab6.com">dave@lab6.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Great stuff Yash!<br>
<br>
The blog needs a bit of attention...<br>
<a href="https://github.com/sugarlabs/edit-fonts-activity/commit/cbdfaf2a2c5320ac417124ebd050087588cd4e6c" rel="noreferrer" target="_blank">https://github.com/sugarlabs/edit-fonts-activity/commit/cbdfaf2a2c5320ac417124ebd050087588cd4e6c</a><br>
is posted 10 days ago, but it wasnt on a friday, which is when we have<br>
to check in with walter, so I renamed it to<br>
<a href="https://sugarlabs.github.io/edit-fonts-activity/bezier-editing-works" rel="noreferrer" target="_blank">https://sugarlabs.github.io/edit-fonts-activity/bezier-editing-works</a><br>
<br>
However, you've then duplicated this file and appended to it -<br>
<a href="https://sugarlabs.github.io/edit-fonts-activity/week-5-work" rel="noreferrer" target="_blank">https://sugarlabs.github.io/edit-fonts-activity/week-5-work</a> - so that<br>
its the same at the top<br>
<br>
Please remove duplicate content :)<br>
<br>
Also <a href="https://github.com/sugarlabs/edit-fonts-activity/blob/gh-pages/_posts/2016-06-28-week-5-work_wip.md" rel="noreferrer" target="_blank">https://github.com/sugarlabs/edit-fonts-activity/blob/gh-pages/_posts/2016-06-28-week-5-work_wip.md</a><br>
seems to be not needed, so I made a PR to remove it,<br>
<a href="https://github.com/sugarlabs/edit-fonts-activity/pull/47" rel="noreferrer" target="_blank">https://github.com/sugarlabs/edit-fonts-activity/pull/47</a><br>
<br>
Since (a) you can set a post to not be published if you want to draft<br>
it on gh-pages, and (b) you can draft it in a git branch and only<br>
merge it when its ready, I suggest not creating such WIP files in<br>
future :) I have been trying to make blog posts with the  (b) process<br>
but (a) is also fine, as you did in<br>
<a href="https://github.com/sugarlabs/edit-fonts-activity/commit/6507a9efbb266d31deadd3d4fccf5f190e6f8af0" rel="noreferrer" target="_blank">https://github.com/sugarlabs/edit-fonts-activity/commit/6507a9efbb266d31deadd3d4fccf5f190e6f8af0</a><br>
:)<br>
<br>
Secondly, the filename of a Jekyll post must be unique after the date,<br>
so I made a PR to rename the latest features checklist,<br>
<a href="https://github.com/sugarlabs/edit-fonts-activity/pull/48" rel="noreferrer" target="_blank">https://github.com/sugarlabs/edit-fonts-activity/pull/48</a><br>
<br>
Please do review and merge these when you can :)<br>
</blockquote></div>
</blockquote>