Author: craig
Date: Sat Oct 12 16:41:29 2013
New Revision: 18548

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=18548
Log:
Fix up most parts of Scripts menu

Modified:
    trunk/Scribus/scribus/menumanager.cpp
    trunk/Scribus/scribus/menumanager.h
    trunk/Scribus/scribus/plugins/scriptplugin/scriptercore.cpp
    trunk/Scribus/scribus/scribus.cpp
    trunk/Scribus/scribus/scribus.h

Modified: trunk/Scribus/scribus/menumanager.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=18548&path=/trunk/Scribus/scribus/menumanager.cpp
==============================================================================
--- trunk/Scribus/scribus/menumanager.cpp (original)
+++ trunk/Scribus/scribus/menumanager.cpp Sat Oct 12 16:41:29 2013
@@ -58,22 +58,22 @@
        return retVal;
 }
 
-bool MenuManager::addMenuToMenu(const QString &child, const QString &parent)
-{
-       bool retVal=false;
-       if (child==parent)
-       {
-               qDebug("%s", QString("Cannot add %1 menu to %2 menu 
(itself)").arg(child, parent).toLatin1().constData());
-               return false;   
-       }
-       if ((menuList.contains(child) && menuList[child]!=NULL) &&
-               (menuList.contains(parent) && menuList[parent]!=NULL))
-       {
-               menuList.insert(child, menuList[child]);
-               retVal=menuList[parent]->insertSubMenu(menuList[child]);
-       }
-       return retVal;
-}
+//bool MenuManager::addMenuToMenu(const QString &child, const QString &parent)
+//{
+//     bool retVal=false;
+//     if (child==parent)
+//     {
+//             qDebug("%s", QString("Cannot add %1 menu to %2 menu 
(itself)").arg(child, parent).toLatin1().constData());
+//             return false;
+//     }
+//     if ((menuList.contains(child) && menuList[child]!=NULL) &&
+//             (menuList.contains(parent) && menuList[parent]!=NULL))
+//     {
+//             menuList.insert(child, menuList[child]);
+//             retVal=menuList[parent]->insertSubMenu(menuList[child]);
+//     }
+//     return retVal;
+//}
 
 bool MenuManager::clearMenu(const QString &menuName)
 {
@@ -117,16 +117,16 @@
        return NULL;
 }
 
-bool MenuManager::deleteMenu(const QString &menuName, const QString &parent)
-{
-       if (!parent.isNull())
-       {
-               if (menuList[parent] && 
menuList[parent]->hasSubMenu(menuList[menuName]))
-                       menuList[parent]->removeSubMenu(menuList[menuName]);
-       }
-       menuList.remove(menuName);
-       return true;
-}
+//bool MenuManager::deleteMenu(const QString &menuName, const QString &parent)
+//{
+//     if (!parent.isNull())
+//     {
+//             if (menuList[parent] && 
menuList[parent]->hasSubMenu(menuList[menuName]))
+//                     menuList[parent]->removeSubMenu(menuList[menuName]);
+//     }
+//     menuList.remove(menuName);
+//     return true;
+//}
 
 void MenuManager::setMenuEnabled(const QString &menuName, const bool enabled)
 {
@@ -134,20 +134,20 @@
                menuList[menuName]->setEnabled(enabled);
 }
 
-bool MenuManager::addMenuToMenuBar(const QString &menuName)
-{
-       bool retVal=false;
-       if (menuList.contains(menuName) && menuList[menuName]!=NULL)
-       {
-               QAction* 
t=scribusMenuBar->addMenu(menuList[menuName]->getLocalPopupMenu());
-               if (t!=NULL)
-               {
-                       t->setText(menuList[menuName]->getMenuText());
-                       retVal=true;
-               }
-       }
-       return retVal;
-}
+//bool MenuManager::addMenuToMenuBar(const QString &menuName)
+//{
+//     bool retVal=false;
+//     if (menuList.contains(menuName) && menuList[menuName]!=NULL)
+//     {
+//             QAction* 
t=scribusMenuBar->addMenu(menuList[menuName]->getLocalPopupMenu());
+//             if (t!=NULL)
+//             {
+//                     t->setText(menuList[menuName]->getMenuText());
+//                     retVal=true;
+//             }
+//     }
+//     return retVal;
+//}
 
 bool MenuManager::addMenuStringToMenuBar(const QString &menuName)
 {
@@ -165,51 +165,53 @@
        return retVal;
 }
 
-bool MenuManager::addMenuToMenuBarBefore(const QString &menuName, const 
QString &afterMenuName)
-{
-       bool retVal=false;
-       if (menuList.contains(menuName) && menuList[menuName]!=NULL)
-       {
-               if (menuList[afterMenuName])
-               {
-                       QAction* 
t=scribusMenuBar->insertMenu(menuList[afterMenuName]->getLocalPopupMenu()->menuAction(),
 menuList[menuName]->getLocalPopupMenu());
-                       if (t!=NULL)
-                       {
-                               t->setText(menuList[menuName]->getMenuText());
-                               retVal=true;
-                       }
-               }
-       }
-       return retVal;
-}
-
-bool MenuManager::addMenuStringToMenuBarBefore(const QString &menuName, const 
QString &afterMenuName)
+bool MenuManager::addMenuStringToMenuBarBefore(const QString &menuName, const 
QString &beforeMenuName)
 {
        bool retVal=false;
        if (menuStrings.contains(menuName))
        {
-               QMenu *m=scribusMenuBar->addMenu(menuStringTexts[menuName]);
+               QList<QAction*> mba=scribusMenuBar->actions();
+               QAction* beforeAct;
+               foreach (beforeAct, mba) {
+                       if 
(beforeMenuName==beforeAct->text().remove('&').remove("..."))
+                               break;
+               }
+               QMenu *m=new QMenu(menuName);
+               scribusMenuBar->insertMenu(beforeAct, m);
                menuBarMenus.insert(menuName, m);
-               if (menuName=="Windows" && m!=NULL)
-               {
-                       windowsMenu=m;
-               }
                retVal=true;
        }
        return retVal;
 }
 
-bool MenuManager::removeMenuFromMenuBar(const QString &menuName)
-{
-       bool retVal=false;
-       if (menuList.contains(menuName) && menuList[menuName]!=NULL)
-       {
-               if (menuList[menuName]->getLocalPopupMenu()->menuAction())
-                       
scribusMenuBar->removeAction(menuList[menuName]->getLocalPopupMenu()->menuAction());
-               retVal=true;
-       }
-       return retVal;
-}
+//bool MenuManager::addMenuToMenuBarBefore(const QString &menuName, const 
QString &beforeMenuName)
+//{
+//     bool retVal=false;
+//     if (menuList.contains(menuName) && menuList[menuName]!=NULL)
+//     {
+//             if (menuList[beforeMenuName])
+//             {
+//                     QAction* 
t=scribusMenuBar->insertMenu(menuList[beforeMenuName]->getLocalPopupMenu()->menuAction(),
 menuList[menuName]->getLocalPopupMenu());
