Revision: 1371
          http://geeqie.svn.sourceforge.net/geeqie/?rev=1371&view=rev
Author:   nadvornik
Date:     2009-02-02 20:32:59 +0000 (Mon, 02 Feb 2009)

Log Message:
-----------
improved placing of external commands in menu
small menu improvements

Modified Paths:
--------------
    trunk/plugins/symlink/symlink.desktop
    trunk/src/layout_util.c

Modified: trunk/plugins/symlink/symlink.desktop
===================================================================
--- trunk/plugins/symlink/symlink.desktop       2009-02-01 22:35:40 UTC (rev 
1370)
+++ trunk/plugins/symlink/symlink.desktop       2009-02-02 20:32:59 UTC (rev 
1371)
@@ -15,7 +15,7 @@
 OnlyShowIn=X-Geeqie;
 
 # Show in menu "File"
-X-Geeqie-Menu-Path=FileMenu
+X-Geeqie-Menu-Path=FileMenu/FileOpsSection
 
 # This is a filter - $GEEQIE_DESTINATION is required
 X-Geeqie-Filter=true

Modified: trunk/src/layout_util.c
===================================================================
--- trunk/src/layout_util.c     2009-02-01 22:35:40 UTC (rev 1370)
+++ trunk/src/layout_util.c     2009-02-02 20:32:59 UTC (rev 1371)
@@ -1254,20 +1254,24 @@
 "      <menuitem action='NewCollection'/>"
 "      <menuitem action='OpenCollection'/>"
 "      <menuitem action='OpenRecent'/>"
+"      <placeholder name='OpenSection'/>"
 "      <separator/>"
 "      <menuitem action='Search'/>"
 "      <menuitem action='FindDupes'/>"
 "      <menuitem action='PanView'/>"
+"      <placeholder name='SearchSection'/>"
 "      <separator/>"
 "      <menuitem action='Print'/>"
+"      <placeholder name='PrintSection'/>"
+"      <separator/>"
 "      <menuitem action='NewFolder'/>"
-"      <separator/>"
 "      <menuitem action='Copy'/>"
 "      <menuitem action='Move'/>"
 "      <menuitem action='Rename'/>"
 "      <menuitem action='Delete'/>"
-"      <menuitem action='CopyPath'/>"
+"      <placeholder name='FileOpsSection'/>"
 "      <separator/>"
+"      <placeholder name='QuitSection'/>"
 "      <menuitem action='CloseWindow'/>"
 "      <menuitem action='Quit'/>"
 "      <separator/>"
@@ -1283,11 +1287,20 @@
 "      <menuitem action='SelectAll'/>"
 "      <menuitem action='SelectNone'/>"
 "      <menuitem action='SelectInvert'/>"
+"      <placeholder name='SelectSection'/>"
 "      <separator/>"
+"      <menuitem action='CopyPath'/>"
+"      <placeholder name='ClipboardSection'/>"
+"      <separator/>"
 "      <menuitem action='ShowMarks'/>"
+"      <placeholder name='MarksSection'/>"
 "      <separator/>"
 "    </menu>"
 "    <menu action='EditMenu'>"
+"      <menu action='ExternalMenu'>"
+"      </menu>"
+"      <placeholder name='EditSection'/>"
+"      <separator/>"
 "      <menu action='AdjustMenu'>"
 "        <menuitem action='RotateCW'/>"
 "        <menuitem action='RotateCCW'/>"
@@ -1298,17 +1311,18 @@
 "        <menuitem action='AlterNone'/>"
 "      </menu>"
 "      <menuitem action='Properties'/>"
+"      <placeholder name='PropertiesSection'/>"
 "      <separator/>"
 "      <menuitem action='Preferences'/>"
 "      <menuitem action='Maintenance'/>"
+"      <placeholder name='PreferencesSection'/>"
 "      <separator/>"
 "      <menuitem action='Wallpaper'/>"
 "      <separator/>"
-"      <menu action='ExternalMenu'>"
-"      </menu>"
 "    </menu>"
 "    <menu action='ViewMenu'>"
 "      <menuitem action='ViewInNewWindow'/>"
