Re: Animated Background

[Alexander Larsson <alexl redhat com>]

On Tue, 2007-11-27 at 05:50 +0100, Soeren Sandmann wrote:
> Soeren Sandmann <sandmann daimi au dk> writes:
> 
> > Here is a patch to Eel that changes EelBackground to use the new
> > GnomeBG class in libgnome-desktop. Along with some changes to
> > control-center that haven't landed yet, this will get us animated
> > backgrounds as discussed on DDL:
> > 
> >    http://mail.gnome.org/archives/desktop-devel-list/2007-September/msg00530.html
> > 
> > and also get rid of a bunch of code that previously was triplicated
> > in eel/control-center/libbackground. For eel, the diffstat is:
> > 
> > dhcp83-49:~/vertigo/eel% svn diff | diffstat
> >  configure.in                    |    2
> >  eel/eel-background.c            | 1521
> >  ++++------------------------------------
> >  eel/eel-background.h            |   89 --
> >  eel/eel-gdk-pixbuf-extensions.c |    2
> >  4 files changed, 196 insertions(+), 1418 deletions(-)
> > 
> > The patch is available here:
> > 
> >    http://www.daimi.au.dk/~sandmann/eel-gnome-bg.patch
> 
> Given that this patch can basically be summarized as "turn
> EelBackground into a thin wrapper around GnomeBG", I'll go ahead and
> commit this soon unless someone wants to review it.

I haven't looked at it recently, but that sounds good. The main reason I
haven't replied is that I'm very busy with the nautilus gio port branch.
But this shouldn't really conflict much when we finally merge that, so
it should be ok to commit to HEAD now.