Circumvent odd stmary font metrics (part of #9990).

[lyx.git] / src / insets / InsetArgument.cpp

diff --git a/src/insets/InsetArgument.cpp b/src/insets/InsetArgument.cpp
index 2e4285e7fef5ca48f341717e3080ed0f3420fdf1..48551db1f528e94f1d6becdc8d38414a188206d1 100644 (file)
--- a/src/insets/InsetArgument.cpp
+++ b/src/insets/InsetArgument.cpp
@@ -39,7 +39,7 @@ namespace lyx {
 InsetArgument::InsetArgument(Buffer * buf, string const & name)
     : InsetCollapsable(buf), name_(name), labelstring_(docstring()),
       font_(inherit_font), labelfont_(inherit_font), decoration_(string()),
-      pass_thru_(false)
+      pass_thru_(false), pass_thru_chars_(docstring())
 {}
 
 
@@ -49,22 +49,24 @@ void InsetArgument::write(ostream & os) const
        InsetCollapsable::write(os);
 }
 
+
 void InsetArgument::read(Lexer & lex)
 {
        lex >> name_;
        InsetCollapsable::read(lex);
 }
 
+
 void InsetArgument::updateBuffer(ParIterator const & it, UpdateType utype)
 {
        Layout::LaTeXArgMap args = it.paragraph().layout().args();
        pass_thru_ = it.paragraph().layout().pass_thru;
-       bool const insetlayout = &it.inset() && args.empty();
+       bool const insetlayout = args.empty();
        if (insetlayout) {
                args = it.inset().getLayout().args();
                pass_thru_ = it.inset().getLayout().isPassThru();
        }
-       
+
        // Handle pre 2.1 ArgInsets (lyx2lyx cannot classify them)
        if (name_ == "999") {
                unsigned int const req = insetlayout ? it.inset().getLayout().requiredArgs()
@@ -110,6 +112,7 @@ void InsetArgument::updateBuffer(ParIterator const & it, UpdateType utype)
                font_ = (*lait).second.font;
                labelfont_ = (*lait).second.labelfont;
                decoration_ = (*lait).second.decoration;
+               pass_thru_chars_ = (*lait).second.pass_thru_chars;
        } else {
                labelstring_ = _("Unknown Argument");
                tooltip_ = _("Argument not known in this Layout. Will be supressed in the output.");
@@ -118,11 +121,13 @@ void InsetArgument::updateBuffer(ParIterator const & it, UpdateType utype)
        InsetCollapsable::updateBuffer(it, utype);
 }
 
+
 void InsetArgument::setButtonLabel()
 {
        setLabel(labelstring_);
 }
 
+
 docstring InsetArgument::toolTip(BufferView const & bv, int, int) const
 {
        if (isOpen(bv))
@@ -130,6 +135,7 @@ docstring InsetArgument::toolTip(BufferView const & bv, int, int) const
        return toolTipText(tooltip_ + from_ascii(":\n"));
 }
 
+
 void InsetArgument::doDispatch(Cursor & cur, FuncRequest & cmd)
 {
        switch (cmd.action()) {
@@ -143,7 +149,7 @@ void InsetArgument::doDispatch(Cursor & cur, FuncRequest & cmd)
                        cur.undispatched();
                        return;
                }
-               cur.recordUndoInset(ATOMIC_UNDO, this);
+               cur.recordUndoInset(this);
                name_ = cmd.getArg(1);
                cur.forceBufferUpdate();
                break;
@@ -157,7 +163,7 @@ void InsetArgument::doDispatch(Cursor & cur, FuncRequest & cmd)
                // with (inherited) pass_thru to avoid call for
                // fixParagraphsFont(), which does not play nicely with
                // inherited pass_thru (see #8471).
-               // FIXME: Once we have implemented genuine pass_thru 
+               // FIXME: Once we have implemented genuine pass_thru
                // option for InsetArgument (not inherited pass_thru),
                // we should probably directly call
                // InsetCollapsable::doDispatch(cur, cmd) for that
@@ -190,7 +196,7 @@ bool InsetArgument::getStatus(Cursor & cur, FuncRequest const & cmd,
                                return true;
                        }
                        Layout::LaTeXArgMap args;
-                       bool const insetlayout = &cur.inset() && cur.paragraph().layout().latexargs().empty();
+                       bool const insetlayout = cur.paragraph().layout().latexargs().empty();
                        if (insetlayout)
                                args = cur.inset().getLayout().latexargs();
                        else
@@ -223,11 +229,13 @@ bool InsetArgument::getStatus(Cursor & cur, FuncRequest const & cmd,
        }
 }
 
+
 string InsetArgument::contextMenuName() const
 {
        return "context-argument";
 }
 
+
 FontInfo InsetArgument::getFont() const
 {
        if (font_ != inherit_font)
@@ -235,6 +243,7 @@ FontInfo InsetArgument::getFont() const
        return getLayout().font();
 }
 
+
 FontInfo InsetArgument::getLabelfont() const
 {
        if (labelfont_ != inherit_font)
@@ -242,6 +251,7 @@ FontInfo InsetArgument::getLabelfont() const
        return getLayout().labelfont();
 }
 
+
 InsetLayout::InsetDecoration InsetArgument::decoration() const
 {
        InsetLayout::InsetDecoration dec = getLayout().decoration();
@@ -250,6 +260,7 @@ InsetLayout::InsetDecoration InsetArgument::decoration() const
        return dec == InsetLayout::DEFAULT ? InsetLayout::CLASSIC : dec;
 }
 
+
 void InsetArgument::latexArgument(otexstream & os,
                OutputParams const & runparams_in, docstring const & ldelim,
                docstring const & rdelim, docstring const & presetarg) const
@@ -258,6 +269,8 @@ void InsetArgument::latexArgument(otexstream & os,
        odocstringstream ss;
        otexstream ots(ss, texrow);
        OutputParams runparams = runparams_in;
+       if (!pass_thru_chars_.empty())
+               runparams.pass_thru_chars += pass_thru_chars_;
        InsetText::latex(ots, runparams);
        docstring str = ss.str();
        docstring const sep = str.empty() ? docstring() : from_ascii(", ");