]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/Menus.cpp
Use <cstdint> instead of <boost/cstdint.hpp>
[lyx.git] / src / frontends / qt4 / Menus.cpp
index a53b6bb8be8c30749cd31efac6c235d7601aafe9..e06ce0fab265cf4c9fdb1979e7b68f1da0763b29 100644 (file)
@@ -60,6 +60,8 @@
 #include "insets/Inset.h"
 #include "insets/InsetCitation.h"
 #include "insets/InsetGraphics.h"
+#include "insets/InsetInfo.h"
+#include "insets/InsetQuotes.h"
 
 #include "support/lassert.h"
 #include "support/convert.h"
@@ -78,9 +80,8 @@
 #include <QProxyStyle>
 #endif
 
-#include "support/shared_ptr.h"
-
 #include <algorithm>
+#include <memory>
 #include <vector>
 
 using namespace std;
@@ -135,6 +136,9 @@ public:
                /** This is a list of exportable formats
                    typically for the File->Export menu. */
                ExportFormats,
+               /** This exports the document default format
+                   typically for the File->Export menu. */
+               ExportFormat,
                /** This is a list of importable formats
                    typically for the File->Import menu. */
                ImportFormats,
@@ -170,6 +174,8 @@ public:
                IndicesListsContext,
                /** Available citation styles for a given citation */
                CiteStyles,
+               /** Available arguments for a given info inset */
+               InfoArguments,
                /** Available graphics groups */
                GraphicsGroups,
                /// Words suggested by the spellchecker.
@@ -189,7 +195,11 @@ public:
                in the InsetCaption context menu. */
                SwitchCaptions,
                /** Commands to separate environments. */
-               EnvironmentSeparators
+               EnvironmentSeparators,
+               /** Commands to separate environments (context menu version). */
+               EnvironmentSeparatorsContext,
+               /** This is the list of quotation marks available */
+               SwitchQuotes
        };
 
        explicit MenuItem(Kind kind) : kind_(kind), optional_(false) {}
@@ -199,8 +209,8 @@ public:
                 QString const & submenu = QString(),
                 QString const & tooltip = QString(),
                 bool optional = false)
-               : kind_(kind), label_(label), submenuname_(submenu),
-                 tooltip_(tooltip), optional_(optional)
+               : kind_(kind), label_(label), func_(make_shared<FuncRequest>()),
+                 submenuname_(submenu), tooltip_(tooltip), optional_(optional)
        {
                LATTEST(kind == Submenu || kind == Help || kind == Info);
        }
@@ -211,15 +221,12 @@ public:
                 QString const & tooltip = QString(),
                 bool optional = false,
                 FuncRequest::Origin origin = FuncRequest::MENU)
-               : kind_(kind), label_(label), func_(func),
+               : kind_(kind), label_(label), func_(make_shared<FuncRequest>(func)),
                  tooltip_(tooltip), optional_(optional)
        {
-               func_.setOrigin(origin);
+               func_->setOrigin(origin);
        }
 
-       // shared_ptr<MenuDefinition> needs this apprently...
-       ~MenuItem() {}
-
        /// The label of a given menuitem
        QString label() const
        {
@@ -238,7 +245,7 @@ public:
        /// The kind of entry
        Kind kind() const { return kind_; }
        /// the action (if relevant)
-       FuncRequest const & func() const { return func_; }
+       shared_ptr<FuncRequest const> func() const { return func_; }
        /// the tooltip
        QString const & tooltip() const { return tooltip_; }
        /// returns true if the entry should be omitted when disabled
@@ -257,13 +264,13 @@ public:
                        return QString();
                // Get the keys bound to this action, but keep only the
                // first one later
-               KeyMap::Bindings bindings = theTopLevelKeymap().findBindings(func_);
+               KeyMap::Bindings bindings = theTopLevelKeymap().findBindings(*func_);
                if (!bindings.empty())
                        return toqstr(bindings.begin()->print(KeySequence::ForGui));
 
                LYXERR(Debug::KBMAP, "No binding for "
-                       << lyxaction.getActionName(func_.action())
-                       << '(' << func_.argument() << ')');
+                       << lyxaction.getActionName(func_->action())
+                       << '(' << func_->argument() << ')');
                return QString();
        }
 
@@ -289,7 +296,7 @@ private:
        ///
        QString label_;
        ///
-       FuncRequest func_;
+       shared_ptr<FuncRequest> func_;// non-null
        ///
        QString submenuname_;
        ///
@@ -323,8 +330,6 @@ public:
        ///
        size_t size() const { return items_.size(); }
        ///
-       MenuItem const & operator[](size_t) const;
-       ///
        const_iterator begin() const { return items_.begin(); }
        ///
        const_iterator end() const { return items_.end(); }
@@ -354,7 +359,8 @@ public:
        void expandFloatListInsert(Buffer const * buf);
        void expandFloatInsert(Buffer const * buf);
        void expandFlexInsert(Buffer const * buf, InsetLayout::InsetLyXType type);
-       void expandToc2(Toc const & toc_list, size_t from, size_t to, int depth);
+       void expandTocSubmenu(std::string const & type, Toc const & toc_list);
+       void expandToc2(Toc const & toc_list, size_t from, size_t to, int depth, string toc_type);
        void expandToc(Buffer const * buf);
        void expandPasteRecent(Buffer const * buf);
        void expandToolbars();
@@ -362,12 +368,14 @@ public:
        void expandIndices(Buffer const * buf, bool listof = false);
        void expandIndicesContext(Buffer const * buf, bool listof = false);
        void expandCiteStyles(BufferView const *);
+       void expandInfoArguments(BufferView const *);
        void expandGraphicsGroups(BufferView const *);
        void expandSpellingSuggestions(BufferView const *);
        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_;
        ///
