[Sugar-devel] [PATCH] New Toolbar for Physics Activity
akashg1611 at gmail.com
Fri Jun 25 02:46:39 EDT 2010
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 Xephyr.
> 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,
> >> 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
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Sugar-devel