]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
more cursor dispatch
[lyx.git] / src / text2.C
index c50e499cc7cc1e8399459981e63378af9ca39ea3..e15775524e164ac99ef5fd351930390432dac074 100644 (file)
 #include "funcrequest.h"
 #include "gettext.h"
 #include "language.h"
+#include "LColor.h"
 #include "lyxrc.h"
 #include "lyxrow.h"
 #include "lyxrow_funcs.h"
 #include "paragraph.h"
 #include "paragraph_funcs.h"
 #include "ParagraphParameters.h"
+#include "PosIterator.h"
 #include "undo.h"
 #include "vspace.h"
 
@@ -70,20 +72,19 @@ using std::ostringstream;
 using std::string;
 
 
-LyXText::LyXText(BufferView * bv, InsetText * inset, bool in_inset,
-         ParagraphList & paragraphs)
+LyXText::LyXText(BufferView * bv, bool in_inset)
        : height(0), width(0), textwidth_(bv ? bv->workWidth() : 100),
-         inset_owner(inset), bv_owner(bv),
-         in_inset_(in_inset), paragraphs_(&paragraphs), xo_(0), yo_(0)
+               background_color_(LColor::background),
+         bv_owner(bv), in_inset_(in_inset), xo_(0), yo_(0)
 {}
 
 
-void LyXText::init(BufferView * bview)
+void LyXText::init(BufferView * bv)
 {
-       bv_owner = bview;
+       bv_owner = bv;
 
-       ParagraphList::iterator const beg = ownerParagraphs().begin();
-       ParagraphList::iterator const end = ownerParagraphs().end();
+       ParagraphList::iterator const beg = paragraphs().begin();
+       ParagraphList::iterator const end = paragraphs().end();
        for (ParagraphList::iterator pit = beg; pit != end; ++pit)
                pit->rows.clear();
 
@@ -93,8 +94,7 @@ void LyXText::init(BufferView * bview)
        current_font = getFont(beg, 0);
 
        redoParagraphs(beg, end);
-       setCursorIntern(0, 0);
-       selection.cursor = cursor;
+       bv->cursor().resetAnchor();
 
        updateCounters();
 }
@@ -117,8 +117,8 @@ LyXFont LyXText::getFont(ParagraphList::iterator pit, pos_type pos) const
        // We specialize the 95% common case:
        if (!pit->getDepth()) {
                LyXFont f = pit->getFontSettings(params, pos);
-               if (pit->inInset())
-                       pit->inInset()->getDrawFont(f);
+               if (in_inset_)
+                       f.realize(font_);
                if (layout->labeltype == LABEL_MANUAL && pos < body_pos)
                        return f.realize(layout->reslabelfont);
                else
@@ -135,11 +135,11 @@ LyXFont LyXText::getFont(ParagraphList::iterator pit, pos_type pos) const
        LyXFont font = pit->getFontSettings(params, pos);
        font.realize(layoutfont);
 
-       if (pit->inInset())
-               pit->inInset()->getDrawFont(font);
+       if (in_inset_)
+               font.realize(font_);
 
        // Realize with the fonts of lesser depth.
-       //font.realize(outerFont(pit, ownerParagraphs()));
+       //font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
@@ -155,7 +155,7 @@ LyXFont LyXText::getLayoutFont(ParagraphList::iterator pit) const
 
        LyXFont font = layout->font;
        // Realize with the fonts of lesser depth.
-       //font.realize(outerFont(pit, ownerParagraphs()));
+       //font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
@@ -171,34 +171,13 @@ LyXFont LyXText::getLabelFont(ParagraphList::iterator pit) const
 
        LyXFont font = layout->labelfont;
        // Realize with the fonts of lesser depth.
-       font.realize(outerFont(pit, ownerParagraphs()));
+       font.realize(outerFont(pit, paragraphs()));
        font.realize(defaultfont_);
 
        return font;
 }
 
 
