[Sugar-devel] [PATCH] New Toolbar for Physics Activity

[Gary Martin]

Hi Akash,

On 25 Jun 2010, at 07:46, Akash Gangil <akashg1611 at gmail.com> wrote:

> On Fri, Jun 25, 2010 at 8:49 AM, Gary Martin <garycmartin at googlemail.com> wrote:
> Hi Akash,
> 
> On 24 Jun 2010, at 21:25, Akash Gangil wrote:
> 
> > Hi Gary,
> >
> >> Make your changes, commit, push, and then request a merge?
> >
> > I have filed the merge request.Please look into it. Also I made it backward compatible to previous sugar versions as you suggested. It would fall back to older toolbar design in case of previous sugar versions.
> 
> I've just tested under Sugar 0.88, 0.84, and 0.82.
> 
> Hey, is there any way I too can test my work on all sugar versions? Currently I use jhbuild on ubuntu lucid, which I think runs 0.88 version of sugar in 

For ease of development I usually use a current Sugar build (0.88) running under Fedora in VirtualBox on a Mac. I'm using F13 at the moment with the Sugar Desktop package made by the SoaS folks (jhbuild broke for me under F11 and I didn't have time to diagnose why - seemed to need a newer version of GTK+ than was shipping in F11).    

For older Sugar versions I have two XO-1's and one XO-B4. It might be possible to run some old builds in VirtualBox for testing, but I've not tried that route myself (I do like to test on XO-1 hardware as that is what 99% of our users are using).

Regards,
--Gary 

> I am going to merge your patch tomorrow, but will also make a couple changes of my own before making the next official release:
> 
> 1) Testing with 0.84 and 0.82 displays multiple create tabs running off the right edge of the screen, the first create tab contains the correct tools, all the others are empty.
> 
> 
> 
> 
> 2) I like your new create icon (shown below), however I'm going to try and place all the tool buttons up in the primary toolbar so we make maximum use of that available space. The Sugar HIG allows for up to 16 icons, so we should be fine putting all the current tools there (and still have room for one or two more new tool button features on my todo list).
> 
> 
> 
> 
> Thanks again for your effort! Hopefully I'll be able to make a new release over this weekend :)
> 
> Kind Regards,
> --Gary
> 
> >> Having a quick scan through the below patch I don't see any fallback to the old toolbar setup if someone is running an older version of Sugar. I don't want to loose compatibility 99% of our current user base if at all possible (check the git reps of Moon, Calculator, TurtleArt, and I'm sure many other activities for dual toolbar support).
> >>
> >> Thanks again for taking the time to submit a patch.
> >>
> >> Kind Regards,
> >> --Gary
> 
> 
> 
> 
> 
> -- 
> Best Regards
> Akash Gangil
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.sugarlabs.org/archive/sugar-devel/attachments/20100625/3e9d3714/attachment.htm