On Tue, 2006-02-14 at 00:02 -0700, Lakin Wecker wrote:
> FIrst off, thanks for the positive feedback. I figured it would be
> too late coming for the current release cycle, but I'm excited to hear
> that it might make it in for the next.
>
> I've addressed all of the issues which Alex raised in his previous
> email, and uploaded the new patch to:
> http://pages.cpsc.ucalgary.ca/~weckerl/nautilus_patch.html
@@ -1224,6 +1226,17 @@
update_extra_location_widgets_visibility (window);
}
+ directory = nautilus_directory_get (window->details->location);
+ if (NAUTILUS_IS_TRASH_DIRECTORY (directory)) {
+ widget = nautilus_location_widget_provider_get_widget
(NAUTILUS_LOCATION_WIDGET_PROVIDER (directory),
+
window->details->location,
+
GTK_WIDGET (window));
+ if (widget != NULL) {
+ nautilus_window_add_extra_location_widget (window,
widget);
+ }
+ update_extra_location_widgets_visibility (window);
+ }
+
This leaks the directory, and isn't what i meant. Something more like:
@@ -1217,6 +1217,10 @@ update_for_new_location (NautilusWindow
} else {
nautilus_window_set_search_mode (window, FALSE, NULL);
}
+ if (NAUTILUS_IS_TRASH_DIRECTORY (directory)) {
+ nautilus_window_add_extra_location_widget (window,
+
nautilus_trash_directory_bar_new ());
+ }
nautilus_directory_unref (directory);
Also, nautilus_trash_directory_bar_get_button() isn't needed anymore.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Alexander Larsson Red Hat, Inc
[EMAIL PROTECTED] [EMAIL PROTECTED]
He's a bookish arachnophobic Green Beret plagued by the memory of his family's
brutal murder. She's a vivacious Buddhist doctor who inherited a spooky
stately manor from her late maiden aunt. They fight crime!
--
nautilus-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/nautilus-list