+"      <placeholder name='WindowSection'/>"
 "      <separator/>"
 "      <menu action='ZoomMenu'>"
 "        <menuitem action='ZoomIn'/>"
@@ -1338,6 +1352,7 @@
 "        <menuitem action='ConnectZoom33'/>"
 "        <menuitem action='ConnectZoom25'/>"
 "      </menu>"
+"      <placeholder name='ZoomSection'/>"
 "      <separator/>"
 "      <menu action='SplitMenu'>"
 "        <menuitem action='SplitHorizontal'/>"
@@ -1346,31 +1361,37 @@
 "        <menuitem action='SplitSingle'/>"
 "      </menu>"
 "      <separator/>"
-"      <menuitem action='Thumbnails'/>"
 "      <menuitem action='ViewList'/>"
 "      <menuitem action='ViewIcons'/>"
+"      <menuitem action='Thumbnails'/>"
+"      <placeholder name='ListSection'/>"
 "      <separator/>"
 "      <menu action='DirMenu'>"
 "        <menuitem action='FolderList'/>"
 "        <menuitem action='FolderTree'/>"
 "      </menu>"
+"      <placeholder name='DirSection'/>"
 "      <separator/>"
 "      <menuitem action='ImageOverlay'/>"
 "      <menuitem action='HistogramChan'/>"
 "      <menuitem action='HistogramLog'/>"
 "      <menuitem action='FullScreen'/>"
+"      <placeholder name='OverlaySection'/>"
 "      <separator/>"
 "      <menuitem action='FloatTools'/>"
 "      <menuitem action='HideTools'/>"
 "      <menuitem action='HideToolbar'/>"
+"      <placeholder name='ToolsSection'/>"
 "      <separator/>"
 "      <menuitem action='SBarKeywords'/>"
 "      <menuitem action='SBarExif'/>"
 "      <menuitem action='SBarSort'/>"
+"      <placeholder name='SideBarSection'/>"
 "      <separator/>"
 "      <menuitem action='SlideShow'/>"
 "      <menuitem action='SlideShowPause'/>"
 "      <menuitem action='Refresh'/>"
+"      <placeholder name='SlideShowSection'/>"
 "      <separator/>"
 "    </menu>"
 "    <menu action='HelpMenu'>"
@@ -1378,6 +1399,7 @@
 "      <menuitem action='HelpContents'/>"
 "      <menuitem action='HelpShortcuts'/>"
 "      <menuitem action='HelpNotes'/>"
+"      <placeholder name='HelpSection'/>"
 "      <separator/>"
 "      <menuitem action='About'/>"
 "      <separator/>"
@@ -1538,14 +1560,45 @@
        to_open = g_list_length(path) - 1;
        to_close = g_list_length(old_path) - 1;
        
+       if (to_close > 0)
+               {
+               old_path = g_list_last(old_path);
+               old_path = old_path->prev;
+               }
+       
        for (i =  0; i < to_close; i++)
                {
-               g_string_append(desc,   "    </menu>");
+               gchar *name = old_path->data;
+               if (g_str_has_suffix(name, "Section"))
+                       {
+                       g_string_append(desc,   "      </placeholder>");
+                       }
+               else if (g_str_has_suffix(name, "Menu"))
+                       {
+                       g_string_append(desc,   "    </menu>");
+                       }
+               else
+                       {
+                       g_warning("invalid menu path item %s", name);
+                       }
+               old_path = old_path->prev;
                }
 
        for (i =  0; i < to_open; i++)
                {
-               g_string_append_printf(desc,    "    <menu action='%s'>", 
(gchar *)path->data);
+               gchar *name = path->data;
+               if (g_str_has_suffix(name, "Section"))
+                       {
+                       g_string_append_printf(desc,    "      <placeholder 
name='%s'>", name);
+                       }
+               else if (g_str_has_suffix(name, "Menu"))
+                       {
+                       g_string_append_printf(desc,    "    <menu 
action='%s'>", name);
+                       }
+               else
+                       {
+                       g_warning("invalid menu path item %s", name);
+                       }
                path = path->next;
                }
        


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
Geeqie-svn mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geeqie-svn

Reply via email to