@@ -402,7 +410,7 @@ void MenuDefinition::addWithStatusCheck(MenuItem const & i)
        switch (i.kind()) {
 
        case MenuItem::Command: {
-               FuncStatus status = lyx::getStatus(i.func());
+               FuncStatus status = lyx::getStatus(*i.func());
                if (status.unknown() || (!status.enabled() && i.optional()))
                        break;
                items_.push_back(i);
@@ -417,8 +425,9 @@ void MenuDefinition::addWithStatusCheck(MenuItem const & i)
                                  cit != i.submenu().end(); ++cit) {
                                // Only these kind of items affect the status of the submenu
                                if ((cit->kind() == MenuItem::Command
-                                       || cit->kind() == MenuItem::Submenu
-                                       || cit->kind() == MenuItem::Help)) {
+                                    || cit->kind() == MenuItem::Submenu
+                                    || cit->kind() == MenuItem::Help)
+                                   && cit->status().enabled()) {
                                        enabled = true;
                                        break;
                                }
@@ -454,12 +463,14 @@ void MenuDefinition::read(Lexer & lex)
                md_custom,
                md_elements,
                md_endmenu,
+               md_exportformat,
                md_exportformats,
                md_importformats,
                md_indices,
                md_indicescontext,
                md_indiceslists,
                md_indiceslistscontext,
+               md_infoarguments,
                md_lastfiles,
                md_optitem,
                md_optsubmenu,
@@ -479,7 +490,9 @@ void MenuDefinition::read(Lexer & lex)
                md_switcharguments,
                md_captions,
                md_switchcaptions,
-               md_env_separators
+               md_env_separators,
+               md_env_separatorscontext,
+               md_switchquotes
        };
 
        LexerKeyword menutags[] = {
@@ -494,6 +507,8 @@ 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 },
                { "floatlistinsert", md_floatlistinsert },
@@ -503,6 +518,7 @@ void MenuDefinition::read(Lexer & lex)
                { "indicescontext", md_indicescontext },
                { "indiceslists", md_indiceslists },
                { "indiceslistscontext", md_indiceslistscontext },
+               { "infoarguments", md_infoarguments },
                { "item", md_item },
                { "languageselector", md_languageselector },
                { "lastfiles", md_lastfiles },
@@ -514,6 +530,7 @@ void MenuDefinition::read(Lexer & lex)
                { "submenu", md_submenu },
                { "switcharguments", md_switcharguments },
                { "switchcaptions", md_switchcaptions },
+               { "switchquotes", md_switchquotes },
                { "toc", md_toc },
                { "toolbars", md_toolbars },
                { "updateformats", md_updateformats },
@@ -585,6 +602,10 @@ void MenuDefinition::read(Lexer & lex)
                        add(MenuItem(MenuItem::ExportFormats));
                        break;
 
+               case md_exportformat:
+                       add(MenuItem(MenuItem::ExportFormat));
+                       break;
+
                case md_importformats:
                        add(MenuItem(MenuItem::ImportFormats));
                        break;
@@ -613,6 +634,11 @@ void MenuDefinition::read(Lexer & lex)
                        add(MenuItem(MenuItem::CiteStyles));
                        break;
 
+               case md_infoarguments:
+                       add(MenuItem(MenuItem::InfoArguments));
+                       break;
+                       
+
                case md_graphicsgroups:
                        add(MenuItem(MenuItem::GraphicsGroups));
                        break;
@@ -661,6 +687,14 @@ 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;
+
                case md_optsubmenu:
                case md_submenu: {
                        lex.next(true);
@@ -685,16 +719,10 @@ void MenuDefinition::read(Lexer & lex)
 }
 
 
-MenuItem const & MenuDefinition::operator[](size_type i) const
-{
-       return items_[i];
-}
-
-
 bool MenuDefinition::hasFunc(FuncRequest const & func) const
 {
        for (const_iterator it = begin(), et = end(); it != et; ++it)
-               if (it->func() == func)
+               if (*it->func() == func)
                        return true;
        return false;
 }
@@ -744,7 +772,7 @@ bool MenuDefinition::searchMenu(FuncRequest const & func, docstring_list & names
        const_iterator m = begin();
        const_iterator m_end = end();
        for (; m != m_end; ++m) {
-               if (m->kind() == MenuItem::Command && m->func() == func) {
+               if (m->kind() == MenuItem::Command && *m->func() == func) {
                        names.push_back(qstring_to_ucs4(m->label()));
                        return true;
                }
@@ -768,11 +796,9 @@ bool MenuDefinition::searchMenu(FuncRequest const & func, docstring_list & names
 QString limitStringLength(docstring const & str)
 {
        size_t const max_item_length = 45;
-
-       if (str.size() > max_item_length)
-               return toqstr(str.substr(0, max_item_length - 3) + "...");
-
-       return toqstr(str);
+       docstring ret = str.substr(0, max_item_length + 1);
+       support::truncateWithEllipsis(ret, max_item_length);
+       return toqstr(ret);
 }
 
 
@@ -788,10 +814,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)));
        }
 }
 
@@ -882,13 +908,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"));
@@ -897,9 +923,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;
@@ -983,6 +1009,8 @@ void MenuDefinition::expandDocuments()
                QString label = toqstr(b.fileName().displayName(20));
                if (!b.isClean())
                        label += "*";
