From: Jean-Marc Lasgouttes <lasgouttes@lyx.org>
Date: Fri, 3 Feb 2017 09:47:33 +0000 (+0100)
Subject: Fixup 89662a68: remove markers that should not be there
X-Git-Tag: 2.3.0alpha1~405
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=ad79ac406f9c8fc85880cfeb5416b0299ee9a617;p=features.git

Fixup 89662a68: remove markers that should not be there

InsetMathDelim does not need markers because the delimiters are
visually enough.

CommandInset (like InsetMathRef) does have cells, but they are hidden
(only for storage). Therefore the default InsetMath::markers() method
mistakenly added markers. In practice, the markers were never active
(since the cursor cannot go into the inset), but some space was
reserved for them.
---

diff --git a/src/mathed/CommandInset.h b/src/mathed/CommandInset.h
index 770ecb388d..68d0d13c53 100644
--- a/src/mathed/CommandInset.h
+++ b/src/mathed/CommandInset.h
@@ -28,6 +28,8 @@ public:
 	explicit CommandInset(Buffer * buf, docstring const & name,
 		bool needs_math_mode = true);
 	///
+	marker_type marker() const { return NO_MARKER; }
+	///
 	void metrics(MetricsInfo & mi, Dimension & dim) const;
 	///
 	void draw(PainterInfo & pi, int x, int y) const;
diff --git a/src/mathed/InsetMathDelim.h b/src/mathed/InsetMathDelim.h
index 5a7e816404..bc49ce3c59 100644
--- a/src/mathed/InsetMathDelim.h
+++ b/src/mathed/InsetMathDelim.h
@@ -32,6 +32,8 @@ public:
 	InsetMathDelim const * asDelimInset() const { return this; }
 	///
 	MathClass mathClass() const { return MC_INNER; }
+	///
+	marker_type marker() const { return NO_MARKER; }
 	/// is it (...)?
 	bool isParenthesis() const;
 	/// is it [...]?