]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/Menus.cpp
QDialogButtonBox for the remaining dialogs.
[lyx.git] / src / frontends / qt4 / Menus.cpp
index 8daec7d5de23a91991fbfc94698f5d3c15b928c0..c68ecbe5a5575f174ce1938d4539602232083b00 100644 (file)
@@ -193,6 +193,8 @@ public:
                SwitchCaptions,
                /** Commands to separate environments. */
                EnvironmentSeparators,
+               /** Commands to separate environments (context menu version). */
+               EnvironmentSeparatorsContext,
                /** This is the list of quotation marks available */
                SwitchQuotes
        };
@@ -368,7 +370,7 @@ public:
        void expandLanguageSelector(Buffer const * buf);
        void expandArguments(BufferView const *, bool switcharg = false);
        void expandCaptions(Buffer const * buf, bool switchcap = false);
-       void expandEnvironmentSeparators(BufferView const *);
+       void expandEnvironmentSeparators(BufferView const *, bool contextmenu = false);
        void expandQuotes(BufferView const *);
        ///
        ItemList items_;
@@ -484,6 +486,7 @@ void MenuDefinition::read(Lexer & lex)
                md_captions,
                md_switchcaptions,
                md_env_separators,
+               md_env_separatorscontext,
                md_switchquotes
        };
 
@@ -499,6 +502,7 @@ void MenuDefinition::read(Lexer & lex)
                { "elements", md_elements },
                { "end", md_endmenu },
                { "environmentseparators", md_env_separators },
+               { "environmentseparatorscontext", md_env_separatorscontext },
                { "exportformat", md_exportformat },
                { "exportformats", md_exportformats },
                { "floatinsert", md_floatinsert },
@@ -672,6 +676,10 @@ void MenuDefinition::read(Lexer & lex)
                        add(MenuItem(MenuItem::EnvironmentSeparators));
                        break;
 
+               case md_env_separatorscontext:
+                       add(MenuItem(MenuItem::EnvironmentSeparatorsContext));
+                       break;
+
                case md_switchquotes:
                        add(MenuItem(MenuItem::SwitchQuotes));
                        break;
@@ -795,10 +803,10 @@ void MenuDefinition::expandGraphicsGroups(BufferView const * bv)
        set<string>::const_iterator it = grp.begin();
        set<string>::const_iterator end = grp.end();
        add(MenuItem(MenuItem::Command, qt_("No Group"),
-                    FuncRequest(LFUN_SET_GRAPHICS_GROUP)));
+                    FuncRequest(LFUN_GRAPHICS_SET_GROUP)));
        for (; it != end; ++it) {
                addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(*it) + '|',
-                               FuncRequest(LFUN_SET_GRAPHICS_GROUP, *it)));
+                               FuncRequest(LFUN_GRAPHICS_SET_GROUP, *it)));
        }
 }
 
@@ -889,13 +897,13 @@ void MenuDefinition::expandLanguageSelector(Buffer const * buf)
        if (languages_buffer.size() < 2)
                return;
 
-       std::set<Language const *, sortLanguageByName> languages;
+       std::set<Language const *, sortLanguageByName> langs;
 
        std::set<Language const *>::const_iterator const beg =
                languages_buffer.begin();
        for (std::set<Language const *>::const_iterator cit = beg;
             cit != languages_buffer.end(); ++cit) {
-               languages.insert(*cit);
+               langs.insert(*cit);
        }
 
        MenuItem item(MenuItem::Submenu, qt_("Language|L"));
@@ -904,9 +912,9 @@ void MenuDefinition::expandLanguageSelector(Buffer const * buf)
        QStringList accelerators;
        if (morelangs.contains('|'))
                accelerators.append(morelangs.section('|', -1));
