GNOME Bugzilla – Bug 141667
Egg recent view that uses the ui manager
Last modified: 2005-03-30 19:53:04 UTC
I've done a egg recent view that uses the ui manager interface to create the menu items.
Created attachment 27309 [details] [review] this is the c file.
Created attachment 27310 [details] [review] the header
Created attachment 29924 [details] [review] support multiple EggRecentViewUIManager As it stands the attached EggRecentViewUIManager is limited to 1 per app (screem keeps 2 lists, 1 for projects, 1 for files worked on outside of a project). This patch allows multiple instances, and also simplifies egg_recent_view_uimanager_get_item() to get the item via g_object_get_data() on the GtkAction.
Created attachment 29927 [details] [review] revised to use merge_id same patch revised to use the obtained merge_id as the unique group identifier. removes view->item_list, not needed as items can be obtained directly from actions anyway, this also fixes a crash as already destroyed EggRecentItems were being unrefed here. make trailing sep names contain the merge_id as well
After asking snorp on IRC I committed the latest version of Paolo Bacchilega's uimanager view and a small test app. I'm leaving the bug open since I have no idea about the status of David's patches attached to this bug: the version I committed is the latest found in File Roller and used in many other projects. I'll leave to Paolo Bacchilega any other decision about changes to egg-recent-view-uimanager.
David's patch has been applied now, closing the bug...