]> git.lyx.org Git - lyx.git/blobdiff - src/text2.C
more dialog merging
[lyx.git] / src / text2.C
index d871e2a82688b85754b98fa5587039bd6e8544ff..a570b387d498bc6d98110e482c25e1f2a57ad88a 100644 (file)
 
 #include "buffer.h"
 #include "buffer_funcs.h"
+#include "bufferlist.h"
 #include "bufferparams.h"
 #include "BufferView.h"
+#include "bufferview_funcs.h"
 #include "Bullet.h"
-#include "counters.h"
 #include "coordcache.h"
 #include "cursor.h"
 #include "CutAndPaste.h"
 #include "debug.h"
 #include "dispatchresult.h"
 #include "errorlist.h"
-#include "Floating.h"
-#include "FloatList.h"
 #include "funcrequest.h"
 #include "gettext.h"
 #include "language.h"
 #include "LColor.h"
+#include "lyxfunc.h"
 #include "lyxrc.h"
 #include "lyxrow.h"
-#include "lyxrow_funcs.h"
 #include "paragraph.h"
+#include "TextMetrics.h"
 #include "paragraph_funcs.h"
 #include "ParagraphParameters.h"
+#include "pariterator.h"
+#include "lyxserver.h"
+#include "lyxsocket.h"
 #include "undo.h"
 #include "vspace.h"
 
-#include "frontends/font_metrics.h"
-#include "frontends/LyXView.h"
+#include "frontends/FontMetrics.h"
 
-#include "insets/insetbibitem.h"
-#include "insets/insetenv.h"
-#include "insets/insetfloat.h"
-#include "insets/insetwrap.h"
+#include "insets/InsetEnv.h"
+
+#include "mathed/InsetMathHull.h"
 
-#include "support/lstrings.h"
 #include "support/textutils.h"
-#include "support/tostr.h"
+
+#include <boost/current_function.hpp>
 
 #include <sstream>
 
