[Sugar-devel] Patch for Jukebox: New toolbar

[Sebastian Silva]

About these patches for new toolbars that we shared back in march, yes I 
should have sent them to the list.
I did not think we were this formal with activities development, but I 
agree we should be, to avoid duplication of work.
I will go now and dispatch my patches to the list, meanwhile you can see 
what was done at 
http://wiki.sugarlabs.org/go/Activity_Team/TODO#Adding_new_Toolbars

This makes me think we should relive the Activities Team and document 
these workflows if we want to make it approachable to build and mantain 
more activities.

Manu, great to see you active around here.
Un gusto haberlos conocido en Montevideo. Saludos!

Sebastian

El 19/05/11 06:41, Gonzalo Odiard escribió:
> Yes, Manu is working with me.
> About the patchs from Sebastian, in what other activities have you worked?
> Can you send the patches to sugar-devel?
> The merge request does not work, because nobody receive a mail and 
> there are not review.
>
> Gonzalo
>
> 2011/5/19 Rafael Ortiz <rafael at activitycentral.com 
> <mailto:rafael at activitycentral.com>>
>
>     Hi Manu.
>
>     2011/5/18 manuel quiñones <manuel.por.aca at gmail.com
>     <mailto:manuel.por.aca at gmail.com>>
>
>         Hello,
>
>         Here is a patch to use the new toolbar for the Jukebox
>         activity.  A
>         few notes about the changes below:
>
>         You will notice that I renamed the ControlToolbar class to just
>         Control, that's because it doesn't inherit anymore from
>         gtk.Toolbar,
>         now the toolbar is a property of that class, that can be the
>         old one
>         or the new one.
>
>         I moved the fullscreen button to a View subtoolbar for consistency
>         with other activities.
>
>         Regards,
>
>
>     I think that Sebastian Silva (on cc) has a merge request to fix
>     this issue also.
>
>     http://git.sugarlabs.org/jukebox/mainline/merge_requests/2
>      Would be nice if you two work with kushal to have this merged
>     into mainline and packaged into a new release.
>
>     Thanks for your patch, gonzalo has a list of activities that
>     need a new toolbar if you want to work on those :).
>
>     http://dev.laptop.org/query?status=assigned&status=new&status=reopened&summary=~has+old+toolbar&col=id&col=summary&col=status&col=type&col=priority&col=milestone&col=component&order=priority
>     <http://dev.laptop.org/query?status=assigned&status=new&status=reopened&summary=%7Ehas+old+toolbar&col=id&col=summary&col=status&col=type&col=priority&col=milestone&col=component&order=priority>
>
>
>
>         --
>         .. manuq ..
>
>         _______________________________________________
>         Sugar-devel mailing list
>         Sugar-devel at lists.sugarlabs.org
>         <mailto:Sugar-devel at lists.sugarlabs.org>
>         http://lists.sugarlabs.org/listinfo/sugar-devel
>
>
>
>     _______________________________________________
>     Sugar-devel mailing list
>     Sugar-devel at lists.sugarlabs.org
>     <mailto:Sugar-devel at lists.sugarlabs.org>
>     http://lists.sugarlabs.org/listinfo/sugar-devel
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20110519/da3233ba/attachment.html>