-void LyXText::setCharFont(ParagraphList::iterator pit,
-                         pos_type pos, LyXFont const & fnt,
-                         bool toggleall)
-{
-       BufferParams const & params = bv()->buffer()->params();
-       LyXFont font = getFont(pit, pos);
-       font.update(fnt, params.language, toggleall);
-       // Let the insets convert their font
-       if (pit->isInset(pos)) {
-               InsetOld * inset = pit->getInset(pos);
-               if (isEditableInset(inset)) {
-                       static_cast<UpdatableInset *>(inset)
-                               ->setFont(bv(), fnt, toggleall, true);
-               }
-       }
-
-       // Plug through to version below:
-       setCharFont(pit, pos, font);
-}
-
-
 void LyXText::setCharFont(
        ParagraphList::iterator pit, pos_type pos, LyXFont const & fnt)
 {
@@ -217,10 +196,10 @@ void LyXText::setCharFont(
        if (pit->getDepth()) {
                ParagraphList::iterator tp = pit;
                while (!layoutfont.resolved() &&
-                      tp != ownerParagraphs().end() &&
+                      tp != paragraphs().end() &&
                       tp->getDepth()) {
-                       tp = outerHook(tp, ownerParagraphs());
-                       if (tp != ownerParagraphs().end())
+                       tp = outerHook(tp, paragraphs());
+                       if (tp != paragraphs().end())
                                layoutfont.realize(tp->layout()->font);
                }
        }
@@ -234,46 +213,6 @@ void LyXText::setCharFont(
 }
 
 
-InsetOld * LyXText::getInset() const
-{
-       ParagraphList::iterator pit = cursorPar();
-       pos_type const pos = cursor.pos();
-
-       if (pos < pit->size() && pit->isInset(pos)) {
-               return pit->getInset(pos);
-       }
-       return 0;
-}
-
-
-void LyXText::toggleInset()
-{
-       InsetOld * inset = getInset();
-       // is there an editable inset at cursor position?
-       if (!isEditableInset(inset)) {
-               // No, try to see if we are inside a collapsable inset
-               if (inset_owner && inset_owner->owner()
-                   && inset_owner->owner()->isOpen()) {
-                       finishUndo();
-                       inset_owner->owner()->close();
-                       bv()->getLyXText()->cursorRight(true);
-                       bv()->updateParagraphDialog();
-               }
-               return;
-       }
-       //bv()->owner()->message(inset->editMessage());
-
-       // do we want to keep this?? (JMarc)
-       if (!isHighlyEditableInset(inset))
-               recUndo(cursor.par());
-
-       if (inset->isOpen())
-               inset->close();
-       else
-               inset->open();
-}
-
-
 // used in setLayout
 // Asger is not sure we want to do this...
 void LyXText::makeFontEntriesLayoutSpecific(BufferParams const & params,
@@ -296,68 +235,52 @@ void LyXText::makeFontEntriesLayoutSpecific(BufferParams const & params,
 }
 
 
-ParagraphList::iterator
-LyXText::setLayout(LyXCursor & cur, LyXCursor & sstart_cur,
-                  LyXCursor & send_cur,
-                  string const & layout)
+// return past-the-last paragraph influenced by a layout change on pit
+ParagraphList::iterator LyXText::undoSpan(ParagraphList::iterator pit)
 {
-       ParagraphList::iterator endpit = boost::next(getPar(send_cur));
-       ParagraphList::iterator undoendpit = endpit;
-       ParagraphList::iterator pars_end = ownerParagraphs().end();
-
-       if (endpit != pars_end && endpit->getDepth()) {
-               while (endpit != pars_end && endpit->getDepth()) {
-                       ++endpit;
-                       undoendpit = endpit;
-               }
-       } else if (endpit != pars_end) {
-               // because of parindents etc.
-               ++endpit;
+       ParagraphList::iterator end = paragraphs().end();
+       ParagraphList::iterator nextpit = boost::next(pit);
+       if (nextpit == end)
+               return nextpit;
+       //because of parindents
+       if (!pit->getDepth())
+               return boost::next(nextpit);
+       //because of depth constrains
+       for (; nextpit != end; ++pit, ++nextpit) {
+               if (!pit->getDepth())
+                       break;
        }
+       return nextpit;
+}
 
-       recUndo(sstart_cur.par(), parOffset(undoendpit) - 1);
 
-       // ok we have a selection. This is always between sstart_cur
-       // and sel_end cursor
-       cur = sstart_cur;
-       ParagraphList::iterator pit = getPar(sstart_cur);
-       ParagraphList::iterator epit = boost::next(getPar(send_cur));
+ParagraphList::iterator
+LyXText::setLayout(ParagraphList::iterator start,
+                  ParagraphList::iterator end,
+                  string const & layout)
+{
+       BOOST_ASSERT(start != end);
+       ParagraphList::iterator undopit = undoSpan(boost::prior(end));
+       recUndo(parOffset(start), parOffset(undopit) - 1);
 
        BufferParams const & bufparams = bv()->buffer()->params();
        LyXLayout_ptr const & lyxlayout =
                bufparams.getLyXTextClass()[layout];
 
-       do {
+       for (ParagraphList::iterator pit = start; pit != end; ++pit) {
                pit->applyLayout(lyxlayout);
                makeFontEntriesLayoutSpecific(bufparams, *pit);
-               pit->params().spaceTop(lyxlayout->fill_top ?
-                                        VSpace(VSpace::VFILL)
-                                        : VSpace(VSpace::NONE));
-               pit->params().spaceBottom(lyxlayout->fill_bottom ?
-                                           VSpace(VSpace::VFILL)
-                                           : VSpace(VSpace::NONE));
                if (lyxlayout->margintype == MARGIN_MANUAL)
                        pit->setLabelWidthString(lyxlayout->labelstring());
-               cur.par(std::distance(ownerParagraphs().begin(), pit));
-               ++pit;
-       } while (pit != epit);
+       }
 
-       return endpit;
+       return undopit;
 }
 
 
 // set layout over selection and make a total rebreak of those paragraphs
 void LyXText::setLayout(string const & layout)
 {
-       LyXCursor tmpcursor = cursor;  // store the current cursor
-
-       // if there is no selection just set the layout
-       // of the current paragraph
-       if (!selection.set()) {
-               selection.start = cursor;  // dummy selection
-               selection.end = cursor;
-       }
-
        // special handling of new environment insets
        BufferParams const & params = bv()->buffer()->params();
        LyXLayout_ptr const & lyxlayout = params.getLyXTextClass()[layout];
@@ -367,29 +290,23 @@ void LyXText::setLayout(string const & layout)
                bv()->owner()->dispatch(FuncRequest(LFUN_HOME));
                bv()->owner()->dispatch(FuncRequest(LFUN_ENDSEL));
                bv()->owner()->dispatch(FuncRequest(LFUN_CUT));
-               InsetOld * inset = new InsetEnvironment(params, layout);
+               InsetBase * inset = new InsetEnvironment(params, layout);
                if (bv()->insertInset(inset)) {
                        //inset->edit(bv());
                        //bv()->owner()->dispatch(FuncRequest(LFUN_PASTE));
-               }
-               else
+               } else
                        delete inset;
                return;
        }
 
-       ParagraphList::iterator endpit = setLayout(cursor, selection.start,
-                                                  selection.end, layout);
-       redoParagraphs(getPar(selection.start), endpit);
+       ParagraphList::iterator start =
+               getPar(bv()->cursor().selBegin().par());
+       ParagraphList::iterator end =
+               boost::next(getPar(bv()->cursor().selEnd().par()));
+       ParagraphList::iterator endpit = setLayout(start, end, layout);
 
-       // we have to reset the selection, because the
-       // geometry could have changed
-       setCursor(selection.start.par(), selection.start.pos(), false);
-       selection.cursor = cursor;
-       setCursor(selection.end.par(), selection.end.pos(), false);
+       redoParagraphs(start, endpit);
        updateCounters();
-       clearSelection();
-       setSelection();
-       setCursor(tmpcursor.par(), tmpcursor.pos(), true);
 }
 
 
@@ -400,12 +317,12 @@ void getSelectionSpan(LyXText & text,
        ParagraphList::iterator & beg,
        ParagraphList::iterator & end)
 {
-       if (!text.selection.set()) {
+       if (!text.bv()->cursor().selection()) {
                beg = text.cursorPar();
                end = boost::next(beg);
        } else {
-               beg = text.getPar(text.selection.start);
-               end = boost::next(text.getPar(text.selection.end));
+               beg = text.getPar(text.bv()->cursor().selBegin());
+               end = boost::next(text.getPar(text.bv()->cursor().selEnd()));
        }
 }
 
@@ -433,7 +350,7 @@ bool LyXText::changeDepthAllowed(bv_funcs::DEPTH_CHANGE type)
        ParagraphList::iterator beg, end; 
        getSelectionSpan(*this, beg, end);
        int max_depth = 0;
-       if (beg != ownerParagraphs().begin())
+       if (beg != paragraphs().begin())
                max_depth = boost::prior(beg)->getMaxDepthAfter();
 
        for (ParagraphList::iterator pit = beg; pit != end; ++pit) {
@@ -453,7 +370,7 @@ void LyXText::changeDepth(bv_funcs::DEPTH_CHANGE type)
        recUndo(parOffset(beg), parOffset(end) - 1);
 
        int max_depth = 0;
-       if (beg != ownerParagraphs().begin())
+       if (beg != paragraphs().begin())
                max_depth = boost::prior(beg)->getMaxDepthAfter();
 
        for (ParagraphList::iterator pit = beg; pit != end; ++pit) {
@@ -469,22 +386,22 @@ void LyXText::changeDepth(bv_funcs::DEPTH_CHANGE type)
        // this handles the counter labels, and also fixes up
        // depth values for follow-on (child) paragraphs
        updateCounters();
-       redoCursor();
 }
 
 
 // set font over selection and make a total rebreak of those paragraphs
 void LyXText::setFont(LyXFont const & font, bool toggleall)
 {
+       LCursor & cur = bv()->cursor();
        // if there is no selection just set the current_font
-       if (!selection.set()) {
+       if (!cur.selection()) {
                // Determine basis font
                LyXFont layoutfont;
-               if (cursor.pos() < cursorPar()->beginOfBody()) {
+               if (cursor().pos() < cursorPar()->beginOfBody())
                        layoutfont = getLabelFont(cursorPar());
-               } else {
+               else
                        layoutfont = getLayoutFont(cursorPar());
-               }
+
                // Update current font
                real_current_font.update(font,
                                         bv()->buffer()->params().language,
@@ -499,76 +416,44 @@ void LyXText::setFont(LyXFont const & font, bool toggleall)
                return;
        }
 
-       LyXCursor tmpcursor = cursor; // store the current cursor
+       // ok we have a selection.
+       recUndo(cur.selBegin().par(), cur.selEnd().par());
+       freezeUndo();
 
-       // ok we have a selection. This is always between sel_start_cursor
-       // and sel_end cursor
+       ParagraphList::iterator beg = getPar(cur.selBegin().par());
+       ParagraphList::iterator end = getPar(cur.selEnd().par());
+       
+       PosIterator pos(&paragraphs(), beg, cur.selBegin().pos());
+       PosIterator posend(&paragraphs(), end, cur.selEnd().pos());
 
-       recUndo(selection.start.par(), selection.end.par());
-       freezeUndo();
-       cursor = selection.start;
-       while (cursor.par() != selection.end.par() ||
-              cursor.pos() < selection.end.pos())
-       {
-               if (cursor.pos() < cursorPar()->size()) {
-                       // an open footnote should behave like a closed one
-                       setCharFont(cursorPar(), cursor.pos(), font, toggleall);
-                       cursor.pos(cursor.pos() + 1);
-               } else {
-                       cursor.pos(0);
-                       cursor.par(cursor.par() + 1);
-               }
+       BufferParams const & params = bv()->buffer()->params();
+
+       for (; pos != posend; ++pos) {
+               LyXFont f = getFont(pos.pit(), pos.pos());
+               f.update(font, params.language, toggleall);
+               setCharFont(pos.pit(), pos.pos(), f);
        }
+       
        unFreezeUndo();
 
-       redoParagraph(getPar(selection.start));
-
-       // we have to reset the selection, because the
-       // geometry could have changed, but we keep
-       // it for user convenience
-       setCursor(selection.start.par(), selection.start.pos());
-       selection.cursor = cursor;
-       setCursor(selection.end.par(), selection.end.pos());
-       setSelection();
-       setCursor(tmpcursor.par(), tmpcursor.pos(), true,
-                 tmpcursor.boundary());
+       redoParagraphs(beg, ++end);
 }
 
 
-// important for the screen
-
-
 // the cursor set functions have a special mechanism. When they
-// realize, that you left an empty paragraph, they will delete it.
-
-// need the selection cursor:
-void LyXText::setSelection()
-{
-       TextCursor::setSelection();
-}
-
-
-void LyXText::clearSelection()
-{
-       TextCursor::clearSelection();
-
-       // reset this in the bv()!
-       if (bv() && bv()->text)
-               bv()->text->xsel_cache.set(false);
-}
-
+// realize you left an empty paragraph, they will delete it.
 
 void LyXText::cursorHome()
 {
        ParagraphList::iterator cpit = cursorPar();
-       setCursor(cpit, cpit->getRow(cursor.pos())->pos());
+       setCursor(cpit, cpit->getRow(cursor().pos())->pos());
 }
 
 
 void LyXText::cursorEnd()
 {
        ParagraphList::iterator cpit = cursorPar();
-       pos_type end = cpit->getRow(cursor.pos())->endpos();
+       pos_type end = cpit->getRow(cursor().pos())->endpos();
        // if not on the last row of the par, put the cursor before
        // the final space
        setCursor(cpit, end == cpit->size() ? end : end - 1);
@@ -577,14 +462,14 @@ void LyXText::cursorEnd()
 
 void LyXText::cursorTop()
 {
-       setCursor(ownerParagraphs().begin(), 0);
+       setCursor(paragraphs().begin(), 0);
 }
 
 
 void LyXText::cursorBottom()
 {
        ParagraphList::iterator lastpit =
-               boost::prior(ownerParagraphs().end());
+               boost::prior(paragraphs().end());
        setCursor(lastpit, lastpit->size());
 }
 
@@ -602,7 +487,7 @@ void LyXText::toggleFree(LyXFont const & font, bool toggleall)
        // Try implicit word selection
        // If there is a change in the language the implicit word selection
        // is disabled.
-       LyXCursor resetCursor = cursor;
+       CursorSlice resetCursor = cursor();
        bool implicitSelection =
                font.language() == ignore_language
                && font.number() == LyXFont::IGNORE
@@ -614,89 +499,62 @@ void LyXText::toggleFree(LyXFont const & font, bool toggleall)
        // Implicit selections are cleared afterwards
        //and cursor is set to the original position.
        if (implicitSelection) {
-               clearSelection();
-               cursor = resetCursor;
-               setCursor(cursorPar(), cursor.pos());
-               selection.cursor = cursor;
+               bv()->cursor().clearSelection();
+               cursor() = resetCursor;
+               bv()->cursor().resetAnchor();
        }
 }
 
 
 string LyXText::getStringToIndex()
 {
+       LCursor & cur = bv()->cursor();
        // Try implicit word selection
        // If there is a change in the language the implicit word selection
        // is disabled.
-       LyXCursor const reset_cursor = cursor;
+       CursorSlice const reset_cursor = cursor();
        bool const implicitSelection =
                selectWordWhenUnderCursor(lyx::PREVIOUS_WORD);
 
        string idxstring;
-       if (!selection.set())
+       if (!cur.selection())
                bv()->owner()->message(_("Nothing to index!"));
-       else if (selection.start.par() != selection.end.par())
+       else if (cur.selBegin().par() != cur.selEnd().par())
                bv()->owner()->message(_("Cannot index more than one paragraph!"));
        else
                idxstring = selectionAsString(*bv()->buffer(), false);
 
        // Reset cursors to their original position.
-       cursor = reset_cursor;
-       setCursor(cursorPar(), cursor.pos());
-       selection.cursor = cursor;
+       cursor() = reset_cursor;
+       cur.resetAnchor();
 
        // Clear the implicit selection.
        if (implicitSelection)
-               clearSelection();
+               cur.clearSelection();
 
        return idxstring;
 }
 
 
-// the DTP switches for paragraphs. LyX will store them in the first
+// the DTP switches for paragraphs(). LyX will store them in the first
 // physical paragraph. When a paragraph is broken, the top settings rest,
 // the bottom settings are given to the new one. So I can make sure,
 // they do not duplicate themself and you cannot play dirty tricks with
 // them!
 
-void LyXText::setParagraph(VSpace const & space_top,
-                          VSpace const & space_bottom,
-                          Spacing const & spacing,
-                          LyXAlignment align,
-                          string const & labelwidthstring,
-                          bool noindent)
-{
-       LyXCursor tmpcursor = cursor;
-       if (!selection.set()) {
-               selection.start = cursor;
-               selection.end = cursor;
-       }
-
+void LyXText::setParagraph(Spacing const & spacing, LyXAlignment align,
+       string const & labelwidthstring, bool noindent)
+{
+       LCursor & cur = bv()->cursor();
        // make sure that the depth behind the selection are restored, too
-       ParagraphList::iterator endpit = boost::next(getPar(selection.end));
-       ParagraphList::iterator undoendpit = endpit;
-       ParagraphList::iterator pars_end = ownerParagraphs().end();
+       ParagraphList::iterator undopit = undoSpan(getPar(cur.selEnd()));
+       recUndo(cur.selBegin().par(), parOffset(undopit) - 1);
 
-       if (endpit != pars_end && endpit->getDepth()) {
-               while (endpit != pars_end && endpit->getDepth()) {
-                       ++endpit;
-                       undoendpit = endpit;
-               }
-       } else if (endpit != pars_end) {
-               // because of parindents etc.
-               ++endpit;
-       }
-
-       recUndo(selection.start.par(), parOffset(undoendpit) - 1);
-
-       int tmppit = selection.end.par();
+       ParagraphList::reverse_iterator pit(getPar(cur.selEnd().par()));
+       ParagraphList::reverse_iterator beg(getPar(cur.selBegin().par()));
 
-       while (tmppit != selection.start.par() - 1) {
-               setCursor(tmppit, 0);
-
-               ParagraphList::iterator const pit = cursorPar();
+       for (--pit; pit != beg; ++pit) {
                ParagraphParameters & params = pit->params();
-               params.spaceTop(space_top);
-               params.spaceBottom(space_bottom);
                params.spacing(spacing);
 
                // does the layout allow the new alignment?
@@ -712,17 +570,9 @@ void LyXText::setParagraph(VSpace const & space_top,
                }
                pit->setLabelWidthString(labelwidthstring);
                params.noindent(noindent);
-               --tmppit;
        }
 
-       redoParagraphs(getPar(selection.start), endpit);
-
-       clearSelection();
-       setCursor(selection.start.par(), selection.start.pos());
-       selection.cursor = cursor;
-       setCursor(selection.end.par(), selection.end.pos());
-       setSelection();
-       setCursor(tmpcursor.par(), tmpcursor.pos());
+       redoParagraphs(getPar(cur.selBegin()), undopit);
 }
 
 
@@ -832,7 +682,7 @@ void LyXText::setCounter(Buffer const & buf, ParagraphList::iterator pit)
        BufferParams const & bufparams = buf.params();
        LyXTextClass const & textclass = bufparams.getLyXTextClass();
        LyXLayout_ptr const & layout = pit->layout();
-       ParagraphList::iterator first_pit = ownerParagraphs().begin();
+       ParagraphList::iterator first_pit = paragraphs().begin();
        Counters & counters = textclass.counters();
 
        // Always reset
@@ -933,16 +783,16 @@ void LyXText::setCounter(Buffer const & buf, ParagraphList::iterator pit)
 
                // the caption hack:
                if (layout->labeltype == LABEL_SENSITIVE) {
-                       ParagraphList::iterator end = ownerParagraphs().end();
+                       ParagraphList::iterator end = paragraphs().end();
                        ParagraphList::iterator tmppit = pit;
-                       InsetOld * in = 0;
+                       InsetBase * in = 0;
                        bool isOK = false;
                        while (tmppit != end && tmppit->inInset()
                               // the single '=' is intended below
                               && (in = tmppit->inInset()->owner()))
                        {
-                               if (in->lyxCode() == InsetOld::FLOAT_CODE ||
-                                   in->lyxCode() == InsetOld::WRAP_CODE) {
+                               if (in->lyxCode() == InsetBase::FLOAT_CODE ||
+                                   in->lyxCode() == InsetBase::WRAP_CODE) {
                                        isOK = true;
                                        break;
                                } else {
@@ -957,9 +807,9 @@ void LyXText::setCounter(Buffer const & buf, ParagraphList::iterator pit)
                        if (isOK) {
                                string type;
 
-                               if (in->lyxCode() == InsetOld::FLOAT_CODE)
+                               if (in->lyxCode() == InsetBase::FLOAT_CODE)
                                        type = static_cast<InsetFloat*>(in)->params().type;
-                               else if (in->lyxCode() == InsetOld::WRAP_CODE)
+                               else if (in->lyxCode() == InsetBase::WRAP_CODE)
                                        type = static_cast<InsetWrap*>(in)->params().type;
                                else
                                        BOOST_ASSERT(false);
@@ -990,8 +840,8 @@ void LyXText::updateCounters()
 
        bool update_pos = false;
        
-       ParagraphList::iterator beg = ownerParagraphs().begin();
-       ParagraphList::iterator end = ownerParagraphs().end();
+       ParagraphList::iterator beg = paragraphs().begin();
+       ParagraphList::iterator end = paragraphs().end();
        for (ParagraphList::iterator pit = beg; pit != end; ++pit) {
                string const oldLabel = pit->params().labelString();
                size_t maxdepth = 0;
@@ -1015,14 +865,14 @@ void LyXText::updateCounters()
 }
 
 
-void LyXText::insertInset(InsetOld * inset)
+void LyXText::insertInset(InsetBase * inset)
 {
        if (!cursorPar()->insetAllowed(inset->lyxCode()))
                return;
 
-       recUndo(cursor.par());
+       recUndo(cursor().par());
        freezeUndo();
-       cursorPar()->insertInset(cursor.pos(), inset);
+       cursorPar()->insertInset(cursor().pos(), inset);
        // Just to rebreak and refresh correctly.
        // The character will not be inserted a second time
        insertChar(Paragraph::META_INSET);
@@ -1039,6 +889,7 @@ void LyXText::insertInset(InsetOld * inset)
 
 void LyXText::cutSelection(bool doclear, bool realcut)
 {
+       LCursor & cur = bv()->cursor();
        // Stuff what we got on the clipboard. Even if there is no selection.
 
        // There is a problem with having the stuffing here in that the
@@ -1050,98 +901,86 @@ void LyXText::cutSelection(bool doclear, bool realcut)
        bv()->stuffClipboard(selectionAsString(*bv()->buffer(), true));
 
        // This doesn't make sense, if there is no selection
-       if (!selection.set())
+       if (!cur.selection())
                return;
 
-       // OK, we have a selection. This is always between selection.start
-       // and selection.end
+       // OK, we have a selection. This is always between cur.selBegin()
+       // and cur.selEnd()
 
        // make sure that the depth behind the selection are restored, too
-       ParagraphList::iterator endpit = boost::next(getPar(selection.end.par()));
-       ParagraphList::iterator undoendpit = endpit;
-       ParagraphList::iterator pars_end = ownerParagraphs().end();
-
-       if (endpit != pars_end && endpit->getDepth()) {
-               while (endpit != pars_end && endpit->getDepth()) {
-                       ++endpit;
-                       undoendpit = endpit;
-               }
-       } else if (endpit != pars_end) {
-               // because of parindents etc.
-               ++endpit;
-       }
-
-       recUndo(selection.start.par(), parOffset(undoendpit) - 1);
+       ParagraphList::iterator begpit = getPar(cur.selBegin().par());
+       ParagraphList::iterator endpit = getPar(cur.selEnd().par());
+       ParagraphList::iterator undopit = undoSpan(endpit);
+       recUndo(cur.selBegin().par(), parOffset(undopit) - 1);
 
-       endpit = getPar(selection.end.par());
-       int endpos = selection.end.pos();
+       int endpos = cur.selEnd().pos();
 
        BufferParams const & bufparams = bv()->buffer()->params();
        boost::tie(endpit, endpos) = realcut ?
                CutAndPaste::cutSelection(bufparams,
-                                         ownerParagraphs(),
-                                         getPar(selection.start.par()), endpit,
-                                         selection.start.pos(), endpos,
+                                         paragraphs(),
+                                         begpit , endpit,
+                                         cur.selBegin().pos(), endpos,
                                          bufparams.textclass,
                                          doclear)
                : CutAndPaste::eraseSelection(bufparams,
-                                             ownerParagraphs(),
-                                             getPar(selection.start.par()), endpit,
-                                             selection.start.pos(), endpos,
+                                             paragraphs(),
+                                             begpit, endpit,
+                                             cur.selBegin().pos(), endpos,
                                              doclear);
        // sometimes necessary
        if (doclear)
-               getPar(selection.start.par())->stripLeadingSpaces();
+               begpit->stripLeadingSpaces();
 
-       redoParagraphs(getPar(selection.start.par()), boost::next(endpit));
+       redoParagraphs(begpit, undopit);
        // cutSelection can invalidate the cursor so we need to set
        // it anew. (Lgb)
        // we prefer the end for when tracking changes
-       cursor.pos(endpos);
-       cursor.par(parOffset(endpit));
+       cursor().pos(endpos);
+       cursor().par(parOffset(endpit));
 
        // need a valid cursor. (Lgb)
-       clearSelection();
-
-       setCursor(cursorPar(), cursor.pos());
-       selection.cursor = cursor;
+       cur.clearSelection();
        updateCounters();
 }
 
 
 void LyXText::copySelection()
 {
+       LCursor & cur = bv()->cursor();
        // stuff the selection onto the X clipboard, from an explicit copy request
        bv()->stuffClipboard(selectionAsString(*bv()->buffer(), true));
 
        // this doesnt make sense, if there is no selection
-       if (!selection.set())
+       if (!cur.selection())
                return;
 
-       // ok we have a selection. This is always between selection.start
+       // ok we have a selection. This is always between cur.selBegin()
        // and sel_end cursor
 
        // copy behind a space if there is one
-       while (getPar(selection.start)->size() > selection.start.pos()
-              && getPar(selection.start)->isLineSeparator(selection.start.pos())
-              && (selection.start.par() != selection.end.par()
-                  || selection.start.pos() < selection.end.pos()))
-               selection.start.pos(selection.start.pos() + 1);
-
-       CutAndPaste::copySelection(getPar(selection.start.par()),
-                                  getPar(selection.end.par()),
-                                  selection.start.pos(), selection.end.pos(),
+       while (getPar(cur.selBegin())->size() > cur.selBegin().pos()
+              && getPar(cur.selBegin())->isLineSeparator(cur.selBegin().pos())
+              && (cur.selBegin().par() != cur.selEnd().par()
+                  || cur.selBegin().pos() < cur.selEnd().pos()))
+               cur.selBegin().pos(cur.selBegin().pos() + 1);
+
+       CutAndPaste::copySelection(getPar(cur.selBegin().par()),
+                                  getPar(cur.selEnd().par()),
+                                  cur.selBegin().pos(), 
+                                  cur.selEnd().pos(),
                                   bv()->buffer()->params().textclass);
 }
 
 
 void LyXText::pasteSelection(size_t sel_index)
 {
+       LCursor & cur = bv()->cursor();
        // this does not make sense, if there is nothing to paste
        if (!CutAndPaste::checkPastePossible())
                return;
 
-       recUndo(cursor.par());
+       recUndo(cursor().par());
 
        ParagraphList::iterator endpit;
        PitPosPair ppp;
@@ -1150,8 +989,8 @@ void LyXText::pasteSelection(size_t sel_index)
 
        boost::tie(ppp, endpit) =
                CutAndPaste::pasteSelection(*bv()->buffer(),
-                                           ownerParagraphs(),
-                                           cursorPar(), cursor.pos(),
+                                           paragraphs(),
+                                           cursorPar(), cursor().pos(),
                                            bv()->buffer()->params().textclass,
                                            sel_index, el);
        bufferErrors(*bv()->buffer(), el);
@@ -1159,12 +998,10 @@ void LyXText::pasteSelection(size_t sel_index)
 
        redoParagraphs(cursorPar(), endpit);
 
-       setCursor(cursor.par(), cursor.pos());
-       clearSelection();
-
-       selection.cursor = cursor;
+       cur.clearSelection();
+       cur.resetAnchor();
        setCursor(ppp.first, ppp.second);
-       setSelection();
+       cur.setSelection();
        updateCounters();
 }
 
@@ -1174,35 +1011,32 @@ void LyXText::setSelectionRange(lyx::pos_type length)
        if (!length)
                return;
 
-       selection.cursor = cursor;
+       LCursor & cur = bv()->cursor();
+       cur.resetAnchor();
        while (length--)
                cursorRight(true);
-       setSelection();
+       cur.setSelection();
 }
 
 
 // simple replacing. The font of the first selected character is used
 void LyXText::replaceSelectionWithString(string const & str)
 {
-       recUndo(cursor.par());
+       LCursor & cur = bv()->cursor();
+       recUndo(cur.par());
        freezeUndo();
 
-       if (!selection.set()) { // create a dummy selection
-               selection.end = cursor;
-               selection.start = cursor;
-       }
-
        // Get font setting before we cut
-       pos_type pos = selection.end.pos();
-       LyXFont const font = getPar(selection.start)
+       pos_type pos = cur.selEnd().pos();
+       LyXFont const font = getPar(cur.selBegin())
                ->getFontSettings(bv()->buffer()->params(),
-                                 selection.start.pos());
+                                 cur.selBegin().pos());
 
        // Insert the new string
        string::const_iterator cit = str.begin();
        string::const_iterator end = str.end();
        for (; cit != end; ++cit) {
-               getPar(selection.end)->insertChar(pos, (*cit), font);
+               getPar(cur.selEnd())->insertChar(pos, (*cit), font);
                ++pos;
        }
 
@@ -1216,27 +1050,25 @@ void LyXText::replaceSelectionWithString(string const & str)
 // needed to insert the selection
 void LyXText::insertStringAsLines(string const & str)
 {
+       LCursor & cur = bv()->cursor();
        ParagraphList::iterator pit = cursorPar();
-       pos_type pos = cursor.pos();
+       pos_type pos = cursor().pos();
        ParagraphList::iterator endpit = boost::next(cursorPar());
-
-       recUndo(cursor.par());
+       recordUndo(cur, Undo::ATOMIC);
 
        // only to be sure, should not be neccessary
-       clearSelection();
-
+       cur.clearSelection();
        bv()->buffer()->insertStringAsLines(pit, pos, current_font, str);
 
        redoParagraphs(cursorPar(), endpit);
-       setCursor(cursorPar(), cursor.pos());
-       selection.cursor = cursor;
+       cur.resetAnchor();
        setCursor(pit, pos);
-       setSelection();
+       cur.setSelection();
 }
 
 
-// turns double-CR to single CR, others where converted into one
-// blank. Then InsertStringAsLines is called
+// turn double CR to single CR, others are converted into one
+// blank. Then insertStringAsLines is called
 void LyXText::insertStringAsParagraphs(string const & str)
 {
        string linestr(str);
@@ -1247,7 +1079,7 @@ void LyXText::insertStringAsParagraphs(string const & str)
                if (linestr[i] == '\n') {
                        if (newline_inserted) {
                                // we know that \r will be ignored by
-                               // InsertStringA. Of course, it is a dirty
+                               // insertStringAsLines. Of course, it is a dirty
                                // trick, but it works...
                                linestr[i - 1] = '\r';
                                linestr[i] = '\n';
@@ -1269,54 +1101,31 @@ void LyXText::setCursor(ParagraphList::iterator pit, pos_type pos)
 }
 
 
-bool LyXText::setCursor(paroffset_type par, pos_type pos, bool setfont, bool boundary)
+bool LyXText::setCursor(paroffset_type par, pos_type pos, bool setfont,
+       bool boundary)
 {
-       LyXCursor old_cursor = cursor;
+       CursorSlice old_cursor = cursor();
        setCursorIntern(par, pos, setfont, boundary);
        return deleteEmptyParagraphMechanism(old_cursor);
 }
 
 
-void LyXText::redoCursor()
-{
-       setCursor(cursor, cursor.par(), cursor.pos(), cursor.boundary());
-
-       if (!selection.set())
-               return;
-
-       LyXCursor tmpcursor = cursor;
-       setCursor(selection.cursor.par(), selection.cursor.pos());
-       selection.cursor = cursor;
-       setCursor(tmpcursor.par(), tmpcursor.pos());
-       setSelection();
-}
-
-
-void LyXText::setCursor(LyXCursor & cur, paroffset_type par,
+void LyXText::setCursor(CursorSlice & cur, paroffset_type par,
        pos_type pos, bool boundary)
 {
-       BOOST_ASSERT(par != int(ownerParagraphs().size()));
+       BOOST_ASSERT(par != int(paragraphs().size()));
 
        cur.par(par);
        cur.pos(pos);
        cur.boundary(boundary);
 
        // no rows, no fun...
-       if (ownerParagraphs().begin()->rows.empty())
+       if (paragraphs().begin()->rows.empty())
                return;
 
-       // get the cursor y position in text
-
-       ParagraphList::iterator pit = getPar(par);
-       Row const & row = *pit->getRow(pos);
-
-       int y = pit->y + row.y_offset();
-
-       // y is now the beginning of the cursor row
-       y += row.baseline();
-       // y is now the cursor baseline
-       cur.y(y);
-
+       // now some strict checking
+       Paragraph & para = *getPar(par);
+       Row const & row = *para.getRow(pos);
        pos_type const end = row.endpos();
 
        // None of these should happen, but we're scaredy-cats
@@ -1325,9 +1134,9 @@ void LyXText::setCursor(LyXCursor & cur, paroffset_type par,
                pos = 0;
                cur.pos(0);
                BOOST_ASSERT(false);
-       } else if (pos > pit->size()) {
+       } else if (pos > para.size()) {
                lyxerr << "dont like 1, pos: " << pos
-                      << " size: " << pit->size()
+                      << " size: " << para.size()
                       << " row.pos():" << row.pos()
                       << " paroffset: " << par << endl;
                pos = 0;
@@ -1341,80 +1150,21 @@ void LyXText::setCursor(LyXCursor & cur, paroffset_type par,
                BOOST_ASSERT(false);
        } else if (pos < row.pos()) {
                lyxerr << "dont like 3 please report pos:" << pos
-                      << " size: " << pit->size()
+                      << " size: " << para.size()
                       << " row.pos():" << row.pos()
                       << " paroffset: " << par << endl;
                pos = row.pos();
                cur.pos(pos);
                BOOST_ASSERT(false);
        }
-       // now get the cursors x position
-       cur.x(int(getCursorX(pit, row, pos, boundary)));
-}
-
-
-float LyXText::getCursorX(ParagraphList::iterator pit, Row const & row,
-                         pos_type pos, bool boundary) const
-{
-       pos_type cursor_vpos    = 0;
-       double x                = row.x();
-       double fill_separator   = row.fill_separator();
-       double fill_hfill       = row.fill_hfill();
-       double fill_label_hfill = row.fill_label_hfill();
-       pos_type const row_pos  = row.pos();
-       pos_type const end = row.endpos();
-
-       if (end <= row_pos)
-               cursor_vpos = row_pos;
-       else if (pos >= end && !boundary)
-               cursor_vpos = (pit->isRightToLeftPar(bv()->buffer()->params()))
-                       ? row_pos : end;
-       else if (pos > row_pos && (pos >= end || boundary))
-               // Place cursor after char at (logical) position pos - 1
-               cursor_vpos = (bidi.level(pos - 1) % 2 == 0)
-                       ? bidi.log2vis(pos - 1) + 1 : bidi.log2vis(pos - 1);
-       else
-               // Place cursor before char at (logical) position pos
-               cursor_vpos = (bidi.level(pos) % 2 == 0)
-                       ? bidi.log2vis(pos) : bidi.log2vis(pos) + 1;
-
-       pos_type body_pos = pit->beginOfBody();
-       if (body_pos > 0 &&
-           (body_pos > end || !pit->isLineSeparator(body_pos - 1)))
-               body_pos = 0;
-
-       for (pos_type vpos = row_pos; vpos < cursor_vpos; ++vpos) {
-               pos_type pos = bidi.vis2log(vpos);
-               if (body_pos > 0 && pos == body_pos - 1) {
-                       x += fill_label_hfill
-                               + font_metrics::width(pit->layout()->labelsep,
-                                                     getLabelFont(pit));
-                       if (pit->isLineSeparator(body_pos - 1))
-                               x -= singleWidth(pit, body_pos - 1);
-               }
-
-               if (hfillExpansion(*pit, row, pos)) {
-                       x += singleWidth(pit, pos);
-                       if (pos >= body_pos)
-                               x += fill_hfill;
-                       else
-                               x += fill_label_hfill;
-               } else if (pit->isSeparator(pos)) {
-                       x += singleWidth(pit, pos);
-                       if (pos >= body_pos)
-                               x += fill_separator;
-               } else
-                       x += singleWidth(pit, pos);
-       }
-       return x;
 }
 
 
 void LyXText::setCursorIntern(paroffset_type par,
                              pos_type pos, bool setfont, bool boundary)
 {
-       setCursor(cursor, par, pos, boundary);
-       bv()->x_target(cursor.x() + xo_);
+       setCursor(cursor(), par, pos, boundary);
+       bv()->cursor().x_target() = cursorX(cursor());
        if (setfont)
                setCurrentFont();
 }
@@ -1422,10 +1172,11 @@ void LyXText::setCursorIntern(paroffset_type par,
 
 void LyXText::setCurrentFont()
 {
-       pos_type pos = cursor.pos();
+       LCursor & cur = bv()->cursor();
+       pos_type pos = cur.pos();
        ParagraphList::iterator pit = cursorPar();
 
-       if (cursor.boundary() && pos > 0)
+       if (cursor().boundary() && pos > 0)
                --pos;
 
        if (pos > 0) {
@@ -1446,9 +1197,9 @@ void LyXText::setCurrentFont()
        current_font = pit->getFontSettings(bufparams, pos);
        real_current_font = getFont(pit, pos);
 
-       if (cursor.pos() == pit->size() &&
-           bidi.isBoundary(*bv()->buffer(), *pit, cursor.pos()) &&
-           !cursor.boundary()) {
+       if (cursor().pos() == pit->size() &&
+           bidi.isBoundary(*bv()->buffer(), *pit, cursor().pos()) &&
+           !cursor().boundary()) {
                Language const * lang =
                        pit->getParLanguage(bufparams);
                current_font.setLanguage(lang);
@@ -1464,6 +1215,7 @@ void LyXText::setCurrentFont()
 pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
        Row const & row, int & x, bool & boundary) const
 {
+       x -= xo_;
        double tmpx             = row.x();
        double fill_separator   = row.fill_separator();
        double fill_hfill       = row.fill_hfill();
@@ -1480,13 +1232,12 @@ pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
        double last_tmpx = tmpx;
 
        if (body_pos > 0 &&
-           (body_pos > end ||
-            !pit->isLineSeparator(body_pos - 1)))
+           (body_pos > end || !pit->isLineSeparator(body_pos - 1)))
                body_pos = 0;
 
        // check for empty row
        if (vc == end) {
-               x = int(tmpx);
+               x = int(tmpx) + xo_;
                return 0;
        }
 
@@ -1530,7 +1281,7 @@ pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
 
        // If lastrow is false, we don't need to compute
        // the value of rtl.
-       bool const rtl = (lastrow)
+       bool const rtl = lastrow
                ? pit->isRightToLeftPar(bv()->buffer()->params())
                : false;
        if (lastrow &&
@@ -1558,47 +1309,68 @@ pos_type LyXText::getColumnNearX(ParagraphList::iterator pit,
                c = end - 1;
        }
 
-       c -= row.pos();
-       x = int(tmpx);
-       return c;
+       x = int(tmpx) + xo_;
+       return c - row.pos();
 }
 
 
 void LyXText::setCursorFromCoordinates(int x, int y)
 {
-       LyXCursor old_cursor = cursor;
-       setCursorFromCoordinates(cursor, x, y);
+       CursorSlice old_cursor = cursor();
+       setCursorFromCoordinates(cursor(), x, y);
        setCurrentFont();
        deleteEmptyParagraphMechanism(old_cursor);
 }
 
+
 // x,y are coordinates relative to this LyXText
-void LyXText::setCursorFromCoordinates(LyXCursor & cur, int x, int y)
+void LyXText::setCursorFromCoordinates(CursorSlice & cur, int x, int y)
 {
-       // Get the row first.
        ParagraphList::iterator pit;
        Row const & row = *getRowNearY(y, pit);
-       y = pit->y + row.y_offset();
-
        bool bound = false;
-       pos_type const column = getColumnNearX(pit, row, x, bound);
-       cur.par(parOffset(pit));
-       cur.pos(row.pos() + column);
-       cur.x(x);
-       cur.y(y + row.baseline());
+       pos_type const pos = row.pos() + getColumnNearX(pit, row, x, bound);
+       cur.par() = parOffset(pit);
+       cur.pos() = pos;
+       cur.boundary() = bound;
+}
 
-       cur.boundary(bound);
+
+// x,y are absolute screen coordinates
+void LyXText::edit(LCursor & cur, int x, int y)
+{
+       int xx = x; // is modified by getColumnNearX
+       ParagraphList::iterator pit;
+       Row const & row = *getRowNearY(y, pit);
+       bool bound = false;
+       pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
+       cur.par() = parOffset(pit);
+       cur.pos() = pos;
+       cur.boundary() = bound;
+
+       // try to descend into nested insets
+       InsetBase * inset = checkInsetHit(x, y);
+       if (inset) {
+               // This should be just before or just behind the cursor position
+               // set above.
+               BOOST_ASSERT((pos != 0 && inset == pit->getInset(pos - 1))
+                            || inset == pit->getInset(pos));
+               // Make sure the cursor points to the position before this inset.
+               if (inset == pit->getInset(pos - 1))
+                       --cur.pos();
+               inset->edit(cur, x, y);
+       }
 }
 
 
 bool LyXText::checkAndActivateInset(bool front)
 {
-       if (cursor.pos() == cursorPar()->size())
+       if (cursor().pos() == cursorPar()->size())
                return false;
-       InsetOld * inset = cursorPar()->getInset(cursor.pos());
+       InsetBase * inset = cursorPar()->getInset(cursor().pos());
        if (!isHighlyEditableInset(inset))
                return false;
-       inset->edit(bv(), front);
+       inset->edit(bv()->cursor(), front);
        return true;
 }
 
@@ -1624,14 +1396,14 @@ DispatchResult LyXText::moveLeft()
 DispatchResult LyXText::moveRightIntern(bool front, bool activate_inset, bool selecting)
 {
        ParagraphList::iterator c_par = cursorPar();
-       if (boost::next(c_par) == ownerParagraphs().end()
-               && cursor.pos() >= c_par->size())
+       if (boost::next(c_par) == paragraphs().end()
+               && cursor().pos() >= c_par->size())
                return DispatchResult(false, FINISHED_RIGHT);
        if (activate_inset && checkAndActivateInset(front))
                return DispatchResult(true, true);
        cursorRight(true);
        if (!selecting)
-               clearSelection();
+               bv()->cursor().clearSelection();
        return DispatchResult(true);
 }
 
@@ -1639,11 +1411,11 @@ DispatchResult LyXText::moveRightIntern(bool front, bool activate_inset, bool se
 DispatchResult LyXText::moveLeftIntern(bool front,
                          bool activate_inset, bool selecting)
 {
-       if (cursor.par() == 0 && cursor.pos() <= 0)
+       if (cursor().par() == 0 && cursor().pos() <= 0)
                return DispatchResult(false, FINISHED);
        cursorLeft(true);
        if (!selecting)
-               clearSelection();
+               bv()->cursor().clearSelection();
        if (activate_inset && checkAndActivateInset(front))
                return DispatchResult(true, true);
        return DispatchResult(true);
@@ -1652,38 +1424,41 @@ DispatchResult LyXText::moveLeftIntern(bool front,
 
 DispatchResult LyXText::moveUp()
 {
-       if (cursorPar() == firstPar() && cursorRow() == firstRow())
+       LCursor & cur = bv()->cursor();
+       if (cur.par() == 0 && cursorRow() == firstRow())
                return DispatchResult(false, FINISHED_UP);
        cursorUp(false);
-       clearSelection();
+       cur.clearSelection();
        return DispatchResult(true);
 }
 
 
 DispatchResult LyXText::moveDown()
 {
-       if (cursorPar() == lastPar() && cursorRow() == lastRow())
+       LCursor & cur = bv()->cursor();
+       if (cur.par() == cur.lastpar() && cursorRow() == lastRow())
                return DispatchResult(false, FINISHED_DOWN);
        cursorDown(false);
-       clearSelection();
+       cur.clearSelection();
        return DispatchResult(true);
 }
 
 
 bool LyXText::cursorLeft(bool internal)
 {
-       if (cursor.pos() > 0) {
-               bool boundary = cursor.boundary();
-               setCursor(cursor.par(), cursor.pos() - 1, true, false);
+       LCursor & cur = bv()->cursor();
+       if (cur.pos() > 0) {
+               bool boundary = cur.boundary();
+               setCursor(cur.par(), cur.pos() - 1, true, false);
                if (!internal && !boundary &&
-                   bidi.isBoundary(*bv()->buffer(), *cursorPar(), cursor.pos() + 1))
-                       setCursor(cursor.par(), cursor.pos() + 1, true, true);
+                   bidi.isBoundary(*bv()->buffer(), cur.paragraph(), cur.pos() + 1))
+                       setCursor(cur.par(), cur.pos() + 1, true, true);
                return true;
        }
 
-       if (cursor.par() != 0) {
+       if (cur.par() != 0) {
                // steps into the paragraph above
-               setCursor(cursor.par() - 1, boost::prior(cursorPar())->size());
+               setCursor(cur.par() - 1, boost::prior(cursorPar())->size());
                return true;
        }
 
@@ -1693,21 +1468,22 @@ bool LyXText::cursorLeft(bool internal)
 
 bool LyXText::cursorRight(bool internal)
 {
-       if (!internal && cursor.boundary()) {
-               setCursor(cursor.par(), cursor.pos(), true, false);
+       LCursor & cur = bv()->cursor();
+       if (!internal && cur.boundary()) {
+               setCursor(cur.par(), cur.pos(), true, false);
                return true;
        }
 
-       if (cursor.pos() != cursorPar()->size()) {
-               setCursor(cursor.par(), cursor.pos() + 1, true, false);
-               if (!internal && bidi.isBoundary(*bv()->buffer(), *cursorPar(),
-                                                cursor.pos()))
-                       setCursor(cursor.par(), cursor.pos(), true, true);
+       if (cur.pos() != cur.lastpos()) {
+               setCursor(cur.par(), cur.pos() + 1, true, false);
+               if (!internal && bidi.isBoundary(*bv()->buffer(), cur.paragraph(),
+                                                cur.pos()))
+                       setCursor(cur.par(), cur.pos(), true, true);
                return true;
        }
 
-       if (cursor.par() + 1 != int(ownerParagraphs().size())) {
-               setCursor(cursor.par() + 1, 0);
+       if (cur.par() + 1 != int(paragraphs().size())) {
+               setCursor(cur.par() + 1, 0);
                return true;
        }
 
@@ -1717,32 +1493,32 @@ bool LyXText::cursorRight(bool internal)
 
 void LyXText::cursorUp(bool selecting)
 {
+       LCursor & cur = bv()->cursor();
        Row const & row = *cursorRow();
-       int x = bv()->x_target() - xo_;
-       int y = cursor.y() - row.baseline() - 1;
+       int x = cur.x_target();
+       int y = cursorY(cur.current()) - row.baseline() - 1;
        setCursorFromCoordinates(x, y);
 
        if (!selecting) {
-               int y_abs = y + yo_ - bv()->top_y();
-               InsetOld * inset_hit = checkInsetHit(bv()->x_target(), y_abs);
+               InsetBase * inset_hit = checkInsetHit(cur.x_target(), y);
                if (inset_hit && isHighlyEditableInset(inset_hit))
-                       inset_hit->edit(bv(), bv()->x_target(), y_abs);
+                       inset_hit->edit(cur, cur.x_target(), y);
        }
 }
 
 
 void LyXText::cursorDown(bool selecting)
 {
+       LCursor & cur = bv()->cursor();
        Row const & row = *cursorRow();
-       int x = bv()->x_target() - xo_;
-       int y = cursor.y() - row.baseline() + row.height() + 1;
+       int x = cur.x_target();
+       int y = cursorY(cur.current()) - row.baseline() + row.height() + 1;
        setCursorFromCoordinates(x, y);
 
        if (!selecting) {
-               int y_abs = y + yo_ - bv()->top_y();
-               InsetOld * inset_hit = checkInsetHit(bv()->x_target(), y_abs);
+               InsetBase * inset_hit = checkInsetHit(cur.x_target(), y);
                if (inset_hit && isHighlyEditableInset(inset_hit))
-                       inset_hit->edit(bv(), bv()->x_target(), y_abs);
+                       inset_hit->edit(cur, cur.x_target(), y);
        }
 }
 
@@ -1750,9 +1526,9 @@ void LyXText::cursorDown(bool selecting)
 void LyXText::cursorUpParagraph()
 {
        ParagraphList::iterator cpit = cursorPar();
-       if (cursor.pos() > 0)
+       if (cursor().pos() > 0)
                setCursor(cpit, 0);
-       else if (cpit != ownerParagraphs().begin())
+       else if (cpit != paragraphs().begin())
                setCursor(boost::prior(cpit), 0);
 }
 
@@ -1762,7 +1538,7 @@ void LyXText::cursorDownParagraph()
        ParagraphList::iterator pit = cursorPar();
        ParagraphList::iterator next_pit = boost::next(pit);
 
-       if (next_pit != ownerParagraphs().end())
+       if (next_pit != paragraphs().end())
                setCursor(next_pit, 0);
        else
                setCursor(pit, pit->size());
@@ -1771,7 +1547,7 @@ void LyXText::cursorDownParagraph()
 
 // fix the cursor `cur' after a characters has been deleted at `where'
 // position. Called by deleteEmptyParagraphMechanism
-void LyXText::fixCursorAfterDelete(LyXCursor & cur, LyXCursor const & where)
+void LyXText::fixCursorAfterDelete(CursorSlice & cur, CursorSlice const & where)
 {
        // if cursor is not in the paragraph where the delete occured,
        // do nothing
@@ -1785,24 +1561,25 @@ void LyXText::fixCursorAfterDelete(LyXCursor & cur, LyXCursor const & where)
 
        // check also if we don't want to set the cursor on a spot behind the
        // pagragraph because we erased the last character.
-       if (cur.pos() > getPar(cur)->size())
-               cur.pos(getPar(cur)->size());
-
-       // recompute row et al. for this cursor
-       setCursor(cur, cur.par(), cur.pos(), cur.boundary());
+       if (cur.pos() > cur.lastpos())
+               cur.pos() = cur.lastpos();
 }
 
 
-bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
+bool LyXText::deleteEmptyParagraphMechanism(CursorSlice const & old_cursor)
 {
+#warning Disabled as it crashes after the cursor data shift... (Andre)
+       return false;
+
        // Would be wrong to delete anything if we have a selection.
-       if (selection.set())
+       if (bv()->cursor().selection())
                return false;
 
        // Don't do anything if the cursor is invalid
        if (old_cursor.par() == -1)
                return false;
 
+#if 0
        // We allow all kinds of "mumbo-jumbo" when freespacing.
        ParagraphList::iterator const old_pit = getPar(old_cursor);
        if (old_pit->isFreeSpacing())
@@ -1814,7 +1591,7 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
           Copy and Paste to hopefully do some sensible things.
           There are still some small problems that can lead to
           double spaces stored in the document file or space at
-          the beginning of paragraphs. This happens if you have
+          the beginning of paragraphs(). This happens if you have
           the cursor between to spaces and then save. Or if you
           cut and paste and the selection have a space at the
           beginning and then save right after the paste. I am
@@ -1832,8 +1609,8 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
        // MISSING
 
        // If the pos around the old_cursor were spaces, delete one of them.
-       if (old_cursor.par() != cursor.par()
-           || old_cursor.pos() != cursor.pos()) {
+       if (old_cursor.par() != cursor().par()
+           || old_cursor.pos() != cursor().pos()) {
 
                // Only if the cursor has really moved
                if (old_cursor.pos() > 0
@@ -1849,19 +1626,17 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
 #warning This will not work anymore when we have multiple views of the same buffer
 // In this case, we will have to correct also the cursors held by
 // other bufferviews. It will probably be easier to do that in a more
-// automated way in LyXCursor code. (JMarc 26/09/2001)
+// automated way in CursorSlice code. (JMarc 26/09/2001)
 #endif
                        // correct all cursors held by the LyXText
-                       fixCursorAfterDelete(cursor, old_cursor);
-                       fixCursorAfterDelete(selection.cursor, old_cursor);
-                       fixCursorAfterDelete(selection.start, old_cursor);
-                       fixCursorAfterDelete(selection.end, old_cursor);
+                       fixCursorAfterDelete(cursor(), old_cursor);
+                       fixCursorAfterDelete(anchor(), old_cursor);
                        return false;
                }
        }
 
        // don't delete anything if this is the ONLY paragraph!
-       if (ownerParagraphs().size() == 1)
+       if (paragraphs().size() == 1)
                return false;
 
        // Do not delete empty paragraphs with keepempty set.
@@ -1869,7 +1644,7 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
                return false;
 
        // only do our magic if we changed paragraph
-       if (old_cursor.par() == cursor.par())
+       if (old_cursor.par() == cursor().par())
                return false;
 
        // record if we have deleted a paragraph
@@ -1879,38 +1654,35 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
        if (old_pit->empty()
            || (old_pit->size() == 1 && old_pit->isLineSeparator(0))) {
                // ok, we will delete something
-               LyXCursor tmpcursor;
+               CursorSlice tmpcursor;
 
                deleted = true;
 
                bool selection_position_was_oldcursor_position =
-                       selection.cursor.par() == old_cursor.par()
-                       && selection.cursor.pos() == old_cursor.pos();
+                       anchor().par() == old_cursor.par()
+                       && anchor().pos() == old_cursor.pos();
 
-               tmpcursor = cursor;
-               cursor = old_cursor; // that undo can restore the right cursor position
+               tmpcursor = cursor();
+               cursor() = old_cursor; // that undo can restore the right cursor position
 
                ParagraphList::iterator endpit = boost::next(old_pit);
-               while (endpit != ownerParagraphs().end() && endpit->getDepth())
+               while (endpit != paragraphs().end() && endpit->getDepth())
                        ++endpit;
 
                recUndo(parOffset(old_pit), parOffset(endpit) - 1);
-               cursor = tmpcursor;
+               cursor() = tmpcursor;
 
                // cache cursor pit
                ParagraphList::iterator tmppit = cursorPar();
                // delete old par
-               ownerParagraphs().erase(old_pit);
+               paragraphs().erase(old_pit);
                // update cursor par offset
-               cursor.par(parOffset(tmppit));
+               cursor().par(parOffset(tmppit));
                redoParagraph();
 
-               // correct cursor y
-               setCursorIntern(cursor.par(), cursor.pos());
-
                if (selection_position_was_oldcursor_position) {
                        // correct selection
-                       selection.cursor = cursor;
+                       bv()->resetAnchor();
                }
        }
 
@@ -1919,41 +1691,58 @@ bool LyXText::deleteEmptyParagraphMechanism(LyXCursor const & old_cursor)
 
        if (old_pit->stripLeadingSpaces()) {
                redoParagraph(old_pit);
-               // correct cursor y
-               setCursorIntern(cursor.par(), cursor.pos());
-               selection.cursor = cursor;
+               bv()->resetAnchor();
        }
        return false;
+#endif
 }
 
 
-ParagraphList & LyXText::ownerParagraphs() const
+ParagraphList & LyXText::paragraphs() const
 {
-       return *paragraphs_;
+       return const_cast<ParagraphList &>(paragraphs_);
 }
 
 
 void LyXText::recUndo(paroffset_type first, paroffset_type last) const
 {
-       recordUndo(Undo::ATOMIC, this, first, last);
+       recordUndo(bv()->cursor(), Undo::ATOMIC, first, last);
 }
 
 
 void LyXText::recUndo(lyx::paroffset_type par) const
 {
-       recordUndo(Undo::ATOMIC, this, par, par);
+       recordUndo(bv()->cursor(), Undo::ATOMIC, par, par);
 }
 
 
 bool LyXText::isInInset() const
 {
-       // Sub-level has non-null bv owner and non-null inset owner.
-       return inset_owner != 0;
+       return in_inset_;
+}
+
+
+bool LyXText::toggleInset()
+{
+       InsetBase * inset = bv()->cursor().nextInset();
+       // is there an editable inset at cursor position?
+       if (!isEditableInset(inset))
+               return false;
+       //bv()->owner()->message(inset->editMessage());
+
+       // do we want to keep this?? (JMarc)
+       if (!isHighlyEditableInset(inset))
+               recUndo(cursor().par());
+
+       if (inset->isOpen())
+               inset->close();
+       else
+               inset->open();
+       return true;
 }
 
 
 int defaultRowHeight()
 {
-       LyXFont const font(LyXFont::ALL_SANE);
-       return int(font_metrics::maxHeight(font) *  1.2);
+       return int(font_metrics::maxHeight(LyXFont(LyXFont::ALL_SANE)) *  1.2);
 }