]> git.lyx.org Git - lyx.git/blobdiff - src/mathed/InsetMathHull.cpp
typo
[lyx.git] / src / mathed / InsetMathHull.cpp
index 24e4397490573b0e117da11cd4a44b956efd2cd1..97d95da03b3876372634d41e2f5113ef15e79b09 100644 (file)
@@ -11,7 +11,6 @@
 #include <config.h>
 
 #include "InsetMathHull.h"
-
 #include "InsetMathChar.h"
 #include "InsetMathColor.h"
 #include "InsetMathFrac.h"
@@ -480,12 +479,9 @@ void InsetMathHull::metrics(MetricsInfo & mi, Dimension & dim) const
 {
        /* Compute \(above|below)displayskip
           true value in LaTeX is 10pt plus 2pt minus 5pt (in normal size at 10pt)
-          FIXME: make this dependent of current size? (minor improvement)
-          FIXME: if would be nice if this was not part of the inset, but
-                 just increased the row ascent/descent.
-          FIXME: even better would be to handle the short skip case.
+          But we use a fixed number of pixels and scale them with zoom.
        */
-       int const bottom_display_margin = mi.base.inPixels(Length(10, Length::PT));
+       int const bottom_display_margin = mi.base.bv->zoomedPixels(6);
        int top_display_margin = bottom_display_margin;
        // at start of paragraph, add an empty line
        if (mi.vmode)
@@ -593,13 +589,9 @@ void InsetMathHull::drawBackground(PainterInfo & pi, int x, int y) const
                    dim.wid, dim.asc + dim.des, backgroundColor(pi));
                return;
        }
-       // If there are numbers, the margins around the (displayed)
-       // equation have to be cleared.
-       if (numberedType())
-               pi.pain.fillRectangle(pi.leftx, y - dim.asc,
-                               pi.rightx - pi.leftx, dim.height(), pi.background_color);
+
        pi.pain.fillRectangle(x + 1, y - dim.asc + 1, dim.wid - 2,
-                       dim.asc + dim.des - 1, pi.backgroundColor(this));
+                             dim.height() - 1, pi.backgroundColor(this));
 }
 
 
@@ -630,11 +622,6 @@ void InsetMathHull::draw(PainterInfo & pi, int x, int y) const
        }
 
        // First draw the numbers
-       ColorCode color = pi.selected && lyxrc.use_system_colors
-                               ? Color_selectiontext : standardColor();
-       bool const really_change_color = pi.base.font.color() == Color_none;
-       Changer dummy0 = really_change_color ? pi.base.font.changeColor(color)
-               : noChange();
        if (numberedType()) {
                BufferParams::MathNumber const math_number = buffer().params().getMathNumber();
                for (row_type row = 0; row < nrows(); ++row) {
@@ -643,18 +630,37 @@ void InsetMathHull::draw(PainterInfo & pi, int x, int y) const
                        Dimension dimnl;
                        mathed_string_dim(pi.base.font, nl, dimnl);
                        if (math_number == BufferParams::LEFT) {
+                               ColorCode const col = pi.selected_left
+                                       ? Color_selectiontext
+                                       : pi.base.font.color();
+                               Changer dummy0 = pi.base.font.changeColor(col);
                                if (dimnl.wid > x - pi.leftx)
                                        yy += rowinfo(row).descent + dimnl.asc;
+                               pi.pain.fillRectangle(pi.leftx, yy - dimnl.asc,
+                                       dimnl.width(), dimnl.height(),
+                                       pi.selected_left ? Color_selection : pi.background_color);
                                pi.draw(pi.leftx, yy, nl);
                        } else {
+                               ColorCode const col = pi.selected_right
+                                       ? Color_selectiontext
+                                       : pi.base.font.color();
+                               Changer dummy0 = pi.base.font.changeColor(col);
                                if (dimnl.wid > pi.rightx - x - dim.wid)
                                        yy += rowinfo(row).descent + dimnl.asc;
+                               pi.pain.fillRectangle(pi.rightx - dimnl.wid, yy - dimnl.asc,
+                                       dimnl.width(), dimnl.height(),
+                                       pi.selected_right ? Color_selection : pi.background_color);
                                pi.draw(pi.rightx - dimnl.wid, yy, nl);
                        }
                }
        }
 
        // Then the equations
+       ColorCode color = pi.selected && lyxrc.use_system_colors
+               ? Color_selectiontext : standardColor();
+       bool const really_change_color = pi.base.font.color() == Color_none;
+       Changer dummy0 = really_change_color ? pi.base.font.changeColor(color)
+               : noChange();
        Changer dummy1 = pi.base.changeFontSet(standardFont());
        Changer dummy2 = pi.base.font.changeStyle(display() ? DISPLAY_STYLE
                                                            : TEXT_STYLE);
@@ -2371,8 +2377,7 @@ int InsetMathHull::plaintext(odocstringstream & os,
 }
 
 