+               if (b.notifiesExternalModification())
+                       label += QChar(0x26a0);
                if (i < 10)
                        label = QString::number(i) + ". " + label + '|' + QString::number(i);
                add(MenuItem(MenuItem::Command, label,
@@ -1000,6 +1028,8 @@ void MenuDefinition::expandDocuments()
                        QString label = toqstr(b->fileName().displayName(20));
                        if (!b->isClean())
                                label += "*";
+                       if (b->notifiesExternalModification())
+                               label += QChar(0x26a0);
                        if (i < 10)
                                label = QString::number(i) + ". " + label + '|' + QString::number(i);
                        item.submenu().add(MenuItem(MenuItem::Command, label,
@@ -1039,8 +1069,7 @@ void MenuDefinition::expandFormats(MenuItem::Kind const kind, Buffer const * buf
        if (!buf && kind != MenuItem::ImportFormats)
                return;
 
-       typedef vector<Format const *> Formats;
-       Formats formats;
+       FormatList formats;
        FuncCode action = LFUN_NOACTION;
 
        switch (kind) {
@@ -1064,7 +1093,6 @@ void MenuDefinition::expandFormats(MenuItem::Kind const kind, Buffer const * buf
                LATTEST(false);
                return;
        }
-       sort(formats.begin(), formats.end(), Format::formatSorter);
 
        bool const view_update = (kind == MenuItem::ViewFormats
                        || kind == MenuItem::UpdateFormats);
@@ -1077,14 +1105,12 @@ void MenuDefinition::expandFormats(MenuItem::Kind const kind, Buffer const * buf
        MenuItem item(MenuItem::Submenu, smenue);
        item.setSubmenu(MenuDefinition(smenue));
 
-       Formats::const_iterator fit = formats.begin();
-       Formats::const_iterator end = formats.end();
-       for (; fit != end ; ++fit) {
-               if ((*fit)->dummy())
+       for (Format const * f : formats) {
+               if (f->dummy())
                        continue;
 
-               docstring lab = from_utf8((*fit)->prettyname());
-               docstring const scut = from_utf8((*fit)->shortcut());
+               docstring lab = f->prettyname();
+               docstring const scut = from_utf8(f->shortcut());
                docstring const tmplab = lab;
 
                if (!scut.empty())
@@ -1101,16 +1127,16 @@ void MenuDefinition::expandFormats(MenuItem::Kind const kind, Buffer const * buf
                        break;
                case MenuItem::ViewFormats:
                case MenuItem::UpdateFormats:
-                       if ((*fit)->name() == buf->params().getDefaultOutputFormat()) {
+                       if (f->name() == buf->params().getDefaultOutputFormat()) {
                                docstring lbl = (kind == MenuItem::ViewFormats
                                        ? bformat(_("View [%1$s]|V"), label)
                                        : bformat(_("Update [%1$s]|U"), label));
                                add(MenuItem(MenuItem::Command, toqstr(lbl), FuncRequest(action)));
                                continue;
                        }
-               // fall through
+                       break;
                case MenuItem::ExportFormats:
-                       if (!(*fit)->inExportMenu())
+                       if (!f->inExportMenu())
                                continue;
                        break;
                default:
@@ -1122,19 +1148,17 @@ void MenuDefinition::expandFormats(MenuItem::Kind const kind, Buffer const * buf
                        label += '|' + shortcut;
 
                if (view_update) {
-                       if (buf)
-                               item.submenu().addWithStatusCheck(MenuItem(MenuItem::Command,
-                                       toqstr(label), FuncRequest(action, (*fit)->name())));
-                       else
-                               item.submenu().add(MenuItem(MenuItem::Command, toqstr(label),
-                                       FuncRequest(action, (*fit)->name())));
+                       // note that at this point, we know that buf is not null
+                       LATTEST(buf);
+                       item.submenu().addWithStatusCheck(MenuItem(MenuItem::Command,
+                               toqstr(label), FuncRequest(action, f->name())));
                } else {
                        if (buf)
                                addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(label),
-                                       FuncRequest(action, (*fit)->name())));
+                                       FuncRequest(action, f->name())));
                        else
                                add(MenuItem(MenuItem::Command, toqstr(label),
-                                       FuncRequest(action, (*fit)->name())));
+                                       FuncRequest(action, f->name())));
                }
        }
        if (view_update)
@@ -1206,25 +1230,36 @@ void MenuDefinition::expandFlexInsert(
                if (cit->second.lyxtype() == type) {
                        if (!cit->second.obsoleted_by().empty())
                                continue;
-                       docstring label = cit->first;
+                       docstring name = cit->first;
                        // we remove the "Flex:" prefix, if it is present
-                       if (prefixIs(label, from_ascii("Flex:")))
-                               label = label.substr(5);
+                       if (prefixIs(name, from_ascii("Flex:")))
+                               name = name.substr(5);
+                       docstring const label = (cit->second.menustring().empty()) ?
+                                               name
+                                             : cit->second.menustring();
                        addWithStatusCheck(MenuItem(MenuItem::Command,
                                toqstr(translateIfPossible(label)),
-                               FuncRequest(LFUN_FLEX_INSERT, Lexer::quoteString(label))));
+                               FuncRequest(LFUN_FLEX_INSERT, Lexer::quoteString(name))));
                }
        }
        // FIXME This is a little clunky.
-       if (items_.empty() && type == InsetLayout::CUSTOM && !buf->isReadonly())
+       if (items_.empty() && type == InsetLayout::CUSTOM && !buf->hasReadonlyFlag())
                add(MenuItem(MenuItem::Help, qt_("No Custom Insets Defined!")));
 }
 
 
-size_t const max_number_of_items = 25;
+// Threshold before we stop displaying sub-items alongside items
+// (for display purposes). Ideally this should fit on a screen.
+size_t const max_number_of_items = 30;
+// Size limit for the menu. This is for performance purposes,
+// because qt already displays a scrollable menu when necessary.
+// Ideally this should be the menu size from which scrollable
+// menus become unpractical.
+size_t const menu_size_limit = 80;
 
 void MenuDefinition::expandToc2(Toc const & toc_list,
-               size_t from, size_t to, int depth)
+                                size_t from, size_t to, int depth,
+                                string toc_type)
 {
        int shortcut_count = 0;
 
@@ -1238,7 +1273,7 @@ void MenuDefinition::expandToc2(Toc const & toc_list,
        if (to - from <= max_number_of_items) {
                for (size_t i = from; i < to; ++i) {
                        QString label(4 * max(0, toc_list[i].depth() - depth), ' ');
-                       label += limitStringLength(toc_list[i].str());
+                       label += limitStringLength(toc_list[i].asString());
                        if (toc_list[i].depth() == depth) {
                                label += '|';
                            if (shortcut_count < 9) {
@@ -1248,16 +1283,20 @@ void MenuDefinition::expandToc2(Toc const & toc_list,
                        }
                        add(MenuItem(MenuItem::Command, label,
                                            FuncRequest(toc_list[i].action())));
+                       // separator after the menu heading
+                       if (toc_list[i].depth() < depth)
+                               add(MenuItem(MenuItem::Separator));
                }
        } else {
                size_t pos = from;
+               size_t size = 1;
                while (pos < to) {
                        size_t new_pos = pos + 1;
                        while (new_pos < to && toc_list[new_pos].depth() > depth)
                                ++new_pos;
 
                        QString label(4 * max(0, toc_list[pos].depth() - depth), ' ');
-                       label += limitStringLength(toc_list[pos].str());
+                       label += limitStringLength(toc_list[pos].asString());
                        if (toc_list[pos].depth() == depth) {
                                label += '|';
                            if (shortcut_count < 9) {
@@ -1265,16 +1304,22 @@ void MenuDefinition::expandToc2(Toc const & toc_list,
                                                label += QString::number(++shortcut_count);
                                }
                        }
+                       if (size >= menu_size_limit) {
+                               FuncRequest f(LFUN_DIALOG_SHOW, "toc " + toc_type);
+                               add(MenuItem(MenuItem::Command, "...", f));
+                               break;
+                       }
                        if (new_pos == pos + 1) {
                                add(MenuItem(MenuItem::Command,
                                                    label, FuncRequest(toc_list[pos].action())));
                        } else {
                                MenuDefinition sub;
-                               sub.expandToc2(toc_list, pos, new_pos, depth + 1);
+                               sub.expandToc2(toc_list, pos, new_pos, depth + 1, toc_type);
                                MenuItem item(MenuItem::Submenu, label);
                                item.setSubmenu(sub);
                                add(item);
                        }
+                       ++size;
                        pos = new_pos;
                }
        }
@@ -1287,12 +1332,10 @@ void MenuDefinition::expandToc(Buffer const * buf)
        // all MenuItem constructors and to expandToc2. However, we
        // know that all the entries in a TOC will be have status_ ==
        // OK, so we avoid this unnecessary overhead (JMarc)
-
        if (!buf) {
-               add(MenuItem(MenuItem::Info, qt_("<No Document Open>")));
+               add(MenuItem(MenuItem::Info, qt_("(No Document Open)")));
                return;
        }
-
        // Add an entry for the master doc if this is a child doc
        Buffer const * const master = buf->masterBuffer();
        if (buf != master) {
@@ -1303,37 +1346,22 @@ void MenuDefinition::expandToc(Buffer const * buf)
        }
 
        MenuDefinition other_lists;
-
-       FloatList const & floatlist = buf->params().documentClass().floats();
-       TocList const & toc_list = buf->tocBackend().tocs();
-       TocList::const_iterator cit = toc_list.begin();
-       TocList::const_iterator end = toc_list.end();
-       for (; cit != end; ++cit) {
-               // Handle this later
-               if (cit->first == "tableofcontents")
+       // In the navigation menu, only add tocs from this document
+       TocBackend const & backend = buf->tocBackend();
+       TocList const & toc_list = backend.tocs();
+       for (pair<string, shared_ptr<Toc>> const & toc : toc_list) {
+               // Handle table of contents later
+               if (toc.first == "tableofcontents" || toc.second->empty())
                        continue;
-
                MenuDefinition submenu;
-               if (cit->second.size() >= 30) {
-                       FuncRequest f(LFUN_DIALOG_SHOW, "toc " + cit->first);
-                       submenu.add(MenuItem(MenuItem::Command, qt_("Open Navigator..."), f));
-               } else {
-                       TocIterator ccit = cit->second.begin();
-                       TocIterator eend = cit->second.end();
-                       for (; ccit != eend; ++ccit) {
-                               submenu.add(MenuItem(MenuItem::Command,
-                                       limitStringLength(ccit->str()) + '|',
-                                       FuncRequest(ccit->action())));
-                       }
-               }
-
-               MenuItem item(MenuItem::Submenu, guiName(cit->first, buf->params()));
+               submenu.expandTocSubmenu(toc.first, *toc.second);
+               docstring const toc_name = backend.outlinerName(toc.first);
+               MenuItem item(MenuItem::Submenu, toqstr(toc_name));
                item.setSubmenu(submenu);
-               if (floatlist.typeExist(cit->first) || cit->first == "child") {
-                       // Those two types deserve to be in the main menu.
-                       item.setSubmenu(submenu);
+               // deserves to be in the main menu?
+               if (!TocBackend::isOther(toc.first))
                        add(item);
-               else
+               else
                        other_lists.add(item);
        }
        if (!other_lists.empty()) {
@@ -1341,20 +1369,32 @@ void MenuDefinition::expandToc(Buffer const * buf)
                item.setSubmenu(other_lists);
                add(item);
        }
-
        // Handle normal TOC
-       cit = toc_list.find("tableofcontents");
-       if (cit == end)
+       add(MenuItem(MenuItem::Separator));
+       TocList::const_iterator cit = toc_list.find("tableofcontents");
+       if (cit == toc_list.end())
                LYXERR(Debug::GUI, "No table of contents.");
        else {
-               if (!cit->second.empty())
-                       expandToc2(cit->second, 0, cit->second.size(), 0);
+               if (!cit->second->empty())
+                       expandToc2(*cit->second, 0, cit->second->size(), 0,
+                                  "tableofcontents");
                else
-                       add(MenuItem(MenuItem::Info, qt_("<Empty Table of Contents>")));
+                       add(MenuItem(MenuItem::Info, qt_("(Empty Table of Contents)")));
        }
 }
 
 
+void MenuDefinition::expandTocSubmenu(std::string const & type, Toc const & toc)
+{
+       // "Open outliner..." entry
+       FuncRequest f(LFUN_DIALOG_SHOW, "toc " + type);
+       add(MenuItem(MenuItem::Command, qt_("Open Outliner..."), f));
+       add(MenuItem(MenuItem::Separator));
+       // add entries
+       expandToc2(toc, 0, toc.size(), 0, type);
+}
+
+
 void MenuDefinition::expandPasteRecent(Buffer const * buf)
 {
        docstring_list const sel = cap::availableSelections(buf);
@@ -1394,54 +1434,62 @@ void MenuDefinition::expandToolbars()
 
 void MenuDefinition::expandBranches(Buffer const * buf)
 {
-       if (!buf || buf->isReadonly())
+       if (!buf || buf->hasReadonlyFlag())
                return;
 
-       BufferParams const & master_params = buf->masterBuffer()->params();
-       BufferParams const & params = buf->params();
-       if (params.branchlist().empty() && master_params.branchlist().empty() ) {
-               add(MenuItem(MenuItem::Help, qt_("No Branches Set for Document!")));
-               return;
-       }
-
-       BranchList::const_iterator cit = master_params.branchlist().begin();
-       BranchList::const_iterator end = master_params.branchlist().end();
+       BranchList const & child_list = buf->params().branchlist();
+       set<docstring> brset;
+       int ii = 1;
+       for (auto const & b : child_list) {
+               docstring const & bname = b.branch();
+               // NUM. Branch Name + "|", which triggers an empty shortcut in
+               // case that character should be in the branch name.
+               docstring label = convert<docstring>(ii) + ". " + bname + char_type('|');
+               // Add NUM as a keyboard shortcut if it's a single digit
+               if (ii < 10)
+                       label += convert<docstring>(ii);
 
-       for (int ii = 1; cit != end; ++cit, ++ii) {
-               docstring label = cit->branch();
-               if (ii < 10) {
-                       label = convert<docstring>(ii) + ". " + label
-                               + char_type('|') + convert<docstring>(ii);
-               }
                addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(label),
-                                   FuncRequest(LFUN_BRANCH_INSERT,
-                                               cit->branch())));
+                       FuncRequest(LFUN_BRANCH_INSERT, bname)));
+
+               brset.insert(bname);
+               ++ii;
        }
 
-       if (buf == buf->masterBuffer())
-               return;
+       set<Buffer const *> bufset;
+       Buffer const * nextbuf = buf->parent();
+       MenuDefinition master_branches;
+       ii = 1;
+       while (nextbuf) {
+               // recursive includes are a bad idea, but let's not crash
+               // if we find one.
+               if (bufset.count(nextbuf))
+                       break;
+
+               for (auto const & b : nextbuf->params().branchlist()) {
+                       docstring const & bname = b.branch();
+                       // do not add it if we've already seen it
+                       if (brset.count(bname))
+                               continue;
 
-       MenuDefinition child_branches;
+                       docstring label = convert<docstring>(ii) + ". " + bname + char_type('|');
+                       if (ii < 10)
+                               label += convert<docstring>(ii);
 
-       BranchList::const_iterator ccit = params.branchlist().begin();
-       BranchList::const_iterator cend = params.branchlist().end();
+                       master_branches.addWithStatusCheck(MenuItem(MenuItem::Command,
+                               toqstr(label), FuncRequest(LFUN_BRANCH_INSERT, bname)));
 
-       for (int ii = 1; ccit != cend; ++ccit, ++ii) {
-               docstring label = ccit->branch();
-               if (ii < 10) {
-                       label = convert<docstring>(ii) + ". " + label
-                               + char_type('|') + convert<docstring>(ii);
-               } else
-                       label += char_type('|');
-               child_branches.addWithStatusCheck(MenuItem(MenuItem::Command,
-                                   toqstr(label),
-                                   FuncRequest(LFUN_BRANCH_INSERT,
-                                               ccit->branch())));
+                       bufset.insert(nextbuf);
+                       brset.insert(bname);
+                       ++ii;
+               }
+
+               nextbuf = nextbuf->parent();
        }
 
-       if (!child_branches.empty()) {
-               MenuItem item(MenuItem::Submenu, qt_("Child Document"));
-               item.setSubmenu(child_branches);
+       if (!master_branches.empty()) {
+               MenuItem item(MenuItem::Submenu, qt_("Master Documents"));
+               item.setSubmenu(master_branches);
                add(item);
        }
 }
@@ -1531,10 +1579,11 @@ void MenuDefinition::expandCiteStyles(BufferView const * bv)
                                    FuncRequest(LFUN_NOACTION)));
                return;
        }
-       InsetCommand const * citinset =
-                               static_cast<InsetCommand const *>(inset);
+       InsetCitation const * citinset =
+                               static_cast<InsetCitation const *>(inset);
 
        Buffer const * buf = &bv->buffer();
+       BufferParams const & bp = buf->masterParams();
        string const cmd = citinset->params().getCmdName();
 
        docstring const & key = citinset->getParam("key");
@@ -1545,36 +1594,127 @@ void MenuDefinition::expandCiteStyles(BufferView const * bv)
                return;
        }
 
-       docstring const & before = citinset->getParam("before");
-       docstring const & after = citinset->getParam("after");
-
        size_t const n = cmd.size();
-       bool const force = cmd[0] == 'C';
-       bool const full = cmd[n] == '*';
+       bool const force = isUpperCase(cmd[0]);
+       bool const star = cmd[n] == '*';
 
        vector<docstring> const keys = getVectorFromString(key);
 
-       vector<CitationStyle> const citeStyleList = buf->params().citeStyles();
-       static const size_t max_length = 40;
-       vector<docstring> citeStrings =
-               buf->masterBibInfo().getCiteStrings(keys, citeStyleList, bv->buffer(),
-               before, after, from_utf8("dialog"), max_length);
-
-       vector<docstring>::const_iterator cit = citeStrings.begin();
-       vector<docstring>::const_iterator end = citeStrings.end();
+       vector<CitationStyle> const citeStyleList = bp.citeStyles();
+
+       CitationStyle cs = citinset->getCitationStyle(bp, cmd, citeStyleList);
+       bool const qualified = cs.hasQualifiedList
+               && (keys.size() > 1
+                   || !citinset->getParam("pretextlist").empty()
+                   || !citinset->getParam("posttextlist").empty());
+       std::map<docstring, docstring> pres =
+               citinset->getQualifiedLists(citinset->getParam("pretextlist"));
+       std::map<docstring, docstring> posts =
+               citinset->getQualifiedLists(citinset->getParam("posttextlist"));
+
+       CiteItem ci;
+       ci.textBefore = citinset->getParam("before");
+       ci.textAfter = citinset->getParam("after");
+       ci.forceUpperCase = force;
+       ci.Starred = star;
+       ci.context = CiteItem::Dialog;
+       ci.max_size = 40;
+       ci.isQualified = qualified;
+       ci.pretexts = pres;
+       ci.posttexts = posts;
+       BiblioInfo::CiteStringMap citeStrings =
+               buf->masterBibInfo().getCiteStrings(keys, citeStyleList, bv->buffer(), ci);
+
+       BiblioInfo::CiteStringMap::const_iterator cit = citeStrings.begin();
+       BiblioInfo::CiteStringMap::const_iterator end = citeStrings.end();
 
        for (int ii = 1; cit != end; ++cit, ++ii) {
-               docstring label = *cit;
-               CitationStyle cs = citeStyleList[ii - 1];
-               cs.forceUpperCase &= force;
-               cs.fullAuthorList &= full;
+               docstring label = cit->second;
+               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) {
+               docstring starred = _("All authors|h");
+               // Check if we have a custom string/tooltip for the starred version
+               if (!cs.stardesc.empty()) {
+                       string val =
+                               bp.documentClass().getCiteMacro(buf->params().citeEngineType(), cs.stardesc);
+                       if (!val.empty())
+                               starred = translateIfPossible(from_utf8(val));
+                       // Transform qt-style accelerators to menu-style
+                       int const amps = count_char(starred, '&');
+                       if (amps > 0) {
+                               if (amps > 1)
+                                       starred = subst(starred, from_ascii("&&"), from_ascii("<:amp:>"));
+                               size_t nn = starred.find('&');
+                               char_type accel = char_type();
+                               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("&&"));
+                               if (accel != char_type())
+                                       starred = starred + '|' + accel;
+                       }
+               }
+               add(MenuItem(MenuItem::Separator));
+               addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(starred),
+                                   FuncRequest(LFUN_INSET_MODIFY, "toggleparam star")));
+       }
+
+       if (cs.forceUpperCase) {
+               if (!cs.hasStarredVersion)
+                       add(MenuItem(MenuItem::Separator));
+               addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(_("Force upper case|u")),
+                                   FuncRequest(LFUN_INSET_MODIFY, "toggleparam casing")));
        }
 }
 
 
