Fix to bug 2362: Deleting superscript also deletes subscript.

[lyx.git] / src / mathed / math_scriptinset.h

diff --git a/src/mathed/math_scriptinset.h b/src/mathed/math_scriptinset.h
index 50a409b799c4bff5345f499287ea1c68f52c2c76..52beb7073f06b2418328cea41ac31fefcea20519 100644 (file)
--- a/src/mathed/math_scriptinset.h
+++ b/src/mathed/math_scriptinset.h
@@ -1,15 +1,23 @@
 // -*- C++ -*-
+/**
+ * \file math_scriptinset.h
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
+ *
+ * \author André Pönitz
+ *
+ * Full author contact details are available in file CREDITS.
+ */
+
 #ifndef MATH_SCRIPTINSET_H
 #define MATH_SCRIPTINSET_H
 
 #include "math_nestinset.h"
 
 
-/** Inset for super- and subscripts
- *  \author André Pönitz
- *
- * Full author contact details are available in file CREDITS
- */
+// An inset for super- and subscripts or both.  The 'nucleus' is always
+// cell 0.  If there is just one script, it's cell 1 and cell_1_is_up_
+// is set accordingly.  If both are used, cell 1 is up and cell 2 is down.
 
 class MathScriptInset : public MathNestInset {
 public:
@@ -20,9 +28,9 @@ public:
        /// create inset with single script and given nucleus
        MathScriptInset(MathAtom const & at, bool up);
        ///
-       MathInset * clone() const;
+       mode_type currentMode() const { return MATH_MODE; }
        ///
-       Dimension metrics(MetricsInfo & mi) const;
+       void metrics(MetricsInfo & mi, Dimension & dim) const;
        ///
        void draw(PainterInfo & pi, int x, int y) const;
        ///
@@ -30,6 +38,17 @@ public:
        ///
        void drawT(TextPainter & pi, int x, int y) const;
 
+       /// move cursor left
+       bool idxLeft(LCursor & cur) const;
+       /// move cursor right
+       bool idxRight(LCursor & cur) const;
+       /// move cursor up or down
+       bool idxUpDown(LCursor & cur, bool up) const;
+       /// Target pos when we enter the inset from the left by pressing "Right"
+       bool idxFirst(LCursor & cur) const;
+       /// Target pos when we enter the inset from the right by pressing "Left"
+       bool idxLast(LCursor & cur) const;
+
        /// write LaTeX and Lyx code
        void write(WriteStream & os) const;
        /// write normalized content
@@ -42,18 +61,6 @@ public:
        void mathmlize(MathMLStream &) const;
        /// write content as something readable by Octave
        void octave(OctaveStream &) const;
-       /// move cursor left
-       bool idxLeft(idx_type &, pos_type &) const;
-       /// move cursor right
-       bool idxRight(idx_type &, pos_type &) const;
-       /// move cursor up or down
-       bool idxUpDown(idx_type & idx, pos_type & pos, bool up, int targetx) const;
-       /// Target pos when we enter the inset from the left by pressing "Right"
-       bool idxFirst(idx_type & idx, pos_type & pos) const;
-       /// Target pos when we enter the inset from the right by pressing "Left"
-       bool idxLast(idx_type & idx, pos_type & pos) const;
-       /// can we enter this cell?
-       bool validCell(idx_type i) const { return i == 2 || script_[i]; }
 
        /// identifies scriptinsets
        MathScriptInset const * asScriptInset() const;
@@ -64,13 +71,13 @@ public:
        void limits(int lim) { limits_ = lim; }
        /// get limits
        int limits() const { return limits_; }
-       /// returns subscript
+       /// returns subscript. Always run 'hasDown' or 'has(false)' before!
        MathArray const & down() const;
-       /// returns subscript
+       /// returns subscript. Always run 'hasDown' or 'has(false)' before!
        MathArray & down();
-       /// returns superscript
+       /// returns superscript. Always run 'hasUp' or 'has(true)' before!
        MathArray const & up() const;
-       /// returns superscript
+       /// returns superscript. Always run 'hasUp' or 'has(true)' before!
        MathArray & up();
        /// returns nucleus
        MathArray const & nuc() const;
@@ -82,6 +89,8 @@ public:
        bool hasDown() const;
        /// do we have a script?
        bool has(bool up) const;
+       /// what idx has super/subscript?
+       idx_type idxOfScript(bool up) const;
        /// remove script
        void removeScript(bool up);
        /// make sure a script is accessible
@@ -90,10 +99,10 @@ public:
        void infoize(std::ostream & os) const;
        /// say whether we have displayed limits
        void infoize2(std::ostream & os) const;
-       /// local dispatcher
-       dispatch_result dispatch(FuncRequest const & cmd, idx_type & idx, pos_type & pos);
-
+protected:
+       virtual void doDispatch(LCursor & cur, FuncRequest & cmd);
 private:
+       virtual std::auto_ptr<InsetBase> doClone() const;
        /// returns x offset for main part
        int dxx() const;
        /// returns width of nucleus if any
@@ -113,10 +122,10 @@ private:
        /// where do we have to draw the scripts?
        bool hasLimits() const;
        /// clean up empty cells
-       void notifyCursorLeaves(idx_type idx);
+       void notifyCursorLeaves(LCursor & cur);
 
        /// possible subscript (index 0) and superscript (index 1)
-       bool script_[2];
+       bool cell_1_is_up_;
        /// 1 - "limits", -1 - "nolimits", 0 - "default"
        int limits_;
 };