]> git.lyx.org Git - lyx.git/blobdiff - src/text.C
more cursor dispatch
[lyx.git] / src / text.C
index 05c612da29cd1006b40232fff5e2aaec164f0f87..5f53767f140a1236abce8e724d44b91e79631c70 100644 (file)
@@ -21,6 +21,7 @@
 #include "buffer.h"
 #include "bufferparams.h"
 #include "BufferView.h"
+#include "cursor.h"
 #include "debug.h"
 #include "dispatchresult.h"
 #include "encoding.h"
@@ -29,6 +30,7 @@
 #include "language.h"
 #include "LColor.h"
 #include "lyxlength.h"
+#include "lyxlex.h"
 #include "lyxrc.h"
 #include "lyxrow.h"
 #include "lyxrow_funcs.h"
@@ -37,7 +39,6 @@
 #include "paragraph_funcs.h"
 #include "ParagraphParameters.h"
 #include "rowpainter.h"
-#include "text_funcs.h"
 #include "undo.h"
 #include "vspace.h"
 #include "WordLangTuple.h"
@@ -50,8 +51,6 @@
 #include "support/lstrings.h"
 #include "support/textutils.h"
 
-using bv_funcs::number;
-
 using lyx::pos_type;
 using lyx::word_location;
 
@@ -59,21 +58,19 @@ using lyx::support::contains;
 using lyx::support::lowercase;
 using lyx::support::uppercase;
 
+using std::advance;
+using std::distance;
 using std::max;
 using std::endl;
 using std::string;
 
 
-/// top, right, bottom pixel margin
-extern int const PAPER_MARGIN = 20;
+/// some space for drawing the 'nested' markers (in pixel)
+extern int const NEST_MARGIN = 20;
 /// margin for changebar
 extern int const CHANGEBAR_MARGIN = 10;
-/// left margin
-extern int const LEFT_MARGIN = PAPER_MARGIN + CHANGEBAR_MARGIN;
-
-
-
-int bibitemMaxWidth(BufferView *, LyXFont const &);
+/// right margin
+extern int const RIGHT_MARGIN = 10;
 
 
 namespace {
@@ -113,10 +110,10 @@ BufferView * LyXText::bv() const
 }
 
 
