Code review comment for lp:~ted/indicator-applet/messages-launchers

Revision history for this message
Ted Gould (ted) wrote :

On Thu, 2009-08-20 at 14:27 +0000, Ted Gould wrote:
> On Thu, 2009-08-20 at 13:59 +0000, David Barth wrote:
> > - does g_app_info_launch manage notification ids? or do you
> > have to trigger that explicitly with g_app_launch_context_get_startup_notify_id ()?
> > it would be good to add that for the release and cancel
> > the s-n in case of errors
>
> I'm not sure. I'll look into the GNOME Panel code and steal it from
> them to see how they do it :)

Turns out they use libgnome :(

I'm not using the GDK App Context which should do this. Updated the
branch.

« Back to merge proposal