On Mon, 2005-02-14 at 00:04 -0700, Elijah Newren wrote:
> As you're probably aware, trying to launch nautilus windows has been
> resulting in windows appearing without focus and below the focused
> window. Further, neither the busy cursor nor an entry in the taskbar
> were being shown while such windows were being launched. All of this
> was due to nautilus not supporting startup-notification as a launchee.
>
> So, I wrote a pair of patches in bug 163046 and bug 166242, which I
> have attached to this email as well. There's probably lots of various
> things to fix up (I haven't written a substantial nautilus patch
> before, I don't grok bonobo, and cutting & pasting always seems to
> cause problems, not to mention the two FIXMEs I put in there...),
> however the patches appear to work and fix the problem in my testing.
> I'd appreciate any feedback and further testing, in hopes that we can
> a cleaned up version of this in for 2.10.
Since this is clearly better than what we currently do, i committed
this. I removed the leak FIXME and freed startup_id in the
NautilusWindow finalizer, and I fixed up the coding style/indentation to
match Nautilus coding style. Also, I removed the g_warning on launching
without a startup id. This was triggering every time you launch nautilus
from a terminal, which I think is a bit harsh. Do other gnome apps do
this too?
Some questions remain:
on existing window presenting you do this:
end_startup_notification (GTK_WIDGET (existing_window),
existing_window->details->startup_id,
startup_id);
Is this really right? I'm not totally sure how startup notification
works, but doesn't this re-trigger the startup_id that was originally
used to start nautilus, instead of the new startup (startup_id) that was
requested?
@@ -345,10 +351,12 @@ restore_one_window_callback (const char
if (eel_strlen (location) > 0) {
window = nautilus_application_present_spatial_window
(shell->details->application,
NULL,
+ NULL, /*
FIXME: Need startup_id? */
location,
screen);
} else {
window = nautilus_application_create_navigation_window
(shell->details->application,
+ NULL,
/* FIXME: Need startup_id? */
screen);
nautilus_window_go_home (window);
}
Does the session launch its processes using startup notification?
--
nautilus-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/nautilus-list