-using lyx::par_type;
-using lyx::pos_type;
-using lyx::support::bformat;
+
+namespace lyx {
 
 using std::endl;
 using std::ostringstream;
 using std::string;
+using std::max;
+using std::min;
 
 
-LyXText::LyXText(BufferView * bv)
-       : width_(0), maxwidth_(bv ? bv->workWidth() : 100), height_(0),
+LyXText::LyXText()
+       : current_font(LyXFont::ALL_INHERIT),
          background_color_(LColor::background),
-         bv_owner(bv), xo_(0), yo_(0)
+         autoBreakRows_(false)
 {}
 
 
-void LyXText::init(BufferView * bv)
+bool LyXText::isMainText(Buffer const & buffer) const
 {
-       BOOST_ASSERT(bv);
-       bv_owner = bv;
-       maxwidth_ = bv->workWidth();
-       width_ = maxwidth_;
-       height_ = 0;
-
-       par_type const end = paragraphs().size();
-       for (par_type pit = 0; pit != end; ++pit)
-               pars_[pit].rows.clear();
-
-       current_font = getFont(0, 0);
-       redoParagraphs(0, end);
-       updateCounters();
+       return &buffer.text() == this;
 }
 
 
-bool LyXText::isMainText() const
+//takes screen x,y coordinates
+InsetBase * LyXText::checkInsetHit(BufferView & bv, int x, int y)
 {
-       return &bv()->buffer()->text() == this;
-}
+       pit_type pit = getPitNearY(bv, y);
+       BOOST_ASSERT(pit != -1);
 
+       Paragraph const & par = pars_[pit];
 
-// takes absolute x,y coordinates
-InsetBase * LyXText::checkInsetHit(int x, int y) const 
-{
-       par_type pit;
-       par_type end;
-
-       getParsInRange(paragraphs(),
-                      bv()->top_y() - yo_,
-                      bv()->top_y() - yo_ + bv()->workHeight(),
-                      pit, end);
-
-       // convert to screen-absolute y coordinate
-       y -= bv()->top_y();
-       lyxerr << "checkInsetHit: x: " << x << " y: " << y << endl;
-       lyxerr << "  pit: " << pit << " end: " << end << endl;
-       for (; pit != end; ++pit) {
-               InsetList::const_iterator iit = pars_[pit].insetlist.begin();
-               InsetList::const_iterator iend = pars_[pit].insetlist.end();
-               for (; iit != iend; ++iit) {
-                       InsetBase * inset = iit->inset;
+       LYXERR(Debug::DEBUG)
+               << BOOST_CURRENT_FUNCTION
+               << ": x: " << x
+               << " y: " << y
+               << "  pit: " << pit
+               << endl;
+       InsetList::const_iterator iit = par.insetlist.begin();
+       InsetList::const_iterator iend = par.insetlist.end();
+       for (; iit != iend; ++iit) {
+               InsetBase * inset = iit->inset;
 #if 1
-                       lyxerr << "examining inset " << inset << endl;
-                       if (theCoords.insets_.has(inset))
-                               lyxerr
-                                       << " xo: " << inset->xo() << "..." << inset->xo() + inset->width()
-                                       << " yo: " << inset->yo() - inset->ascent() << "..."
-                                       << inset->yo() + inset->descent() << endl;
-                       else
-                               lyxerr << " inset has no cached position";
+               LYXERR(Debug::DEBUG)
+                       << BOOST_CURRENT_FUNCTION
+                       << ": examining inset " << inset << endl;
+
+               if (bv.coordCache().getInsets().has(inset))
+                       LYXERR(Debug::DEBUG)
+                               << BOOST_CURRENT_FUNCTION
+                               << ": xo: " << inset->xo(bv) << "..."
+                               << inset->xo(bv) + inset->width()
+                               << " yo: " << inset->yo(bv) - inset->ascent()
+                               << "..."
+                               << inset->yo(bv) + inset->descent()
+                               << endl;
+               else
+                       LYXERR(Debug::DEBUG)
+                               << BOOST_CURRENT_FUNCTION
+                               << ": inset has no cached position" << endl;
 #endif
-                       if (inset->covers(x, y)) {
-                               lyxerr << "Hit inset: " << inset << endl;
-                               return inset;
-                       }
+               if (inset->covers(bv, x, y)) {
+                       LYXERR(Debug::DEBUG)
+                               << BOOST_CURRENT_FUNCTION
+                               << ": Hit inset: " << inset << endl;
+                       return inset;
                }
        }
-       lyxerr << "No inset hit. " << endl;
+       LYXERR(Debug::DEBUG)
+               << BOOST_CURRENT_FUNCTION
+               << ": No inset hit. " << endl;
        return 0;
 }
 
@@ -149,27 +143,36 @@ InsetBase * LyXText::checkInsetHit(int x, int y) const
 // The difference is that this one is used for displaying, and thus we
 // are allowed to make cosmetic improvements. For instance make footnotes
 // smaller. (Asger)
-LyXFont LyXText::getFont(par_type const pit, pos_type const pos) const
+LyXFont LyXText::getFont(Buffer const & buffer, Paragraph const & par,
+               pos_type const pos) const
 {
        BOOST_ASSERT(pos >= 0);
 
-       Paragraph const & par = pars_[pit];
        LyXLayout_ptr const & layout = par.layout();
 #ifdef WITH_WARNINGS
 #warning broken?
 #endif
-       BufferParams const & params = bv()->buffer()->params();
+       BufferParams const & params = buffer.params();
        pos_type const body_pos = par.beginOfBody();
 
        // We specialize the 95% common case:
        if (!par.getDepth()) {
                LyXFont f = par.getFontSettings(params, pos);
-               if (!isMainText())
-                       f.realize(font_);
-               if (layout->labeltype == LABEL_MANUAL && pos < body_pos)
-                       return f.realize(layout->reslabelfont);
-               else
-                       return f.realize(layout->resfont);
+               if (!isMainText(buffer))
+                       applyOuterFont(buffer, f);
+               LyXFont lf;
+               LyXFont rlf;
+               if (layout->labeltype == LABEL_MANUAL && pos < body_pos) {
+                       lf = layout->labelfont;
+                       rlf = layout->reslabelfont;
+               } else {
+                       lf = layout->font;
+                       rlf = layout->resfont;
+               }
+               // In case the default family has been customized
+               if (lf.family() == LyXFont::INHERIT_FAMILY)
+                       rlf.setFamily(params.getFont().family());
+               return f.realize(rlf);
        }
 
        // The uncommon case need not be optimized as much
@@ -182,50 +185,90 @@ LyXFont LyXText::getFont(par_type const pit, pos_type const pos) const
        LyXFont font = par.getFontSettings(params, pos);
        font.realize(layoutfont);
 
-       if (!isMainText())
-               font.realize(font_);
+       if (!isMainText(buffer))
+               applyOuterFont(buffer, font);
+
+       // Find the pit value belonging to paragraph. This will not break
+       // even if pars_ would not be a vector anymore.
+       // Performance appears acceptable.
+
+       pit_type pit = pars_.size();
+       for (pit_type it = 0; it < pit; ++it)
+               if (&pars_[it] == &par) {
+                       pit = it;
+                       break;
+               }
+       // Realize against environment font information
+       // NOTE: the cast to pit_type should be removed when pit_type
+       // changes to a unsigned integer.
+       if (pit < pit_type(pars_.size()))
+               font.realize(outerFont(pit, pars_));
 
        // Realize with the fonts of lesser depth.
-       //font.realize(outerFont(pit, paragraphs()));
-       font.realize(defaultfont_);
+       font.realize(params.getFont());
 
        return font;
 }
 
+// There are currently two font mechanisms in LyX:
+// 1. The font attributes in a lyxtext, and
+// 2. The inset-specific font properties, defined in an inset's
+// metrics() and draw() methods and handed down the inset chain through
+// the pi/mi parameters, and stored locally in a lyxtext in font_.
+// This is where the two are integrated in the final fully realized
+// font.
+void LyXText::applyOuterFont(Buffer const & buffer, LyXFont & font) const {
+       LyXFont lf(font_);
+       lf.reduce(buffer.params().getFont());
+       lf.realize(font);
+       lf.setLanguage(font.language());
+       font = lf;
+}
+
 
-LyXFont LyXText::getLayoutFont(par_type const pit) const
+LyXFont LyXText::getLayoutFont(Buffer const & buffer, pit_type const pit) const
 {
        LyXLayout_ptr const & layout = pars_[pit].layout();
 
-       if (!pars_[pit].getDepth())
-               return layout->resfont;
+       if (!pars_[pit].getDepth())  {
+               LyXFont lf = layout->resfont;
+               // In case the default family has been customized
+               if (layout->font.family() == LyXFont::INHERIT_FAMILY)
+                       lf.setFamily(buffer.params().getFont().family());
+               return lf;
+       }
 
        LyXFont font = layout->font;
        // Realize with the fonts of lesser depth.
        //font.realize(outerFont(pit, paragraphs()));
-       font.realize(defaultfont_);
+       font.realize(buffer.params().getFont());
 
        return font;
 }
 
 
-LyXFont LyXText::getLabelFont(par_type pit) const
+LyXFont LyXText::getLabelFont(Buffer const & buffer, Paragraph const & par) const
 {
-       LyXLayout_ptr const & layout = pars_[pit].layout();
+       LyXLayout_ptr const & layout = par.layout();
 
-       if (!pars_[pit].getDepth())
-               return layout->reslabelfont;
+       if (!par.getDepth()) {
+               LyXFont lf = layout->reslabelfont;
+               // In case the default family has been customized
+               if (layout->labelfont.family() == LyXFont::INHERIT_FAMILY)
+                       lf.setFamily(buffer.params().getFont().family());
+               return lf;
+       }
 
        LyXFont font = layout->labelfont;
        // Realize with the fonts of lesser depth.
-       font.realize(outerFont(pit, paragraphs()));
-       font.realize(defaultfont_);
+       font.realize(buffer.params().getFont());
 
        return font;
 }
 
 
-void LyXText::setCharFont(par_type pit, pos_type pos, LyXFont const & fnt)
+void LyXText::setCharFont(Buffer const & buffer, pit_type pit,
+               pos_type pos, LyXFont const & fnt)
 {
        LyXFont font = fnt;
        LyXLayout_ptr const & layout = pars_[pit].layout();
@@ -240,17 +283,22 @@ void LyXText::setCharFont(par_type pit, pos_type pos, LyXFont const & fnt)
 
        // Realize against environment font information
        if (pars_[pit].getDepth()) {
-               par_type tp = pit;
+               pit_type tp = pit;
                while (!layoutfont.resolved() &&
-                      tp != par_type(paragraphs().size()) &&
+                      tp != pit_type(paragraphs().size()) &&
                       pars_[tp].getDepth()) {
                        tp = outerHook(tp, paragraphs());
-                       if (tp != par_type(paragraphs().size()))
+                       if (tp != pit_type(paragraphs().size()))
                                layoutfont.realize(pars_[tp].layout()->font);
                }
        }
 
-       layoutfont.realize(defaultfont_);
+       // Inside inset, apply the inset's font attributes if any
+       // (charstyle!)
+       if (!isMainText(buffer))
+               layoutfont.realize(font_);
+
+       layoutfont.realize(buffer.params().getFont());
 
        // Now, reduce font against full layout font
        font.reduce(layoutfont);
@@ -259,33 +307,11 @@ void LyXText::setCharFont(par_type pit, pos_type pos, LyXFont const & fnt)
 }
 
 
-// used in setLayout
-// Asger is not sure we want to do this...
-void LyXText::makeFontEntriesLayoutSpecific(BufferParams const & params,
-                                           Paragraph & par)
-{
-       LyXLayout_ptr const & layout = par.layout();
-       pos_type const psize = par.size();
-
-       LyXFont layoutfont;
-       for (pos_type pos = 0; pos < psize; ++pos) {
-               if (pos < par.beginOfBody())
-                       layoutfont = layout->labelfont;
-               else
-                       layoutfont = layout->font;
-
-               LyXFont tmpfont = par.getFontSettings(params, pos);
-               tmpfont.reduce(layoutfont);
-               par.setFont(pos, tmpfont);
-       }
-}
-
-
 // return past-the-last paragraph influenced by a layout change on pit
-par_type LyXText::undoSpan(par_type pit)
+pit_type LyXText::undoSpan(pit_type pit)
 {
-       par_type end = paragraphs().size();
-       par_type nextpit = pit + 1;
+       pit_type end = paragraphs().size();
+       pit_type nextpit = pit + 1;
        if (nextpit == end)
                return nextpit;
        //because of parindents
@@ -300,23 +326,21 @@ par_type LyXText::undoSpan(par_type pit)
 }
 
 
-par_type LyXText::setLayout(par_type start, par_type end, string const & layout)
+void LyXText::setLayout(Buffer const & buffer, pit_type start, pit_type end,
+               string const & layout)
 {
        BOOST_ASSERT(start != end);
-       par_type undopit = undoSpan(end - 1);
-       recUndo(start, undopit - 1);
 
-       BufferParams const & bufparams = bv()->buffer()->params();
+       BufferParams const & bufparams = buffer.params();
        LyXLayout_ptr const & lyxlayout = bufparams.getLyXTextClass()[layout];
 
-       for (par_type pit = start; pit != end; ++pit) {
-               pars_[pit].applyLayout(lyxlayout);
-               makeFontEntriesLayoutSpecific(bufparams, pars_[pit]);
+       for (pit_type pit = start; pit != end; ++pit) {
+               Paragraph & par = pars_[pit];
+               par.applyLayout(lyxlayout);
                if (lyxlayout->margintype == MARGIN_MANUAL)
-                       pars_[pit].setLabelWidthString(lyxlayout->labelstring());
+                       par.setLabelWidthString(par.translateIfPossible(
+                               lyxlayout->labelstring(), buffer.params()));
        }
-
-       return undopit;
 }
 
 
@@ -330,42 +354,28 @@ void LyXText::setLayout(LCursor & cur, string const & layout)
        LyXLayout_ptr const & lyxlayout = params.getLyXTextClass()[layout];
        if (lyxlayout->is_environment) {
                // move everything in a new environment inset
-               lyxerr[Debug::DEBUG] << "setting layout " << layout << endl;
-               bv.owner()->dispatch(FuncRequest(LFUN_HOME));
-               bv.owner()->dispatch(FuncRequest(LFUN_ENDSEL));
-               bv.owner()->dispatch(FuncRequest(LFUN_CUT));
+               LYXERR(Debug::DEBUG) << "setting layout " << layout << endl;
+               lyx::dispatch(FuncRequest(LFUN_LINE_BEGIN));
+               lyx::dispatch(FuncRequest(LFUN_LINE_END_SELECT));
+               lyx::dispatch(FuncRequest(LFUN_CUT));
                InsetBase * inset = new InsetEnvironment(params, layout);
                insertInset(cur, inset);
                //inset->edit(cur, true);
-               //bv.owner()->dispatch(FuncRequest(LFUN_PASTE));
+               //lyx::dispatch(FuncRequest(LFUN_PASTE));
                return;
        }
 
-       par_type start = cur.selBegin().par();
-       par_type end = cur.selEnd().par() + 1;
-       par_type endpit = setLayout(start, end, layout);
-       redoParagraphs(start, endpit);
-       updateCounters();
-}
-
-
-namespace {
-
-
-void getSelectionSpan(LCursor & cur, par_type & beg, par_type & end)
-{
-       if (!cur.selection()) {
-               beg = cur.par();
-               end = cur.par() + 1;
-       } else {
-               beg = cur.selBegin().par();
-               end = cur.selEnd().par() + 1;
-       }
+       pit_type start = cur.selBegin().pit();
+       pit_type end = cur.selEnd().pit() + 1;
+       pit_type undopit = undoSpan(end - 1);
+       recUndo(cur, start, undopit - 1);
+       setLayout(cur.buffer(), start, end, layout);
+       updateLabels(cur.buffer());
 }
 
 
-bool changeDepthAllowed(LyXText::DEPTH_CHANGE type,
-       Paragraph const & par, int max_depth)
+static bool changeDepthAllowed(LyXText::DEPTH_CHANGE type,
+                       Paragraph const & par, int max_depth)
 {
        if (par.layout()->labeltype == LABEL_BIBLIO)
                return false;
@@ -378,20 +388,19 @@ bool changeDepthAllowed(LyXText::DEPTH_CHANGE type,
 }
 
 
-}
-
-
 bool LyXText::changeDepthAllowed(LCursor & cur, DEPTH_CHANGE type) const
 {
        BOOST_ASSERT(this == cur.text());
-       par_type beg, end;
-       getSelectionSpan(cur, beg, end);
-       int max_depth = 0;
-       if (beg != 0)
-               max_depth = pars_[beg - 1].getMaxDepthAfter();
-
-       for (par_type pit = beg; pit != end; ++pit) {
-               if (::changeDepthAllowed(type, pars_[pit], max_depth))
+       // this happens when selecting several cells in tabular (bug 2630)
+       if (cur.selBegin().idx() != cur.selEnd().idx())
+               return false;
+
+       pit_type const beg = cur.selBegin().pit();
+       pit_type const end = cur.selEnd().pit() + 1;
+       int max_depth = (beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0);
+
+       for (pit_type pit = beg; pit != end; ++pit) {
+               if (lyx::changeDepthAllowed(type, pars_[pit], max_depth))
                        return true;
                max_depth = pars_[pit].getMaxDepthAfter();
        }
@@ -402,27 +411,25 @@ bool LyXText::changeDepthAllowed(LCursor & cur, DEPTH_CHANGE type) const
 void LyXText::changeDepth(LCursor & cur, DEPTH_CHANGE type)
 {
        BOOST_ASSERT(this == cur.text());
-       par_type beg, end;
-       getSelectionSpan(cur, beg, end);
+       pit_type const beg = cur.selBegin().pit();
+       pit_type const end = cur.selEnd().pit() + 1;
        recordUndoSelection(cur);
+       int max_depth = (beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0);
 
-       int max_depth = 0;
-       if (beg != 0)
-               max_depth = pars_[beg - 1].getMaxDepthAfter();
-
-       for (par_type pit = beg; pit != end; ++pit) {
-               if (::changeDepthAllowed(type, pars_[pit], max_depth)) {
-                       int const depth = pars_[pit].params().depth();
+       for (pit_type pit = beg; pit != end; ++pit) {
+               Paragraph & par = pars_[pit];
+               if (lyx::changeDepthAllowed(type, par, max_depth)) {
+                       int const depth = par.params().depth();
                        if (type == INC_DEPTH)
-                               pars_[pit].params().depth(depth + 1);
+                               par.params().depth(depth + 1);
                        else
-                               pars_[pit].params().depth(depth - 1);
+                               par.params().depth(depth - 1);
                }
-               max_depth = pars_[pit].getMaxDepthAfter();
+               max_depth = par.getMaxDepthAfter();
        }
        // this handles the counter labels, and also fixes up
        // depth values for follow-on (child) paragraphs
-       updateCounters();
+       updateLabels(cur.buffer());
 }
 
 
@@ -434,11 +441,11 @@ void LyXText::setFont(LCursor & cur, LyXFont const & font, bool toggleall)
        if (!cur.selection()) {
                // Determine basis font
                LyXFont layoutfont;
-               par_type pit = cur.par();
+               pit_type pit = cur.pit();
                if (cur.pos() < pars_[pit].beginOfBody())
-                       layoutfont = getLabelFont(pit);
+                       layoutfont = getLabelFont(cur.buffer(), pars_[pit]);
                else
-                       layoutfont = getLayoutFont(pit);
+                       layoutfont = getLayoutFont(cur.buffer(), pit);
 
                // Update current font
                real_current_font.update(font,
@@ -457,62 +464,70 @@ void LyXText::setFont(LCursor & cur, LyXFont const & font, bool toggleall)
        // Ok, we have a selection.
        recordUndoSelection(cur);
 
-       par_type const beg = cur.selBegin().par();
-       par_type const end = cur.selEnd().par();
-
-       DocIterator pos = cur.selectionBegin();
-       DocIterator posend = cur.selectionEnd();
-
-       lyxerr[Debug::DEBUG] << "pos: " << pos << " posend: " << posend
-                            << endl;
+       DocIterator dit = cur.selectionBegin();
+       DocIterator ditend = cur.selectionEnd();
 
        BufferParams const & params = cur.buffer().params();
 
-       // Don't use forwardChar here as posend might have
+       // Don't use forwardChar here as ditend might have
        // pos() == lastpos() and forwardChar would miss it.
-       for (; pos != posend; pos.forwardPos()) {
-               if (pos.pos() != pos.lastpos()) {
-                       LyXFont f = getFont(pos.par(), pos.pos());
+       // Can't use forwardPos either as this descends into
+       // nested insets.
+       for (; dit != ditend; dit.forwardPosNoDescend()) {
+               if (dit.pos() != dit.lastpos()) {
+                       LyXFont f = getFont(cur.buffer(), dit.paragraph(), dit.pos());
                        f.update(font, params.language, toggleall);
-                       setCharFont(pos.par(), pos.pos(), f);
+                       setCharFont(cur.buffer(), dit.pit(), dit.pos(), f);
                }
        }
-
-       redoParagraphs(beg, end + 1);
 }
 
 
 // the cursor set functions have a special mechanism. When they
 // realize you left an empty paragraph, they will delete it.
 
-void LyXText::cursorHome(LCursor & cur)
+bool LyXText::cursorHome(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
-       setCursor(cur, cur.par(), cur.textRow().pos());
+       ParagraphMetrics const & pm = cur.bv().parMetrics(this, cur.pit());
+       Row const & row = pm.getRow(cur.pos(),cur.boundary());
+       return setCursor(cur, cur.pit(), row.pos());
 }
 
 
-void LyXText::cursorEnd(LCursor & cur)
+bool LyXText::cursorEnd(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
        // if not on the last row of the par, put the cursor before
-       // the final space
-       pos_type const end = cur.textRow().endpos();
-       setCursor(cur, cur.par(), end == cur.lastpos() ? end : end - 1);
+       // the final space exept if I have a spanning inset or one string
+       // is so long that we force a break.
+       pos_type end = cur.textRow().endpos();
+       if (end == 0)
+               // empty text, end-1 is no valid position
+               return false;
+       bool boundary = false;
+       if (end != cur.lastpos()) {
+               if (!cur.paragraph().isLineSeparator(end-1)
+                   && !cur.paragraph().isNewline(end-1))
+                       boundary = true;
+               else
+                       --end;
+       }
+       return setCursor(cur, cur.pit(), end, true, boundary);
 }
 
 
-void LyXText::cursorTop(LCursor & cur)
+bool LyXText::cursorTop(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
-       setCursor(cur, 0, 0);
+       return setCursor(cur, 0, 0);
 }
 
 
-void LyXText::cursorBottom(LCursor & cur)
+bool LyXText::cursorBottom(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
-       setCursor(cur, cur.lastpar(), boost::prior(paragraphs().end())->size());
+       return setCursor(cur, cur.lastpit(), boost::prior(paragraphs().end())->size());
 }
 
 
@@ -523,7 +538,7 @@ void LyXText::toggleFree(LCursor & cur, LyXFont const & font, bool toggleall)
        if (font == LyXFont(LyXFont::ALL_IGNORE)) {
                // Could only happen with user style
                cur.message(_("No font change defined. "
-                       "Use Character under the Layout menu to define font change."));
+                                          "Use Character under the Layout menu to define font change."));
                return;
        }
 
@@ -534,7 +549,7 @@ void LyXText::toggleFree(LCursor & cur, LyXFont const & font, bool toggleall)
        bool implicitSelection =
                font.language() == ignore_language
                && font.number() == LyXFont::IGNORE
-               && selectWordWhenUnderCursor(cur, lyx::WHOLE_WORD_STRICT);
+               && selectWordWhenUnderCursor(cur, WHOLE_WORD_STRICT);
 
        // Set font
        setFont(cur, font, toggleall);
@@ -549,47 +564,43 @@ void LyXText::toggleFree(LCursor & cur, LyXFont const & font, bool toggleall)
 }
 
 
-string LyXText::getStringToIndex(LCursor & cur)
+docstring LyXText::getStringToIndex(LCursor const & cur)
 {
        BOOST_ASSERT(this == cur.text());
-       // Try implicit word selection
-       // If there is a change in the language the implicit word selection
-       // is disabled.
-       CursorSlice const reset_cursor = cur.top();
-       bool const implicitSelection =
-               selectWordWhenUnderCursor(cur, lyx::PREVIOUS_WORD);
-
-       string idxstring;
-       if (!cur.selection())
-               cur.message(_("Nothing to index!"));
-       else if (cur.selBegin().par() != cur.selEnd().par())
-               cur.message(_("Cannot index more than one paragraph!"));
-       else
-               idxstring = cur.selectionAsString(false);
 
-       // Reset cursors to their original position.
-       cur.top() = reset_cursor;
-       cur.resetAnchor();
-
-       // Clear the implicit selection.
-       if (implicitSelection)
-               cur.clearSelection();
+       docstring idxstring;
+       if (cur.selection())
+               idxstring = cur.selectionAsString(false);
+       else {
+               // Try implicit word selection. If there is a change
+               // in the language the implicit word selection is
+               // disabled.
+               LCursor tmpcur = cur;
+               selectWord(tmpcur, PREVIOUS_WORD);
+
+               if (!tmpcur.selection())
+                       cur.message(_("Nothing to index!"));
+               else if (tmpcur.selBegin().pit() != tmpcur.selEnd().pit())
+                       cur.message(_("Cannot index more than one paragraph!"));
+               else
+                       idxstring = tmpcur.selectionAsString(false);
+       }
 
        return idxstring;
 }
 
 
 void LyXText::setParagraph(LCursor & cur,
-       Spacing const & spacing, LyXAlignment align,
-       string const & labelwidthstring, bool noindent)
+                          Spacing const & spacing, LyXAlignment align,
+                          docstring const & labelwidthstring, bool noindent)
 {
        BOOST_ASSERT(cur.text());
        // make sure that the depth behind the selection are restored, too
-       par_type undopit = undoSpan(cur.selEnd().par());
-       recUndo(cur.selBegin().par(), undopit - 1);
+       pit_type undopit = undoSpan(cur.selEnd().pit());
+       recUndo(cur, cur.selBegin().pit(), undopit - 1);
 
-       for (par_type pit = cur.selBegin().par(), end = cur.selEnd().par();
-                       pit <= end; ++pit) {
+       for (pit_type pit = cur.selBegin().pit(), end = cur.selEnd().pit();
+            pit <= end; ++pit) {
                Paragraph & par = pars_[pit];
                ParagraphParameters & params = par.params();
                params.spacing(spacing);
@@ -608,330 +619,33 @@ void LyXText::setParagraph(LCursor & cur,
                par.setLabelWidthString(labelwidthstring);
                params.noindent(noindent);
        }
-
-       redoParagraphs(cur.selBegin().par(), undopit);
-}
-
-
-string expandLabel(LyXTextClass const & textclass,
-       LyXLayout_ptr const & layout, bool appendix)
-{
-       string fmt = appendix ?
-               layout->labelstring_appendix() : layout->labelstring();
-
-       // handle 'inherited level parts' in 'fmt',
-       // i.e. the stuff between '@' in   '@Section@.\arabic{subsection}'
-       size_t const i = fmt.find('@', 0);
-       if (i != string::npos) {
-               size_t const j = fmt.find('@', i + 1);
-               if (j != string::npos) {
-                       string parent(fmt, i + 1, j - i - 1);
-                       string label = expandLabel(textclass, textclass[parent], appendix);
-                       fmt = string(fmt, 0, i) + label + string(fmt, j + 1, string::npos);
-               }
-       }
-
-       return textclass.counters().counterLabel(fmt);
-}
-
-
-namespace {
-
-void incrementItemDepth(ParagraphList & pars, par_type pit, par_type first_pit)
-{
-       int const cur_labeltype = pars[pit].layout()->labeltype;
-
-       if (cur_labeltype != LABEL_ENUMERATE && cur_labeltype != LABEL_ITEMIZE)
-               return;
-
-       int const cur_depth = pars[pit].getDepth();
-
-       par_type prev_pit = pit - 1;
-       while (true) {
-               int const prev_depth = pars[prev_pit].getDepth();
-               int const prev_labeltype = pars[prev_pit].layout()->labeltype;
-               if (prev_depth == 0 && cur_depth > 0) {
-                       if (prev_labeltype == cur_labeltype) {
-                               pars[pit].itemdepth = pars[prev_pit].itemdepth + 1;
-                       }
-                       break;
-               } else if (prev_depth < cur_depth) {
-                       if (prev_labeltype == cur_labeltype) {
-                               pars[pit].itemdepth = pars[prev_pit].itemdepth + 1;
-                               break;
-                       }
-               } else if (prev_depth == cur_depth) {
-                       if (prev_labeltype == cur_labeltype) {
-                               pars[pit].itemdepth = pars[prev_pit].itemdepth;
-                               break;
-                       }
-               }
-               if (prev_pit == first_pit)
-                       break;
-
-               --prev_pit;
-       }
 }
 
 
-void resetEnumCounterIfNeeded(ParagraphList & pars, par_type pit,
-       par_type firstpit, Counters & counters)
-{
-       if (pit == firstpit)
-               return;
-
-       int const cur_depth = pars[pit].getDepth();
-       par_type prev_pit = pit - 1;
-       while (true) {
-               int const prev_depth = pars[prev_pit].getDepth();
-               int const prev_labeltype = pars[prev_pit].layout()->labeltype;
-               if (prev_depth <= cur_depth) {
-                       if (prev_labeltype != LABEL_ENUMERATE) {
-                               switch (pars[pit].itemdepth) {
-                               case 0:
-                                       counters.reset("enumi");
-                               case 1:
-                                       counters.reset("enumii");
-                               case 2:
-                                       counters.reset("enumiii");
-                               case 3:
-                                       counters.reset("enumiv");
-                               }
-                       }
-                       break;
-               }
-
-               if (prev_pit == firstpit)
-                       break;
-
-               --prev_pit;
-       }
-}
-
-} // anon namespace
-
-
-// set the counter of a paragraph. This includes the labels
-void LyXText::setCounter(Buffer const & buf, par_type pit)
-{
-       BufferParams const & bufparams = buf.params();
-       LyXTextClass const & textclass = bufparams.getLyXTextClass();
-       LyXLayout_ptr const & layout = pars_[pit].layout();
-       par_type first_pit = 0;
-       Counters & counters = textclass.counters();
-
-       // Always reset
-       pars_[pit].itemdepth = 0;
-
-       if (pit == first_pit) {
-               pars_[pit].params().appendix(pars_[pit].params().startOfAppendix());
-       } else {
-               pars_[pit].params().appendix(pars_[pit - 1].params().appendix());
-               if (!pars_[pit].params().appendix() &&
-                   pars_[pit].params().startOfAppendix()) {
-                       pars_[pit].params().appendix(true);
-                       textclass.counters().reset();
-               }
-
-               // Maybe we have to increment the item depth.
-               incrementItemDepth(pars_, pit, first_pit);
-       }
-
-       // erase what was there before
-       pars_[pit].params().labelString(string());
-
-       if (layout->margintype == MARGIN_MANUAL) {
-               if (pars_[pit].params().labelWidthString().empty())
-                       pars_[pit].setLabelWidthString(layout->labelstring());
-       } else {
-               pars_[pit].setLabelWidthString(string());
-       }
-
-       // is it a layout that has an automatic label?
-       if (layout->labeltype == LABEL_COUNTER) {
-               BufferParams const & bufparams = buf.params();
-               LyXTextClass const & textclass = bufparams.getLyXTextClass();
-               counters.step(layout->counter);
-               string label = expandLabel(textclass, layout, pars_[pit].params().appendix());
-               pars_[pit].params().labelString(label);
-       } else if (layout->labeltype == LABEL_ITEMIZE) {
-               // At some point of time we should do something more
-               // clever here, like:
-               //   pars_[pit].params().labelString(
-               //    bufparams.user_defined_bullet(pars_[pit].itemdepth).getText());
-               // for now, use a simple hardcoded label
-               string itemlabel;
-               switch (pars_[pit].itemdepth) {
-               case 0:
-                       itemlabel = "*";
-                       break;
-               case 1:
-                       itemlabel = "-";
-                       break;
-               case 2:
-                       itemlabel = "@";
-                       break;
-               case 3:
-                       itemlabel = "·";
-                       break;
-               }
-
-               pars_[pit].params().labelString(itemlabel);
-       } else if (layout->labeltype == LABEL_ENUMERATE) {
-               // Maybe we have to reset the enumeration counter.
-               resetEnumCounterIfNeeded(pars_, pit, first_pit, counters);
-
-               // FIXME
-               // Yes I know this is a really, really! bad solution
-               // (Lgb)
-               string enumcounter = "enum";
-
-               switch (pars_[pit].itemdepth) {
-               case 2:
-                       enumcounter += 'i';
-               case 1:
-                       enumcounter += 'i';
-               case 0:
-                       enumcounter += 'i';
-                       break;
-               case 3:
-                       enumcounter += "iv";
-                       break;
-               default:
-                       // not a valid enumdepth...
-                       break;
-               }
-
-               counters.step(enumcounter);
-
-               pars_[pit].params().labelString(counters.enumLabel(enumcounter));
-       } else if (layout->labeltype == LABEL_BIBLIO) {// ale970302
-               counters.step("bibitem");
-               int number = counters.value("bibitem");
-               if (pars_[pit].bibitem()) {
-                       pars_[pit].bibitem()->setCounter(number);
-                       pars_[pit].params().labelString(layout->labelstring());
-               }
-               // In biblio should't be following counters but...
-       } else {
-               string s = buf.B_(layout->labelstring());
-
-               // the caption hack:
-               if (layout->labeltype == LABEL_SENSITIVE) {
-                       par_type end = paragraphs().size();
-                       par_type tmppit = pit;
-                       InsetBase * in = 0;
-                       bool isOK = false;
-                       while (tmppit != end) {
-                               in = pars_[tmppit].inInset();
-                               if (in->lyxCode() == InsetBase::FLOAT_CODE ||
-                                   in->lyxCode() == InsetBase::WRAP_CODE) {
-                                       isOK = true;
-                                       break;
-                               } else {
-                                       Paragraph const * owner = &ownerPar(buf, in);
-                                       tmppit = first_pit;
-                                       for ( ; tmppit != end; ++tmppit)
-                                               if (&pars_[tmppit] == owner)
-                                                       break;
-                               }
-                       }
-
-                       if (isOK) {
-                               string type;
-
-                               if (in->lyxCode() == InsetBase::FLOAT_CODE)
-                                       type = static_cast<InsetFloat*>(in)->params().type;
-                               else if (in->lyxCode() == InsetBase::WRAP_CODE)
-                                       type = static_cast<InsetWrap*>(in)->params().type;
-                               else
-                                       BOOST_ASSERT(false);
-
-                               Floating const & fl = textclass.floats().getType(type);
-
-                               counters.step(fl.type());
-
-                               // Doesn't work... yet.
-                               s = bformat(_("%1$s #:"), buf.B_(fl.name()));
-                       } else {
-                               // par->SetLayout(0);
-                               // s = layout->labelstring;
-                               s = _("Senseless: ");
-                       }
-               }
-               pars_[pit].params().labelString(s);
-
-       }
-}
-
-
-// Updates all counters.
-void LyXText::updateCounters()
-{
-       // start over
-       bv()->buffer()->params().getLyXTextClass().counters().reset();
-
-       bool update_pos = false;
-
-       par_type end = paragraphs().size();
-       for (par_type pit = 0; pit != end; ++pit) {
-               string const oldLabel = pars_[pit].params().labelString();
-               size_t maxdepth = 0;
-               if (pit != 0)
-                       maxdepth = pars_[pit - 1].getMaxDepthAfter();
-
-               if (pars_[pit].params().depth() > maxdepth)
-                       pars_[pit].params().depth(maxdepth);
-
-               // setCounter can potentially change the labelString.
-               setCounter(*bv()->buffer(), pit);
-               string const & newLabel = pars_[pit].params().labelString();
-               if (oldLabel != newLabel) {
-                       //lyxerr[Debug::DEBUG] << "changing labels: old: " << oldLabel << " new: "
-                       //      << newLabel << endl;
-                       redoParagraphInternal(pit);
-                       update_pos = true;
-               }
-       }
-       if (update_pos)
-               updateParPositions();
-}
-
-
-// this really should just inset the inset and not move the cursor.
+// this really should just insert the inset and not move the cursor.
 void LyXText::insertInset(LCursor & cur, InsetBase * inset)
 {
        BOOST_ASSERT(this == cur.text());
        BOOST_ASSERT(inset);
-       cur.paragraph().insertInset(cur.pos(), inset);
-       redoParagraph(cur);
+       cur.paragraph().insertInset(cur.pos(), inset, 
+                                   Change(cur.buffer().params().trackChanges ?
+                                          Change::INSERTED : Change::UNCHANGED));
 }
 
 
 // needed to insert the selection
-void LyXText::insertStringAsLines(LCursor & cur, string const & str)
+void LyXText::insertStringAsLines(LCursor & cur, docstring const & str)
 {
-       par_type pit = cur.par();
-       par_type endpit = cur.par() + 1;
-       pos_type pos = cur.pos();
-       recordUndo(cur);
-
-       // only to be sure, should not be neccessary
-       cur.clearSelection();
-       cur.buffer().insertStringAsLines(pars_, pit, pos, current_font, str);
-
-       redoParagraphs(cur.par(), endpit);
-       cur.resetAnchor();
-       setCursor(cur, cur.par(), pos);
-       cur.setSelection();
+       cur.buffer().insertStringAsLines(pars_, cur.pit(), cur.pos(),
+                                        current_font, str, autoBreakRows_);
 }
 
 
 // turn double CR to single CR, others are converted into one
 // blank. Then insertStringAsLines is called
-void LyXText::insertStringAsParagraphs(LCursor & cur, string const & str)
+void LyXText::insertStringAsParagraphs(LCursor & cur, docstring const & str)
 {
-       string linestr = str;
+       docstring linestr = str;
        bool newline_inserted = false;
 
        for (string::size_type i = 0, siz = linestr.size(); i < siz; ++i) {
@@ -946,7 +660,7 @@ void LyXText::insertStringAsParagraphs(LCursor & cur, string const & str)
                                linestr[i] = ' ';
                                newline_inserted = true;
                        }
-               } else if (IsPrintable(linestr[i])) {
+               } else if (isPrintable(linestr[i])) {
                        newline_inserted = false;
                }
        }
@@ -954,32 +668,23 @@ void LyXText::insertStringAsParagraphs(LCursor & cur, string const & str)
 }
 
 
-bool LyXText::setCursor(LCursor & cur, par_type par, pos_type pos,
-       bool setfont, bool boundary)
+bool LyXText::setCursor(LCursor & cur, pit_type par, pos_type pos,
+                       bool setfont, bool boundary)
 {
        LCursor old = cur;
        setCursorIntern(cur, par, pos, setfont, boundary);
-       return deleteEmptyParagraphMechanism(cur, old);
+       return cur.bv().checkDepm(cur, old);
 }
 
 
-void LyXText::setCursor(CursorSlice & cur, par_type par,
-       pos_type pos, bool boundary)
+void LyXText::setCursor(CursorSlice & cur, pit_type par, pos_type pos)
 {
        BOOST_ASSERT(par != int(paragraphs().size()));
-
-       cur.par() = par;
+       cur.pit() = par;
        cur.pos() = pos;
-       cur.boundary() = boundary;
-
-       // no rows, no fun...
-       if (paragraphs().begin()->rows.empty())
-               return;
 
        // 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
        if (pos < 0) {
@@ -990,24 +695,6 @@ void LyXText::setCursor(CursorSlice & cur, par_type par,
        if (pos > para.size()) {
                lyxerr << "dont like 1, pos: " << pos
                       << " size: " << para.size()
-                      << " row.pos():" << row.pos()
-                      << " par: " << par << endl;
-               BOOST_ASSERT(false);
-       }
-
-       if (pos > end) {
-               lyxerr << "dont like 2, pos: " << pos
-                      << " size: " << para.size()
-                      << " row.pos():" << row.pos()
-                      << " par: " << par << endl;
-               // This shouldn't happen.
-               BOOST_ASSERT(false);
-       }
-
-       if (pos < row.pos()) {
-               lyxerr << "dont like 3 please report pos:" << pos
-                      << " size: " << para.size()
-                      << " row.pos():" << row.pos()
                       << " par: " << par << endl;
                BOOST_ASSERT(false);
        }
@@ -1015,10 +702,11 @@ void LyXText::setCursor(CursorSlice & cur, par_type par,
 
 
 void LyXText::setCursorIntern(LCursor & cur,
-       par_type par, pos_type pos, bool setfont, bool boundary)
+                             pit_type par, pos_type pos, bool setfont, bool boundary)
 {
-       setCursor(cur.top(), par, pos, boundary);
-       cur.x_target() = cursorX(cur.top());
+       BOOST_ASSERT(this == cur.text());
+       cur.boundary(boundary);
+       setCursor(cur.top(), par, pos);
        if (setfont)
                setCurrentFont(cur);
 }
@@ -1028,7 +716,7 @@ void LyXText::setCurrentFont(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
        pos_type pos = cur.pos();
-       par_type pit = cur.par();
+       Paragraph & par = cur.paragraph();
 
        if (cur.boundary() && pos > 0)
                --pos;
@@ -1037,7 +725,7 @@ void LyXText::setCurrentFont(LCursor & cur)
                if (pos == cur.lastpos())
                        --pos;
                else // potentional bug... BUG (Lgb)
-                       if (pars_[pit].isSeparator(pos)) {
+                       if (par.isSeparator(pos)) {
                                if (pos > cur.textRow().pos() &&
                                    bidi.level(pos) % 2 ==
                                    bidi.level(pos - 1) % 2)
@@ -1048,13 +736,13 @@ void LyXText::setCurrentFont(LCursor & cur)
        }
 
        BufferParams const & bufparams = cur.buffer().params();
-       current_font = pars_[pit].getFontSettings(bufparams, pos);
-       real_current_font = getFont(pit, pos);
+       current_font = par.getFontSettings(bufparams, pos);
+       real_current_font = getFont(cur.buffer(), par, pos);
 
        if (cur.pos() == cur.lastpos()
-           && bidi.isBoundary(cur.buffer(), pars_[pit], cur.pos())
+           && bidi.isBoundary(cur.buffer(), par, cur.pos())
            && !cur.boundary()) {
-               Language const * lang = pars_[pit].getParLanguage(bufparams);
+               Language const * lang = par.getParLanguage(bufparams);
                current_font.setLanguage(lang);
                current_font.setNumber(LyXFont::OFF);
                real_current_font.setLanguage(lang);
@@ -1062,176 +750,152 @@ void LyXText::setCurrentFont(LCursor & cur)
        }
 }
 
-
-// x is an absolute screen coord
-// returns the column near the specified x-coordinate of the row
-// x is set to the real beginning of this column
-pos_type LyXText::getColumnNearX(par_type pit,
-       Row const & row, int & x, bool & boundary) const
+// y is screen coordinate
+pit_type LyXText::getPitNearY(BufferView & bv, int y) const
 {
-       x -= xo_;
-       RowMetrics const r = computeRowMetrics(pit, row);
-
-       pos_type vc = row.pos();
-       pos_type end = row.endpos();
-       pos_type c = 0;
-       LyXLayout_ptr const & layout = pars_[pit].layout();
-
-       bool left_side = false;
-
-       pos_type body_pos = pars_[pit].beginOfBody();
-
-       double tmpx = r.x;
-       double last_tmpx = tmpx;
-
-       if (body_pos > 0 &&
-           (body_pos > end || !pars_[pit].isLineSeparator(body_pos - 1)))
-               body_pos = 0;
-
-       // check for empty row
-       if (vc == end) {
-               x = int(tmpx) + xo_;
-               return 0;
+       BOOST_ASSERT(!paragraphs().empty());
+       BOOST_ASSERT(bv.coordCache().getParPos().find(this) != bv.coordCache().getParPos().end());
+       CoordCache::InnerParPosCache const & cc = bv.coordCache().getParPos().find(this)->second;
+       LYXERR(Debug::DEBUG)
+               << BOOST_CURRENT_FUNCTION
+               << ": y: " << y << " cache size: " << cc.size()
+               << endl;
+
+       // look for highest numbered paragraph with y coordinate less than given y
+       pit_type pit = 0;
+       int yy = -1;
+       CoordCache::InnerParPosCache::const_iterator it = cc.begin();
+       CoordCache::InnerParPosCache::const_iterator et = cc.end();
+       CoordCache::InnerParPosCache::const_iterator last = et; last--;
+
+       TextMetrics & tm = bv.textMetrics(this);
+       ParagraphMetrics const & pm = tm.parMetrics(it->first);
+
+       // If we are off-screen (before the visible part)
+       if (y < 0
+               // and even before the first paragraph in the cache.
+               && y < it->second.y_ - int(pm.ascent())) {
+               //  and we are not at the first paragraph in the inset.
+               if (it->first == 0)
+                       return 0;
+               // then this is the paragraph we are looking for.
+               pit = it->first - 1;
+               // rebreak it and update the CoordCache.
+               tm.redoParagraph(pit);
+               bv.coordCache().parPos()[this][pit] =
+                       Point(0, it->second.y_ - pm.descent());
+               return pit;
        }
 
-       while (vc < end && tmpx <= x) {
-               c = bidi.vis2log(vc);
-               last_tmpx = tmpx;
-               if (body_pos > 0 && c == body_pos - 1) {
-                       tmpx += r.label_hfill +
-                               font_metrics::width(layout->labelsep, getLabelFont(pit));
-                       if (pars_[pit].isLineSeparator(body_pos - 1))
-                               tmpx -= singleWidth(pit, body_pos - 1);
-               }
-
-               if (hfillExpansion(pars_[pit], row, c)) {
-                       tmpx += singleWidth(pit, c);
-                       if (c >= body_pos)
-                               tmpx += r.hfill;
-                       else
-                               tmpx += r.label_hfill;
-               } else if (pars_[pit].isSeparator(c)) {
-                       tmpx += singleWidth(pit, c);
-                       if (c >= body_pos)
-                               tmpx += r.separator;
-               } else {
-                       tmpx += singleWidth(pit, c);
-               }
-               ++vc;
+       ParagraphMetrics const & pm_last = bv.parMetrics(this, last->first);
+
+       // If we are off-screen (after the visible part)
+       if (y > bv.workHeight()
+               // and even after the first paragraph in the cache.
+               && y >= last->second.y_ + int(pm_last.descent())) {
+               pit = last->first + 1;
+               //  and we are not at the last paragraph in the inset.
+               if (pit == int(pars_.size()))
+                       return last->first;
+               // then this is the paragraph we are looking for.
+               // rebreak it and update the CoordCache.
+               tm.redoParagraph(pit);
+               bv.coordCache().parPos()[this][pit] =
+                       Point(0, last->second.y_ + pm_last.ascent());
+               return pit;
        }
 
-       if ((tmpx + last_tmpx) / 2 > x) {
-               tmpx = last_tmpx;
-               left_side = true;
-       }
+       for (; it != et; ++it) {
+               LYXERR(Debug::DEBUG)
+                       << BOOST_CURRENT_FUNCTION
+                       << "  examining: pit: " << it->first
+                       << " y: " << it->second.y_
+                       << endl;
+
+               ParagraphMetrics const & pm = bv.parMetrics(this, it->first);
 
-       BOOST_ASSERT(vc <= end);  // This shouldn't happen.
-
-       boundary = false;
-       // This (rtl_support test) is not needed, but gives
-       // some speedup if rtl_support == false
-       bool const lastrow = lyxrc.rtl_support && row.endpos() == pars_[pit].size();
-
-       // If lastrow is false, we don't need to compute
-       // the value of rtl.
-       bool const rtl = lastrow ? isRTL(pars_[pit]) : false;
-       if (lastrow &&
-                ((rtl  &&  left_side && vc == row.pos() && x < tmpx - 5) ||
-                 (!rtl && !left_side && vc == end  && x > tmpx + 5)))
-               c = end;
-       else if (vc == row.pos()) {
-               c = bidi.vis2log(vc);
-               if (bidi.level(c) % 2 == 1)
-                       ++c;
-       } else {
-               c = bidi.vis2log(vc - 1);
-               bool const rtl = (bidi.level(c) % 2 == 1);
-               if (left_side == rtl) {
-                       ++c;
-                       boundary = bidi.isBoundary(*bv()->buffer(), pars_[pit], c);
+               if (it->first >= pit && int(it->second.y_) - int(pm.ascent()) <= y) {
+                       pit = it->first;
+                       yy = it->second.y_;
                }
        }
 
-       if (row.pos() < end && c >= end && pars_[pit].isNewline(end - 1)) {
-               if (bidi.level(end -1) % 2 == 0)
-                       tmpx -= singleWidth(pit, end - 1);
-               else
-                       tmpx += singleWidth(pit, end - 1);
-               c = end - 1;
-       }
+       LYXERR(Debug::DEBUG)
+               << BOOST_CURRENT_FUNCTION
+               << ": found best y: " << yy << " for pit: " << pit
+               << endl;
 
-       x = int(tmpx) + xo_;
-       return c - row.pos();
+       return pit;
 }
 
 
-// y is relative to this LyXText's top
-// this is only used in the two functions below
-Row const & LyXText::getRowNearY(int y, par_type & pit) const
+Row const & LyXText::getRowNearY(BufferView const & bv, int y, pit_type pit) const
 {
-       BOOST_ASSERT(!paragraphs().empty());
-       BOOST_ASSERT(!paragraphs().begin()->rows.empty());
-       par_type const pend = paragraphs().size() - 1;
-       pit = 0;
-       while (int(pars_[pit].y + pars_[pit].height) < y && pit != pend)
-               ++pit;
-
-       RowList::iterator rit = pars_[pit].rows.end();
-       RowList::iterator const rbegin = pars_[pit].rows.begin();
-       do {
-               --rit;
-       } while (rit != rbegin && int(pars_[pit].y + rit->y_offset()) > y);
-
+       ParagraphMetrics const & pm = bv.parMetrics(this, pit);
+
+       int yy = bv.coordCache().get(this, pit).y_ - pm.ascent();
+       BOOST_ASSERT(!pm.rows().empty());
+       RowList::const_iterator rit = pm.rows().begin();
+       RowList::const_iterator const rlast = boost::prior(pm.rows().end());
+       for (; rit != rlast; yy += rit->height(), ++rit)
+               if (yy + rit->height() > y)
+                       break;
        return *rit;
 }
 
 
-// x,y are absolute coordinates
-// sets cursor only within this LyXText
-void LyXText::setCursorFromCoordinates(LCursor & cur, int x, int y)
-{
-       x -= xo_;
-       y -= yo_;
-       par_type pit;
-       Row const & row = getRowNearY(y, pit);
-       lyxerr[Debug::DEBUG] << "setCursorFromCoordinates:: hit row at: "
-                            << row.pos() << endl;
-       bool bound = false;
-       int xx = x + xo_; // getRowNearX get absolute x coords
-       pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
-       setCursor(cur, pit, pos, true, bound);
-}
-
-
 // x,y are absolute screen coordinates
 // sets cursor recursively descending into nested editable insets
-InsetBase * LyXText::editXY(LCursor & cur, int x, int y) const
+InsetBase * LyXText::editXY(LCursor & cur, int x, int y)
 {
-       par_type pit;
-       Row const & row = getRowNearY(y - yo_, pit);
+       if (lyxerr.debugging(Debug::WORKAREA)) {
+               lyxerr << "LyXText::editXY(cur, " << x << ", " << y << ")" << std::endl;
+               cur.bv().coordCache().dump();
+       }
+       pit_type pit = getPitNearY(cur.bv(), y);
+       BOOST_ASSERT(pit != -1);
+
+       Row const & row = getRowNearY(cur.bv(), y, pit);
        bool bound = false;
 
+       TextMetrics const & tm = cur.bv().textMetrics(this);
        int xx = x; // is modified by getColumnNearX
-       pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
-       cur.par() = pit;
+       pos_type const pos = row.pos()
+               + tm.getColumnNearX(pit, row, xx, bound);
+       cur.pit() = pit;
        cur.pos() = pos;
-       cur.boundary() = bound;
+       cur.boundary(bound);
+       cur.x_target() = x;
 
        // try to descend into nested insets
-       InsetBase * inset = checkInsetHit(x, y);
-       lyxerr << "inset " << inset << " hit at x: " << x << " y: " << y << endl;
-       if (!inset)
+       InsetBase * inset = checkInsetHit(cur.bv(), x, y);
+       //lyxerr << "inset " << inset << " hit at x: " << x << " y: " << y << endl;
+       if (!inset) {
+               // Either we deconst editXY or better we move current_font
+               // and real_current_font to LCursor
+               setCurrentFont(cur);
                return 0;
+       }
+
+       InsetBase * insetBefore = pos? pars_[pit].getInset(pos - 1): 0;
+       //InsetBase * insetBehind = pars_[pit].getInset(pos);
 
        // This should be just before or just behind the
        // cursor position set above.
-       BOOST_ASSERT((pos != 0 && inset == pars_[pit].getInset(pos - 1))
-                    || inset == pars_[pit].getInset(pos));
+       BOOST_ASSERT((pos != 0 && inset == insetBefore)
+               || inset == pars_[pit].getInset(pos));
+
        // Make sure the cursor points to the position before
        // this inset.
-       if (inset == pars_[pit].getInset(pos - 1))
+       if (inset == insetBefore)
                --cur.pos();
-       return inset->editXY(cur, x, y);
+
+       // Try to descend recursively inside the inset.
+       inset = inset->editXY(cur, x, y);
+
+       if (cur.top().text() == this)
+               setCurrentFont(cur);
+       return inset;
 }
 
 
@@ -1249,93 +913,210 @@ bool LyXText::checkAndActivateInset(LCursor & cur, bool front)
 }
 
 
-void LyXText::cursorLeft(LCursor & cur)
+bool LyXText::cursorLeft(LCursor & cur)
 {
+       // Tell BufferView to test for FitCursor in any case!
+       cur.updateFlags(Update::FitCursor);
+
+       if (!cur.boundary() && cur.pos() > 0 &&
+           cur.textRow().pos() == cur.pos() &&
+           !cur.paragraph().isLineSeparator(cur.pos()-1) &&
+           !cur.paragraph().isNewline(cur.pos()-1)) {
+               return setCursor(cur, cur.pit(), cur.pos(), true, true);
+       }
        if (cur.pos() != 0) {
-               bool boundary = cur.boundary();
-               setCursor(cur, cur.par(), cur.pos() - 1, true, false);
+               bool updateNeeded = setCursor(cur, cur.pit(), cur.pos() - 1, true, false);
                if (!checkAndActivateInset(cur, false)) {
+                       /** FIXME: What's this cause purpose???
+                       bool boundary = cur.boundary();
                        if (false && !boundary &&
-                                       bidi.isBoundary(cur.buffer(), cur.paragraph(), cur.pos() + 1))
-                               setCursor(cur, cur.par(), cur.pos() + 1, true, true);
+                           bidi.isBoundary(cur.buffer(), cur.paragraph(), cur.pos() + 1))
+                               updateNeeded |=
+                                       setCursor(cur, cur.pit(), cur.pos() + 1, true, true);
+                       */
                }
-               return;
+               return updateNeeded;
        }
 
-       if (cur.par() != 0) {
-               // steps into the paragraph above
-               setCursor(cur, cur.par() - 1, getPar(cur.par() - 1).size());
+       if (cur.pit() != 0) {
+               // Steps into the paragraph above
+               return setCursor(cur, cur.pit() - 1, getPar(cur.pit() - 1).size());
        }
+       return false;
 }
 
 
-void LyXText::cursorRight(LCursor & cur)
+bool LyXText::cursorRight(LCursor & cur)
 {
-       if (false && cur.boundary()) {
-               setCursor(cur, cur.par(), cur.pos(), true, false);
-               return;
-       }
+       // Tell BufferView to test for FitCursor in any case!
+       cur.updateFlags(Update::FitCursor);
 
        if (cur.pos() != cur.lastpos()) {
+               if (cur.boundary())
+                       return setCursor(cur, cur.pit(), cur.pos(),
+                                        true, false);
+
+               bool updateNeeded = false;
                if (!checkAndActivateInset(cur, true)) {
-                       setCursor(cur, cur.par(), cur.pos() + 1, true, false);
+                       if (cur.textRow().endpos() == cur.pos() + 1 &&
+                           cur.textRow().endpos() != cur.lastpos() &&
+                           !cur.paragraph().isLineSeparator(cur.pos()) &&
+                           !cur.paragraph().isNewline(cur.pos())) {
+                               cur.boundary(true);
+                       }
+                       updateNeeded |= setCursor(cur, cur.pit(), cur.pos() + 1, true, cur.boundary());
                        if (false && bidi.isBoundary(cur.buffer(), cur.paragraph(),
-                                                        cur.pos()))
-                               setCursor(cur, cur.par(), cur.pos(), true, true);
+                                                    cur.pos()))
+                               updateNeeded |= setCursor(cur, cur.pit(), cur.pos(), true, true);
                }
-               return;
+               return updateNeeded;
        }
 
-       if (cur.par() != cur.lastpar())
-               setCursor(cur, cur.par() + 1, 0);
+       if (cur.pit() != cur.lastpit())
+               return setCursor(cur, cur.pit() + 1, 0);
+       return false;
 }
 
 
-void LyXText::cursorUp(LCursor & cur)
+bool LyXText::cursorUp(LCursor & cur)
 {
-       Row const & row = cur.textRow();
-       int x = cur.x_target();
-       int y = cursorY(cur.top()) - row.baseline() - 1;
-       setCursorFromCoordinates(cur, x, y);
+       // Tell BufferView to test for FitCursor in any case!
+       cur.updateFlags(Update::FitCursor);
+
+       TextMetrics const & tm = cur.bv().textMetrics(this);
+       ParagraphMetrics const & pm = tm.parMetrics(cur.pit());
+
+       int row;
+       if (cur.pos() && cur.boundary())
+               row = pm.pos2row(cur.pos()-1);
+       else
+               row = pm.pos2row(cur.pos());
+
+       // remember current position only if we are not at the end of a row.
+       if (cur.pos() != pm.rows()[row].endpos())
+               cur.setTargetX();
+       int const x = cur.targetX();
 
        if (!cur.selection()) {
-               InsetBase * inset_hit = checkInsetHit(cur.x_target(), y);
-               if (inset_hit && isHighlyEditableInset(inset_hit))
-                       inset_hit->editXY(cur, cur.x_target(), y);
+               int const y = bv_funcs::getPos(cur.bv(), cur, cur.boundary()).y_;
+               LCursor old = cur;
+               // Go to middle of previous row. 16 found to work OK;
+               // 12 = top/bottom margin of display math
+               int const margin = 3 * InsetMathHull::displayMargin() / 2;
+               editXY(cur, x, y - pm.rows()[row].ascent() - margin);
+               cur.clearSelection();
+
+               // This happens when you move out of an inset.
+               // And to give the DEPM the possibility of doing
+               // something we must provide it with two different
+               // cursors. (Lgb)
+               LCursor dummy = cur;
+               if (dummy == old)
+                       ++dummy.pos();
+
+               cur.bv().checkDepm(dummy, old);
+               return false;
+       }
+
+       bool updateNeeded = false;
+
+       if (row > 0) {
+               updateNeeded |= setCursor(cur, cur.pit(),
+                       tm.x2pos(cur.pit(), row - 1, x));
+       } else if (cur.pit() > 0) {
+               --cur.pit();
+               //cannot use 'par' now
+               ParagraphMetrics const & pmcur = cur.bv().parMetrics(this, cur.pit());
+               updateNeeded |= setCursor(cur, cur.pit(),
+                       tm.x2pos(cur.pit(), pmcur.rows().size() - 1, x));
        }
+
+       cur.x_target() = x;
+
+       return updateNeeded;
 }
 
 
-void LyXText::cursorDown(LCursor & cur)
+bool LyXText::cursorDown(LCursor & cur)
 {
-       Row const & row = cur.textRow();
-       int x = cur.x_target();
-       int y = cursorY(cur.top()) - row.baseline() + row.height() + 1;
-       setCursorFromCoordinates(cur, x, y);
+       // Tell BufferView to test for FitCursor in any case!
+       cur.updateFlags(Update::FitCursor);
+
+       TextMetrics const & tm = cur.bv().textMetrics(this);
+       ParagraphMetrics const & pm = tm.parMetrics(cur.pit());
+
+       int row;
+       if (cur.pos() && cur.boundary())
+               row = pm.pos2row(cur.pos()-1);
+       else
+               row = pm.pos2row(cur.pos());
+
+       // remember current position only if we are not at the end of a row.
+       if (cur.pos() != pm.rows()[row].endpos())
+               cur.setTargetX();
+       int const x = cur.targetX();
 
        if (!cur.selection()) {
-               InsetBase * inset_hit = checkInsetHit(cur.x_target(), y);
-               if (inset_hit && isHighlyEditableInset(inset_hit))
-                       inset_hit->editXY(cur, cur.x_target(), y);
+               int const y = bv_funcs::getPos(cur.bv(), cur, cur.boundary()).y_;
+               LCursor old = cur;
+               // To middle of next row
+               int const margin = 3 * InsetMathHull::displayMargin() / 2;
+               editXY(cur, x, y + pm.rows()[row].descent() + margin);
+               cur.clearSelection();
+
+               // This happens when you move out of an inset.
+               // And to give the DEPM the possibility of doing
+               // something we must provide it with two different
+               // cursors. (Lgb)
+               LCursor dummy = cur;
+               if (dummy == old)
+                       ++dummy.pos();
+               
+               bool const changed = cur.bv().checkDepm(dummy, old);
+
+               // Make sure that cur gets back whatever happened to dummy(Lgb)
+               if (changed)
+                       cur = dummy;
+
+               return false;
+       }
+
+       bool updateNeeded = false;
+
+       if (row + 1 < int(pm.rows().size())) {
+               updateNeeded |= setCursor(cur, cur.pit(),
+                       tm.x2pos(cur.pit(), row + 1, x));
+       } else if (cur.pit() + 1 < int(paragraphs().size())) {
+               ++cur.pit();
+               updateNeeded |= setCursor(cur, cur.pit(),
+                       tm.x2pos(cur.pit(), 0, x));
        }
+
+       cur.x_target() = x;
+
+       return updateNeeded;
 }
 
 
-void LyXText::cursorUpParagraph(LCursor & cur)
+bool LyXText::cursorUpParagraph(LCursor & cur)
 {
+       bool updated = false;
        if (cur.pos() > 0)
-               setCursor(cur, cur.par(), 0);
-       else if (cur.par() != 0)
-               setCursor(cur, cur.par() - 1, 0);
+               updated = setCursor(cur, cur.pit(), 0);
+       else if (cur.pit() != 0)
+               updated = setCursor(cur, cur.pit() - 1, 0);
+       return updated;
 }
 
 
-void LyXText::cursorDownParagraph(LCursor & cur)
+bool LyXText::cursorDownParagraph(LCursor & cur)
 {
-       if (cur.par() != cur.lastpar())
-               setCursor(cur, cur.par() + 1, 0);
+       bool updated = false;
+       if (cur.pit() != cur.lastpit())
+               updated = setCursor(cur, cur.pit() + 1, 0);
        else
-               setCursor(cur, cur.par(), cur.lastpos());
+               updated = setCursor(cur, cur.pit(), cur.lastpos());
+       return updated;
 }
 
 
@@ -1343,49 +1124,40 @@ void LyXText::cursorDownParagraph(LCursor & cur)
 // position. Called by deleteEmptyParagraphMechanism
 void LyXText::fixCursorAfterDelete(CursorSlice & cur, CursorSlice const & where)
 {
-       // do notheing if cursor is not in the paragraph where the
+       // Do nothing if cursor is not in the paragraph where the
        // deletion occured,
-       if (cur.par() != where.par())
+       if (cur.pit() != where.pit())
                return;
 
-       // if cursor position is after the deletion place update it
+       // If cursor position is after the deletion place update it
        if (cur.pos() > where.pos())
                --cur.pos();
 
-       // check also if we don't want to set the cursor on a spot behind the
+       // 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() > cur.lastpos())
                cur.pos() = cur.lastpos();
 }
 
 
-bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
+bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur,
+               LCursor & old, bool & need_anchor_change)
 {
-       BOOST_ASSERT(cur.size() == old.size());
-       // Would be wrong to delete anything if we have a selection.
-       if (cur.selection())
-               return false;
+       //LYXERR(Debug::DEBUG) << "DEPM: cur:\n" << cur << "old:\n" << old << endl;
 
-       //lyxerr[Debug::DEBUG] << "DEPM: cur:\n" << cur << "old:\n" << old << endl;
-       Paragraph const & oldpar = pars_[old.par()];
+       Paragraph & oldpar = old.paragraph();
 
        // We allow all kinds of "mumbo-jumbo" when freespacing.
        if (oldpar.isFreeSpacing())
                return false;
 
        /* Ok I'll put some comments here about what is missing.
-          I have fixed BackSpace (and thus Delete) to not delete
-          double-spaces automagically. I have also changed Cut,
-          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 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
-          sure none of these are very hard to fix, but I will
-          put out 1.1.4pre2 with FIX_DOUBLE_SPACE defined so
-          that I can get some feedback. (Lgb)
+          beginning and then save right after the paste. (Lgb)
        */
 
        // If old.pos() == 0 and old.pos()(1) == LineSeparator
@@ -1396,15 +1168,19 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
        // delete the LineSeparator.
        // MISSING
 
-       // If the chars around the old cursor were spaces, delete one of them.
-       if (old.par() != cur.par() || old.pos() != cur.pos()) {
+       bool const same_inset = &old.inset() == &cur.inset();
+       bool const same_par = same_inset && old.pit() == cur.pit();
+       bool const same_par_pos = same_par && old.pos() == cur.pos();
 
+       // If the chars around the old cursor were spaces, delete one of them.
+       if (!same_par_pos) {
                // Only if the cursor has really moved.
                if (old.pos() > 0
                    && old.pos() < oldpar.size()
                    && oldpar.isLineSeparator(old.pos())
-                   && oldpar.isLineSeparator(old.pos() - 1)) {
-                       pars_[old.par()].erase(old.pos() - 1);
+                   && oldpar.isLineSeparator(old.pos() - 1)
+                   && !oldpar.isDeleted(old.pos() - 1)) {
+                       oldpar.eraseChar(old.pos() - 1, cur.buffer().params().trackChanges);
 #ifdef WITH_WARNINGS
 #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
@@ -1412,93 +1188,112 @@ bool LyXText::deleteEmptyParagraphMechanism(LCursor & cur, LCursor const & old)
 // automated way in CursorSlice code. (JMarc 26/09/2001)
 #endif
                        // correct all cursor parts
-                       fixCursorAfterDelete(cur.top(), old.top());
-#ifdef WITH_WARNINGS
-#warning DEPM, look here
-#endif
-                       //fixCursorAfterDelete(cur.anchor(), old.top());
-                       return false;
+                       if (same_par) {
+                               fixCursorAfterDelete(cur.top(), old.top());
+                               need_anchor_change = true;
+                       }
+                       return true;
                }
        }
 
        // only do our magic if we changed paragraph
-       if (old.par() == cur.par())
+       if (same_par)
                return false;
 
        // don't delete anything if this is the ONLY paragraph!
-       if (pars_.size() == 1)
+       if (old.lastpit() == 0)
                return false;
 
        // Do not delete empty paragraphs with keepempty set.
        if (oldpar.allowEmpty())
                return false;
 
-       // record if we have deleted a paragraph
-       // we can't possibly have deleted a paragraph before this point
-       bool deleted = false;
-
        if (oldpar.empty() || (oldpar.size() == 1 && oldpar.isLineSeparator(0))) {
-               // ok, we will delete something
-               CursorSlice tmpcursor;
-
-               deleted = true;
-
-               bool selection_position_was_oldcursor_position =
-                       cur.anchor().par() == old.par() && cur.anchor().pos() == old.pos();
-
-               // This is a bit of a overkill. We change the old and the cur par
-               // at max, certainly not everything in between...
-               recUndo(old.par(), cur.par());
-
                // Delete old par.
-               pars_.erase(pars_.begin() + old.par());
-
-               // Update cursor par offset if necessary.
-               // Some 'iterator registration' would be nice that takes care of
-               // such events. Maybe even signal/slot?
-               if (cur.par() > old.par())
-                       --cur.par();
-#ifdef WITH_WARNINGS
-#warning DEPM, look here
-#endif
-//             if (cur.anchor().par() > old.par())
-//                     --cur.anchor().par();
-
-               if (selection_position_was_oldcursor_position) {
-                       // correct selection
-                       cur.resetAnchor();
+               recordUndo(old, Undo::ATOMIC,
+                          max(old.pit() - 1, pit_type(0)),
+                          min(old.pit() + 1, old.lastpit()));
+               ParagraphList & plist = old.text()->paragraphs();
+               plist.erase(boost::next(plist.begin(), old.pit()));
+
+               // see #warning above
+               if (cur.depth() >= old.depth()) {
+                       CursorSlice & curslice = cur[old.depth() - 1];
+                       if (&curslice.inset() == &old.inset()
+                           && curslice.pit() > old.pit()) {
+                               --curslice.pit();
+                               // since a paragraph has been deleted, all the
+                               // insets after `old' have been copied and
+                               // their address has changed. Therefore we
+                               // need to `regenerate' cur. (JMarc)
+                               cur.updateInsets(&(cur.bottom().inset()));
+                               need_anchor_change = true;
+                       }
                }
+               return true;
        }
 
-       if (deleted)
+       if (oldpar.stripLeadingSpaces(cur.buffer().params().trackChanges)) {
+               need_anchor_change = true;
+               // We return true here because the Paragraph contents changed and
+               // we need a redraw before further action is processed.
                return true;
-
-       if (pars_[old.par()].stripLeadingSpaces())
-               cur.resetAnchor();
+       }
 
        return false;
 }
 
 
-ParagraphList & LyXText::paragraphs() const
+void LyXText::deleteEmptyParagraphMechanism(pit_type first, pit_type last, bool trackChanges)
 {
-       return const_cast<ParagraphList &>(pars_);
-}
+       BOOST_ASSERT(first >= 0 && first <= last && last < (int) pars_.size());
 
+       for (pit_type pit = first; pit <= last; ++pit) {
+               Paragraph & par = pars_[pit];
 
-void LyXText::recUndo(par_type first, par_type last) const
-{
-       recordUndo(bv()->cursor(), Undo::ATOMIC, first, last);
+               // We allow all kinds of "mumbo-jumbo" when freespacing.
+               if (par.isFreeSpacing())
+                       continue;
+
+               for (pos_type pos = 1; pos < par.size(); ++pos) {
+                       if (par.isLineSeparator(pos) && par.isLineSeparator(pos - 1)
+                           && !par.isDeleted(pos - 1)) {
+                               if (par.eraseChar(pos - 1, trackChanges)) {
+                                       --pos;
+                               }
+                       }
+               }
+
+               // don't delete anything if this is the only remaining paragraph within the given range
+               // note: LyXText::acceptOrRejectChanges() sets the cursor to 'first' after calling DEPM 
+               if (first == last)
+                       continue;
+
+               // don't delete empty paragraphs with keepempty set
+               if (par.allowEmpty())
+                       continue;
+
+               if (par.empty() || (par.size() == 1 && par.isLineSeparator(0))) {
+                       pars_.erase(boost::next(pars_.begin(), pit));
+                       --pit;
+                       --last;
+                       continue;
+               }
+
+               par.stripLeadingSpaces(trackChanges);
+       }
 }
 
 
-void LyXText::recUndo(par_type par) const
+void LyXText::recUndo(LCursor & cur, pit_type first, pit_type last) const
 {
-       recordUndo(bv()->cursor(), Undo::ATOMIC, par, par);
+       recordUndo(cur, Undo::ATOMIC, first, last);
 }
 
 
-int defaultRowHeight()
+void LyXText::recUndo(LCursor & cur, pit_type par) const
 {
-       return int(font_metrics::maxHeight(LyXFont(LyXFont::ALL_SANE)) *  1.2);
+       recordUndo(cur, Undo::ATOMIC, par, par);
 }
+
+} // namespace lyx