[sugar] frame redesign

Marco Pesenti Gritti mpgritti
Fri Feb 29 20:48:36 EST 2008

On Fri, Feb 29, 2008 at 4:59 AM, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
> Hi Marco,
>  have started moving the items in the frame around as Eben defined.
>  Can you give it a look and tell me what you think?
>  http://dev.laptop.org/git?p=users/tomeu/sugar;a=summary
>  First commit is "ClipboardBox to ClipboardTray. Make it a VTray
>  instead of hippo.CanvasBox.".


here are some notes which I put together this morning on the train.
Traveling tomorrow, it will probably take a couple of days for me to
follow back, be patient :)


+        return True;

Patch looks fine. Please fix the ; above while you are changing that code.




Talked with Eben. The zoom buttons should actually stay in a toolbar. The
activity icons should be in a separate Tray, on the right of the zoom toolbar.
That way if activities scroll, the zoom buttons stays in screen.


+        if new_level == ShellModel.ZOOM_MESH:
+            self._mesh_button.props.active = True
+        elif new_level == ShellModel.ZOOM_FRIENDS:
+            self._groups_button.props.active = True
+        elif new_level == ShellModel.ZOOM_HOME:
+            self._home_button.props.active = True
+        elif new_level == ShellModel.ZOOM_ACTIVITY:
+            self._activity_button.props.active = True

I think we should start using the new style gobject properties in new code.

There will be conflicts with the Tray vs Toolbar thing in the previous one.


What problems do you have there? I'll just note that sometimes it's easier
to use toggle buttons then radio, not sure if this is the case.


        activitiestray.py               \
+       activitiestray2.py              \

I suppose the 2 is temporary. Any reason to do this? The rename will mess up
the history. Hmm, I guess I'm confused by this one, You are modifying both
versions of the tray and I'm not sure which one we are going to use...


+        if home_activity.props.launching:
+            palette = Palette(_('Starting...'))
+            palette.props.invoker = FrameWidgetInvoker(self)
+            palette.set_group_id('frame')
+            self.set_palette(palette)
+            #self._start_pulsing()
+            home_activity.connect('notify::launching',
+        else:
+            self._setup_palette()

Maybe it would be cleaner to have the launching case inside _setup_palette too?


-        self.props.icon_name = name
+        self.get_icon().props.icon_name = name

Can we fix the toolkit and make this:

self.icon.props.icon_name = name

(should get even more convenient if we start using the new style gobject props).

+        view = deviceview.create(device)
+        self.pack_end(view, expand=False, fill=False, padding=0)

I'd rather pack_end the container then each widget, if that works as well.

+        """
         activities_tray = activitiestray2.ActivitiesTray(self._shell)
+        """

Please remove this.


More information about the Sugar-devel mailing list