+//                     if (t!=NULL)
+//                     {
+//                             t->setText(menuList[menuName]->getMenuText());
+//                             retVal=true;
+//                     }
+//             }
+//     }
+//     return retVal;
+//}
+//bool MenuManager::removeMenuFromMenuBar(const QString &menuName)
+//{
+//     bool retVal=false;
+//     if (menuList.contains(menuName) && menuList[menuName]!=NULL)
+//     {
+//             if (menuList[menuName]->getLocalPopupMenu()->menuAction())
+//                     
scribusMenuBar->removeAction(menuList[menuName]->getLocalPopupMenu()->menuAction());
+//             retVal=true;
+//     }
+//     return retVal;
+//}
 
 void MenuManager::addMenuItemStringstoMenuBar(const QString &menuName, const 
QMap<QString, QPointer<ScrAction> > &menuActions)
 {

Modified: trunk/Scribus/scribus/menumanager.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=18548&path=/trunk/Scribus/scribus/menumanager.h
==============================================================================
--- trunk/Scribus/scribus/menumanager.h (original)
+++ trunk/Scribus/scribus/menumanager.h Sat Oct 12 16:41:29 2013
@@ -51,18 +51,18 @@
                enum MenuType {Normal, DLL};
 
                bool createMenu(const QString &menuName, const QString 
&menuText = QString::null, const QString parent = QString::null, bool checkable 
= false);
-               bool addMenuToMenu(const QString & child, const QString 
&parent);
-               bool deleteMenu(const QString &menuName, const QString &parent 
= QString::null);
+//             bool addMenuToMenu(const QString & child, const QString 
&parent);
+//             bool deleteMenu(const QString &menuName, const QString &parent 
= QString::null);
                bool clearMenu(const QString &menuName);
                void setText(const QString &menuName, const QString &menuText);
                void setMenuIcon(const QString &menuName, const QIcon 
&menuIcon);
                QMenu *getLocalPopupMenu(const QString &menuName);
                void setMenuEnabled(const QString &menuName, const bool 
enabled);
-               bool addMenuToMenuBar(const QString &menuName);
+//             bool addMenuToMenuBar(const QString &menuName);
                bool addMenuStringToMenuBar(const QString &menuName);