-void InsetMathHull::docbook(XMLStream & xs, OutputParams const & runparams) const
-{
+void InsetMathHull::docbook(XMLStream & xs, OutputParams const & runparams) const {
        // Choose the tag around the MathML equation.
        docstring name;
        bool doCR = false;
@@ -2387,9 +2392,17 @@ void InsetMathHull::docbook(XMLStream & xs, OutputParams const & runparams) cons
        // TODO: recognise \tag from amsmath? This would allow having <equation> with a proper title.
 
        docstring attr;
+
+       bool mathmlNamespaceInline = buffer().params().docbook_mathml_prefix == BufferParams::NoPrefix;
+       if (mathmlNamespaceInline)
+               attr += "xmlns=\"http://www.w3.org/1998/Math/MathML\"";
+
        for (row_type i = 0; i < nrows(); ++i) {
                if (!label(i).empty()) {
-                       attr = "xml:id=\"" + xml::cleanID(label(i)) + "\"";
+                       if (!attr.empty())
+                               attr += " ";
+
+                       attr += "xml:id=\"" + xml::cleanID(label(i)) + "\"";
                        break;
                }
        }
@@ -2401,20 +2414,28 @@ void InsetMathHull::docbook(XMLStream & xs, OutputParams const & runparams) cons
        xs << xml::StartTag(name, attr);
        xs << xml::CR();
 
-       // With DocBook 5, MathML must be within its own namespace; defined in Buffer.cpp::writeDocBookSource as "m".
+       // With DocBook 5, MathML must be within its own namespace (defined in Buffer.cpp::writeDocBookSource, except when
+       // it should be inlined).
        // Output everything in a separate stream so that this does not interfere with the standard flow of DocBook tags.
+       std::string mathmlNamespacePrefix;
+       if (!mathmlNamespaceInline) {
+               if (buffer().params().docbook_mathml_prefix == BufferParams::MPrefix)
+                       mathmlNamespacePrefix = "m";
+               else if (buffer().params().docbook_mathml_prefix == BufferParams::MMLPrefix)
+                       mathmlNamespacePrefix = "mml";
+       }
+
        odocstringstream osmath;
-       MathMLStream ms(osmath, "m", true);
+       MathMLStream ms(osmath, mathmlNamespacePrefix, true);
 
        // Output the MathML subtree.
-       odocstringstream ls;
-       otexstream ols(ls);
-
        // TeX transcription. Avoid MTag/ETag so that there are no extraneous spaces.
        ms << "<" << from_ascii("alt") << " role='tex'" << ">";
        // Workaround for db2latex: db2latex always includes equations with
        // \ensuremath{} or \begin{display}\end{display}
        // so we strip LyX' math environment
+       odocstringstream ls;
+       otexstream ols(ls);
        TeXMathStream wi(ols, false, false, TeXMathStream::wsDefault, runparams.encoding);
        InsetMathGrid::write(wi);
        ms << from_utf8(subst(subst(to_utf8(ls.str()), "&", "&amp;"), "<", "&lt;"));
@@ -2425,19 +2446,28 @@ void InsetMathHull::docbook(XMLStream & xs, OutputParams const & runparams) cons
        // temporary stream; then, if it is possible without error, then copy it back to the "real" stream. Otherwise,
        // some incomplete tags might be put into the real stream.
        try {
-               // First, generate the MathML expression.
+               // First, generate the MathML expression. If there is an error in the generation, this block is not fully
+               // executed, and the formula is not output to the DocBook stream.
                odocstringstream ostmp;
                MathMLStream mstmp(ostmp, ms.xmlns(), ms.xmlMode());
-               InsetMathGrid::mathmlize(mstmp);
+               mathmlize(mstmp);
+
+               // Choose the display style for the formula, to be output as an attribute near the formula root.
+               std::string mathmlAttr;
+               if (getType() == hullSimple)
+                       mathmlAttr = "display=\"inline\"";
+               else
+                       mathmlAttr = "display=\"block\"";
 
-               // Then, output it (but only if the generation can be done without errors!).
-               ms << MTag("math");
+               // Then, output the formula.
+               ms << MTag("math", mathmlAttr);
                ms.cr();
                osmath << ostmp.str(); // osmath is not a XMLStream, so no need for XMLStream::ESCAPE_NONE.
                ms << ETag("math");
        } catch (MathExportException const &) {
                ms.cr();
-               osmath << "<mathphrase>MathML export failed. Please report this as a bug.</mathphrase>";
+               osmath << "<mathphrase>MathML export failed. Please report this as a bug to the LyX developers: "
+                       "https://www.lyx.org/trac.</mathphrase>";
        }
 
        // Output the complete formula to the DocBook stream.
@@ -2510,8 +2540,15 @@ void InsetMathHull::mathmlize(MathMLStream & ms) const
        bool const havenumbers = haveNumbers();
        bool const havetable = havenumbers || nrows() > 1 || ncols() > 1;
 
-       if (havetable)
-               ms << MTag("mtable");
+    if (havetable) {
+        if (getType() == hullSimple)
+            ms << MTag("mtable");
+        else if (getType() >= hullAlign && getType() <= hullXXAlignAt)
+            ms << MTag("mtable", "displaystyle='true' columnalign='right left'");
+        else
+            ms << MTag("mtable", "displaystyle='true'");
+    }
+
        char const * const celltag = havetable ? "mtd" : "mrow";
        // FIXME There does not seem to be wide support at the moment
        // for mlabeledtr, so we have to use just mtr for now.
@@ -2530,7 +2567,7 @@ void InsetMathHull::mathmlize(MathMLStream & ms) const
                        ms << MTag("mtd");
                        docstring const & num = numbers_[row];
                        if (!num.empty())
-                               ms << '(' << num << ')';
+                               ms << MTagInline("mtext") << '(' << num << ')' << ETagInline("mtext");
                  ms << ETag("mtd");
                }
                if (havetable)