toolbar: change ownership of menu popover to the toolbar

Prior changes to merge the view and action menus essentially moved the
action menu items into the view menu. This was the path of least
resistance; the view has a lot of hooks on items in the view menu,
whereas the there are very few hooks on items in the action menu,
meaning the latter could be moved more easily.

However, previously the view menu was disabled for Other Locations and
the action menu wasn't. So the side effect of the changes is the
remaining menu is now disabled completely on Other Locations. There are
a couple of items that could be shown for Other Locations (e.g. New
Tab), so we still want to show a menu, but this involves some
refactoring so has been deferred until now.

This commit is the first part of the refactoring; the files view owns
the menu popover, meaning that the Other Locations view doesn't have
access to it. This commit moves the ownership of the menu popover to the
toolbar. Future commits will move the common items into the popover so
all views will show them.

https://bugzilla.gnome.org/show_bug.cgi?id=764632

1 files changed, 1 insertions, 1 deletions

diff --git a/src/nautilus-places-view.c b/src/nautilus-places-view.c
index f6214f880..ed46f2059 100644
--- a/src/nautilus-places-view.c
+++ b/src/nautilus-places-view.c
@@ -283,7 +283,7 @@ nautilus_places_view_set_search_query (NautilusView  *view,
 static GtkWidget*
 nautilus_places_view_get_view_widget (NautilusView *view)
 {
-        /* By returning NULL, the view menu button turns insensitive */
+        /* By returning NULL, no section will be added to the toolbar menu when this view is active */
         return NULL;
 }