-               bool addMenuToMenuBarBefore(const QString &, const QString &);
-               bool addMenuStringToMenuBarBefore(const QString &, const 
QString &);
-               bool removeMenuFromMenuBar(const QString &menuName);
+//             bool addMenuToMenuBarBefore(const QString &, const QString 
&beforeMenuName);
+               bool addMenuStringToMenuBarBefore(const QString &, const 
QString &beforeMenuName);
+//             bool removeMenuFromMenuBar(const QString &menuName);
                bool addMenuToWidgetOfAction(const QString &menuName, ScrAction 
*action);
 
                bool addMenuItem(ScrAction *menuAction, const QString &parent, 
bool enabled);

Modified: trunk/Scribus/scribus/plugins/scriptplugin/scriptercore.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=18548&path=/trunk/Scribus/scribus/plugins/scriptplugin/scriptercore.cpp
==============================================================================
--- trunk/Scribus/scribus/plugins/scriptplugin/scriptercore.cpp (original)
+++ trunk/Scribus/scribus/plugins/scriptplugin/scriptercore.cpp Sat Oct 12 
16:41:29 2013
@@ -77,15 +77,23 @@
 {
        menuMgr = mw->scrMenuMgr;
        menuMgr->createMenu("Scripter", QObject::tr("&Script"));
-       menuMgr->addMenuToMenuBarBefore("Scripter","Windows");
        menuMgr->createMenu("ScribusScripts", QObject::tr("&Scribus Scripts"), 
"Scripter");
-       menuMgr->addMenuItem(scrScripterActions["scripterExecuteScript"], 
"Scripter", true);
+       menuMgr->addMenuItemString("ScribusScripts", "Scripter");
+       menuMgr->addMenuItemString("scripterExecuteScript", "Scripter");
        menuMgr->createMenu("RecentScripts", QObject::tr("&Recent Scripts"), 
"Scripter");
-       menuMgr->addMenuSeparator("Scripter");
-       menuMgr->addMenuItem(scrScripterActions["scripterShowConsole"], 
"Scripter", true);
-       menuMgr->addMenuItem(scrScripterActions["scripterAboutScript"], 
"Scripter", true);
+       menuMgr->addMenuItemString("scripterExecuteScript", "Scripter");
+       menuMgr->addMenuItemString("SEPARATOR", "Scripter");
+//     menuMgr->addMenuItem(scrScripterActions["scripterShowConsole"], 
"Scripter", true);
+//     menuMgr->addMenuItem(scrScripterActions["scripterAboutScript"], 
"Scripter", true);
+       menuMgr->addMenuItemString("scripterShowConsole", "Scripter");
+       menuMgr->addMenuItemString("scripterAboutScript", "Scripter");
+
+
        buildScribusScriptsMenu();
        buildRecentScriptsMenu();
+
+       menuMgr->addMenuStringToMenuBarBefore("Scripter","Windows");
+       menuMgr->addMenuItemStringstoMenuBar("Scripter", scrScripterActions);
 }
 
 void ScripterCore::enableMainWindowMenu()
