Fix bug 2481

[lyx.git] / src / mathed / math_fracinset.C

diff --git a/src/mathed/math_fracinset.C b/src/mathed/math_fracinset.C
index ea05d41c6698960f23252b0e68d95586c9a22208..2ba15ab46282200371bb2206f7519db53eb01e91 100644 (file)
--- a/src/mathed/math_fracinset.C
+++ b/src/mathed/math_fracinset.C
@@ -1,82 +1,125 @@
+/**
+ * \file math_fracinset.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
+ *
+ * \author Alejandro Aguilar Sierra
+ * \author André Pönitz
+ *
+ * Full author contact details are available in file CREDITS.
+ */
+
+#include <config.h>
 
 #include "math_fracinset.h"
-#include "math_support.h"
-#include "frontends/Painter.h"
+#include "math_data.h"
 #include "math_mathmlstream.h"
 #include "textpainter.h"
+#include "LaTeXFeatures.h"
+#include "LColor.h"
+#include "frontends/Painter.h"
 
 
+using std::string;
 using std::max;
+using std::auto_ptr;
 
 
-MathFracInset::MathFracInset(bool atop)
-       : atop_(atop)
+MathFracInset::MathFracInset(Kind kind)
+       : kind_(kind)
 {}
 
 
-MathInset * MathFracInset::clone() const
+auto_ptr<InsetBase> MathFracInset::doClone() const
 {
-       return new MathFracInset(*this);
+       return auto_ptr<InsetBase>(new MathFracInset(*this));
 }
 
 
 MathFracInset * MathFracInset::asFracInset()
 {
-       return atop_ ? 0 : this;
+       return kind_ == ATOP ? 0 : this;
 }
 
 
 MathFracInset const * MathFracInset::asFracInset() const
 {
-       return atop_ ? 0 : this;
+       return kind_ == ATOP ? 0 : this;
 }
 
 
-void MathFracInset::metrics(MetricsInfo & mi) const
+void MathFracInset::metrics(MetricsInfo & mi, Dimension & dim) const
 {
        FracChanger dummy(mi.base);
        cell(0).metrics(mi);
        cell(1).metrics(mi);
-       dim_.wid = max(cell(0).width(), cell(1).width()) + 2;
-       dim_.asc = cell(0).height() + 2 + 5;
-       dim_.des = cell(1).height() + 2 - 5;
+       if (kind_ == NICEFRAC) {
+               dim.wid =  cell(0).width() + cell(1).width() + 5;
+               dim.asc = cell(0).height() + 5;
+               dim.des = cell(1).height() - 5;
+       } else {
+               dim.wid = max(cell(0).width(), cell(1).width()) + 2;
+               dim.asc = cell(0).height() + 2 + 5;
+               dim.des = cell(1).height() + 2 - 5;
+       }
+       metricsMarkers(dim);
+       dim_ = dim;
 }
 
 
 void MathFracInset::draw(PainterInfo & pi, int x, int y) const
 {
-       int m = x + width() / 2;
+       setPosCache(pi, x, y);
+       int m = x + dim_.wid / 2;
        FracChanger dummy(pi.base);
-       cell(0).draw(pi, m - cell(0).width() / 2, y - cell(0).descent() - 2 - 5);
-       cell(1).draw(pi, m - cell(1).width() / 2, y + cell(1).ascent()  + 2 - 5);
-       if (!atop_)
-               pi.pain.line(x + 1, y - 5, x + width() - 2, y - 5, LColor::math);
+       if (kind_ == NICEFRAC) {
+               cell(0).draw(pi, x + 2, 
+                               y - cell(0).descent() - 5);
+               cell(1).draw(pi, x + cell(0).width() + 5,
+                               y + cell(1).ascent() / 2);
+       } else {
+               cell(0).draw(pi, m - cell(0).width() / 2, 
+                               y - cell(0).descent() - 2 - 5);
+               cell(1).draw(pi, m - cell(1).width() / 2,
+                               y + cell(1).ascent()  + 2 - 5);
+       }
+       if (kind_ == NICEFRAC)
+               pi.pain.line(x + cell(0).width(), 
+                               y + dim_.des - 2, 
+                               x + cell(0).width() + 5, 
+                               y - dim_.asc + 2, LColor::math);
+       if (kind_ == FRAC)
+               pi.pain.line(x + 1, y - 5, 
+                               x + dim_.wid - 2, y - 5, LColor::math);
+       drawMarkers(pi, x, y);
 }
 
 
-void MathFracInset::metricsT(TextMetricsInfo const & mi) const
+void MathFracInset::metricsT(TextMetricsInfo const & mi, Dimension & dim) const
 {
-       cell(0).metricsT(mi);
-       cell(1).metricsT(mi);
-       dim_.wid = max(cell(0).width(), cell(1).width());
-       dim_.asc = cell(0).height() + 1;
-       dim_.des = cell(1).height();
+       cell(0).metricsT(mi, dim);
+       cell(1).metricsT(mi, dim);
+       dim.wid = max(cell(0).width(), cell(1).width());
+       dim.asc = cell(0).height() + 1;
+       dim.des = cell(1).height();
+       //dim = dim_;
 }
 
 
 void MathFracInset::drawT(TextPainter & pain, int x, int y) const
 {
-       int m = x + width() / 2;
+       int m = x + dim_.width() / 2;
        cell(0).drawT(pain, m - cell(0).width() / 2, y - cell(0).descent() - 1);
        cell(1).drawT(pain, m - cell(1).width() / 2, y + cell(1).ascent());
-       if (!atop_)
-               pain.horizontalLine(x, y, width());
+       // ASCII art: ignore niceties
+       if (kind_ == FRAC || kind_ == NICEFRAC)
+               pain.horizontalLine(x, y, dim_.width());
 }
 
 
 void MathFracInset::write(WriteStream & os) const
 {
-       if (atop_)
+       if (kind_ == ATOP)
                os << '{' << cell(0) << "\\atop " << cell(1) << '}';
        else // it's \\frac
                MathNestInset::write(os);
@@ -85,7 +128,22 @@ void MathFracInset::write(WriteStream & os) const
 
 string MathFracInset::name() const
 {
-       return atop_ ? "atop" : "frac";
+       switch (kind_) {
+       case FRAC:
+               return "frac";
+       case NICEFRAC:
+               return "nicefrac";
+       case ATOP:
+               return "atop";
+       default:
+               return string();
+       }
+}
+
+
+bool MathFracInset::extraBraces() const
+{
+       return kind_ == ATOP;
 }
 
 
@@ -111,3 +169,12 @@ void MathFracInset::mathmlize(MathMLStream & os) const
 {
        os << MTag("mfrac") << cell(0) << cell(1) << ETag("mfrac");
 }
+
+
+void MathFracInset::validate(LaTeXFeatures & features) const
+{
+       if (kind_ == NICEFRAC)
+               features.require("nicefrac");
+       MathNestInset::validate(features);
+}
+