+void MenuDefinition::expandInfoArguments(BufferView const * bv)
+{
+       if (!bv)
+               return;
+
+       Inset const * inset = bv->cursor().nextInset();
+       if (!inset || inset->lyxCode() != INFO_CODE) {
+               add(MenuItem(MenuItem::Command,
+                                   qt_("No Text Field in Scope!"),
+                                   FuncRequest(LFUN_NOACTION)));
+               return;
+       }
+       InsetInfo const * iinset = static_cast<InsetInfo const *>(inset);
+
+       string const type = iinset->params().infoType();
+       vector<pair<string,docstring>> const args =
+                       iinset->params().getArguments(&bv->buffer(), type);
+
+       // Don't generate a menu for big lists (such as lfuns and rcs)
+       if (args.size() > 15)
+               return;
+
+       for (auto const & p : args) {
+               if (p.first == "invalid")
+                       // non-selectable
+                       continue;
+               if (p.first == "custom") {
+                       add(MenuItem(MenuItem::Command, qt_("Custom..."),
+                                    FuncRequest(LFUN_INSET_SETTINGS, "info")));
+                       continue;
+               }
+               docstring label = p.second;
+               addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(label),
+                                   FuncRequest(LFUN_INSET_MODIFY, type + " " + p.first)));
+       }
+}
+
+
+
 void MenuDefinition::expandArguments(BufferView const * bv, bool switcharg)
 {
        if (!bv)
@@ -1613,47 +1753,37 @@ void MenuDefinition::expandCaptions(Buffer const * buf, bool switchcap)
        if (!buf)
                return;
 
-       vector<docstring> caps;
        DocumentClass const & dc = buf->params().documentClass();
-       TextClass::InsetLayouts::const_iterator lit = dc.insetLayouts().begin();
-       TextClass::InsetLayouts::const_iterator len = dc.insetLayouts().end();
-       for (; lit != len; ++lit) {
-               if (prefixIs(lit->first, from_ascii("Caption:")))
-                       caps.push_back(lit->first);
+       vector< pair<docstring, FuncRequest> > caps;
+       for (pair<docstring, InsetLayout> const & il : dc.insetLayouts()) {
+               docstring instype;
+               docstring const type = split(il.first, instype, ':');
+               if (instype == from_ascii("Caption")) {
+                       // skip forbidden caption types
+                       FuncRequest const cmd = switchcap
+                               ? FuncRequest(LFUN_INSET_MODIFY, from_ascii("changetype ") + type)
+                               : FuncRequest(LFUN_CAPTION_INSERT, type);
+                       if (getStatus(cmd).enabled())
+                               caps.push_back(make_pair(type, cmd));
+               }
        }
 
        if (caps.empty() || (switchcap && caps.size() == 1))
                return;
        if (caps.size() == 1) {
-               docstring dummy;
-               docstring const type = split(*caps.begin(), dummy, ':');
-               add(MenuItem(MenuItem::Command, qt_("Caption"),
-                        FuncRequest(LFUN_CAPTION_INSERT, translateIfPossible(type))));
+               add(MenuItem(MenuItem::Command, qt_("Caption"), caps.front().second));
                return;
        }
 
        MenuDefinition captions;
-
-       vector<docstring>::const_iterator cit = caps.begin();
-       vector<docstring>::const_iterator end = caps.end();
-
-       for (int ii = 1; cit != end; ++cit, ++ii) {
-               docstring dummy;
-               docstring const type = split(*cit, dummy, ':');
+       for (pair<docstring, FuncRequest> const & cap : caps) {
+               docstring const type = cap.first;
                docstring const trtype = translateIfPossible(type);
                docstring const cmitem = bformat(_("Caption (%1$s)"), trtype);
-               // make menu item optional, otherwise we would also see
-               // forbidden caption types
                if (switchcap)
-                       addWithStatusCheck(MenuItem(MenuItem::Command, toqstr(cmitem),
-                                    FuncRequest(LFUN_INSET_MODIFY,
-                                                from_ascii("changetype ")
-                                                + type), QString(), true));
+                       add(MenuItem(MenuItem::Command, toqstr(cmitem), cap.second));
                else
-                       captions.addWithStatusCheck(MenuItem(MenuItem::Command,
-                                                            toqstr(trtype),
-                                                            FuncRequest(LFUN_CAPTION_INSERT,
-                                                            type), QString(), true));
+                       captions.add(MenuItem(MenuItem::Command, toqstr(trtype), cap.second));
        }
        if (!captions.empty()) {
                MenuItem item(MenuItem::Submenu, qt_("Caption"));
@@ -1663,7 +1793,130 @@ void MenuDefinition::expandCaptions(Buffer const * buf, bool switchcap)
 }
 
 
-void MenuDefinition::expandEnvironmentSeparators(BufferView const * bv)
+void MenuDefinition::expandQuotes(BufferView const * bv)
+{
+       if (!bv)
+               return;
+
+       if (!bv->cursor().inTexted())
+               return;
+
+       Inset const * inset = bv->cursor().nextInset();
+       if (!inset || inset->lyxCode() != QUOTE_CODE) {
+               add(MenuItem(MenuItem::Command,
+                                   qt_("No Quote in Scope!"),
+                                   FuncRequest(LFUN_NOACTION)));
+               return;
+       }
+       InsetQuotes const * qinset =
+               static_cast<InsetQuotes const *>(inset);
+
+       map<string, docstring> styles = quoteparams.getTypes();
+       string const qtype = qinset->getType();
+
+       map<string, docstring>::const_iterator qq = styles.begin();
+       map<string, docstring>::const_iterator end = styles.end();
+
+       MenuDefinition aqs;
+
+       BufferParams const & bp = bv->buffer().masterBuffer()->params();
+
+       // The global setting
+       InsetQuotesParams::QuoteStyle globalqs = bp.quotes_style;
+       char const globalqsc = quoteparams.getStyleChar(globalqs);
+
+       // The current language's default
+       InsetQuotesParams::QuoteStyle langdefqs =
+               bp.getQuoteStyle(bv->cursor().current_font.language()->quoteStyle());
+       char const langqs = quoteparams.getStyleChar(langdefqs);
+
+       bool main_global_qs = false;
+       bool main_langdef_qs = false;
+       bool main_dynamic_qs = false;
+       docstring const subcmd = from_ascii("changetype ");
+       docstring const wildcards = from_ascii("..");
+       // Add the items
+       // First the top level menu (all glyphs of the current style) ...
+       // Begin with dynamic (if they are current style),
+       if (qtype[0] == 'x') {
+               FuncRequest cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + from_ascii("xld"));
+               docstring desc = bformat(_("%1$s (dynamic)"),
+                       quoteparams.getShortGuiLabel(globalqsc + from_ascii("ld")));
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+               cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + from_ascii("xls"));
+               desc = bformat(_("%1$s (dynamic)"),
+                       quoteparams.getShortGuiLabel(globalqsc + from_ascii("ls")));
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+               cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + from_ascii("xrd"));
+               desc = bformat(_("%1$s (dynamic)"),
+                       quoteparams.getShortGuiLabel(globalqsc + from_ascii("rd")));
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+               cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + from_ascii("xrs"));
+               desc = bformat(_("%1$s (dynamic)"),
+                       quoteparams.getShortGuiLabel(globalqsc + from_ascii("rs")));
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+               main_dynamic_qs = true;
+       }
+       // now traverse through the static styles ...
+       for (; qq != end; ++qq) {
+               docstring const style = from_ascii(qq->first);
+               bool langdef = (style[0] == langqs);
+               bool globaldef = (style[0] == globalqsc);
+
+               if (prefixIs(style, qtype[0])) {
+                       FuncRequest cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + style);
+                       docstring const desc = quoteparams.getShortGuiLabel(style);
+                       add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+                       main_global_qs = globaldef;
+                       main_langdef_qs = langdef;
+               }
+               else if (!langdef && !globaldef && suffixIs(style, from_ascii("ld"))) {
+                       docstring const desc =
+                               quoteparams.getGuiLabel(quoteparams.getQuoteStyle(to_ascii(style)));
+                       FuncRequest cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + style[0] + "..");
+                       aqs.add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+               }
+       }
+
+       add(MenuItem(MenuItem::Separator));
+
+       bool display_static = false;
+       // ... then potentially items to reset to the defaults and to dynamic style ...
+       if (!main_dynamic_qs && globalqsc != 'x') {
+               FuncRequest cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + 'x' + wildcards);
+               docstring const desc = bformat(_("Use dynamic quotes (%1$s)|d"),
+                                               quoteparams.getGuiLabel(globalqs));
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+               display_static = true;
+       }
+       if (!main_global_qs && langdefqs != globalqs) {
+               docstring const variant = main_dynamic_qs ? _("dynamic[[Quotes]]") : _("static[[Quotes]]");
+               FuncRequest cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + globalqsc + wildcards);
+               docstring const desc = bformat(_("Reset to document default (%1$s, %2$s)|o"),
+                                               quoteparams.getGuiLabel(globalqs), variant);
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+       }
+       if (!main_langdef_qs) {
+               FuncRequest cmd = FuncRequest(LFUN_INSET_MODIFY, subcmd + globalqsc + wildcards);
+               docstring const desc = (main_dynamic_qs || display_static)
+                                       ? bformat(_("Reset to language default (%1$s, %2$s)|l"),
+                                                 quoteparams.getGuiLabel(langdefqs), _("static[[Quotes]]"))
+                                       : bformat(_("Reset to language default (%1$s)|l"),
+                                                 quoteparams.getGuiLabel(langdefqs));
+               add(MenuItem(MenuItem::Command, toqstr(desc), cmd));
+       }
+
+       add(MenuItem(MenuItem::Separator));
+
+       // ... and a subitem with the rest
+       MenuItem item(MenuItem::Submenu, qt_("Change Style|y"));
+       item.setSubmenu(aqs);
+       add(item);
+}
+
+
+void MenuDefinition::expandEnvironmentSeparators(BufferView const * bv,
+                                                bool contextmenu)
 {
        if (!bv)
                return;
@@ -1676,38 +1929,82 @@ void MenuDefinition::expandEnvironmentSeparators(BufferView const * bv)
        Paragraph const & par = text->getPar(pit);
        docstring const curlayout = par.layout().name();
        docstring outerlayout;
+       docstring prevlayout;
        depth_type current_depth = par.params().depth();
-       // check if we have an environment in our nesting hierarchy
+       // check if we have an environment in our scope
        Paragraph cpar = par;
        while (true) {
-               if (pit == 0 || cpar.params().depth() == 0)
+               if (pit == 0)
                        break;
                --pit;
                cpar = text->getPar(pit);
+               if (cpar.layout().isEnvironment() && prevlayout.empty()
+                   && cpar.params().depth() <= current_depth)
+                               prevlayout = cpar.layout().name();
                if (cpar.params().depth() < current_depth
                    && cpar.layout().isEnvironment()) {
                                outerlayout = cpar.layout().name();
                                current_depth = cpar.params().depth();
                }
+               if (cpar.params().depth() == 0)
+                       break;
        }
        if (par.layout().isEnvironment()) {
-               docstring const label =
-                       bformat(_("Start 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)));
+                            FuncRequest(LFUN_ENVIRONMENT_SPLIT,
+                                        from_ascii("before"))));
+               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 = contextmenu ?
+                       bformat(_("Insert Separated %1$s Below"),
+                               translateIfPossible(prevlayout)) :
+                       bformat(_("Separated %1$s Below"),
+                               translateIfPossible(prevlayout));
+               // 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(_("Start 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"))));
        }
 }
 