@@ -120,7 +128,8 @@
                        QString strippedName=fs.baseName();
                        scrScripterActions.insert(strippedName, new ScrAction( 
ScrAction::RecentScript, strippedName, QKeySequence(), this));
                        connect( scrScripterActions[strippedName], 
SIGNAL(triggeredData(QString)), this, SLOT(StdScript(QString)) );
-                       menuMgr->addMenuItem(scrScripterActions[strippedName], 
"ScribusScripts", true);
+                       
//menuMgr->addMenuItem(scrScripterActions[strippedName], "ScribusScripts", 
true);
+                       menuMgr->addMenuItemString(strippedName, 
"ScribusScripts");
                }
        }
 
@@ -140,7 +149,8 @@
                strippedName.remove(QDir::separator());
                scrRecentScriptActions.insert(strippedName, new ScrAction( 
ScrAction::RecentScript, RecentScripts[m], QKeySequence(), this));
                connect( scrRecentScriptActions[strippedName], 
SIGNAL(triggeredData(QString)), this, SLOT(RecentScript(QString)) );
-               menuMgr->addMenuItem(scrRecentScriptActions[strippedName], 
"RecentScripts", true);
+//             menuMgr->addMenuItem(scrRecentScriptActions[strippedName], 
"RecentScripts", true);
+               menuMgr->addMenuItemString(strippedName, "RecentScripts");
        }
 }
 

Modified: trunk/Scribus/scribus/scribus.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=18548&path=/trunk/Scribus/scribus/scribus.cpp
==============================================================================
--- trunk/Scribus/scribus/scribus.cpp (original)
+++ trunk/Scribus/scribus/scribus.cpp Sat Oct 12 16:41:29 2013
@@ -361,10 +361,12 @@
 //     prefsManager->applyLoadedShortCuts();
 //     initKeyboardShortcuts();
        initMenuBar();
+       createMenuBar();
        initToolBars();
        ScCore->pluginManager->setupPluginActions(this);
        ScCore->pluginManager->enableOnlyStartupPluginActions(this);
        ScCore->pluginManager->languageChange();
+
        if (primaryMainWindow)
                ScCore->setSplashStatus( tr("Applying User Shortcuts") );
        prefsManager->applyLoadedShortCuts();
@@ -1099,7 +1101,10 @@
        scrMenuMgr->addMenuItemString("helpAboutScribus", "Help");
        scrMenuMgr->addMenuItemString("helpAboutPlugins", "Help");
        scrMenuMgr->addMenuItemString("helpAboutQt", "Help");
-
+}
+
+void ScribusMainWindow::createMenuBar()
+{
        scrMenuMgr->addMenuStringToMenuBar("File");
        scrMenuMgr->addMenuItemStringstoMenuBar("File", scrActions);
        scrMenuMgr->addMenuStringToMenuBar("Edit");
@@ -1121,7 +1126,6 @@
        menuBar()->addSeparator();
        scrMenuMgr->addMenuStringToMenuBar("Help");
        scrMenuMgr->addMenuItemStringstoMenuBar("Help", scrActions);
-
 }
 
 

Modified: trunk/Scribus/scribus/scribus.h
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=18548&path=/trunk/Scribus/scribus/scribus.h
==============================================================================
--- trunk/Scribus/scribus/scribus.h (original)
+++ trunk/Scribus/scribus/scribus.h Sat Oct 12 16:41:29 2013
@@ -560,6 +560,7 @@
     /** init methods */
        void initSplash(bool showSplash);
        void initMenuBar(); // initMenuBar creates the menu_bar and inserts the 
menuitems
+       void createMenuBar();
        void addDefaultWindowMenuItems(); // addDefaultWindowMenuItems adds the 
basic Windows menu items, excluding the actual list of windows
        void initStatusBar(); // setup the statusbar
        void initToolBars(); // setup the toolbars


_______________________________________________
scribus-commit mailing list
[email protected]
http://lists.scribus.net/mailman/listinfo/scribus-commit

Reply via email to