]> git.lyx.org Git - lyx.git/blobdiff - src/cursor.C
the 'fitCursor fix'
[lyx.git] / src / cursor.C
index 6f8a9111916a380ffb90ca2cac882a35bd5ab044..4642a2ee3aee666f998e813c880cea116c285866 100644 (file)
 #include "BufferView.h"
 #include "buffer.h"
 #include "cursor.h"
+#include "CutAndPaste.h"
 #include "debug.h"
 #include "dispatchresult.h"
 #include "encoding.h"
 #include "funcrequest.h"
-#include "iterators.h"
 #include "language.h"
 #include "lfuns.h"
 #include "lyxfont.h"
@@ -29,6 +29,7 @@
 #include "lyxtext.h"
 #include "paragraph.h"
 #include "paragraph_funcs.h"
+#include "pariterator.h"
 
 #include "insets/updatableinset.h"
 #include "insets/insettabular.h"
@@ -45,6 +46,8 @@
 
 #include <boost/assert.hpp>
 
+using lyx::par_type;
+
 using std::string;
 using std::vector;
 using std::endl;
@@ -81,8 +84,8 @@ void region(CursorSlice const & i1, CursorSlice const & i2,
 
 
 LCursor::LCursor(BufferView & bv)
-       : DocumentIterator(), bv_(&bv), anchor_(),
-         cached_y_(0), x_target_(-1), selection_(false), mark_(false)
+       : DocIterator(), bv_(&bv), anchor_(), x_target_(-1),
+         selection_(false), mark_(false)
 {}
 
 
@@ -90,31 +93,28 @@ void LCursor::reset(InsetBase & inset)
 {
        clear();
        push_back(CursorSlice(inset));
-       anchor_.clear();
-       anchor_.push_back(CursorSlice(inset));
-       cached_y_ = 0;
+       anchor_ = DocIterator(inset);
        clearTargetX();
        selection_ = false;
        mark_ = false;
 }
 
 
-void LCursor::setCursor(DocumentIterator const & cur, bool sel)
+void LCursor::setCursor(DocIterator const & cur, bool sel)
 {
        // this (intentionally) does not touch the anchor
-       DocumentIterator::operator=(cur);
+       DocIterator::operator=(cur);
        selection() = sel;
 }
 
 
 DispatchResult LCursor::dispatch(FuncRequest const & cmd0)
 {
+       lyxerr << "\nLCursor::dispatch: cmd: " << cmd0 << endl << *this << endl;
        if (empty())
                return DispatchResult();
 
-       //lyxerr << "\nLCursor::dispatch: cmd: " << cmd0 << endl << *this << endl;
        FuncRequest cmd = cmd0;
-       nopop_ = false;
        LCursor safe = *this;
 
        for ( ; size(); pop()) {
@@ -123,7 +123,7 @@ DispatchResult LCursor::dispatch(FuncRequest const & cmd0)
                BOOST_ASSERT(idx() <= lastidx());
                BOOST_ASSERT(par() <= lastpar());
 
-               // The common case is 'LFUN handled, need update', so make the 
+               // The common case is 'LFUN handled, need update', so make the
                // LFUN handler's life easier by assuming this as default value.
                // The handler can reset the update and val flags if necessary.
                disp_.update(true);
@@ -134,15 +134,22 @@ DispatchResult LCursor::dispatch(FuncRequest const & cmd0)
        }
        // it completely to get a 'bomb early' behaviour in case this
        // object will be used again.
-       if (nopop_ || !disp_.dispatched())
+       if (!disp_.dispatched()) {
+               lyxerr << "RESTORING OLD CURSOR!" << endl;
                operator=(safe);
+               disp_.dispatched(false);
+       }
        return disp_;
 }
 
 
 bool LCursor::getStatus(FuncRequest const & cmd, FuncStatus & status)
 {
+       // This is, of course, a mess. Better create a new doc iterator and use
+       // this in Inset::getStatus. This might require an additional
+       // BufferView * arg, though (which should be avoided)
        LCursor safe = *this;
+       bool res = false;
        for ( ; size(); pop()) {
                //lyxerr << "\nLCursor::getStatus: cmd: " << cmd << endl << *this << endl;
                BOOST_ASSERT(pos() <= lastpos());
@@ -153,11 +160,13 @@ bool LCursor::getStatus(FuncRequest const & cmd, FuncStatus & status)
                // a definitive decision on whether it want to handle the
                // request or not. The result of this decision is put into
                // the 'status' parameter.
-               if (inset().getStatus(*this, cmd, status))
+               if (inset().getStatus(*this, cmd, status)) {
+                       res = true;
                        break;
+               }
        }
        operator=(safe);
-       return true;
+       return res;
 }
 
 
@@ -171,7 +180,6 @@ void LCursor::pop()
 {
        BOOST_ASSERT(size() >= 1);
        pop_back();
-       anchor_.pop_back();
 }
 
 
@@ -194,7 +202,7 @@ bool LCursor::popLeft()
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "Leaving inset to the left" << endl;
-       inset().notifyCursorLeaves(idx());
+       inset().notifyCursorLeaves(*this);
        if (depth() == 1)
                return false;
        pop();
@@ -206,7 +214,7 @@ bool LCursor::popRight()
 {
        BOOST_ASSERT(!empty());
        //lyxerr << "Leaving inset to the right" << endl;
-       inset().notifyCursorLeaves(idx());
+       inset().notifyCursorLeaves(*this);
        if (depth() == 1)
                return false;
        pop();
@@ -227,15 +235,6 @@ int LCursor::currentMode()
 }
 
 
-void LCursor::updatePos()
-{
-       BOOST_ASSERT(!empty());
-       if (size() > 1)
-               cached_y_ = bv().top_y() + back().inset().yo();
-               //cached_y_ = back().inset().yo();
-}
-
-
 void LCursor::getDim(int & asc, int & des) const
 {
        if (inMathed()) {
@@ -261,16 +260,6 @@ void LCursor::getPos(int & x, int & y) const
        y = 0;
        if (!empty())
                inset().getCursorPos(back(), x, y);
-       // getCursorPos gives _screen_ coordinates. We need to add
-       // top_y to get document coordinates. This is hidden in cached_y_.
-       //y += cached_y_ - inset().yo();
-       // The rest is non-obvious. The reason we have to have these
-       // extra computation is that the getCursorPos() calls rely
-       // on the inset's own knowledge of its screen position.
-       // If we scroll up or down in a big enough increment,
-       // inset->draw() is not called: this doesn't update
-       // inset.yo_, so getCursor() returns an old value.
-       // Ugly as you like.
 }
 
 
@@ -305,19 +294,19 @@ bool LCursor::posRight()
 }
 
 
-CursorSlice & LCursor::anchor()
+CursorSlice LCursor::anchor() const
 {
-       return anchor_.back();
-}
-
-
-CursorSlice const & LCursor::anchor() const
-{
-       return anchor_.back();
+       BOOST_ASSERT(anchor_.size() >= size());
+       CursorSlice normal = anchor_[size() - 1];
+       if (size() < anchor_.size() && back() <= normal) {
+               // anchor is behind cursor -> move anchor behind the inset
+               ++normal.pos();
+       }
+       return normal;
 }
 
 
-CursorSlice const & LCursor::selBegin() const
+CursorSlice LCursor::selBegin() const
 {
        if (!selection())
                return back();
@@ -325,29 +314,27 @@ CursorSlice const & LCursor::selBegin() const
 }
 
 
-CursorSlice & LCursor::selBegin()
+CursorSlice LCursor::selEnd() const
 {
        if (!selection())
                return back();
-       // can't use std::min as this returns a const ref
-       return anchor() < back() ? anchor() : back();
+       return anchor() > back() ? anchor() : back();
 }
 
 
-CursorSlice const & LCursor::selEnd() const
+DocIterator LCursor::selectionBegin() const
 {
        if (!selection())
-               return back();
-       return anchor() > back() ? anchor() : back();
+               return *this;
+       return anchor() < back() ? anchor_ : *this;
 }
 
 
-CursorSlice & LCursor::selEnd()
+DocIterator LCursor::selectionEnd() const
 {
        if (!selection())
-               return back();
-       // can't use std::min as this returns a const ref
-       return anchor() > back() ? anchor() : back();
+               return *this;
+       return anchor() > back() ? anchor_ : *this;
 }
 
 
@@ -355,15 +342,15 @@ void LCursor::setSelection()
 {
        selection() = true;
        // a selection with no contents is not a selection
+#warning doesnt look ok
        if (par() == anchor().par() && pos() == anchor().pos())
                selection() = false;
 }
 
 
-void LCursor::setSelection(DocumentIterator const & where, size_t n)
+void LCursor::setSelection(DocIterator const & where, size_t n)
 {
-       selection() = true;
-       setCursor(where, false);
+       setCursor(where, true);
        anchor_ = where;
        pos() += n;
 }
@@ -454,7 +441,9 @@ void LCursor::eraseSelection()
        //lyxerr << "LCursor::eraseSelection" << endl;
        CursorSlice const & i1 = selBegin();
        CursorSlice const & i2 = selEnd();
+#ifdef WITH_WARNINGS
 #warning FIXME
+#endif
        if (i1.inset().asMathInset()) {
                if (i1.idx() == i2.idx()) {
                        i1.cell().erase(i1.pos(), i2.pos());
@@ -486,13 +475,6 @@ string LCursor::grabAndEraseSelection()
 }
 
 
-void LCursor::selClear()
-{
-       resetAnchor();
-       clearSelection();
-}
-
-
 void LCursor::selCopy()
 {
        if (selection()) {
@@ -533,8 +515,12 @@ void LCursor::selPaste(size_t n)
 void LCursor::selHandle(bool sel)
 {
        //lyxerr << "LCursor::selHandle" << endl;
-       if (sel == selection())
+       if (sel == selection()) {
+               if (!sel)
+                       noUpdate();
                return;
+       }
+       
        resetAnchor();
        selection() = sel;
 }
@@ -552,9 +538,19 @@ void LCursor::selClearOrDel()
 
 std::ostream & operator<<(std::ostream & os, LCursor const & cur)
 {
-       for (size_t i = 0, n = cur.size(); i != n; ++i)
-               os << " " << cur.operator[](i) << " | " << cur.anchor_[i] << "\n";
-       os << " selection: " << cur.selection_ << endl;
+       for (size_t i = 0, n = cur.size(); i != n; ++i) {
+               os << " " << cur.operator[](i) << " | ";
+               if (i < cur.anchor_.size())
+                       os << cur.anchor_[i];
+               else
+                       os << "-------------------------------";
+               os << "\n";
+       }
+       for (size_t i = cur.size(), n = cur.anchor_.size(); i < n; ++i) {
+               os << "------------------------------- | " << cur.anchor_[i] << "\n";
+       }
+       os << " selection: " << cur.selection_
+          << " x_target: " << cur.x_target_ << endl;
        return os;
 }
 
@@ -573,7 +569,6 @@ std::ostream & operator<<(std::ostream & os, LCursor const & cur)
 #include "mathed/math_factory.h"
 #include "mathed/math_gridinset.h"
 #include "mathed/math_macroarg.h"
-#include "mathed/math_macrotemplate.h"
 #include "mathed/math_mathmlstream.h"
 #include "mathed/math_scriptinset.h"
 #include "mathed/math_support.h"
@@ -603,7 +598,7 @@ bool LCursor::openable(MathAtom const & t) const
                return true;
 
        // we can't move into anything new during selection
-       if (depth() == anchor_.size())
+       if (depth() >= anchor_.size())
                return false;
        if (!ptr_cmp(t.nucleus(), &anchor_[depth()].inset()))
                return false;
@@ -612,8 +607,8 @@ bool LCursor::openable(MathAtom const & t) const
 }
 
 
-bool positionable(DocumentIterator const & cursor,
-       DocumentIterator const & anchor)
+bool positionable(DocIterator const & cursor,
+       DocIterator const & anchor)
 {
        // avoid deeper nested insets when selecting
        if (cursor.size() > anchor.size())
@@ -645,7 +640,7 @@ void LCursor::plainErase()
 
 void LCursor::markInsert()
 {
-       cell().insert(pos(), MathAtom(new MathCharInset(0)));
+       insert(char(0));
 }
 
 
@@ -664,7 +659,7 @@ void LCursor::plainInsert(MathAtom const & t)
 
 void LCursor::insert(string const & str)
 {
-       lyxerr << "LCursor::insert str '" << str << "'" << endl;
+       //lyxerr << "LCursor::insert str '" << str << "'" << endl;
        for (string::const_iterator it = str.begin(); it != str.end(); ++it)
                insert(*it);
 }
@@ -676,7 +671,7 @@ void LCursor::insert(char c)
        BOOST_ASSERT(!empty());
        if (inMathed()) {
                selClearOrDel();
-               plainInsert(MathAtom(new MathCharInset(c)));
+               insert(new MathCharInset(c));
        } else {
                text()->insertChar(*this, c);
        }
@@ -689,6 +684,7 @@ void LCursor::insert(MathAtom const & t)
        macroModeClose();
        selClearOrDel();
        plainInsert(t);
+       lyxerr << "LCursor::insert MathAtom: cur:\n" << *this << endl;
 }
 
 
@@ -765,6 +761,7 @@ bool LCursor::backspace()
        if (pos() != 0 && prevAtom()->nargs() > 0) {
                // let's require two backspaces for 'big stuff' and
                // highlight on the first
+               resetAnchor();
                selection() = true;
                --pos();
        } else {
@@ -803,7 +800,9 @@ bool LCursor::erase()
                return true;
        }
 
+       // 'clever' UI hack: only erase large items if previously slected
        if (pos() != lastpos() && inset().nargs() > 0) {
+               resetAnchor();
                selection() = true;
                ++pos();
        } else {
@@ -817,7 +816,7 @@ bool LCursor::erase()
 bool LCursor::up()
 {
        macroModeClose();
-       DocumentIterator save = *this;
+       DocIterator save = *this;
        if (goUpDown(true))
                return true;
        setCursor(save, false);
@@ -829,7 +828,7 @@ bool LCursor::up()
 bool LCursor::down()
 {
        macroModeClose();
-       DocumentIterator save = *this;
+       DocIterator save = *this;
        if (goUpDown(false))
                return true;
        setCursor(save, false);
@@ -891,18 +890,6 @@ int LCursor::targetX() const
 }
 
 
-void LCursor::adjust(pos_type from, int diff)
-{
-       if (pos() > from)
-               pos() += diff;
-       if (anchor().pos() > from)
-               anchor().pos() += diff;
-       // just to be on the safe side
-       // theoretically unecessary
-       normalize();
-}
-
-
 bool LCursor::inMacroMode() const
 {
        if (!pos() != 0)
@@ -918,31 +905,11 @@ MathUnknownInset * LCursor::activeMacro()
 }
 
 
-bool LCursor::inMacroArgMode() const
-{
-       return pos() > 0 && prevAtom()->getChar() == '#';
-}
-
-
-MathGridInset * LCursor::enclosingGrid(idx_type & idx) const
-{
-       for (MathInset::difference_type i = depth() - 1; i >= 0; --i) {
-               MathInset * m = operator[](i).inset().asMathInset();
-               if (!m)
-                       return 0;
-               MathGridInset * p = m->asGridInset();
-               if (p) {
-                       idx = operator[](i).idx();
-                       return p;
-               }
-       }
-       return 0;
-}
-
-
 void LCursor::pullArg()
 {
+#ifdef WITH_WARNINGS
 #warning Look here
+#endif
        MathArray ar = cell();
        if (popLeft() && inMathed()) {
                plainErase();
@@ -956,10 +923,12 @@ void LCursor::pullArg()
 
 void LCursor::touch()
 {
+#ifdef WITH_WARNINGS
 #warning look here
+#endif
 #if 0
-       DocumentIterator::const_iterator it = begin();
-       DocumentIterator::const_iterator et = end();
+       DocIterator::const_iterator it = begin();
+       DocIterator::const_iterator et = end();
        for ( ; it != et; ++it)
                it->cell().touch();
 #endif
@@ -987,22 +956,6 @@ void LCursor::normalize()
 }
 
 
-char LCursor::valign()
-{
-       idx_type idx;
-       MathGridInset * p = enclosingGrid(idx);
-       return p ? p->valign() : '\0';
-}
-
-
-char LCursor::halign()
-{
-       idx_type idx;
-       MathGridInset * p = enclosingGrid(idx);
-       return p ? p->halign(idx % p->ncols()) : '\0';
-}
-
-
 bool LCursor::goUpDown(bool up)
 {
        // Be warned: The 'logic' implemented in this function is highly
@@ -1093,24 +1046,23 @@ bool LCursor::goUpDown(bool up)
 bool LCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhigh)
 {
        BOOST_ASSERT(!empty());
-       ParagraphList::iterator beg;
-       ParagraphList::iterator end;
-       CursorSlice bottom = operator[](0); 
+       par_type beg, end;
+       CursorSlice bottom = operator[](0);
        LyXText * text = bottom.text();
        BOOST_ASSERT(text);
        getParsInRange(text->paragraphs(), ylow, yhigh, beg, end);
 
-       DocumentIterator it(bv().buffer()->inset());
-       DocumentIterator et;
-       lyxerr << "x: " << x << " y: " << y << endl;
-       lyxerr << "xlow: " << xlow << " ylow: " << ylow << endl;
-       lyxerr << "xhigh: " << xhigh << " yhigh: " << yhigh << endl;
+       DocIterator it = doc_iterator_begin(bv().buffer()->inset());
+       DocIterator et = doc_iterator_end(bv().buffer()->inset());
+       //lyxerr << "x: " << x << " y: " << y << endl;
+       //lyxerr << "xlow: " << xlow << " ylow: " << ylow << endl;
+       //lyxerr << "xhigh: " << xhigh << " yhigh: " << yhigh << endl;
 
-       it.par() = text->parOffset(beg);
+       it.par() = beg;
        //et.par() = text->parOffset(end);
 
        double best_dist = 10e10;
-       DocumentIterator best_cursor = it;
+       DocIterator best_cursor = it;
 
        for ( ; it != et; it.forwardPos()) {
                // avoid invalid nesting when selecting
@@ -1120,11 +1072,11 @@ bool LCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhigh)
                        cur.inset().getCursorPos(cur, xo, yo);
                        if (xlow <= xo && xo <= xhigh && ylow <= yo && yo <= yhigh) {
                                double d = (x - xo) * (x - xo) + (y - yo) * (y - yo);
-                               lyxerr << "xo: " << xo << " yo: " << yo << " d: " << d << endl;
+                               //lyxerr << "xo: " << xo << " yo: " << yo << " d: " << d << endl;
                                // '<=' in order to take the last possible position
                                // this is important for clicking behind \sum in e.g. '\sum_i a'
                                if (d <= best_dist) {
-                                       lyxerr << "*" << endl;
+                                       //lyxerr << "*" << endl;
                                        best_dist   = d;
                                        best_cursor = it;
                                }
@@ -1132,7 +1084,7 @@ bool LCursor::bruteFind(int x, int y, int xlow, int xhigh, int ylow, int yhigh)
                }
        }
 
-       lyxerr << "best_dist: " << best_dist << " cur:\n" << best_cursor << endl;
+       //lyxerr << "best_dist: " << best_dist << " cur:\n" << best_cursor << endl;
        if (best_dist < 1e10)
                setCursor(best_cursor, false);
        return best_dist < 1e10;
@@ -1143,9 +1095,9 @@ void LCursor::bruteFind2(int x, int y)
 {
        double best_dist = 1e10;
 
-       DocumentIterator it = *this;
+       DocIterator it = *this;
        it.back().pos() = 0;
-       DocumentIterator et = *this;
+       DocIterator et = *this;
        et.back().pos() = et.back().asMathInset()->cell(et.back().idx()).size();
        for (int i = 0; ; ++i) {
                int xo, yo;
@@ -1166,25 +1118,6 @@ void LCursor::bruteFind2(int x, int y)
 }
 
 
-CursorSlice LCursor::normalAnchor()
-{
-       if (anchor_.size() < depth()) {
-               resetAnchor();
-               lyxerr << "unusual Anchor size" << endl;
-       }
-       //lyx::BOOST_ASSERT(Anchor_.size() >= cursor.depth());
-       // use Anchor on the same level as Cursor
-       CursorSlice normal = anchor_[size() - 1];
-#if 0
-       if (depth() < anchor_.size() && !(normal < xx())) {
-               // anchor is behind cursor -> move anchor behind the inset
-               ++normal.pos_;
-       }
-#endif
-       return normal;
-}
-
-
 void LCursor::handleFont(string const & font)
 {
        lyxerr << "LCursor::handleFont: " << font << endl;
@@ -1239,32 +1172,36 @@ string LCursor::selectionAsString(bool label) const
 
        if (inTexted()) {
                Buffer const & buffer = *bv().buffer();
+               ParagraphList & pars = text()->paragraphs();
 
                // should be const ...
-               ParagraphList::iterator startpit = text()->getPar(selBegin());
-               ParagraphList::iterator endpit = text()->getPar(selEnd());
+               par_type startpit = selBegin().par();
+               par_type endpit = selEnd().par();
                size_t const startpos = selBegin().pos();
                size_t const endpos = selEnd().pos();
 
                if (startpit == endpit)
-                       return startpit->asString(buffer, startpos, endpos, label);
+                       return pars[startpit].asString(buffer, startpos, endpos, label);
 
                // First paragraph in selection
-               string result =
-                       startpit->asString(buffer, startpos, startpit->size(), label) + "\n\n";
+               string result = pars[startpit].
+                       asString(buffer, startpos, pars[startpit].size(), label) + "\n\n";
 
                // The paragraphs in between (if any)
-               ParagraphList::iterator pit = startpit;
-               for (++pit; pit != endpit; ++pit)
-                       result += pit->asString(buffer, 0, pit->size(), label) + "\n\n";
+               for (par_type pit = startpit + 1; pit != endpit; ++pit) {
+                       Paragraph & par = pars[pit];
+                       result += par.asString(buffer, 0, par.size(), label) + "\n\n";
+               }
 
                // Last paragraph in selection
-               result += endpit->asString(buffer, 0, endpos, label);
+               result += pars[endpit].asString(buffer, 0, endpos, label);
 
                return result;
        }
 
+#ifdef WITH_WARNINGS
 #warning and mathed?
+#endif
        return string();
 }
 
@@ -1284,27 +1221,6 @@ string LCursor::currentState()
 }
 
 
-// only used by the spellchecker
-void LCursor::replaceWord(string const & replacestring)
-{
-       LyXText * t = text();
-       BOOST_ASSERT(t);
-
-       t->replaceSelectionWithString(*this, replacestring);
-       t->setSelectionRange(*this, replacestring.length());
-
-       // Go back so that replacement string is also spellchecked
-       for (string::size_type i = 0; i < replacestring.length() + 1; ++i)
-               t->cursorLeft(*this);
-}
-
-
-void LCursor::update()
-{
-       bv().update();
-}
-
-
 string LCursor::getPossibleLabel()
 {
        return inMathed() ? "eq:" : text()->getPossibleLabel(*this);
@@ -1325,9 +1241,8 @@ Encoding const * LCursor::getEncoding() const
                        break;
        CursorSlice const & sl = operator[](s);
        LyXText & text = *sl.text();
-       ParagraphList::iterator pit = text.getPar(sl.par());
-       LyXFont font = pit->getFont(
-               bv().buffer()->params(), sl.pos(), outerFont(pit, text.paragraphs()));  
+       LyXFont font = text.getPar(sl.par()).getFont(
+               bv().buffer()->params(), sl.pos(), outerFont(sl.par(), text.paragraphs()));
        return font.language()->encoding();
 }
 
@@ -1338,6 +1253,12 @@ void LCursor::undispatched()
 }
 
 
+void LCursor::dispatched()
+{
+       disp_.dispatched(true);
+}
+
+
 void LCursor::noUpdate()
 {
        disp_.update(false);