-       std::set<Language const *, sortLanguageByName>::const_iterator const begin = languages.begin();
+       std::set<Language const *, sortLanguageByName>::const_iterator const begin = langs.begin();
        for (std::set<Language const *, sortLanguageByName>::const_iterator cit = begin;
-            cit != languages.end(); ++cit) {
+                        cit != langs.end(); ++cit) {
                QString label = qt_((*cit)->display());
                // try to add an accelerator
                bool success = false;
@@ -1115,7 +1123,7 @@ void MenuDefinition::expandFormats(MenuItem::Kind const kind, Buffer const * buf
                                add(MenuItem(MenuItem::Command, toqstr(lbl), FuncRequest(action)));
                                continue;
                        }
-               // fall through
+                       break;
                case MenuItem::ExportFormats:
                        if (!f->inExportMenu())
                                continue;
@@ -1600,12 +1608,12 @@ void MenuDefinition::expandCiteStyles(BufferView const * bv)
 
        for (int ii = 1; cit != end; ++cit, ++ii) {
                docstring label = cit->second;
-               CitationStyle cs = citeStyleList[ii - 1];
-               cs.forceUpperCase &= force;
-               cs.hasStarredVersion &= star;
+               CitationStyle ccs = citeStyleList[ii - 1];
+               ccs.forceUpperCase &= force;
+               ccs.hasStarredVersion &= star;
                addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(label),
                                    FuncRequest(LFUN_INSET_MODIFY,
-                                               "changetype " + from_utf8(citationStyleToString(cs)))));
+                                               "changetype " + from_utf8(citationStyleToString(ccs)))));
        }
 
        if (cs.hasStarredVersion) {
@@ -1621,10 +1629,10 @@ void MenuDefinition::expandCiteStyles(BufferView const * bv)
                        if (amps > 0) {
                                if (amps > 1)
                                        starred = subst(starred, from_ascii("&&"), from_ascii("<:amp:>"));
-                               size_t n = starred.find('&');
+                               size_t nn = starred.find('&');
                                char_type accel = char_type();
-                               if (n != docstring::npos && n < starred.size() - 1)
-                                       accel = starred[n + 1];
+                               if (nn != docstring::npos && nn < starred.size() - 1)
+                                       accel = starred[nn + 1];
                                starred = subst(starred, from_ascii("&"), from_ascii(""));
                                if (amps > 1)
                                        starred = subst(starred, from_ascii("<:amp:>"), from_ascii("&&"));
@@ -1846,7 +1854,8 @@ void MenuDefinition::expandQuotes(BufferView const * bv)
 }
 
 
-void MenuDefinition::expandEnvironmentSeparators(BufferView const * bv)
+void MenuDefinition::expandEnvironmentSeparators(BufferView const * bv,
+                                                bool contextmenu)
 {
        if (!bv)
                return;
@@ -1880,28 +1889,54 @@ void MenuDefinition::expandEnvironmentSeparators(BufferView const * bv)
                        break;
        }
        if (par.layout().isEnvironment()) {
-               docstring label = bformat(_("Prepend New Environment (%1$s)"),
-                               translateIfPossible(curlayout));
+               docstring label = contextmenu ?
+                                       bformat(_("Insert Separated %1$s Above"),
+                                               translateIfPossible(curlayout)) :
+                                       bformat(_("Separated %1$s Above"),
+                                               translateIfPossible(curlayout));
                add(MenuItem(MenuItem::Command, toqstr(label),
                             FuncRequest(LFUN_ENVIRONMENT_SPLIT,
                                         from_ascii("before"))));
-               label = bformat(_("Append New Environment (%1$s)"),
-                               translateIfPossible(curlayout));
-               add(MenuItem(MenuItem::Command, toqstr(label),
-                            FuncRequest(LFUN_ENVIRONMENT_SPLIT)));
+               label = contextmenu ?
+                               bformat(_("Insert Separated %1$s Below"),
+                                       translateIfPossible(curlayout)):
+                               bformat(_("Separated %1$s Below"),
+                                       translateIfPossible(curlayout));
+               // We use command-alternatives here since this is how the binding is defined
+               // (otherwise, the binding is not displayed in the menu)
+               if (getStatus(FuncRequest(LFUN_ENVIRONMENT_SPLIT)).enabled())
+                       add(MenuItem(MenuItem::Command, toqstr(label),
+                                    FuncRequest(LFUN_COMMAND_ALTERNATIVES,
+                                                from_ascii("environment-split ; environment-split previous"))));
        }
        else if (!prevlayout.empty()) {
-               docstring const label =
-                       bformat(_("Append New Environment (%1$s)"),
+               docstring const label = contextmenu ?
+                       bformat(_("Insert Separated %1$s Below"),
+                               translateIfPossible(prevlayout)) :
+                       bformat(_("Separated %1$s Below"),
                                translateIfPossible(prevlayout));
-               add(MenuItem(MenuItem::Command, toqstr(label),
-                            FuncRequest(LFUN_ENVIRONMENT_SPLIT,
-                                        from_ascii("previous"))));
+               // We use command-alternatives here since this is how the binding is defined
+               // (otherwise, the binding is not displayed in the menu)
+               if (getStatus(FuncRequest(LFUN_ENVIRONMENT_SPLIT)).enabled())
+                       add(MenuItem(MenuItem::Command, toqstr(label),
+                                    FuncRequest(LFUN_COMMAND_ALTERNATIVES,
+                                                from_ascii("environment-split ; environment-split previous"))));
        }
        if (!outerlayout.empty()) {
-               docstring const label =
-                       bformat(_("Append New Parent Environment (%1$s)"),
-                               translateIfPossible(outerlayout));
+               docstring label;
+               if (contextmenu) {
+                       label = (outerlayout == curlayout) ?
+                               bformat(_("Insert Separated Outer %1$s Below"),
+                                       translateIfPossible(outerlayout)) :
+                               bformat(_("Insert Separated %1$s Below"),
+                                       translateIfPossible(outerlayout));
+               } else {
+                       label = (outerlayout == curlayout) ?
+                               bformat(_("Separated Outer %1$s Below"),
+                                       translateIfPossible(outerlayout)) :
+                               bformat(_("Separated %1$s Below"),
+                                       translateIfPossible(outerlayout));
+               }
                add(MenuItem(MenuItem::Command, toqstr(label),
                             FuncRequest(LFUN_ENVIRONMENT_SPLIT,
                                         from_ascii("outer"))));
@@ -2307,6 +2342,10 @@ void Menus::Impl::expand(MenuDefinition const & frommenu,
                        tomenu.expandEnvironmentSeparators(bv);
                        break;
 
+               case MenuItem::EnvironmentSeparatorsContext:
+                       tomenu.expandEnvironmentSeparators(bv, true);
+                       break;
+
                case MenuItem::SwitchQuotes:
                        tomenu.expandQuotes(bv);
                        break;
@@ -2507,19 +2546,19 @@ void Menus::fillMenuBar(QMenuBar * qmb, GuiView * view, bool initial)
                        continue;
                }
 
-               Menu * menu = new Menu(view, m->submenuname(), true);
-               menu->setTitle(label(*m));
+               Menu * menuptr = new Menu(view, m->submenuname(), true);
+               menuptr->setTitle(label(*m));
 
 #if defined(Q_OS_MAC) && (defined(QT_MAC_USE_COCOA) || (QT_VERSION >= 0x050000))
                // On Mac OS with QT/cocoa, the menu is not displayed if there is no action
                // so we create a temporary one here
-               QAction * action = new QAction(menu);
-               menu->addAction(action);
+               QAction * action = new QAction(menuptr);
+               menuptr->addAction(action);
 #endif
 
-               qmb->addMenu(menu);
+               qmb->addMenu(menuptr);
 
-               d->name_map_[view][name] = menu;
+               d->name_map_[view][name] = menuptr;
        }
 }