-} // namespace anon
+} // namespace
 
 
 /////////////////////////////////////////////////////////////////////
@@ -1718,7 +2015,7 @@ struct Menu::Impl
 {
        /// populates the menu or one of its submenu
        /// This is used as a recursive function
-       void populate(QMenu & qMenu, MenuDefinition const & menu);
+       void populate(QMenu * qMenu, MenuDefinition const & menu);
 
        /// Only needed for top level menus.
        MenuDefinition * top_level_menu;
@@ -1751,7 +2048,7 @@ static QString label(MenuItem const & mi)
        return label;
 }
 
-void Menu::Impl::populate(QMenu & qMenu, MenuDefinition const & menu)
+void Menu::Impl::populate(QMenu * qMenu, MenuDefinition const & menu)
 {
        LYXERR(Debug::GUI, "populating menu " << menu.name());
        if (menu.empty()) {
@@ -1759,21 +2056,26 @@ void Menu::Impl::populate(QMenu & qMenu, MenuDefinition const & menu)
                return;
        }
        LYXERR(Debug::GUI, " *****  menu entries " << menu.size());
-       MenuDefinition::const_iterator m = menu.begin();
-       MenuDefinition::const_iterator end = menu.end();
-       for (; m != end; ++m) {
-               if (m->kind() == MenuItem::Separator)
-                       qMenu.addSeparator();
-               else if (m->kind() == MenuItem::Submenu) {
-                       QMenu * subMenu = qMenu.addMenu(label(*m));
-                       populate(*subMenu, m->submenu());
-                       subMenu->setEnabled(m->status().enabled());
-               } else {
-                       // we have a MenuItem::Command
-                       qMenu.addAction(new Action(QIcon(), label(*m),
-                               m->func(), m->tooltip(), &qMenu));
+       for (MenuItem const & m : menu)
+               switch (m.kind()) {
+               case MenuItem::Separator:
+                       qMenu->addSeparator();
+                       break;
+               case MenuItem::Submenu: {
+                       QMenu * subMenu = qMenu->addMenu(label(m));
+                       populate(subMenu, m.submenu());
+                       subMenu->setEnabled(!subMenu->isEmpty());
+                       break;
+               }
+               case MenuItem::Command:
+               default:
+                       // FIXME: A previous comment assured that MenuItem::Command was the
+                       // only possible case in practice, but this is wrong.  It would be
+                       // good to document which cases are actually treated here.
+                       qMenu->addAction(new Action(m.func(), QIcon(), label(m),
+                                                   m.tooltip(), qMenu));
+                       break;
                }
-       }
 }
 
 #if (defined(Q_OS_WIN) || defined(Q_CYGWIN_WIN)) && (QT_VERSION >= 0x040600)
@@ -1931,7 +2233,7 @@ void Menus::Impl::macxMenuBarInit(QMenuBar * qmb)
                QAction::MenuRole role;
        };
 
