[sugar] [PATCH] screenshots hurt

[Erik Garrison]

On Fri, Sep 05, 2008 at 08:11:53PM +0200, Marco Pesenti Gritti wrote:
> On Fri, Sep 5, 2008 at 7:29 PM, Erik Garrison <erik at laptop.org> wrote:
> > I'm somewhat bothered by the completely black screenshot but see no
> > immediate way to alleviate it.
> >
> > Another issue with the current patchset is that if the activity is
> > closed by a signal from the window manager (e.g. when the user selects
> > the 'shutdown' option from the system palette in the home view) then the
> > screenshots are not taken, as activity.Activity.close() is not hit.
> 
> Are you sure? wm close should cause a delete-event, which is implemented as:
> 
>     def __delete_event_cb(self, widget, event):
>         self.close()
>         return True
> 

Ok.  Thanks for this information.  When is this callback hit?:
http://dev.laptop.org/git?p=sugar-toolkit;a=blob;f=src/sugar/activity/activity.py;h=73eeea7499dcd4dcd28cea5c55648b8f414654e9;hb=HEAD#l487

Erik