[sugar] [PATCH] Add RandomLayout and make it the default.

Tomeu Vizoso tomeu at tomeuvizoso.net
Mon Jun 16 12:10:53 EDT 2008


On Mon, Jun 16, 2008 at 4:42 PM, Marco Pesenti Gritti
<mpgritti at gmail.com> wrote:
> On Wed, Jun 11, 2008 at 9:06 PM, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
>> Hi,
>>
>> this patch renames activitiesring.py to favoritesview.py and moves the
>> layout stuff to favoriteslayout.py. Also adds to favoriteslayout.py
>> RandomLayout.
>
> I'm not fully convinced about the FavoritesLayout/FavoritesView
> naming. It's not immediately clear that these are activities. Also it
> might be a bit too specific, since we even considered dropping
> favorites and showing all activities in that view. I can think of
> Activities* or Home*, better ideas?
>
> +            width = gtk.gdk.screen_width()
> +            height = gtk.gdk.screen_height() - style.GRID_CELL_SIZE
>
> +        width = gtk.gdk.screen_width()
> +        height = gtk.gdk.screen_height() - style.GRID_CELL_SIZE
>
> Would probably be better to use size requisition instead of "hardcoding" these.

Ok.

> +            self.fixed_positions[icon] = (relative_x * 1000 / width,
> +                                           relative_y * 1000 / height)
>
> Is 1000 big enough. We will lose precision. Should we just store it as a float?

Ok, will store as float.

> +        hot_spot = style.zoom(10)
> +        context.set_icon_pixbuf(pixbuf, hot_spot, hot_spot)
>
> Seems arbitrary, but I don't have a better idea atm.

Yeah, this part needs to be improved in general.

Thanks,

Tomeu


More information about the Sugar mailing list