-       static MacMenuEntry entries[] = {
+       static const MacMenuEntry entries[] = {
                {LFUN_DIALOG_SHOW, "aboutlyx", "About LyX",
                 QAction::AboutRole},
                {LFUN_DIALOG_SHOW, "prefs", "Preferences",
@@ -1960,13 +2262,13 @@ void Menus::Impl::macxMenuBarInit(QMenuBar * qmb)
        // add the entries to a QMenu that will eventually be empty
        // and therefore invisible.
        QMenu * qMenu = qmb->addMenu("special");
-       MenuDefinition::const_iterator cit = mac_special_menu_.begin();
-       MenuDefinition::const_iterator end = mac_special_menu_.end();
-       for (size_t i = 0 ; cit != end ; ++cit, ++i) {
-               Action * action = new Action(QIcon(), cit->label(),
-                       cit->func(), QString(), qMenu);
+       size_t i = 0;
+       for (MenuItem const & m : mac_special_menu_) {
+               Action * action = new Action(m.func(), QIcon(), m.label(),
+                                            QString(), qMenu);
                action->setMenuRole(entries[i].role);
                qMenu->addAction(action);
+               ++i;
        }
 }
 
@@ -2000,6 +2302,19 @@ void Menus::Impl::expand(MenuDefinition const & frommenu,
                        tomenu.expandFormats(cit->kind(), buf);
                        break;
 
+               case MenuItem::ExportFormat: {
+                       if (!buf)
+                               break;
+                       string const format = buf->params().getDefaultOutputFormat();
+                       Format const * f = theFormats().getFormat(format);
+                       docstring const name = f ? f->prettyname() : from_utf8(format);
+                       docstring const label = bformat(_("Export [%1$s]|E"), name);
+                       MenuItem item(MenuItem::Command, toqstr(label),
+                                     FuncRequest(LFUN_BUFFER_EXPORT));
+                       tomenu.addWithStatusCheck(item);
+                       break;
+               }
+
                case MenuItem::CharStyles:
                        tomenu.expandFlexInsert(buf, InsetLayout::CHARSTYLE);
                        break;
@@ -2052,6 +2367,11 @@ void Menus::Impl::expand(MenuDefinition const & frommenu,
                        tomenu.expandCiteStyles(bv);
                        break;
 
+               case MenuItem::InfoArguments:
+                       tomenu.expandInfoArguments(bv);
+                       break;
+                       
+
                case MenuItem::Toc:
                        tomenu.expandToc(buf);
                        break;
@@ -2088,6 +2408,14 @@ 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;
+
                case MenuItem::Submenu: {
                        MenuItem item(*cit);
                        item.setSubmenu(MenuDefinition(cit->submenuname()));
@@ -2103,7 +2431,7 @@ void Menus::Impl::expand(MenuDefinition const & frommenu,
                        break;
 
                case MenuItem::Command:
-                       if (!mac_special_menu_.hasFunc(cit->func()))
+                       if (!mac_special_menu_.hasFunc(*cit->func()))
                                tomenu.addWithStatusCheck(*cit);
                }
        }
@@ -2223,10 +2551,10 @@ void Menus::read(Lexer & lex)
 
 
 bool Menus::searchMenu(FuncRequest const & func,
-       docstring_list & names) const
+       docstring_list & names, BufferView const * bv) const
 {
        MenuDefinition menu;
-       d->expand(d->menubar_, menu, 0);
+       d->expand(d->menubar_, menu, bv);
        return menu.searchMenu(func, names);
 }
 
@@ -2284,19 +2612,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;
        }
 }
 
@@ -2342,7 +2670,7 @@ void Menus::updateMenu(Menu * qmenu)
        if (qmenu->d->view)
                bv = qmenu->d->view->currentBufferView();
        d->expand(fromLyxMenu, *qmenu->d->top_level_menu, bv);
-       qmenu->d->populate(*qmenu, *qmenu->d->top_level_menu);
+       qmenu->d->populate(qmenu, *qmenu->d->top_level_menu);
 }