Am Dienstag, den 17.05.2005, 12:54 +0200 schrieb Alexander Larsson: > On Tue, 2005-05-17 at 12:45 +0200, Christian Neumair wrote: > > Am Dienstag, den 17.05.2005, 12:30 +0200 schrieb Alexander Larsson: > > > On Tue, 2005-05-17 at 11:06 +0200, Christian Neumair wrote: > > > > Am Montag, den 16.05.2005, 15:49 +0200 schrieb Alexander Larsson: > > > > > On Fri, 2005-04-29 at 22:06 +0200, Christian Neumair wrote: > > > > > > This is the second version of a patch I've submitted about 2.5 > > > > > > months > > > > > > ago to nautilus-list [1]. There were no direct followups, but I was > > > > > > now > > > > > > asked to update the patch, though, so it looks like people are > > > > > > interested in it. > > > > > > The new version only has the Cut, Copy, Paste, Trash + Delete and > > > > > > Properties items in the context menu, but - in contrast to the last > > > > > > version - doesn't make nautilus spit out g_warnings. Also, I've > > > > > > added a > > > > > > context menu to the navigation view "Location:" label. > > > > > > Comments, suggestions? > > > > > > > > > > > > [1] > > > > > > http://mail.gnome.org/archives/nautilus-list/2005-February/msg00040.html > > > > > > > > > > I like this idea. Is there a later version of this patch fixing the > > > > > various bugs pointed out somewhere? > > > > > > > > Here comes a new version. It has grown somewhat large. It adds some > > > > context menu code to the NautilusView interface so that other views can > > > > react appropriately as well and ports the directory view-related code to > > > > that semantics. Also note that I've added a new item on top of the > > > > context menu for opening the currently open folder in a new nav window. > > > > Are you OK with the popup menu code being potentially async or should we > > > > force to return a gboolean? Currently, we just return "FALSE" for the > > > > button_press_event handlers, since we can't know whether the popup > > > > succeeded, but who cares? :) > > > > I've adapted to the old > > > > fm_directory_view_pop_up_background/selection_context_menu semantics > > > > here. > > > > > > I don't see why you've added a nautilus_view_popup_menu(). This > > > hardcodes a set of popup menus that may not fit for all types of views, > > > and is not needed by the general window code. Having a specific popup > > > function for the location context button seems better, and keeping the > > > specific ones in FMDirectoryView (although at some point FMDirectoryView > > > needs to be split in two so that parts of it can be used even for views > > > that show multiple directories). > > > > How should the location button know that the current view is an > > FMDirectoryView? On IRC, I was told that NautilusView is an additional > > layer of abstraction and that I should never ever do things like if > > (FM_IS_DIRECTORY_VIEW (view)) in NautilusWindows. > > ...or should we add a "location_widget_pressed" signal to the > > NautilusWindow, which the view can connect to? > > You'd have to add something, say > "nautilus_view_popup_location_context_menu()", just not something that > exposes details of FMDirectoryView (like its selection and background > context menus).
Hrm I thought that every view would expose a selection, considering that we have nautilus_view_get_selection_count/nautilus_view_get_selection/nautilus_view_set_selection, so I thought this might be a good idea to add NAUTILUS_VIEW_MENU_SELECTION. Whatever, I'm going to adapt the patch. -- Christian Neumair <[EMAIL PROTECTED]> -- nautilus-list mailing list [email protected] http://mail.gnome.org/mailman/listinfo/nautilus-list