-void LyXText::updateRowPositions()
+void LyXText::updateParPositions()
 {
-       ParagraphList::iterator pit = ownerParagraphs().begin();
-       ParagraphList::iterator end = ownerParagraphs().end();
+       ParagraphList::iterator pit = paragraphs().begin();
+       ParagraphList::iterator end = paragraphs().end();
        for (height = 0; pit != end; ++pit) {
                pit->y = height;
                height += pit->height;
@@ -124,54 +121,19 @@ void LyXText::updateRowPositions()
 }
 
 
-int LyXText::workWidth() const
+int LyXText::textWidth() const
 {
-       return inset_owner ? inset_owner->textWidth() : bv()->workWidth();
+       return textwidth_;
 }
 
 
-int LyXText::getRealCursorX() const
-{
-       int x = cursor.x();
-       if (the_locking_inset && the_locking_inset->getLyXText(bv()) != this)
-               x = the_locking_inset->getLyXText(bv())->getRealCursorX();
-       return x;
-}
-
-
-// This is the comments that some of the warnings below refers to.
-// There are some issues in this file and I don't think they are
-// really related to the FIX_DOUBLE_SPACE patch. I'd rather think that
-// this is a problem that has been here almost from day one and that a
-// larger userbase with different access patters triggers the bad
-// behaviour. (segfaults.) What I think happen is: In several places
-// we store the paragraph in the current cursor and then moves the
-// cursor. This movement of the cursor will delete paragraph at the
-// old position if it is now empty. This will make the temporary
-// pointer to the old cursor paragraph invalid and dangerous to use.
-// And is some cases this will trigger a segfault. I have marked some
-// of the cases where this happens with a warning, but I am sure there
-// are others in this file and in text2.C. There is also a note in
-// Delete() that you should read. In Delete I store the paragraph->id
-// instead of a pointer to the paragraph. I am pretty sure this faulty
-// use of temporary pointers to paragraphs that might have gotten
-// invalidated (through a cursor movement) before they are used, are
-// the cause of the strange crashes we get reported often.
-//
-// It is very tiresom to change this code, especially when it is as
-// hard to read as it is. Help to fix all the cases where this is done
-// would be greately appreciated.
-//
-// Lgb
-
 int LyXText::singleWidth(ParagraphList::iterator pit, pos_type pos) const
 {
        if (pos >= pit->size())
                return 0;
 
        char const c = pit->getChar(pos);
-       LyXFont const & font = getFont(pit, pos);
-       return singleWidth(pit, pos, c, font);
+       return singleWidth(pit, pos, c, getFont(pit, pos));
 }
 
 
@@ -210,7 +172,13 @@ int LyXText::singleWidth(ParagraphList::iterator pit,
 }
 
 
-int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
+int LyXText::leftMargin(ParagraphList::iterator pit) const
+{
+       return leftMargin(pit, pit->size());
+}
+
+
+int LyXText::leftMargin(ParagraphList::iterator pit, pos_type pos) const
 {
        LyXTextClass const & tclass =
                bv()->buffer()->params().getLyXTextClass();
@@ -218,19 +186,19 @@ int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
 
        string parindent = layout->parindent;
 
-       int x = LEFT_MARGIN;
+       int x = NEST_MARGIN + CHANGEBAR_MARGIN;
 
        x += font_metrics::signedWidth(tclass.leftmargin(), tclass.defaultfont());
 
-       // this is the way, LyX handles the LaTeX-Environments.
+       // This is the way LyX handles LaTeX-Environments.
        // I have had this idea very late, so it seems to be a
        // later added hack and this is true
        if (pit->getDepth() == 0) {
                if (pit->layout() == tclass.defaultLayout()) {
                        // find the previous same level paragraph
-                       if (pit != ownerParagraphs().begin()) {
+                       if (pit != paragraphs().begin()) {
                                ParagraphList::iterator newpit =
-                                       depthHook(pit, ownerParagraphs(), pit->getDepth());
+                                       depthHook(pit, paragraphs(), pit->getDepth());
                                if (newpit == pit && newpit->layout()->nextnoindent)
                                        parindent.erase();
                        }
@@ -238,16 +206,16 @@ int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
        } else {
                // find the next level paragraph
                ParagraphList::iterator newpar =
-                       outerHook(pit, ownerParagraphs());
+                       outerHook(pit, paragraphs());
 
-               // make a corresponding row. Needed to call leftMargin()
-               // check wether it is a sufficent paragraph
-               if (newpar != ownerParagraphs().end()
+               // Make a corresponding row. Need to call leftMargin()
+               // to check whether it is a sufficent paragraph.
+               if (newpar != paragraphs().end()
                    && newpar->layout()->isEnvironment()) {
-                       x = leftMargin(newpar, Row(newpar->size()));
+                       x = leftMargin(newpar);
                }
 
-               if (newpar != ownerParagraphs().end()
+               if (newpar != paragraphs().end()
                    && pit->layout() == tclass.defaultLayout()) {
                        if (newpar->params().noindent())
                                parindent.erase();
@@ -274,7 +242,7 @@ int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
        case MARGIN_MANUAL:
                x += font_metrics::signedWidth(layout->labelindent, labelfont);
                // The width of an empty par, even with manual label, should be 0
-               if (!pit->empty() && row.pos() >= pit->beginningOfBody()) {
+               if (!pit->empty() && pos >= pit->beginOfBody()) {
                        if (!pit->getLabelWidthString().empty()) {
                                x += font_metrics::width(pit->getLabelWidthString(),
                                               labelfont);
@@ -290,19 +258,19 @@ int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
 
        case MARGIN_FIRST_DYNAMIC:
                if (layout->labeltype == LABEL_MANUAL) {
-                       if (row.pos() >= pit->beginningOfBody()) {
+                       if (pos >= pit->beginOfBody()) {
                                x += font_metrics::signedWidth(layout->leftmargin,
                                                          labelfont);
                        } else {
                                x += font_metrics::signedWidth(layout->labelindent,
                                                          labelfont);
                        }
-               } else if (row.pos()
+               } else if (pos != 0
                           // Special case to fix problems with
                           // theorems (JMarc)
                           || (layout->labeltype == LABEL_STATIC
                               && layout->latextype == LATEX_ENVIRONMENT
-                              && !isFirstInSequence(pit, ownerParagraphs()))) {
+                              && !isFirstInSequence(pit, paragraphs()))) {
                        x += font_metrics::signedWidth(layout->leftmargin,
                                                  labelfont);
                } else if (layout->labeltype != LABEL_TOP_ENVIRONMENT
@@ -318,28 +286,28 @@ int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
                break;
 
        case MARGIN_RIGHT_ADDRESS_BOX: {
+#if 0
                // ok, a terrible hack. The left margin depends on the widest
                // row in this paragraph.
                RowList::iterator rit = pit->rows.begin();
                RowList::iterator end = pit->rows.end();
 #warning This is wrong.
-               int minfill = workWidth() / 2;
+               int minfill = textwidth_;
                for ( ; rit != end; ++rit)
                        if (rit->fill() < minfill)
                                minfill = rit->fill();
                x += font_metrics::signedWidth(layout->leftmargin,
                        tclass.defaultfont());
                x += minfill;
+#endif
+               // also wrong, but much shorter.
+               x += textwidth_ / 2;
+               break;
        }
-       break;
        }
 
-       if (workWidth() > 0 && !pit->params().leftIndent().zero()) {
-               LyXLength const len = pit->params().leftIndent();
-               int const tw = inset_owner ?
-                       inset_owner->latexTextWidth(bv()) : workWidth();
-               x += len.inPixels(tw);
-       }
+       if (!pit->params().leftIndent().zero())
+               x += pit->params().leftIndent().inPixels(textWidth());
 
        LyXAlignment align;
 
@@ -349,58 +317,52 @@ int LyXText::leftMargin(ParagraphList::iterator pit, Row const & row) const
                align = pit->params().align();
 
        // set the correct parindent
-       if (row.pos() == 0) {
-               if ((layout->labeltype == LABEL_NO_LABEL
-                    || layout->labeltype == LABEL_TOP_ENVIRONMENT
-                    || layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT
-                    || (layout->labeltype == LABEL_STATIC
-                        && layout->latextype == LATEX_ENVIRONMENT
-                        && !isFirstInSequence(pit, ownerParagraphs())))
-                   && align == LYX_ALIGN_BLOCK
-                   && !pit->params().noindent()
-                       // in tabulars and ert paragraphs are never indented!
-                       && (!pit->inInset() || !pit->inInset()->owner() ||
-                               (pit->inInset()->owner()->lyxCode() != InsetOld::TABULAR_CODE &&
-                                pit->inInset()->owner()->lyxCode() != InsetOld::ERT_CODE))
-                   && (pit->layout() != tclass.defaultLayout() ||
-                       bv()->buffer()->params().paragraph_separation ==
-                       BufferParams::PARSEP_INDENT)) {
-                       x += font_metrics::signedWidth(parindent,
-                                                 tclass.defaultfont());
-               } else if (layout->labeltype == LABEL_BIBLIO) {
-                       // ale970405 Right width for bibitems
-                       x += bibitemMaxWidth(bv(), tclass.defaultfont());
-               }
+       if (pos == 0
+           && (layout->labeltype == LABEL_NO_LABEL
+              || layout->labeltype == LABEL_TOP_ENVIRONMENT
+              || layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT
+              || (layout->labeltype == LABEL_STATIC
+                  && layout->latextype == LATEX_ENVIRONMENT
+                  && !isFirstInSequence(pit, paragraphs())))
+           && align == LYX_ALIGN_BLOCK
+           && !pit->params().noindent()
+           // in tabulars and ert paragraphs are never indented!
+           && (!pit->inInset()
+               || !pit->inInset()->owner()
+               || (pit->inInset()->owner()->lyxCode() != InsetOld::TABULAR_CODE
+                   && pit->inInset()->owner()->lyxCode() != InsetOld::ERT_CODE))
+           && (pit->layout() != tclass.defaultLayout()
+               || bv()->buffer()->params().paragraph_separation ==
+                  BufferParams::PARSEP_INDENT))
+       {
+               x += font_metrics::signedWidth(parindent, tclass.defaultfont());
        }
 
        return x;
 }
 
 
-int LyXText::rightMargin(Paragraph const & par, Buffer const & buf) const
+int LyXText::rightMargin(Paragraph const & par) const
 {
-       LyXTextClass const & tclass = buf.params().getLyXTextClass();
-       LyXLayout_ptr const & layout = par.layout();
+       LyXTextClass const & tclass = bv()->buffer()->params().getLyXTextClass();
 
-       return PAPER_MARGIN
+       return
+               RIGHT_MARGIN
                + font_metrics::signedWidth(tclass.rightmargin(),
                                       tclass.defaultfont())
-               + font_metrics::signedWidth(layout->rightmargin,
+               + font_metrics::signedWidth(par.layout()->rightmargin,
                                       tclass.defaultfont())
                * 4 / (par.getDepth() + 4);
 }
 
 
-int LyXText::labelEnd(ParagraphList::iterator pit, Row const & row) const
+int LyXText::labelEnd(ParagraphList::iterator pit) const
 {
        // labelEnd is only needed if the layout fills a flushleft label.
        if (pit->layout()->margintype != MARGIN_MANUAL)
                return 0;
-
-       Row tmprow = row;
-       tmprow.pos(pit->size());
        // return the beginning of the body
-       return leftMargin(pit, tmprow);
+       return leftMargin(pit);
 }
 
 
@@ -430,12 +392,8 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                return;
        }
 
-       // maximum pixel width of a row.
-       int width = workWidth() - rightMargin(*pit, *bv()->buffer());
-//             - leftMargin(pit, row);
-
-       // inset->textWidth() returns -1 via workWidth(),
-       // but why ?
+       // maximum pixel width of a row
+       int width = textWidth() - rightMargin(*pit); // - leftMargin(pit, row);
        if (width < 0) {
                row.endpos(end);
                return;
@@ -448,14 +406,14 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                return;
        }
 
-       pos_type const body_pos = pit->beginningOfBody();
+       pos_type const body_pos = pit->beginOfBody();
 
 
        // Now we iterate through until we reach the right margin
        // or the end of the par, then choose the possible break
        // nearest that.
 
-       int const left = leftMargin(pit, row);
+       int const left = leftMargin(pit, pos);
        int x = left;
 
        // pixel width since last breakpoint
@@ -475,14 +433,12 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                }
                // Break before...
                if (i + 1 < end) {
-                       InsetOld * in = pit->getInset(i + 1);
-                       if (in && in->display()) {
+                       if (pit->isInset(i + 1) && pit->getInset(i + 1)->display()) {
                                point = i + 1;
                                break;
                        }
                        // ...and after.
-                       in = pit->getInset(i);
-                       if (in && in->display()) {
+                       if (pit->isInset(i) && pit->getInset(i)->display()) {
                                point = i + 1;
                                break;
                        }
@@ -504,7 +460,7 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                                if (pit->isLineSeparator(i - 1))
                                        add -= singleWidth(pit, i - 1);
 
-                               add = std::max(add, labelEnd(pit, row) - x);
+                               add = std::max(add, labelEnd(pit) - x);
                                thiswidth += add;
                        }
 
@@ -514,7 +470,6 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                        chunkwidth += thiswidth;
                }
 
-
                // break before a character that will fall off
                // the right of the row
                if (x >= width) {
@@ -526,15 +481,10 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                                }
                        }
                        // exit on last registered breakpoint:
-#if 0
-                       // hack removed:
-                       if (i + 1 < end)
-#endif
                        break;
                }
 
-               InsetOld * in = pit->getInset(i);
-               if (!in || in->isChar()) {
+               if (!pit->isInset(i) || pit->getInset(i)->isChar()) {
                        // some insets are line separators too
                        if (pit->isLineSeparator(i)) {
                                // register breakpoint:
@@ -544,13 +494,6 @@ void LyXText::rowBreakPoint(ParagraphList::iterator pit, Row & row) const
                }
        }
 
-#if 0
-       // hack removed (connected with the #if 0 above):
-       if (point == end && i != end && x >= width) {
-               // didn't find one, break at the point we reached the edge
-               point = i + 1;
-       } else
-#endif
        if (i == end && x < width) {
                // maybe found one, but the par is short enough.
                point = end;
@@ -572,10 +515,10 @@ void LyXText::fill(ParagraphList::iterator pit, Row & row, int workwidth) const
        // get the pure distance
        pos_type const end = row.endpos();
 
-       LyXLayout_ptr const & layout = pit->layout();
-       int w = leftMargin(pit, row);
+       string labelsep = pit->layout()->labelsep;
+       int w = leftMargin(pit, row.pos());
 
-       pos_type const body_pos = pit->beginningOfBody();
+       pos_type const body_pos = pit->beginOfBody();
        pos_type i = row.pos();
 
        if (i < end) {
@@ -584,10 +527,10 @@ void LyXText::fill(ParagraphList::iterator pit, Row & row, int workwidth) const
                lyx::pos_type endPosOfFontSpan = pit->getEndPosOfFontSpan(i);
                for ( ; i < end; ++i) {
                        if (body_pos > 0 && i == body_pos) {
-                               w += font_metrics::width(layout->labelsep, getLabelFont(pit));
+                               w += font_metrics::width(labelsep, getLabelFont(pit));
                                if (pit->isLineSeparator(i - 1))
                                        w -= singleWidth(pit, i - 1);
-                               w = max(w, labelEnd(pit, row));
+                               w = max(w, labelEnd(pit));
                        }
                        char const c = pit->getChar(i);
                        if (IsPrintable(c) && i > endPosOfFontSpan) {
@@ -600,13 +543,13 @@ void LyXText::fill(ParagraphList::iterator pit, Row & row, int workwidth) const
        }
 
        if (body_pos > 0 && body_pos >= end) {
-               w += font_metrics::width(layout->labelsep, getLabelFont(pit));
+               w += font_metrics::width(labelsep, getLabelFont(pit));
                if (end > 0 && pit->isLineSeparator(end - 1))
                        w -= singleWidth(pit, end - 1);
-               w = max(w, labelEnd(pit, row));
+               w = max(w, labelEnd(pit));
        }
 
-       int const fill = workwidth - w - rightMargin(*pit, *bv()->buffer());
+       int const fill = workwidth - w - rightMargin(*pit);
        row.fill(fill);
        row.width(workwidth - fill);
 }
@@ -615,7 +558,7 @@ void LyXText::fill(ParagraphList::iterator pit, Row & row, int workwidth) const
 // returns the minimum space a manual label needs on the screen in pixel
 int LyXText::labelFill(ParagraphList::iterator pit, Row const & row) const
 {
-       pos_type last = pit->beginningOfBody();
+       pos_type last = pit->beginOfBody();
 
        BOOST_ASSERT(last > 0);
 
@@ -640,9 +583,7 @@ int LyXText::labelFill(ParagraphList::iterator pit, Row const & row) const
 
 LColor_color LyXText::backgroundColor() const
 {
-       if (inset_owner)
-               return inset_owner->backgroundColor();
-       return LColor::background;
+       return LColor_color(LColor::color(background_color_));
 }
 
 
@@ -658,9 +599,10 @@ void LyXText::setHeightOfRow(ParagraphList::iterator pit, Row & row)
        // are taken from the layoutfont. Nicer on the screen :)
        LyXLayout_ptr const & layout = pit->layout();
 
-       // as max get the first character of this row then it can increase but not
-       // decrease the height. Just some point to start with so we don't have to
-       // do the assignment below too often.
+       // as max get the first character of this row then it can
+       // increase but not decrease the height. Just some point to
+       // start with so we don't have to do the assignment below too
+       // often.
        LyXFont font = getFont(pit, row.pos());
        LyXFont::FONT_SIZE const tmpsize = font.size();
        font = getLayoutFont(pit);
@@ -711,7 +653,7 @@ void LyXText::setHeightOfRow(ParagraphList::iterator pit, Row & row)
                // some parksips VERY EASY IMPLEMENTATION
                if (bv()->buffer()->params().paragraph_separation
                    == BufferParams::PARSEP_SKIP
-                       && pit != ownerParagraphs().begin()
+                       && pit != paragraphs().begin()
                        && ((layout->isParagraph() && pit->getDepth() == 0)
                            || (boost::prior(pit)->layout()->isParagraph()
                                && boost::prior(pit)->getDepth() == 0)))
@@ -719,13 +661,6 @@ void LyXText::setHeightOfRow(ParagraphList::iterator pit, Row & row)
                                maxasc += bufparams.getDefSkip().inPixels(*bv());
                }
 
-               // the top margin
-               if (pit == ownerParagraphs().begin() && !isInInset())
-                       maxasc += PAPER_MARGIN;
-
-               // add user added vertical space
-               maxasc += getLengthMarkerHeight(*bv(), pit->params().spaceTop());
-
                if (pit->params().startOfAppendix())
                        maxasc += int(3 * dh);
 
@@ -740,7 +675,7 @@ void LyXText::setHeightOfRow(ParagraphList::iterator pit, Row & row)
                if ((layout->labeltype == LABEL_TOP_ENVIRONMENT
                     || layout->labeltype == LABEL_BIBLIO
                     || layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT)
-                   && isFirstInSequence(pit, ownerParagraphs())
+                   && isFirstInSequence(pit, paragraphs())
                    && !pit->getLabelstring().empty())
                {
                        labeladdon = int(
@@ -750,27 +685,27 @@ void LyXText::setHeightOfRow(ParagraphList::iterator pit, Row & row)
                                + (layout->topsep + layout->labelbottomsep) * dh);
                }
 
-               // And now the layout spaces, for example before and after
+               // Add the layout spaces, for example before and after
                // a section, or between the items of a itemize or enumerate
                // environment.
 
                ParagraphList::iterator prev =
-                       depthHook(pit, ownerParagraphs(), pit->getDepth());
+                       depthHook(pit, paragraphs(), pit->getDepth());
                if (prev != pit
                    && prev->layout() == layout
                    && prev->getDepth() == pit->getDepth()
                    && prev->getLabelWidthString() == pit->getLabelWidthString())
                {
                        layoutasc = layout->itemsep * dh;
-               } else if (pit != ownerParagraphs().begin() || row.pos() != 0) {
+               } else if (pit != paragraphs().begin() || row.pos() != 0) {
                        if (layout->topsep > 0)
                                layoutasc = layout->topsep * dh;
                }
 
-               prev = outerHook(pit, ownerParagraphs());
-               if (prev != ownerParagraphs().end()) {
+               prev = outerHook(pit, paragraphs());
+               if (prev != paragraphs().end()) {
                        maxasc += int(prev->layout()->parsep * dh);
-               } else if (pit != ownerParagraphs().begin()) {
+               } else if (pit != paragraphs().begin()) {
                        ParagraphList::iterator prior_pit = boost::prior(pit);
                        if (prior_pit->getDepth() != 0 ||
                                        prior_pit->layout() == layout) {
@@ -781,25 +716,18 @@ void LyXText::setHeightOfRow(ParagraphList::iterator pit, Row & row)
 
        // is it a bottom line?
        if (row.endpos() >= pit->size()) {
-               // the bottom margin
-               ParagraphList::iterator nextpit = boost::next(pit);
-               if (nextpit == ownerParagraphs().end() && !isInInset())
-                       maxdesc += PAPER_MARGIN;
-
-               // add the vertical spaces, that the user added
-               maxdesc += getLengthMarkerHeight(*bv(), pit->params().spaceBottom());
-
-               // and now the layout spaces, for example before and after
+               // add the layout spaces, for example before and after
                // a section, or between the items of a itemize or enumerate
                // environment
-               if (nextpit != ownerParagraphs().end()) {
+               ParagraphList::iterator nextpit = boost::next(pit);
+               if (nextpit != paragraphs().end()) {
                        ParagraphList::iterator cpit = pit;
                        double usual = 0;
                        double unusual = 0;
 
                        if (cpit->getDepth() > nextpit->getDepth()) {
                                usual = cpit->layout()->bottomsep * dh;
-                               cpit = depthHook(cpit, ownerParagraphs(), nextpit->getDepth());
+                               cpit = depthHook(cpit, paragraphs(), nextpit->getDepth());
                                if (cpit->layout() != nextpit->layout()
                                        || nextpit->getLabelWidthString() != cpit->getLabelWidthString())
                                {
@@ -828,27 +756,27 @@ void LyXText::breakParagraph(ParagraphList & paragraphs, char keep_layout)
 {
        // allow only if at start or end, or all previous is new text
        ParagraphList::iterator cpit = cursorPar();
-       if (cursor.pos() && cursor.pos() != cpit->size()
-           && cpit->isChangeEdited(0, cursor.pos()))
+       if (cursor().pos() && cursor().pos() != cpit->size()
+           && cpit->isChangeEdited(0, cursor().pos()))
                return;
 
        LyXTextClass const & tclass =
                bv()->buffer()->params().getLyXTextClass();
        LyXLayout_ptr const & layout = cpit->layout();
 
-       // this is only allowed, if the current paragraph is not empty or caption
-       // and if it has not the keepempty flag active
+       // this is only allowed, if the current paragraph is not empty
+       // or caption and if it has not the keepempty flag active
        if (cpit->empty() && !cpit->allowEmpty()
           && layout->labeltype != LABEL_SENSITIVE)
                return;
 
-       recUndo(cursor.par());
+       // a layout change may affect also the following paragraph
+       recUndo(cursor().par(), parOffset(undoSpan(cpit)) - 1);
 
        // Always break behind a space
-       //
        // It is better to erase the space (Dekel)
-       if (cursor.pos() < cpit->size() && cpit->isLineSeparator(cursor.pos()))
-          cpit->erase(cursor.pos());
+       if (cursor().pos() < cpit->size() && cpit->isLineSeparator(cursor().pos()))
+               cpit->erase(cursor().pos());
 
        // break the paragraph
        if (keep_layout)
@@ -862,18 +790,14 @@ void LyXText::breakParagraph(ParagraphList & paragraphs, char keep_layout)
        // but we can fix this in 1.3.0 (Jug 20020509)
        bool const isempty = cpit->allowEmpty() && cpit->empty();
        ::breakParagraph(bv()->buffer()->params(), paragraphs, cpit,
-                        cursor.pos(), keep_layout);
+                        cursor().pos(), keep_layout);
 
-#warning Trouble Point! (Lgb)
-       // When ::breakParagraph is called from within an inset we must
-       // ensure that the correct ParagraphList is used. Today that is not
-       // the case and the Buffer::paragraphs is used. Not good. (Lgb)
        cpit = cursorPar();
        ParagraphList::iterator next_par = boost::next(cpit);
 
        // well this is the caption hack since one caption is really enough
        if (layout->labeltype == LABEL_SENSITIVE) {
-               if (!cursor.pos())
+               if (!cursor().pos())
                        // set to standard-layout
                        cpit->applyLayout(tclass.defaultLayout());
                else
@@ -885,9 +809,9 @@ void LyXText::breakParagraph(ParagraphList & paragraphs, char keep_layout)
        // move one row up!
        // This touches only the screen-update. Otherwise we would may have
        // an empty row on the screen
-       RowList::iterator crit = cpit->getRow(cursor.pos());
-       if (cursor.pos() && crit->pos() == cursor.pos()
-           && !cpit->isNewline(cursor.pos() - 1))
+       RowList::iterator crit = cpit->getRow(cursor().pos());
+       if (cursor().pos() && crit->pos() == cursor().pos()
+           && !cpit->isNewline(cursor().pos() - 1))
        {
                cursorLeft(bv());
        }
@@ -901,7 +825,7 @@ void LyXText::breakParagraph(ParagraphList & paragraphs, char keep_layout)
 
        // This check is necessary. Otherwise the new empty paragraph will
        // be deleted automatically. And it is more friendly for the user!
-       if (cursor.pos() || isempty)
+       if (cursor().pos() || isempty)
                setCursor(next_par, 0);
        else
                setCursor(cpit, 0);
@@ -911,9 +835,9 @@ void LyXText::breakParagraph(ParagraphList & paragraphs, char keep_layout)
 // convenience function
 void LyXText::redoParagraph()
 {
-       clearSelection();
+       bv()->cursor().clearSelection();
        redoParagraph(cursorPar());
-       setCursorIntern(cursor.par(), cursor.pos());
+       setCursorIntern(cursor().par(), cursor().pos());
 }
 
 
@@ -921,7 +845,7 @@ void LyXText::redoParagraph()
 // same Paragraph one to the right and make a rebreak
 void LyXText::insertChar(char c)
 {
-       recordUndo(Undo::INSERT, this, cursor.par(), cursor.par());
+       recordUndo(bv()->cursor(), Undo::INSERT);
 
        // When the free-spacing option is set for the current layout,
        // disable the double-space checking
@@ -937,35 +861,35 @@ void LyXText::insertChar(char c)
                if (current_font.number() == LyXFont::ON) {
                        if (!IsDigit(c) && !contains(number_operators, c) &&
                            !(contains(number_seperators, c) &&
-                             cursor.pos() >= 1 &&
-                             cursor.pos() < cursorPar()->size() &&
-                             getFont(cursorPar(), cursor.pos()).number() == LyXFont::ON &&
-                             getFont(cursorPar(), cursor.pos() - 1).number() == LyXFont::ON)
+                             cursor().pos() >= 1 &&
+                             cursor().pos() < cursorPar()->size() &&
+                             getFont(cursorPar(), cursor().pos()).number() == LyXFont::ON &&
+                             getFont(cursorPar(), cursor().pos() - 1).number() == LyXFont::ON)
                           )
-                               number(bv()); // Set current_font.number to OFF
+                               number(); // Set current_font.number to OFF
                } else if (IsDigit(c) &&
                           real_current_font.isVisibleRightToLeft()) {
-                       number(bv()); // Set current_font.number to ON
+                       number(); // Set current_font.number to ON
 
-                       if (cursor.pos() > 0) {
-                               char const c = cursorPar()->getChar(cursor.pos() - 1);
+                       if (cursor().pos() > 0) {
+                               char const c = cursorPar()->getChar(cursor().pos() - 1);
                                if (contains(number_unary_operators, c) &&
-                                   (cursor.pos() == 1 ||
-                                    cursorPar()->isSeparator(cursor.pos() - 2) ||
-                                    cursorPar()->isNewline(cursor.pos() - 2))
+                                   (cursor().pos() == 1 ||
+                                    cursorPar()->isSeparator(cursor().pos() - 2) ||
+                                    cursorPar()->isNewline(cursor().pos() - 2))
                                  ) {
                                        setCharFont(
                                                    cursorPar(),
-                                                   cursor.pos() - 1,
+                                                   cursor().pos() - 1,
                                                    current_font);
                                } else if (contains(number_seperators, c) &&
-                                          cursor.pos() >= 2 &&
+                                          cursor().pos() >= 2 &&
                                           getFont(
                                                   cursorPar(),
-                                                  cursor.pos() - 2).number() == LyXFont::ON) {
+                                                  cursor().pos() - 2).number() == LyXFont::ON) {
                                        setCharFont(
                                                    cursorPar(),
-                                                   cursor.pos() - 1,
+                                                   cursor().pos() - 1,
                                                    current_font);
                                }
                        }
@@ -989,14 +913,14 @@ void LyXText::insertChar(char c)
        LyXFont rawtmpfont = current_font;
 
        if (!freeSpacing && IsLineSeparatorChar(c)) {
-               if ((cursor.pos() > 0
-                    && cursorPar()->isLineSeparator(cursor.pos() - 1))
-                   || (cursor.pos() > 0
-                       && cursorPar()->isNewline(cursor.pos() - 1))
-                   || (cursor.pos() == 0)) {
+               if ((cursor().pos() > 0
+                    && cursorPar()->isLineSeparator(cursor().pos() - 1))
+                   || (cursor().pos() > 0
+                       && cursorPar()->isNewline(cursor().pos() - 1))
+                   || (cursor().pos() == 0)) {
                        static bool sent_space_message = false;
                        if (!sent_space_message) {
-                               if (cursor.pos() == 0)
+                               if (cursor().pos() == 0)
                                        bv()->owner()->message(_("You cannot insert a space at the beginning of a paragraph. Please read the Tutorial."));
                                else
                                        bv()->owner()->message(_("You cannot type two spaces this way. Please read the Tutorial."));
@@ -1009,14 +933,14 @@ void LyXText::insertChar(char c)
 
        // Here case LyXText::InsertInset already inserted the character
        if (c != Paragraph::META_INSET)
-               cursorPar()->insertChar(cursor.pos(), c);
+               cursorPar()->insertChar(cursor().pos(), c);
 
-       setCharFont(cursorPar(), cursor.pos(), rawtmpfont);
+       setCharFont(cursorPar(), cursor().pos(), rawtmpfont);
 
        current_font = rawtmpfont;
        real_current_font = realtmpfont;
        redoParagraph(cursorPar());
-       setCursor(cursor.par(), cursor.pos() + 1, false, cursor.boundary());
+       setCursor(cursor().par(), cursor().pos() + 1, false, cursor().boundary());
 
        charInserted();
 }
@@ -1047,9 +971,9 @@ void LyXText::prepareToPrint(ParagraphList::iterator pit, Row & row) const
        bool const is_rtl =
                pit->isRightToLeftPar(bv()->buffer()->params());
        if (is_rtl)
-               x = workWidth() > 0 ? rightMargin(*pit, *bv()->buffer()) : 0;
+               x = rightMargin(*pit);
        else
-               x = workWidth() > 0 ? leftMargin(pit, row) : 0;
+               x = leftMargin(pit, row.pos());
 
        // is there a manual margin with a manual label
        LyXLayout_ptr const & layout = pit->layout();
@@ -1080,22 +1004,14 @@ void LyXText::prepareToPrint(ParagraphList::iterator pit, Row & row) const
        // we don't have to look at the alignment if it is ALIGN_LEFT and
        // if the row is already larger then the permitted width as then
        // we force the LEFT_ALIGN'edness!
-       } else if (int(row.width()) < workWidth()) {
+       } else if (int(row.width()) < textWidth()) {
                // is it block, flushleft or flushright?
                // set x how you need it
                int align;
-               if (pit->params().align() == LYX_ALIGN_LAYOUT) {
+               if (pit->params().align() == LYX_ALIGN_LAYOUT)
                        align = layout->align;
-               } else {
+               else
                        align = pit->params().align();
-               }
-               // ERT insets should always be LEFT ALIGNED on screen
-               InsetOld * inset = pit->inInset();
-               if (inset && inset->owner() &&
-                       inset->owner()->lyxCode() == InsetOld::ERT_CODE)
-               {
-                       align = LYX_ALIGN_LEFT;
-               }
 
                // Display-style insets should always be on a centred row
                // The test on pit->size() is to catch zero-size pars, which
@@ -1113,7 +1029,7 @@ void LyXText::prepareToPrint(ParagraphList::iterator pit, Row & row) const
                                int const ns = numberOfSeparators(*pit, row);
                                bool disp_inset = false;
                                if (row.endpos() < pit->size()) {
-                                       InsetOld * in = pit->getInset(row.endpos());
+                                       InsetBase * in = pit->getInset(row.endpos());
                                        if (in)
                                                disp_inset = in->display();
                                }
@@ -1142,7 +1058,7 @@ void LyXText::prepareToPrint(ParagraphList::iterator pit, Row & row) const
 
        bidi.computeTables(*pit, *bv()->buffer(), row);
        if (is_rtl) {
-               pos_type body_pos = pit->beginningOfBody();
+               pos_type body_pos = pit->beginOfBody();
                pos_type end = row.endpos();
 
                if (body_pos > 0
@@ -1161,17 +1077,13 @@ void LyXText::prepareToPrint(ParagraphList::iterator pit, Row & row) const
 }
 
 
-// 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.
-// They also delete the corresponding row
 
 void LyXText::cursorRightOneWord()
 {
-       ::cursorRightOneWord(*this, cursor, ownerParagraphs());
-       setCursor(cursorPar(), cursor.pos());
+       cursorRightOneWord(cursor());
+       setCursor(cursorPar(), cursor().pos());
 }
 
 
@@ -1179,24 +1091,24 @@ void LyXText::cursorRightOneWord()
 // of prior word, not to end of next prior word.
 void LyXText::cursorLeftOneWord()
 {
-       LyXCursor tmpcursor = cursor;
-       ::cursorLeftOneWord(*this, tmpcursor, ownerParagraphs());
+       CursorSlice tmpcursor = cursor();
+       cursorLeftOneWord(tmpcursor);
        setCursor(getPar(tmpcursor), tmpcursor.pos());
 }
 
 
 void LyXText::selectWord(word_location loc)
 {
-       LyXCursor from = cursor;
-       LyXCursor to = cursor;
-       ::getWord(*this, from, to, loc, ownerParagraphs());
-       if (cursor != from)
+       CursorSlice from = cursor();
+       CursorSlice to = cursor();
+       getWord(from, to, loc);
+       if (cursor() != from)
                setCursor(from.par(), from.pos());
        if (to == from)
                return;
-       selection.cursor = cursor;
+       bv()->cursor().resetAnchor();
        setCursor(to.par(), to.pos());
-       setSelection();
+       bv()->cursor().setSelection();
 }
 
 
@@ -1204,26 +1116,26 @@ void LyXText::selectWord(word_location loc)
 // selection is currently set
 bool LyXText::selectWordWhenUnderCursor(word_location loc)
 {
-       if (!selection.set()) {
-               selectWord(loc);
-               return selection.set();
-       }
-       return false;
+       if (bv()->cursor().selection())
+               return false;
+       selectWord(loc);
+       return bv()->cursor().selection();
 }
 
 
 void LyXText::acceptChange()
 {
-       if (!selection.set() && cursorPar()->size())
+       LCursor & cur = bv()->cursor();
+       if (!cur.selection() && cursorPar()->size())
                return;
 
-       if (selection.start.par() == selection.end.par()) {
-               LyXCursor & startc = selection.start;
-               LyXCursor & endc = selection.end;
-               recordUndo(Undo::INSERT, this, startc.par());
+       if (cur.selBegin().par() == cur.par()) {
+               CursorSlice const & startc = cur.selBegin();
+               CursorSlice const & endc = cur.selEnd();
+               recordUndo(cur, Undo::INSERT, cur.anchor().par());
                getPar(startc)->acceptChange(startc.pos(), endc.pos());
                finishUndo();
-               clearSelection();
+               cur.clearSelection();
                redoParagraph(getPar(startc));
                setCursorIntern(startc.par(), 0);
        }
@@ -1233,16 +1145,17 @@ void LyXText::acceptChange()
 
 void LyXText::rejectChange()
 {
-       if (!selection.set() && cursorPar()->size())
+       LCursor & cur = bv()->cursor();
+       if (!cur.selection() && cursorPar()->size())
                return;
 
-       if (selection.start.par() == selection.end.par()) {
-               LyXCursor & startc = selection.start;
-               LyXCursor & endc = selection.end;
-               recordUndo(Undo::INSERT, this, startc.par());
+       if (cur.selBegin().par() == cur.selEnd().par()) {
+               CursorSlice const & startc = cur.selBegin();
+               CursorSlice const & endc = cur.selEnd();
+               recordUndo(cur, Undo::INSERT, cur.anchor().par());
                getPar(startc)->rejectChange(startc.pos(), endc.pos());
                finishUndo();
-               clearSelection();
+               cur.clearSelection();
                redoParagraph(getPar(startc));
                setCursorIntern(startc.par(), 0);
        }
@@ -1250,141 +1163,20 @@ void LyXText::rejectChange()
 }
 
 
-// This function is only used by the spellchecker for NextWord().
-// It doesn't handle LYX_ACCENTs and probably never will.
-WordLangTuple const LyXText::selectNextWordToSpellcheck(float & value)
-{
-       if (the_locking_inset) {
-               WordLangTuple word =
-                       the_locking_inset->selectNextWordToSpellcheck(bv(), value);
-               if (!word.word().empty()) {
-                       value += float(cursor.y());
-                       value /= float(height);
-                       return word;
-               }
-               // we have to go on checking so move cursor to the next char
-               if (cursor.pos() == cursorPar()->size()) {
-                       if (cursor.par() + 1 == int(ownerParagraphs().size()))
-                               return word;
-                       cursor.par(cursor.par() + 1);
-                       cursor.pos(0);
-               } else {
-                       cursor.pos(cursor.pos() + 1);
-               }
-       }
-       int const tmppar = cursor.par();
-
-       // If this is not the very first word, skip rest of
-       // current word because we are probably in the middle
-       // of a word if there is text here.
-       if (cursor.pos() || cursor.par() != 0) {
-               while (cursor.pos() < cursorPar()->size()
-                      && cursorPar()->isLetter(cursor.pos()))
-                       cursor.pos(cursor.pos() + 1);
-       }
-
-       // Now, skip until we have real text (will jump paragraphs)
-       while (true) {
-               ParagraphList::iterator cpit = cursorPar();
-               pos_type const cpos = cursor.pos();
-
-               if (cpos == cpit->size()) {
-                       if (cursor.par() + 1 != int(ownerParagraphs().size())) {
-                               cursor.par(cursor.par() + 1);
-                               cursor.pos(0);
-                               continue;
-                       }
-                       break;
-               }
-
-               bool const is_good_inset = cpit->isInset(cpos)
-                       && cpit->getInset(cpos)->allowSpellcheck();
-
-               if (!isDeletedText(*cpit, cpos)
-                   && (is_good_inset || cpit->isLetter(cpos)))
-                       break;
-
-               cursor.pos(cpos + 1);
-       }
-
-       // now check if we hit an inset so it has to be a inset containing text!
-       if (cursor.pos() < cursorPar()->size() &&
-           cursorPar()->isInset(cursor.pos())) {
-               // lock the inset!
-               FuncRequest cmd(bv(), LFUN_INSET_EDIT, "left");
-               cursorPar()->getInset(cursor.pos())->dispatch(cmd);
-               // now call us again to do the above trick
-               // but obviously we have to start from down below ;)
-               return bv()->text->selectNextWordToSpellcheck(value);
-       }
-
-       // Update the value if we changed paragraphs
-       if (cursor.par() != tmppar) {
-               setCursor(cursor.par(), cursor.pos());
-               value = float(cursor.y())/float(height);
-       }
-
-       // Start the selection from here
-       selection.cursor = cursor;
-
-       string lang_code = getFont(cursorPar(), cursor.pos()).language()->code();
-       // and find the end of the word (insets like optional hyphens
-       // and ligature break are part of a word)
-       while (cursor.pos() < cursorPar()->size()
-              && cursorPar()->isLetter(cursor.pos())
-              && !isDeletedText(*cursorPar(), cursor.pos()))
-               cursor.pos(cursor.pos() + 1);
-
-       // Finally, we copy the word to a string and return it
-       string str;
-       if (selection.cursor.pos() < cursor.pos()) {
-               for (pos_type i = selection.cursor.pos(); i < cursor.pos(); ++i) {
-                       if (!cursorPar()->isInset(i))
-                               str += cursorPar()->getChar(i);
-               }
-       }
-       return WordLangTuple(str, lang_code);
-}
-
-
-// This one is also only for the spellchecker
-void LyXText::selectSelectedWord()
-{
-       if (the_locking_inset) {
-               the_locking_inset->selectSelectedWord(bv());
-               return;
-       }
-       // move cursor to the beginning
-       setCursor(selection.cursor.par(), selection.cursor.pos());
-
-       // set the sel cursor
-       selection.cursor = cursor;
-
-       // now find the end of the word
-       while (cursor.pos() < cursorPar()->size()
-              && cursorPar()->isLetter(cursor.pos()))
-               cursor.pos(cursor.pos() + 1);
-
-       setCursor(cursorPar(), cursor.pos());
-
-       // finally set the selection
-       setSelection();
-}
-
-
 // Delete from cursor up to the end of the current or next word.
 void LyXText::deleteWordForward()
 {
+       LCursor & cur = bv()->cursor();
        if (cursorPar()->empty())
-               cursorRight(bv());
+               cursorRight(true);
        else {
-               LyXCursor tmpcursor = cursor;
-               selection.set(true); // to avoid deletion
+               CursorSlice tmpcursor = cursor();
+               cur.selection() = true; // to avoid deletion
                cursorRightOneWord();
                setCursor(tmpcursor, tmpcursor.par(), tmpcursor.pos());
-               selection.cursor = cursor;
-               cursor = tmpcursor;
-               setSelection();
+               cur.resetAnchor();
+               cursor() = tmpcursor;
+               cur.setSelection();
                cutSelection(true, false);
        }
 }
@@ -1393,16 +1185,17 @@ void LyXText::deleteWordForward()
 // Delete from cursor to start of current or prior word.
 void LyXText::deleteWordBackward()
 {
+       LCursor & cur = bv()->cursor();
        if (cursorPar()->empty())
-               cursorLeft(bv());
+               cursorLeft(true);
        else {
-               LyXCursor tmpcursor = cursor;
-               selection.set(true); // to avoid deletion
+               CursorSlice tmpcursor = cursor();
+               cur.selection() = true; // to avoid deletion
                cursorLeftOneWord();
                setCursor(tmpcursor, tmpcursor.par(), tmpcursor.pos());
-               selection.cursor = cursor;
-               cursor = tmpcursor;
-               setSelection();
+               cur.resetAnchor();
+               cursor() = tmpcursor;
+               cur.setSelection();
                cutSelection(true, false);
        }
 }
@@ -1411,21 +1204,20 @@ void LyXText::deleteWordBackward()
 // Kill to end of line.
 void LyXText::deleteLineForward()
 {
+       LCursor & cur = bv()->cursor();
        if (cursorPar()->empty()) {
                // Paragraph is empty, so we just go to the right
-               cursorRight(bv());
+               cursorRight(true);
        } else {
-               LyXCursor tmpcursor = cursor;
-               // We can't store the row over a regular setCursor
-               // so we set it to 0 and reset it afterwards.
-               selection.set(true); // to avoid deletion
+               CursorSlice tmpcursor = cursor();
+               cur.selection() = true; // to avoid deletion
                cursorEnd();
                setCursor(tmpcursor, tmpcursor.par(), tmpcursor.pos());
-               selection.cursor = cursor;
-               cursor = tmpcursor;
-               setSelection();
+               cur.resetAnchor();
+               cursor() = tmpcursor;
+               cur.setSelection();
                // What is this test for ??? (JMarc)
-               if (!selection.set())
+               if (!cur.selection())
                        deleteWordForward();
                else
                        cutSelection(true, false);
@@ -1435,24 +1227,25 @@ void LyXText::deleteLineForward()
 
 void LyXText::changeCase(LyXText::TextCase action)
 {
-       LyXCursor from;
-       LyXCursor to;
+       LCursor & cur = bv()->cursor();
+       CursorSlice from;
+       CursorSlice to;
 
-       if (selection.set()) {
-               from = selection.start;
-               to = selection.end;
+       if (cur.selection()) {
+               from = cur.selBegin();
+               to = cur.selEnd();
        } else {
-               from = cursor;
-               ::getWord(*this, from, to, lyx::PARTIAL_WORD, ownerParagraphs());
+               from = cursor();
+               getWord(from, to, lyx::PARTIAL_WORD);
                setCursor(to.par(), to.pos() + 1);
        }
 
-       recordUndo(Undo::ATOMIC, this, from.par(), to.par());
+       recordUndo(cur, Undo::ATOMIC, cur.anchor().par());
 
        pos_type pos = from.pos();
        int par = from.par();
 
-       while (par != int(ownerParagraphs().size()) &&
+       while (par != int(paragraphs().size()) &&
               (pos != to.pos() || par != to.par())) {
                ParagraphList::iterator pit = getPar(par);
                if (pos == pit->size()) {
@@ -1485,30 +1278,15 @@ void LyXText::changeCase(LyXText::TextCase action)
 void LyXText::Delete()
 {
        // this is a very easy implementation
-
-       LyXCursor old_cursor = cursor;
-       int const old_cur_par_id = cursorPar()->id();
-       int const old_cur_par_prev_id =
-               old_cursor.par() ? getPar(old_cursor.par() - 1)->id() : -1;
+       CursorSlice old_cursor = cursor();
 
        // just move to the right
-       cursorRight(bv());
-
-       // CHECK Look at the comment here.
-       // This check is not very good...
-       // The cursorRightIntern calls DeleteEmptyParagraphMechanism
-       // and that can very well delete the par or par->previous in
-       // old_cursor. Will a solution where we compare paragraph id's
-       //work better?
-       int iid = cursor.par() ? getPar(cursor.par() - 1)->id() : -1;
-       if (iid == old_cur_par_prev_id && cursorPar()->id() != old_cur_par_id) {
-               // delete-empty-paragraph-mechanism has done it
-               return;
-       }
+       cursorRight(true);
 
        // if you had success make a backspace
-       if (old_cursor.par() != cursor.par() || old_cursor.pos() != cursor.pos()) {
-               recordUndo(Undo::DELETE, this, old_cursor.par());
+       if (old_cursor.par() != cursor().par()
+           || old_cursor.pos() != cursor().pos()) {
+               recordUndo(bv()->cursor(), Undo::DELETE, old_cursor.par());
                backspace();
        }
 }
@@ -1520,9 +1298,10 @@ void LyXText::backspace()
        ParagraphList::iterator pit = cursorPar();
        pos_type lastpos = pit->size();
 
-       if (cursor.pos() == 0) {
-               // The cursor is at the beginning of a paragraph,
-               // so the the backspace will collapse two paragraphs into one.
+       if (cursor().pos() == 0) {
+               // The cursor is at the beginning of a paragraph, so
+               // the the backspace will collapse two paragraphs into
+               // one.
 
                // but it's not allowed unless it's new
                if (pit->isChangeEdited(0, pit->size()))
@@ -1531,42 +1310,32 @@ void LyXText::backspace()
                // we may paste some paragraphs
 
                // is it an empty paragraph?
-
                if (lastpos == 0 || (lastpos == 1 && pit->isSeparator(0))) {
                        // This is an empty paragraph and we delete it just
                        // by moving the cursor one step
                        // left and let the DeleteEmptyParagraphMechanism
                        // handle the actual deletion of the paragraph.
 
-                       if (cursor.par()) {
-                               ParagraphList::iterator tmppit = getPar(cursor.par() - 1);
-                               if (cursorPar()->layout() == tmppit->layout()
-                                   && cursorPar()->getAlign() == tmppit->getAlign()) {
-                                       // Inherit bottom DTD from the paragraph below.
-                                       // (the one we are deleting)
-                                       tmppit->params().spaceBottom(cursorPar()->params().spaceBottom());
-                               }
-
+                       if (cursor().par()) {
                                cursorLeft(bv());
-
                                // the layout things can change the height of a row !
                                redoParagraph();
                                return;
                        }
                }
 
-               if (cursor.par() != 0)
-                       recordUndo(Undo::DELETE, this, cursor.par() - 1, cursor.par());
+               if (cursor().par() != 0)
+                       recordUndo(bv()->cursor(), Undo::DELETE, cursor().par() - 1);
 
                ParagraphList::iterator tmppit = cursorPar();
                // We used to do cursorLeftIntern() here, but it is
                // not a good idea since it triggers the auto-delete
                // mechanism. So we do a cursorLeftIntern()-lite,
                // without the dreaded mechanism. (JMarc)
-               if (cursor.par() != 0) {
+               if (cursor().par() != 0) {
                        // steps into the above paragraph.
-                       setCursorIntern(cursor.par() - 1,
-                                       getPar(cursor.par() - 1)->size(),
+                       setCursorIntern(cursor().par() - 1,
+                                       getPar(cursor().par() - 1)->size(),
                                        false);
                }
 
@@ -1585,52 +1354,50 @@ void LyXText::backspace()
                    && cpit->getAlign() == tmppit->getAlign()) {
                        mergeParagraph(bufparams, buf.paragraphs(), cpit);
 
-                       if (cursor.pos() && cpit->isSeparator(cursor.pos() - 1))
-                               cursor.pos(cursor.pos() - 1);
+                       if (cursor().pos() && cpit->isSeparator(cursor().pos() - 1))
+                               cursor().pos(cursor().pos() - 1);
 
                        // the counters may have changed
                        updateCounters();
-                       setCursor(cursor.par(), cursor.pos(), false);
+                       setCursor(cursor().par(), cursor().pos(), false);
                }
        } else {
                // this is the code for a normal backspace, not pasting
                // any paragraphs
-               recordUndo(Undo::DELETE, this, cursor.par());
+               recordUndo(bv()->cursor(), Undo::DELETE);
                // We used to do cursorLeftIntern() here, but it is
                // not a good idea since it triggers the auto-delete
                // mechanism. So we do a cursorLeftIntern()-lite,
                // without the dreaded mechanism. (JMarc)
-               setCursorIntern(cursor.par(), cursor.pos() - 1,
-                               false, cursor.boundary());
-               cursorPar()->erase(cursor.pos());
+               setCursorIntern(cursor().par(), cursor().pos() - 1,
+                               false, cursor().boundary());
+               cursorPar()->erase(cursor().pos());
        }
 
        lastpos = cursorPar()->size();
-       if (cursor.pos() == lastpos)
+       if (cursor().pos() == lastpos)
                setCurrentFont();
 
        redoParagraph();
-       setCursor(cursor.par(), cursor.pos(), false, cursor.boundary());
+       setCursor(cursor().par(), cursor().pos(), false, cursor().boundary());
 }
 
 
 ParagraphList::iterator LyXText::cursorPar() const
 {
-       if (cursor.par() != cache_pos_) {
-               cache_pos_ = cursor.par();
-               cache_par_ = getPar(cache_pos_);
-       }
-       return cache_par_;
+       //lyxerr << "### cursorPar: cursor: " << bv()->cursor() << endl;
+       //lyxerr << "xxx cursorPar: cursor: " << cursor() << endl;
+       return getPar(cursor().par());
 }
 
 
 RowList::iterator LyXText::cursorRow() const
 {
-       return cursorPar()->getRow(cursor.pos());
+       return cursorPar()->getRow(cursor().pos());
 }
 
 
-ParagraphList::iterator LyXText::getPar(LyXCursor const & cur) const
+ParagraphList::iterator LyXText::getPar(CursorSlice const & cur) const
 {
        return getPar(cur.par());
 }
@@ -1638,10 +1405,11 @@ ParagraphList::iterator LyXText::getPar(LyXCursor const & cur) const
 
 ParagraphList::iterator LyXText::getPar(int par) const
 {
+       //lyxerr << "getPar: " << par << " from " << paragraphs().size() << endl;
        BOOST_ASSERT(par >= 0);
-       BOOST_ASSERT(par < int(ownerParagraphs().size()));
-       ParagraphList::iterator pit = ownerParagraphs().begin();
-       std::advance(pit, par);
+       BOOST_ASSERT(par < int(paragraphs().size()));
+       ParagraphList::iterator pit = paragraphs().begin();
+       advance(pit, par);
        return pit;
 }
 
@@ -1649,21 +1417,22 @@ ParagraphList::iterator LyXText::getPar(int par) const
 RowList::iterator
 LyXText::getRowNearY(int y, ParagraphList::iterator & pit) const
 {
-       //lyxerr << "getRowNearY: y " << y << endl;
-#if 0
-       ParagraphList::iterator const pend = ownerParagraphs().end();
-       pit = ownerParagraphs().begin();
+#if 1
+       ParagraphList::iterator const
+               pend = boost::prior(paragraphs().end());
+       pit = paragraphs().begin();
        while (int(pit->y + pit->height) < y && pit != pend)
                ++pit;
 
-       RowList::iterator rit = pit->rows.begin();
-       RowList::iterator const rend = pit->rows.end();
-       while (int(pit->y + rit->y_offset()) < y && rit != rend)
-               ++rit;
-       return rit;
+       RowList::iterator rit = pit->rows.end();
+       RowList::iterator const rbegin = pit->rows.begin();
+       do {
+               --rit;
+       } while (rit != rbegin && int(pit->y + rit->y_offset()) > y);
 
+       return rit;
 #else
-       pit = boost::prior(ownerParagraphs().end());
+       pit = boost::prior(paragraphs().end());
 
        RowList::iterator rit = lastRow();
        RowList::iterator rbegin = firstRow();
@@ -1684,7 +1453,7 @@ int LyXText::getDepth() const
 
 RowList::iterator LyXText::firstRow() const
 {
-       return ownerParagraphs().front().rows.begin();
+       return paragraphs().front().rows.begin();
 }
 
 
@@ -1696,7 +1465,7 @@ RowList::iterator LyXText::lastRow() const
 
 RowList::iterator LyXText::endRow() const
 {
-       return ownerParagraphs().back().rows.end();
+       return paragraphs().back().rows.end();
 }
 
 
@@ -1706,7 +1475,7 @@ void LyXText::nextRow(ParagraphList::iterator & pit,
        ++rit;
        if (rit == pit->rows.end()) {
                ++pit;
-               if (pit == ownerParagraphs().end())
+               if (pit == paragraphs().end())
                        --pit;
                else
                        rit = pit->rows.begin();
@@ -1720,7 +1489,7 @@ void LyXText::previousRow(ParagraphList::iterator & pit,
        if (rit != pit->rows.begin())
                --rit;
        else {
-               BOOST_ASSERT(pit != ownerParagraphs().begin());
+               BOOST_ASSERT(pit != paragraphs().begin());
                --pit;
                rit = boost::prior(pit->rows.end());
        }
@@ -1729,14 +1498,15 @@ void LyXText::previousRow(ParagraphList::iterator & pit,
 
 string LyXText::selectionAsString(Buffer const & buffer, bool label) const
 {
-       if (!selection.set())
+       LCursor & cur = bv()->cursor();
+       if (!cur.selection())
                return string();
 
        // should be const ...
-       ParagraphList::iterator startpit = getPar(selection.start);
-       ParagraphList::iterator endpit = getPar(selection.end);
-       size_t const startpos = selection.start.pos();
-       size_t const endpos = selection.end.pos();
+       ParagraphList::iterator startpit = getPar(cur.selBegin());
+       ParagraphList::iterator endpit = getPar(cur.selEnd());
+       size_t const startpos = cur.selBegin().pos();
+       size_t const endpos = cur.selEnd().pos();
 
        if (startpit == endpit)
                return startpit->asString(buffer, startpos, endpos, label);
@@ -1759,7 +1529,7 @@ string LyXText::selectionAsString(Buffer const & buffer, bool label) const
 
 int LyXText::parOffset(ParagraphList::iterator pit) const
 {
-       return std::distance(ownerParagraphs().begin(), pit);
+       return distance(paragraphs().begin(), pit);
 }
 
 
@@ -1778,18 +1548,19 @@ void LyXText::redoParagraphInternal(ParagraphList::iterator pit)
        InsetList::iterator iend = pit->insetlist.end();
        for (; ii != iend; ++ii) {
                Dimension dim;
-               MetricsInfo mi(bv(), getFont(pit, ii->pos), workWidth());
+               int const w = textWidth() - leftMargin(pit) - rightMargin(*pit);
+               MetricsInfo mi(bv(), getFont(pit, ii->pos), w);
                ii->inset->metrics(mi, dim);
        }
 
        // rebreak the paragraph
-       int const ww = workWidth();
+       pit->setBeginOfBody();
        pos_type z = 0;
        do {
                Row row(z);
                rowBreakPoint(pit, row);
                z = row.endpos();
-               fill(pit, row, ww);
+               fill(pit, row, textwidth_);
                prepareToPrint(pit, row);
                setHeightOfRow(pit, row);
                row.y_offset(pit->height);
@@ -1808,51 +1579,362 @@ void LyXText::redoParagraphs(ParagraphList::iterator pit,
 {
        for ( ; pit != end; ++pit)
                redoParagraphInternal(pit);
-       updateRowPositions();
+       updateParPositions();
 }
 
 
 void LyXText::redoParagraph(ParagraphList::iterator pit)
 {
        redoParagraphInternal(pit);
-       updateRowPositions();
+       updateParPositions();
 }
 
 
 void LyXText::fullRebreak()
 {
-       redoParagraphs(ownerParagraphs().begin(), ownerParagraphs().end());
-       redoCursor();
-       selection.cursor = cursor;
+       redoParagraphs(paragraphs().begin(), paragraphs().end());
+       bv()->cursor().resetAnchor();
 }
 
 
 void LyXText::metrics(MetricsInfo & mi, Dimension & dim)
 {
-       //lyxerr << "LyXText::metrics: width: " << mi.base.textwidth
-       //      << " workWidth: " << workWidth() << "\nfont: " << mi.base.font << endl;
        //BOOST_ASSERT(mi.base.textwidth);
+       if (mi.base.textwidth)
+               textwidth_ = mi.base.textwidth;
+       //lyxerr << "LyXText::metrics: width: " << mi.base.textwidth
+       //      << " textWidth: " << textWidth() << "\nfont: " << mi.base.font << endl;
 
-       // rebuild row cache. This recomputes height as well.
-       redoParagraphs(ownerParagraphs().begin(), ownerParagraphs().end());
+       // Rebuild row cache. This recomputes height as well.
+       redoParagraphs(paragraphs().begin(), paragraphs().end());
 
-       width = maxParagraphWidth(ownerParagraphs());
+       width = maxParagraphWidth(paragraphs());
 
        // final dimension
        dim.asc = firstRow()->ascent_of_text();
        dim.des = height - dim.asc;
-       dim.wid = std::max(mi.base.textwidth, int(width));
+       dim.wid = width;
+}
+
+
+// only used for inset right now. should also be used for main text
+void LyXText::draw(PainterInfo & pi, int x, int y) const
+{
+       xo_ = x;
+       yo_ = y;
+       paintTextInset(*this, pi, x, y);
 }
 
 
 bool LyXText::isLastRow(ParagraphList::iterator pit, Row const & row) const
 {
        return row.endpos() >= pit->size()
-              && boost::next(pit) == ownerParagraphs().end();
+              && boost::next(pit) == paragraphs().end();
 }
 
 
 bool LyXText::isFirstRow(ParagraphList::iterator pit, Row const & row) const
 {
-       return row.pos() == 0 && pit == ownerParagraphs().begin();
+       return row.pos() == 0 && pit == paragraphs().begin();
+}
+
+
+void LyXText::cursorLeftOneWord(CursorSlice & cur)
+{
+       // treat HFills, floats and Insets as words
+
+       ParagraphList::iterator pit = cursorPar();
+       size_t pos = cur.pos();
+
+       while (pos &&
+              (pit->isSeparator(pos - 1) ||
+               pit->isKomma(pos - 1) ||
+               pit->isNewline(pos - 1)) &&
+              !(pit->isHfill(pos - 1) ||
+                pit->isInset(pos - 1)))
+               --pos;
+
+       if (pos &&
+           (pit->isInset(pos - 1) ||
+            pit->isHfill(pos - 1))) {
+               --pos;
+       } else if (!pos) {
+               if (pit != paragraphs().begin()) {
+                       --pit;
+                       pos = pit->size();
+               }
+       } else {                // Here, cur != 0
+               while (pos > 0 && pit->isWord(pos - 1))
+                       --pos;
+       }
+
+       cur.par(parOffset(pit));
+       cur.pos(pos);
+}
+
+
+void LyXText::cursorRightOneWord(CursorSlice & cur)
+{
+       // treat floats, HFills and Insets as words
+       ParagraphList::iterator pit = cursorPar();
+       pos_type pos = cur.pos();
+
+       if (pos == pit->size() &&
+               boost::next(pit) != paragraphs().end()) {
+               ++pit;
+               pos = 0;
+       } else {
+               // Skip through initial nonword stuff.
+               while (pos < pit->size() && !pit->isWord(pos)) {
+                       ++pos;
+               }
+               // Advance through word.
+               while (pos < pit->size() && pit->isWord(pos)) {
+                       ++pos;
+               }
+       }
+
+       cur.par(parOffset(pit));
+       cur.pos(pos);
+}
+
+
+void LyXText::getWord(CursorSlice & from, CursorSlice & to, word_location const loc)
+{
+       ParagraphList::iterator from_par = getPar(from);
+       switch (loc) {
+       case lyx::WHOLE_WORD_STRICT:
+               if (from.pos() == 0 || from.pos() == from_par->size()
+                   || from_par->isSeparator(from.pos())
+                   || from_par->isKomma(from.pos())
+                   || from_par->isNewline(from.pos())
+                   || from_par->isSeparator(from.pos() - 1)
+                   || from_par->isKomma(from.pos() - 1)
+                   || from_par->isNewline(from.pos() - 1)) {
+                       to = from;
+                       return;
+               }
+               // no break here, we go to the next
+
+       case lyx::WHOLE_WORD:
+               // Move cursor to the beginning, when not already there.
+               if (from.pos() && !from_par->isSeparator(from.pos() - 1)
+                   && !(from_par->isKomma(from.pos() - 1)
+                        || from_par->isNewline(from.pos() - 1)))
+                       cursorLeftOneWord(from);
+               break;
+       case lyx::PREVIOUS_WORD:
+               // always move the cursor to the beginning of previous word
+               cursorLeftOneWord(from);
+               break;
+       case lyx::NEXT_WORD:
+               lyxerr << "LyXText::getWord: NEXT_WORD not implemented yet"
+                      << endl;
+               break;
+       case lyx::PARTIAL_WORD:
+               break;
+       }
+       to = from;
+       ParagraphList::iterator to_par = getPar(to);
+       while (to.pos() < to_par->size()
+              && !to_par->isSeparator(to.pos())
+              && !to_par->isKomma(to.pos())
+              && !to_par->isNewline(to.pos())
+              && !to_par->isHfill(to.pos())
+              && !to_par->isInset(to.pos()))
+       {
+               to.pos(to.pos() + 1);
+       }
+}
+
+
+void LyXText::write(Buffer const & buf, std::ostream & os) const
+{
+       ParagraphList::const_iterator pit = paragraphs().begin();
+       ParagraphList::const_iterator end = paragraphs().end();
+       Paragraph::depth_type dth = 0;
+       for (; pit != end; ++pit)
+               pit->write(buf, os, buf.params(), dth);
+}
+
+
+bool LyXText::read(Buffer const & buf, LyXLex & lex)
+{
+       static Change current_change;
+
+       bool the_end_read = false;
+       ParagraphList::iterator pit = paragraphs().begin();
+       Paragraph::depth_type depth = 0;
+
+       while (lex.isOK()) {
+               lex.nextToken();
+               string token = lex.getString();
+
+               if (token.empty())
+                       continue;
+
+               if (in_inset_) {
+
+                       if (token == "\\end_inset") {
+                               the_end_read = true;
+                               break;
+                       }
+
+                       if (token == "\\end_document") {
+                               lex.printError("\\end_document read in inset! Error in document!");
+                               return false;
+                       }
+
+               } else {
+
+                       if (token == "\\end_document") {
+                               the_end_read = true;
+                               continue;
+                       }
+
+               }
+
+               // FIXME: ugly.
+               int unknown = 0;
+
+               if (token == "\\begin_layout") {
+                       lex.pushToken(token);
+
+                       Paragraph par;
+                       par.params().depth(depth);
+                       if (buf.params().tracking_changes)
+                               par.trackChanges();
+                       LyXFont f(LyXFont::ALL_INHERIT, buf.params().language);
+                       par.setFont(0, f);
+
+                       // insert after
+                       if (pit != paragraphs().end())
+                               ++pit;
+
+                       pit = paragraphs().insert(pit, par);
+
+                       // FIXME: goddamn InsetTabular makes us pass a Buffer
+                       // not BufferParams
+                       ::readParagraph(buf, *pit, lex);
+
+               } else if (token == "\\begin_deeper") {
+                       ++depth;
+               } else if (token == "\\end_deeper") {
+                       if (!depth) {
+                               lex.printError("\\end_deeper: " "depth is already null");
+                       } else {
+                               --depth;
+                       }
+               } else {
+                       ++unknown;
+               }
+
+       }
+       return the_end_read;
+}
+
+
+int LyXText::ascent() const
+{
+       return firstRow()->ascent_of_text();
+}
+
+
+int LyXText::descent() const
+{
+       return height - firstRow()->ascent_of_text();
+}
+
+
+int LyXText::cursorX(CursorSlice const & cur) const
+{
+       ParagraphList::iterator pit = getPar(cur);
+       if (pit->rows.empty())
+               return xo_;
+       Row const & row         = *pit->getRow(cur.pos());
+       pos_type pos            = cur.pos();
+       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)
+               cursor_vpos = (pit->isRightToLeftPar(bv()->buffer()->params()))
+                       ? row_pos : end;
+       else if (pos > row_pos && pos >= end)
+               // 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 xo_ + int(x);
+}
+
+
+int LyXText::cursorY(CursorSlice const & cur) const
+{
+       Paragraph & par = *getPar(cur);
+       Row & row = *par.getRow(cur.pos());
+       return yo_ + par.y + row.y_offset() + row.baseline();
+}
+
+
+CursorSlice & LyXText::cursor()
+{
+       //lyxerr << "# accessing slice " << findText(this) << endl;
+       if (this != bv()->cursor().text()) {
+               lyxerr << "cursor: " << bv()->cursor()
+                       << "\ntext: " << bv()->cursor().text() 
+                       << "\nthis: " << this << endl;
+               BOOST_ASSERT(false);
+       }
+       return bv()->cursor().current();
+}
+
+
+CursorSlice const & LyXText::cursor() const
+{
+       if (this != bv()->cursor().text()) {
+               lyxerr << "cursor: " << bv()->cursor()
+                       << "\ntext: " << bv()->cursor().text() 
+                       << "\nthis: " << this << endl;
+               BOOST_ASSERT(false);
+       }
+       return bv()->cursor().current();
 }