]> git.lyx.org Git - lyx.git/blobdiff - src/text.C
fix compiler warnings about unused parameter
[lyx.git] / src / text.C
index 8d1b5187acdadadedd3cb78e1627dfc2c06972ed..93b31bbec3ec4b351bc11a0d5071bed4493525db 100644 (file)
@@ -23,6 +23,7 @@
 #include "buffer_funcs.h"
 #include "bufferparams.h"
 #include "BufferView.h"
+#include "bufferview_funcs.h"
 #include "cursor.h"
 #include "pariterator.h"
 #include "coordcache.h"
@@ -52,6 +53,7 @@
 
 #include "frontends/FontMetrics.h"
 #include "frontends/Painter.h"
+#include "frontends/Selection.h"
 
 #include "insets/insettext.h"
 #include "insets/insetbibitem.h"
 
 #include <sstream>
 
+using std::auto_ptr;
+using std::advance;
+using std::distance;
+using std::max;
+using std::min;
+using std::endl;
+using std::string;
 
 namespace lyx {
 
@@ -87,73 +96,8 @@ using cap::pasteParagraphList;
 
 using frontend::FontMetrics;
 
-using std::auto_ptr;
-using std::advance;
-using std::distance;
-using std::max;
-using std::min;
-using std::endl;
-using std::string;
-
-
 namespace {
 
-int numberOfSeparators(Paragraph const & par, Row const & row)
-{
-       pos_type const first = max(row.pos(), par.beginOfBody());
-       pos_type const last = row.endpos() - 1;
-       int n = 0;
-       for (pos_type p = first; p < last; ++p) {
-               if (par.isSeparator(p))
-                       ++n;
-       }
-       return n;
-}
-
-
-int numberOfLabelHfills(Paragraph const & par, Row const & row)
-{
-       pos_type last = row.endpos() - 1;
-       pos_type first = row.pos();
-
-       // hfill *DO* count at the beginning of paragraphs!
-       if (first) {
-               while (first < last && par.isHfill(first))
-                       ++first;
-       }
-
-       last = min(last, par.beginOfBody());
-       int n = 0;
-       for (pos_type p = first; p < last; ++p) {
-               if (par.isHfill(p))
-                       ++n;
-       }
-       return n;
-}
-
-
-int numberOfHfills(Paragraph const & par, Row const & row)
-{
-       pos_type const last = row.endpos();
-       pos_type first = row.pos();
-
-       // hfill *DO* count at the beginning of paragraphs!
-       if (first) {
-               while (first < last && par.isHfill(first))
-                       ++first;
-       }
-
-       first = max(first, par.beginOfBody());
-
-       int n = 0;
-       for (pos_type p = first; p < last; ++p) {
-               if (par.isHfill(p))
-                       ++n;
-       }
-       return n;
-}
-
-
 void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
        string const & token, LyXFont & font, Change & change, ErrorList & errorList)
 {
@@ -218,7 +162,7 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                        par.insertInset(par.size(), inset, font, change);
                else {
                        lex.eatLine();
-                       docstring line = from_utf8(lex.getString());
+                       docstring line = lex.getDocString();
                        errorList.push_back(ErrorItem(_("Unknown Inset"), line,
                                            par.id(), 0, par.size()));
                }
@@ -316,10 +260,11 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                par.insertInset(par.size(), new InsetLine, font, change);
        } else if (token == "\\newpage") {
                par.insertInset(par.size(), new InsetPagebreak, font, change);
+       } else if (token == "\\clearpage") {
+               par.insertInset(par.size(), new InsetClearPage, font, change);
+       } else if (token == "\\cleardoublepage") {
+               par.insertInset(par.size(), new InsetClearDoublePage, font, change);
        } else if (token == "\\change_unchanged") {
-               // Hack ! Needed for empty paragraphs :/
-               // FIXME: change tracking (MG)
-               // set empty 'par' to INSERTED???
                change = Change(Change::UNCHANGED);
        } else if (token == "\\change_inserted") {
                lex.eatLine();
@@ -331,7 +276,6 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                        errorList.push_back(ErrorItem(_("Change tracking error"),
                                            bformat(_("Unknown author index for insertion: %1$d\n"), aid),
                                            par.id(), 0, par.size()));
-
                        change = Change(Change::UNCHANGED);
                } else
                        change = Change(Change::INSERTED, bp.author_map[aid], ct);
@@ -345,7 +289,6 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                        errorList.push_back(ErrorItem(_("Change tracking error"),
                                            bformat(_("Unknown author index for deletion: %1$d\n"), aid),
                                            par.id(), 0, par.size()));
-
                        change = Change(Change::UNCHANGED);
                } else
                        change = Change(Change::DELETED, bp.author_map[aid], ct);
@@ -353,7 +296,7 @@ void readParToken(Buffer const & buf, Paragraph & par, LyXLex & lex,
                lex.eatLine();
                errorList.push_back(ErrorItem(_("Unknown token"),
                        bformat(_("Unknown token: %1$s %2$s\n"), from_utf8(token),
-                       from_utf8(lex.getString())),
+                       lex.getDocString()),
                        par.id(), 0, par.size()));
        }
 }
@@ -368,7 +311,6 @@ void readParagraph(Buffer const & buf, Paragraph & par, LyXLex & lex,
        Change change(Change::UNCHANGED);
 
        while (lex.isOK()) {
-
                readParToken(buf, par, lex, token, font, change, errorList);
 
                lex.nextToken();
@@ -406,36 +348,20 @@ void readParagraph(Buffer const & buf, Paragraph & par, LyXLex & lex,
 
 
 
-BufferView * LyXText::bv() const
-{
-       BOOST_ASSERT(bv_owner != 0);
-       return bv_owner;
-}
-
-
-double LyXText::spacing(Paragraph const & par) const
+double LyXText::spacing(Buffer const & buffer,
+               Paragraph const & par) const
 {
        if (par.params().spacing().isDefault())
-               return bv()->buffer()->params().spacing().getValue();
+               return buffer.params().spacing().getValue();
        return par.params().spacing().getValue();
 }
 
 
-int LyXText::width() const
-{
-       return dim_.wid;
-}
-
-
-int LyXText::height() const
-{
-       return dim_.height();
-}
-
-
-int LyXText::singleWidth(Paragraph const & par, pos_type pos) const
+int LyXText::singleWidth(Buffer const & buffer, Paragraph const & par,
+               pos_type pos) const
 {
-       return singleWidth(par, pos, par.getChar(pos), getFont(par, pos));
+       return singleWidth(par, pos, par.getChar(pos),
+               getFont(buffer, par, pos));
 }
 
 
@@ -466,15 +392,16 @@ int LyXText::singleWidth(Paragraph const & par,
 }
 
 
-int LyXText::leftMargin(pit_type pit) const
+int LyXText::leftMargin(Buffer const & buffer, int max_width, pit_type pit) const
 {
        BOOST_ASSERT(pit >= 0);
        BOOST_ASSERT(pit < int(pars_.size()));
-       return leftMargin(pit, pars_[pit].size());
+       return leftMargin(buffer, max_width, pit, pars_[pit].size());
 }
 
 
-int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
+int LyXText::leftMargin(Buffer const & buffer, int max_width,
+               pit_type const pit, pos_type const pos) const
 {
        BOOST_ASSERT(pit >= 0);
        BOOST_ASSERT(pit < int(pars_.size()));
@@ -482,27 +409,26 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
        BOOST_ASSERT(pos >= 0);
        BOOST_ASSERT(pos <= par.size());
        //lyxerr << "LyXText::leftMargin: pit: " << pit << " pos: " << pos << endl;
-       BufferParams const & params = bv()->buffer()->params();
-       LyXTextClass const & tclass = params.getLyXTextClass();
+       LyXTextClass const & tclass = buffer.params().getLyXTextClass();
        LyXLayout_ptr const & layout = par.layout();
 
        string parindent = layout->parindent;
 
        int l_margin = 0;
 
-       if (isMainText())
+       if (isMainText(buffer))
                l_margin += changebarMargin();
 
        // FIXME UNICODE
        docstring leftm = from_utf8(tclass.leftmargin());
-       l_margin += theFontMetrics(params.getFont()).signedWidth(leftm);
+       l_margin += theFontMetrics(buffer.params().getFont()).signedWidth(leftm);
 
        if (par.getDepth() != 0) {
                // find the next level paragraph
                pit_type newpar = outerHook(pit, pars_);
                if (newpar != pit_type(pars_.size())) {
                        if (pars_[newpar].layout()->isEnvironment()) {
-                               l_margin = leftMargin(newpar);
+                               l_margin = leftMargin(buffer, max_width, newpar);
                        }
                        if (par.layout() == tclass.defaultLayout()) {
                                if (pars_[newpar].params().noindent())
@@ -520,7 +446,7 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
            && pit > 0 && pars_[pit - 1].layout()->nextnoindent)
                parindent.erase();
 
-       LyXFont const labelfont = getLabelFont(par);
+       LyXFont const labelfont = getLabelFont(buffer, par);
        FontMetrics const & labelfont_metrics = theFontMetrics(labelfont);
 
        switch (layout->margintype) {
@@ -528,7 +454,7 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
                if (!layout->leftmargin.empty()) {
                        // FIXME UNICODE
                        docstring leftm = from_utf8(layout->leftmargin);
-                       l_margin += theFontMetrics(params.getFont()).signedWidth(leftm);
+                       l_margin += theFontMetrics(buffer.params().getFont()).signedWidth(leftm);
                }
                if (!par.getLabelstring().empty()) {
                        // FIXME UNICODE
@@ -560,7 +486,7 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
        case MARGIN_STATIC: {
                // FIXME UNICODE
                docstring leftm = from_utf8(layout->leftmargin);
-               l_margin += theFontMetrics(params.getFont()).signedWidth(leftm)
+               l_margin += theFontMetrics(buffer.params().getFont()).signedWidth(leftm)
                        * 4     / (par.getDepth() + 4);
                break;
        }
@@ -603,7 +529,7 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
 #ifdef WITH_WARNINGS
 #warning This is wrong.
 #endif
-               int minfill = maxwidth_;
+               int minfill = max_width;
                for ( ; rit != end; ++rit)
                        if (rit->fill() < minfill)
                                minfill = rit->fill();
@@ -611,13 +537,13 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
                l_margin += minfill;
 #endif
                // also wrong, but much shorter.
-               l_margin += maxwidth_ / 2;
+               l_margin += max_width / 2;
                break;
        }
        }
 
        if (!par.params().leftIndent().zero())
-               l_margin += par.params().leftIndent().inPixels(maxwidth_);
+               l_margin += par.params().leftIndent().inPixels(max_width);
 
        LyXAlignment align;
 
@@ -637,447 +563,29 @@ int LyXText::leftMargin(pit_type const pit, pos_type const pos) const
            && align == LYX_ALIGN_BLOCK
            && !par.params().noindent()
            // in some insets, paragraphs are never indented
-           && !(par.inInset() && par.inInset()->neverIndent())
+           && !(par.inInset() && par.inInset()->neverIndent(buffer))
            // display style insets are always centered, omit indentation
            && !(!par.empty()
                    && par.isInset(pos)
                    && par.getInset(pos)->display())
            && (par.layout() != tclass.defaultLayout()
-               || bv()->buffer()->params().paragraph_separation ==
+               || buffer.params().paragraph_separation ==
                   BufferParams::PARSEP_INDENT))
        {
                docstring din = from_utf8(parindent);
-               l_margin += theFontMetrics(params.getFont()).signedWidth(din);
+               l_margin += theFontMetrics(buffer.params().getFont()).signedWidth(din);
        }
 
        return l_margin;
 }
 
 
-int LyXText::rightMargin(Paragraph const & par) const
-{
-       // FIXME: the correct way is to only call rightMargin() only
-       // within the main LyXText. The following test is thus bogus.
-       LyXText const & text = bv()->buffer()->text();
-       // We do not want rightmargins on inner texts.
-       if (&text != this)
-               return 0;
-
-       BufferParams const & params = bv()->buffer()->params();
-       LyXTextClass const & tclass = params.getLyXTextClass();
-       docstring trmarg = from_utf8(tclass.rightmargin());
-       docstring lrmarg = from_utf8(par.layout()->rightmargin);
-       FontMetrics const & fm = theFontMetrics(params.getFont());
-       int const r_margin =
-               lyx::rightMargin()
-               + fm.signedWidth(trmarg)
-               + fm.signedWidth(lrmarg)
-               * 4 / (par.getDepth() + 4);
-
-       return r_margin;
-}
-
-
-int LyXText::labelEnd(pit_type const pit) const
-{
-       // labelEnd is only needed if the layout fills a flushleft label.
-       if (pars_[pit].layout()->margintype != MARGIN_MANUAL)
-               return 0;
-       // return the beginning of the body
-       return leftMargin(pit);
-}
-
-
-namespace {
-
-// this needs special handling - only newlines count as a break point
-pos_type addressBreakPoint(pos_type i, Paragraph const & par)
-{
-       pos_type const end = par.size();
-
-       for (; i < end; ++i)
-               if (par.isNewline(i))
-                       return i + 1;
-
-       return end;
-}
-
-};
-
-
-void LyXText::rowBreakPoint(pit_type const pit, Row & row) const
-{
-       Paragraph const & par = pars_[pit];
-       pos_type const end = par.size();
-       pos_type const pos = row.pos();
-       if (pos == end) {
-               row.endpos(end);
-               return;
-       }
-
-       // maximum pixel width of a row
-       int width = maxwidth_ - rightMargin(par); // - leftMargin(pit, row);
-       if (width < 0) {
-               row.endpos(end);
-               return;
-       }
-
-       LyXLayout_ptr const & layout = par.layout();
-
-       if (layout->margintype == MARGIN_RIGHT_ADDRESS_BOX) {
-               row.endpos(addressBreakPoint(pos, par));
-               return;
-       }
-
-       pos_type const body_pos = par.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, pos);
-       int x = left;
-
-       // pixel width since last breakpoint
-       int chunkwidth = 0;
-
-       FontIterator fi = FontIterator(*this, par, pos);
-       pos_type point = end;
-       pos_type i = pos;
-       FontMetrics const & fm = theFontMetrics(getLabelFont(par));
-       for ( ; i < end; ++i, ++fi) {
-               char_type const c = par.getChar(i);
-               int thiswidth = singleWidth(par, i, c, *fi);
-
-               // add the auto-hfill from label end to the body
-               if (body_pos && i == body_pos) {
-                       docstring lsep = from_utf8(layout->labelsep);
-                       int add = fm.width(lsep);
-                       if (par.isLineSeparator(i - 1))
-                               add -= singleWidth(par, i - 1);
-
-                       add = std::max(add, labelEnd(pit) - x);
-                       thiswidth += add;
-               }
-
-               x += thiswidth;
-               chunkwidth += thiswidth;
-
-               // break before a character that will fall off
-               // the right of the row
-               if (x >= width) {
-                       // if no break before, break here
-                       if (point == end || chunkwidth >= width - left) {
-                               if (i > pos)
-                                       point = i;
-                               else
-                                       point = i + 1;
-
-                       }
-                       // exit on last registered breakpoint:
-                       break;
-               }
-
-               if (par.isNewline(i)) {
-                       point = i + 1;
-                       break;
-               }
-               // Break before...
-               if (i + 1 < end) {
-                       if (par.isInset(i + 1) && par.getInset(i + 1)->display()) {
-                               point = i + 1;
-                               break;
-                       }
-                       // ...and after.
-                       if (par.isInset(i) && par.getInset(i)->display()) {
-                               point = i + 1;
-                               break;
-                       }
-               }
-
-               if (!par.isInset(i) || par.getInset(i)->isChar()) {
-                       // some insets are line separators too
-                       if (par.isLineSeparator(i)) {
-                               // register breakpoint:
-                               point = i + 1;
-                               chunkwidth = 0;
-                       }
-               }
-       }
-
-       // maybe found one, but the par is short enough.
-       if (i == end && x < width)
-               point = end;
-
-       // manual labels cannot be broken in LaTeX. But we
-       // want to make our on-screen rendering of footnotes
-       // etc. still break
-       if (body_pos && point < body_pos)
-               point = body_pos;
-
-       row.endpos(point);
-}
-
-
-void LyXText::setRowWidth(pit_type const pit, Row & row) const
-{
-       // get the pure distance
-       pos_type const end = row.endpos();
-
-       Paragraph const & par = pars_[pit];
-       docstring const labelsep = from_utf8(par.layout()->labelsep);
-       int w = leftMargin(pit, row.pos());
-
-       pos_type const body_pos = par.beginOfBody();
-       pos_type i = row.pos();
-
-       FontMetrics const & fm = theFontMetrics(getLabelFont(par));
-
-       if (i < end) {
-               FontIterator fi = FontIterator(*this, par, i);
-               for ( ; i < end; ++i, ++fi) {
-                       if (body_pos > 0 && i == body_pos) {
-                               w += fm.width(labelsep);
-                               if (par.isLineSeparator(i - 1))
-                                       w -= singleWidth(par, i - 1);
-                               w = max(w, labelEnd(pit));
-                       }
-                       char_type const c = par.getChar(i);
-                       w += singleWidth(par, i, c, *fi);
-               }
-       }
-
-       if (body_pos > 0 && body_pos >= end) {
-               w += fm.width(labelsep);
-               if (end > 0 && par.isLineSeparator(end - 1))
-                       w -= singleWidth(par, end - 1);
-               w = max(w, labelEnd(pit));
-       }
-
-       row.width(w + rightMargin(par));
-}
-
-
-// returns the minimum space a manual label needs on the screen in pixel
-int LyXText::labelFill(Paragraph const & par, Row const & row) const
-{
-       pos_type last = par.beginOfBody();
-
-       BOOST_ASSERT(last > 0);
-
-       // -1 because a label ends with a space that is in the label
-       --last;
-
-       // a separator at this end does not count
-       if (par.isLineSeparator(last))
-               --last;
-
-       int w = 0;
-       for (pos_type i = row.pos(); i <= last; ++i)
-               w += singleWidth(par, i);
-
-       docstring const & label = par.params().labelWidthString();
-       if (label.empty())
-               return 0;
-
-       FontMetrics const & fm = theFontMetrics(getLabelFont(par));
-
-       return max(0, fm.width(label) - w);
-}
-
-
 LColor_color LyXText::backgroundColor() const
 {
        return LColor_color(LColor::color(background_color_));
 }
 
 
-void LyXText::setHeightOfRow(pit_type const pit, Row & row)
-{
-       Paragraph const & par = pars_[pit];
-       // get the maximum ascent and the maximum descent
-       double layoutasc = 0;
-       double layoutdesc = 0;
-       double const dh = defaultRowHeight();
-
-       // ok, let us initialize the maxasc and maxdesc value.
-       // Only the fontsize count. The other properties
-       // are taken from the layoutfont. Nicer on the screen :)
-       LyXLayout_ptr const & layout = par.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.
-       LyXFont font = getFont(par, row.pos());
-       LyXFont::FONT_SIZE const tmpsize = font.size();
-       font = getLayoutFont(pit);
-       LyXFont::FONT_SIZE const size = font.size();
-       font.setSize(tmpsize);
-
-       LyXFont labelfont = getLabelFont(par);
-
-       FontMetrics const & labelfont_metrics = theFontMetrics(labelfont);
-       FontMetrics const & fontmetrics = theFontMetrics(font);
-
-       // these are minimum values
-       double const spacing_val = layout->spacing.getValue() * spacing(par);
-       //lyxerr << "spacing_val = " << spacing_val << endl;
-       int maxasc  = int(fontmetrics.maxAscent()  * spacing_val);
-       int maxdesc = int(fontmetrics.maxDescent() * spacing_val);
-
-       // insets may be taller
-       InsetList::const_iterator ii = par.insetlist.begin();
-       InsetList::const_iterator iend = par.insetlist.end();
-       for ( ; ii != iend; ++ii) {
-               if (ii->pos >= row.pos() && ii->pos < row.endpos()) {
-                       maxasc  = max(maxasc,  ii->inset->ascent());
-                       maxdesc = max(maxdesc, ii->inset->descent());
-               }
-       }
-
-       // Check if any custom fonts are larger (Asger)
-       // This is not completely correct, but we can live with the small,
-       // cosmetic error for now.
-       int labeladdon = 0;
-       pos_type const pos_end = row.endpos();
-
-       LyXFont::FONT_SIZE maxsize =
-               par.highestFontInRange(row.pos(), pos_end, size);
-       if (maxsize > font.size()) {
-               font.setSize(maxsize);
-               maxasc  = max(maxasc,  fontmetrics.maxAscent());
-               maxdesc = max(maxdesc, fontmetrics.maxDescent());
-       }
-
-       // This is nicer with box insets:
-       ++maxasc;
-       ++maxdesc;
-
-       row.ascent(maxasc);
-
-       // is it a top line?
-       if (row.pos() == 0) {
-               BufferParams const & bufparams = bv()->buffer()->params();
-               // some parksips VERY EASY IMPLEMENTATION
-               if (bv()->buffer()->params().paragraph_separation
-                   == BufferParams::PARSEP_SKIP
-                       && pit != 0
-                       && ((layout->isParagraph() && par.getDepth() == 0)
-                           || (pars_[pit - 1].layout()->isParagraph()
-                               && pars_[pit - 1].getDepth() == 0)))
-               {
-                               maxasc += bufparams.getDefSkip().inPixels(*bv());
-               }
-
-               if (par.params().startOfAppendix())
-                       maxasc += int(3 * dh);
-
-               // This is special code for the chapter, since the label of this
-               // layout is printed in an extra row
-               if (layout->counter == "chapter"
-                   && !par.params().labelString().empty()) {
-                       labeladdon = int(labelfont_metrics.maxHeight()
-                                    * layout->spacing.getValue()
-                                    * spacing(par));
-               }
-
-               // special code for the top label
-               if ((layout->labeltype == LABEL_TOP_ENVIRONMENT
-                    || layout->labeltype == LABEL_BIBLIO
-                    || layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT)
-                   && isFirstInSequence(pit, paragraphs())
-                   && !par.getLabelstring().empty())
-               {
-                       labeladdon = int(
-                                 labelfont_metrics.maxHeight()
-                                       * layout->spacing.getValue()
-                                       * spacing(par)
-                               + (layout->topsep + layout->labelbottomsep) * dh);
-               }
-
-               // Add the layout spaces, for example before and after
-               // a section, or between the items of a itemize or enumerate
-               // environment.
-
-               pit_type prev = depthHook(pit, pars_, par.getDepth());
-               if (prev != pit
-                   && pars_[prev].layout() == layout
-                   && pars_[prev].getDepth() == par.getDepth()
-                   && pars_[prev].getLabelWidthString() == par.getLabelWidthString())
-               {
-                       layoutasc = layout->itemsep * dh;
-               } else if (pit != 0 || row.pos() != 0) {
-                       if (layout->topsep > 0)
-                               layoutasc = layout->topsep * dh;
-               }
-
-               prev = outerHook(pit, pars_);
-               if (prev != pit_type(pars_.size())) {
-                       maxasc += int(pars_[prev].layout()->parsep * dh);
-               } else if (pit != 0) {
-                       if (pars_[pit - 1].getDepth() != 0 ||
-                                       pars_[pit - 1].layout() == layout) {
-                               maxasc += int(layout->parsep * dh);
-                       }
-               }
-       }
-
-       // is it a bottom line?
-       if (row.endpos() >= par.size()) {
-               // add the layout spaces, for example before and after
-               // a section, or between the items of a itemize or enumerate
-               // environment
-               pit_type nextpit = pit + 1;
-               if (nextpit != pit_type(pars_.size())) {
-                       pit_type cpit = pit;
-                       double usual = 0;
-                       double unusual = 0;
-
-                       if (pars_[cpit].getDepth() > pars_[nextpit].getDepth()) {
-                               usual = pars_[cpit].layout()->bottomsep * dh;
-                               cpit = depthHook(cpit, paragraphs(), pars_[nextpit].getDepth());
-                               if (pars_[cpit].layout() != pars_[nextpit].layout()
-                                       || pars_[nextpit].getLabelWidthString() != pars_[cpit].getLabelWidthString())
-                               {
-                                       unusual = pars_[cpit].layout()->bottomsep * dh;
-                               }
-                               layoutdesc = max(unusual, usual);
-                       } else if (pars_[cpit].getDepth() == pars_[nextpit].getDepth()) {
-                               if (pars_[cpit].layout() != pars_[nextpit].layout()
-                                       || pars_[nextpit].getLabelWidthString() != pars_[cpit].getLabelWidthString())
-                                       layoutdesc = int(pars_[cpit].layout()->bottomsep * dh);
-                       }
-               }
-       }
-
-       // incalculate the layout spaces
-       maxasc  += int(layoutasc  * 2 / (2 + pars_[pit].getDepth()));
-       maxdesc += int(layoutdesc * 2 / (2 + pars_[pit].getDepth()));
-
-       // FIXME: the correct way is to do the following is to move the 
-       // following code in another method specially tailored for the 
-       // main LyXText. The following test is thus bogus.
-       LyXText const & text = bv_owner->buffer()->text();
-       // Top and bottom margin of the document (only at top-level)
-       if (&text == this) {
-               if (pit == 0 && row.pos() == 0)
-                       maxasc += 20;
-               if (pit + 1 == pit_type(pars_.size()) &&
-                   row.endpos() == par.size())
-                       maxdesc += 20;
-       }
-
-       row.ascent(maxasc + labeladdon);
-       row.descent(maxdesc);
-}
-
-
-namespace {
-
-}
-
 void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
 {
        BOOST_ASSERT(this == cur.text());
@@ -1090,20 +598,19 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
 
        // this is only allowed, if the current paragraph is not empty
        // or caption and if it has not the keepempty flag active
-       if (cur.lastpos() == 0 && !cpar.allowEmpty()
-          && layout->labeltype != LABEL_SENSITIVE)
+       if (cur.lastpos() == 0 && !cpar.allowEmpty() &&
+           layout->labeltype != LABEL_SENSITIVE)
                return;
 
        // a layout change may affect also the following paragraph
-       recUndo(cur.pit(), undoSpan(cur.pit()) - 1);
+       recUndo(cur, cur.pit(), undoSpan(cur.pit()) - 1);
 
        // Always break behind a space
        // It is better to erase the space (Dekel)
        if (cur.pos() != cur.lastpos() && cpar.isLineSeparator(cur.pos()))
-               // FIXME: change tracking (MG)
                cpar.eraseChar(cur.pos(), cur.buffer().params().trackChanges);
 
-       // How should the layout for the new paragraph be?
+       // What should the layout for the new paragraph be?
        int preserve_layout = 0;
        if (keep_layout)
                preserve_layout = 2;
@@ -1135,9 +642,10 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
                        pars_[next_par].applyLayout(tclass.defaultLayout());
        }
 
-       while (!pars_[next_par].empty() && pars_[next_par].isNewline(0))
-               // FIXME: change tracking (MG)
-               pars_[next_par].eraseChar(0, cur.buffer().params().trackChanges);
+       while (!pars_[next_par].empty() && pars_[next_par].isNewline(0)) {
+               if (!pars_[next_par].eraseChar(0, cur.buffer().params().trackChanges))
+                       break; // the character couldn't be deleted physically due to change tracking
+       }
 
        ParIterator current_it(cur);
        ParIterator last_it(cur);
@@ -1146,19 +654,15 @@ void LyXText::breakParagraph(LCursor & cur, bool keep_layout)
 
        updateLabels(cur.buffer(), current_it, last_it);
 
-       // Mark "carriage return" as inserted if change tracking:
-       if (cur.buffer().params().trackChanges) {
-               // FIXME: Change tracking (MG)
-               cur.paragraph().setChange(cur.paragraph().size(),
-                       Change(Change::INSERTED));
-       }
+       // A singlePar update is not enough in this case.
+       cur.updateFlags(Update::Force);
 
        // This check is necessary. Otherwise the new empty paragraph will
        // be deleted automatically. And it is more friendly for the user!
        if (cur.pos() != 0 || isempty)
-               setCursor(cur, cur.pit() + 1, 0);
+               setCursor(cur.top(), cur.pit() + 1, 0);
        else
-               setCursor(cur, cur.pit(), 0);
+               setCursor(cur.top(), cur.pit(), 0);
 }
 
 
@@ -1171,6 +675,7 @@ void LyXText::insertChar(LCursor & cur, char_type c)
 
        recordUndo(cur, Undo::INSERT);
 
+       Buffer const & buffer = cur.buffer();
        Paragraph & par = cur.paragraph();
        // try to remove this
        pit_type const pit = cur.pit();
@@ -1188,8 +693,8 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                            !(contains(number_seperators, c) &&
                              cur.pos() != 0 &&
                              cur.pos() != cur.lastpos() &&
-                             getFont(par, cur.pos()).number() == LyXFont::ON &&
-                             getFont(par, cur.pos() - 1).number() == LyXFont::ON)
+                             getFont(buffer, par, cur.pos()).number() == LyXFont::ON &&
+                             getFont(buffer, par, cur.pos() - 1).number() == LyXFont::ON)
                           )
                                number(cur); // Set current_font.number to OFF
                } else if (isDigit(c) &&
@@ -1203,11 +708,11 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                                     || par.isSeparator(cur.pos() - 2)
                                     || par.isNewline(cur.pos() - 2))
                                  ) {
-                                       setCharFont(pit, cur.pos() - 1, current_font);
+                                       setCharFont(buffer, pit, cur.pos() - 1, current_font);
                                } else if (contains(number_seperators, c)
                                     && cur.pos() >= 2
-                                    && getFont(par, cur.pos() - 2).number() == LyXFont::ON) {
-                                       setCharFont(pit, cur.pos() - 1, current_font);
+                                    && getFont(buffer, par, cur.pos() - 2).number() == LyXFont::ON) {
+                                       setCharFont(buffer, pit, cur.pos() - 1, current_font);
                                }
                        }
                }
@@ -1232,10 +737,8 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                        return;
                }
                BOOST_ASSERT(cur.pos() > 0);
-               if ((par.isLineSeparator(cur.pos() - 1)
-                   || par.isNewline(cur.pos() - 1))
-                   // FIXME: change tracking (MG)
-                   && par.lookupChange(cur.pos() - 1) != Change(Change::DELETED)) {
+               if ((par.isLineSeparator(cur.pos() - 1) || par.isNewline(cur.pos() - 1))
+                   && !par.isDeleted(cur.pos() - 1)) {
                        static bool sent_space_message = false;
                        if (!sent_space_message) {
                                cur.message(_("You cannot type two spaces this way. "
@@ -1246,9 +749,10 @@ void LyXText::insertChar(LCursor & cur, char_type c)
                }
        }
 
-       // FIXME: change tracking (MG)
-       par.insertChar(cur.pos(), c, current_font, Change(Change::INSERTED));
-       setCursor(cur, cur.pit(), cur.pos() + 1, false, cur.boundary());
+       par.insertChar(cur.pos(), c, current_font, cur.buffer().params().trackChanges);
+
+//             cur.updateFlags(Update::Force);
+       setCursor(cur.top(), cur.pit(), cur.pos() + 1);
        charInserted();
 }
 
@@ -1267,120 +771,6 @@ void LyXText::charInserted()
 }
 
 
-RowMetrics
-LyXText::computeRowMetrics(pit_type const pit, Row const & row) const
-{
-       RowMetrics result;
-       Paragraph const & par = pars_[pit];
-
-       double w = dim_.wid - row.width();
-
-       bool const is_rtl = isRTL(par);
-       if (is_rtl)
-               result.x = rightMargin(par);
-       else
-               result.x = leftMargin(pit, row.pos());
-
-       // is there a manual margin with a manual label
-       LyXLayout_ptr const & layout = par.layout();
-
-       if (layout->margintype == MARGIN_MANUAL
-           && layout->labeltype == LABEL_MANUAL) {
-               /// We might have real hfills in the label part
-               int nlh = numberOfLabelHfills(par, row);
-
-               // A manual label par (e.g. List) has an auto-hfill
-               // between the label text and the body of the
-               // paragraph too.
-               // But we don't want to do this auto hfill if the par
-               // is empty.
-               if (!par.empty())
-                       ++nlh;
-
-               if (nlh && !par.getLabelWidthString().empty())
-                       result.label_hfill = labelFill(par, row) / double(nlh);
-       }
-
-       // are there any hfills in the row?
-       int const nh = numberOfHfills(par, row);
-
-       if (nh) {
-               if (w > 0)
-                       result.hfill = w / nh;
-       // 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()) < maxwidth_) {
-               // is it block, flushleft or flushright?
-               // set x how you need it
-               int align;
-               if (par.params().align() == LYX_ALIGN_LAYOUT)
-                       align = layout->align;
-               else
-                       align = par.params().align();
-
-               // Display-style insets should always be on a centred row
-               // The test on par.size() is to catch zero-size pars, which
-               // would trigger the assert in Paragraph::getInset().
-               //inset = par.size() ? par.getInset(row.pos()) : 0;
-               if (!par.empty()
-                   && par.isInset(row.pos())
-                   && par.getInset(row.pos())->display())
-               {
-                       align = LYX_ALIGN_CENTER;
-               }
-
-               switch (align) {
-               case LYX_ALIGN_BLOCK: {
-                       int const ns = numberOfSeparators(par, row);
-                       bool disp_inset = false;
-                       if (row.endpos() < par.size()) {
-                               InsetBase const * in = par.getInset(row.endpos());
-                               if (in)
-                                       disp_inset = in->display();
-                       }
-                       // If we have separators, this is not the last row of a
-                       // par, does not end in newline, and is not row above a
-                       // display inset... then stretch it
-                       if (ns
-                           && row.endpos() < par.size()
-                           && !par.isNewline(row.endpos() - 1)
-                           && !disp_inset
-                               ) {
-                               result.separator = w / ns;
-                       } else if (is_rtl) {
-                               result.x += w;
-                       }
-                       break;
-               }
-               case LYX_ALIGN_RIGHT:
-                       result.x += w;
-                       break;
-               case LYX_ALIGN_CENTER:
-                       result.x += w / 2;
-                       break;
-               }
-       }
-
-       bidi.computeTables(par, *bv()->buffer(), row);
-       if (is_rtl) {
-               pos_type body_pos = par.beginOfBody();
-               pos_type end = row.endpos();
-
-               if (body_pos > 0
-                   && (body_pos > end || !par.isLineSeparator(body_pos - 1)))
-               {
-                       docstring const lsep = from_utf8(layout->labelsep);
-                       result.x += theFontMetrics(getLabelFont(par)).width(lsep);
-                       if (body_pos <= end)
-                               result.x += result.label_hfill;
-               }
-       }
-
-       return result;
-}
-
-
 // the cursor set functions have a special mechanism. When they
 // realize, that you left an empty paragraph, they will delete it.
 
@@ -1439,6 +829,7 @@ void LyXText::selectWord(LCursor & cur, word_location loc)
        cur.resetAnchor();
        setCursor(cur, to.pit(), to.pos());
        cur.setSelection();
+       theSelection().haveSelection(cur.selection());
 }
 
 
@@ -1454,78 +845,111 @@ bool LyXText::selectWordWhenUnderCursor(LCursor & cur, word_location loc)
 }
 
 
-void LyXText::acceptChange(LCursor & cur)
+void LyXText::acceptOrRejectChange(LCursor & cur, bool accept)
 {
        BOOST_ASSERT(this == cur.text());
-       if (!cur.selection() && cur.lastpos() != 0)
+
+       if (!cur.selection())
                return;
 
-       recordUndoSelection(cur, Undo::INSERT);
-
-       DocIterator it = cur.selectionBegin();
-       DocIterator et = cur.selectionEnd();
-       pit_type pit = it.pit();
-       Change::Type const type = pars_[pit].lookupChange(it.pos()).type;
-       for (; pit <= et.pit(); ++pit) {
-               pos_type left  = ( pit == it.pit() ? it.pos() : 0 );
-               pos_type right =
-                   ( pit == et.pit() ? et.pos() : pars_[pit].size() + 1 );
-               pars_[pit].acceptChange(left, right);
-       }
-       if (type == Change::DELETED) {
-               ParagraphList & plist = paragraphs();
-               if (it.pit() + 1 < et.pit())
-                       pars_.erase(boost::next(plist.begin(), it.pit() + 1),
-                                   boost::next(plist.begin(), et.pit()));
+       recordUndoSelection(cur, Undo::ATOMIC);
 
-               // Paragraph merge if appropriate:
-               // FIXME: change tracking (MG)
-               if (pars_[it.pit()].lookupChange(pars_[it.pit()].size())
-                       == Change(Change::DELETED)) {
-                       setCursorIntern(cur, it.pit() + 1, 0);
-                       backspacePos0(cur);
-               }
-       }
-       finishUndo();
-       cur.clearSelection();
-       setCursorIntern(cur, it.pit(), 0);
-}
+       pit_type begPit = cur.selectionBegin().pit();
+       pit_type endPit = cur.selectionEnd().pit();
 
+       pos_type begPos = cur.selectionBegin().pos();
+       pos_type endPos = cur.selectionEnd().pos();
 
-void LyXText::rejectChange(LCursor & cur)
-{
-       BOOST_ASSERT(this == cur.text());
-       if (!cur.selection() && cur.lastpos() != 0)
-               return;
+       // keep selection info, because endPos becomes invalid after the first loop
+       bool endsBeforeEndOfPar = (endPos < pars_[endPit].size());
+
+       // first, accept/reject changes within each individual paragraph (do not consider end-of-par)
+       
+       for (pit_type pit = begPit; pit <= endPit; ++pit) {
+               pos_type parSize = pars_[pit].size();
 
-       recordUndoSelection(cur, Undo::INSERT);
-
-       DocIterator it = cur.selectionBegin();
-       DocIterator et = cur.selectionEnd();
-       pit_type pit = it.pit();
-       Change::Type const type = pars_[pit].lookupChange(it.pos()).type;
-       for (; pit <= et.pit(); ++pit) {
-               pos_type left  = ( pit == it.pit() ? it.pos() : 0 );
-               pos_type right =
-                   ( pit == et.pit() ? et.pos() : pars_[pit].size() + 1 );
-               pars_[pit].rejectChange(left, right);
+               // ignore empty paragraphs; otherwise, an assertion will fail for
+               // acceptChanges(bparams, 0, 0) or rejectChanges(bparams, 0, 0)
+               if (parSize == 0)
+                       continue;
+
+               // do not consider first paragraph if the cursor starts at pos size()
+               if (pit == begPit && begPos == parSize)
+                       continue;
+
+               // do not consider last paragraph if the cursor ends at pos 0
+               if (pit == endPit && endPos == 0) 
+                       break; // last iteration anyway
+
+               pos_type left  = (pit == begPit ? begPos : 0);
+               pos_type right = (pit == endPit ? endPos : parSize);
+
+               if (accept) {
+                       pars_[pit].acceptChanges(cur.buffer().params(), left, right);
+               } else {
+                       pars_[pit].rejectChanges(cur.buffer().params(), left, right);
+               }
        }
-       if (type == Change::INSERTED) {
-               ParagraphList & plist = paragraphs();
-               if (it.pit() + 1 < et.pit())
-                       pars_.erase(boost::next(plist.begin(), it.pit() + 1),
-                                   boost::next(plist.begin(), et.pit()));
-               // Paragraph merge if appropriate:
-               // FIXME: change tracking (MG)
-               if (pars_[it.pit()].lookupChange(pars_[it.pit()].size())
-                       == Change(Change::INSERTED)) {
-                       setCursorIntern(cur, it.pit() + 1, 0);
-                       backspacePos0(cur);
+       
+       // next, accept/reject imaginary end-of-par characters
+       for (pit_type pit = begPit; pit <= endPit; ++pit) {
+               pos_type pos = pars_[pit].size();
+
+               // skip if the selection ends before the end-of-par
+               if (pit == endPit && endsBeforeEndOfPar)
+                       break; // last iteration anyway
+
+               // skip if this is not the last paragraph of the document
+               // note: the user should be able to accept/reject the par break of the last par!
+               if (pit == endPit && pit != pars_.size() - 1)
+                       break; // last iteration anway
+
+               if (accept) {
+                       if (pars_[pit].isInserted(pos)) {
+                               pars_[pit].setChange(pos, Change(Change::UNCHANGED));
+                       } else if (pars_[pit].isDeleted(pos)) {
+                               if (pit == pars_.size() - 1) {
+                                       // we cannot remove a par break at the end of the last paragraph;
+                                       // instead, we mark it unchanged
+                                       pars_[pit].setChange(pos, Change(Change::UNCHANGED));
+                               } else {
+                                       mergeParagraph(cur.buffer().params(), pars_, pit);
+                                       --endPit;
+                                       --pit;
+                               }
+                       }
+               } else {
+                       if (pars_[pit].isDeleted(pos)) {
+                               pars_[pit].setChange(pos, Change(Change::UNCHANGED));
+                       } else if (pars_[pit].isInserted(pos)) {
+                               if (pit == pars_.size() - 1) {
+                                       // we mark the par break at the end of the last paragraph unchanged
+                                       pars_[pit].setChange(pos, Change(Change::UNCHANGED));
+                               } else {
+                                       mergeParagraph(cur.buffer().params(), pars_, pit);
+                                       --endPit;
+                                       --pit;
+                               }
+                       }
                }
        }
+
+       // finally, invoke the DEPM
+       // FIXME: the following code will be changed in the near future
+       setCursorIntern(cur, endPit, 0);
+       for (pit_type pit = endPit - 1; pit >= begPit; --pit) {
+               bool dummy;
+               LCursor old = cur;
+               setCursorIntern(cur, pit, 0);
+               deleteEmptyParagraphMechanism(cur, old, dummy);
+       }
+
        finishUndo();
        cur.clearSelection();
-       setCursorIntern(cur, it.pit(), 0);
+       setCursorIntern(cur, begPit, begPos);
+       cur.updateFlags(Update::Force);
+       updateLabels(cur.buffer());
 }
 
 
@@ -1638,35 +1062,37 @@ bool LyXText::erase(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
        bool needsUpdate = false;
+       Paragraph & par = cur.paragraph();
 
        if (cur.pos() != cur.lastpos()) {
-               recordUndo(cur, Undo::DELETE, cur.pit());
-               setCursorIntern(cur, cur.pit(), cur.pos() + 1, false, cur.boundary());
-               needsUpdate = backspace(cur);
-               // FIXME: change tracking (MG)
-               if (cur.paragraph().lookupChange(cur.pos()) == Change(Change::DELETED))
-                       cur.posRight();
-       } else if (cur.pit() != cur.lastpit()) {
-               LCursor scur = cur;
-
-               setCursorIntern(cur, cur.pit() + 1, 0, false, false);
-               if (pars_[cur.pit()].layout() == pars_[scur.pit()].layout()) {
-                       recordUndo(scur, Undo::DELETE, scur.pit());
-                       needsUpdate = backspace(cur);
-                       if (cur.buffer().params().trackChanges) {
-                               // FIXME: Change tracking (MG)
-                               // move forward after the paragraph break is DELETED
-                               Paragraph & par = cur.paragraph();
-                               // FIXME: change tracking (MG)
-                               if (par.lookupChange(par.size()) == Change(Change::DELETED))
-                                       setCursorIntern(cur, cur.pit() + 1, 0);
-                               }
+               // this is the code for a normal delete, not pasting
+               // any paragraphs
+               recordUndo(cur, Undo::DELETE);
+               if(!par.eraseChar(cur.pos(), cur.buffer().params().trackChanges)) {
+                       // the character has been logically deleted only => skip it
+                       cur.forwardPosNoDescend();
+               }
+               needsUpdate = true;
+       } else {
+               if (cur.pit() == cur.lastpit())
+                       return dissolveInset(cur);
+
+               if (!par.isMergedOnEndOfParDeletion(cur.buffer().params().trackChanges)) {
+                       par.setChange(cur.pos(), Change(Change::DELETED));
+                       cur.forwardPos();
+                       needsUpdate = true;
                } else {
-                       setCursorIntern(scur, scur.pit(), scur.pos(), false, scur.boundary());
+                       setCursorIntern(cur, cur.pit() + 1, 0);
+                       needsUpdate = backspacePos0(cur);
                }
-       } else
-               needsUpdate = dissolveInset(cur);
+       }
 
+       if (needsUpdate) {
+               // Make sure the cursor is correct. Is this really needed?
+               // No, not really... at least not here!
+               cur.text()->setCursor(cur.top(), cur.pit(), cur.pos());
+       }
+       
        return needsUpdate;
 }
 
@@ -1674,75 +1100,50 @@ bool LyXText::erase(LCursor & cur)
 bool LyXText::backspacePos0(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
+       if (cur.pit() == 0)
+               return false;
+
        bool needsUpdate = false;
 
-       Paragraph & par = cur.paragraph();
-       // is it an empty paragraph?
-       pos_type lastpos = cur.lastpos();
-       if (lastpos == 0 || (lastpos == 1 && par.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 (cur.pit() != 0) {
-                       // For KeepEmpty layouts we need to get
-                       // rid of the keepEmpty setting first.
-                       // And the only way to do this is to
-                       // reset the layout to something
-                       // else: f.ex. the default layout.
-                       if (par.allowEmpty()) {
-                               Buffer & buf = cur.buffer();
-                               BufferParams const & bparams = buf.params();
-                               par.layout(bparams.getLyXTextClass().defaultLayout());
-                       }
+       BufferParams const & bufparams = cur.buffer().params();
+       LyXTextClass const & tclass = bufparams.getLyXTextClass();
+       ParagraphList & plist = cur.text()->paragraphs();
+       Paragraph const & par = cur.paragraph();
+       LCursor prevcur = cur;
+       --prevcur.pit();
+       prevcur.pos() = prevcur.lastpos();
+       Paragraph const & prevpar = prevcur.paragraph();
 
-                       cursorLeft(cur);
-                       return true;
-               }
+       // is it an empty paragraph?
+       if (cur.lastpos() == 0 
+           || (cur.lastpos() == 1 && par.isSeparator(0))) {
+               recordUndo(cur, Undo::ATOMIC, prevcur.pit(), cur.pit());
+               plist.erase(boost::next(plist.begin(), cur.pit()));
+               needsUpdate = true;
        }
-
-       if (cur.pit() != 0)
-               recordUndo(cur, Undo::DELETE, cur.pit() - 1);
-
-       pit_type tmppit = cur.pit();
-       // 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 (cur.pit() != 0) {
-               // steps into the above paragraph.
-               setCursorIntern(cur, cur.pit() - 1,
-                               pars_[cur.pit() - 1].size(),
-                               false);
+       // is previous par empty?
+       else if (prevcur.lastpos() == 0 
+                || (prevcur.lastpos() == 1 && prevpar.isSeparator(0))) {
+               recordUndo(cur, Undo::ATOMIC, prevcur.pit(), cur.pit());
+               plist.erase(boost::next(plist.begin(), prevcur.pit()));
+               needsUpdate = true;
        }
-
        // Pasting is not allowed, if the paragraphs have different
-       // layout. I think it is a real bug of all other
+       // layouts. I think it is a real bug of all other
        // word processors to allow it. It confuses the user.
        // Correction: Pasting is always allowed with standard-layout
-       // Correction (Jug 20050717): Remove check about alignment!
-       Buffer & buf = cur.buffer();
-       BufferParams const & bufparams = buf.params();
-       LyXTextClass const & tclass = bufparams.getLyXTextClass();
-       pit_type const cpit = cur.pit();
-
-       if (cpit != tmppit
-           && (pars_[cpit].layout() == pars_[tmppit].layout()
-               || pars_[tmppit].layout() == tclass.defaultLayout()))
-       {
-               mergeParagraph(bufparams, pars_, cpit);
+       else if (par.layout() == prevpar.layout()
+                || par.layout() == tclass.defaultLayout()) {
+               recordUndo(cur, Undo::ATOMIC, prevcur.pit());
+               mergeParagraph(bufparams, plist, prevcur.pit());
                needsUpdate = true;
+       }
 
-               if (cur.pos() != 0 && pars_[cpit].isSeparator(cur.pos() - 1))
-                               --cur.pos();
-
-               // the counters may have changed
-               ParIterator par_it(cur);
-               updateLabels(cur.buffer(), par_it);
-
-               setCursor(cur, cur.pit(), cur.pos(), false);
+       if (needsUpdate) {
+               updateLabels(cur.buffer());
+               setCursorIntern(cur, prevcur.pit(), prevcur.pos());
        }
+
        return needsUpdate;
 }
 
@@ -1755,25 +1156,15 @@ bool LyXText::backspace(LCursor & cur)
                if (cur.pit() == 0)
                        return dissolveInset(cur);
 
-               // The cursor is at the beginning of a paragraph, so
-               // the the backspace will collapse two paragraphs into
-               // one.
-
-               if (cur.pit() != 0 && cur.buffer().params().trackChanges) {
-                       // FIXME: Change tracking (MG)
-                       // Previous paragraph, mark "carriage return" as
-                       // deleted:
-                       Paragraph & par = pars_[cur.pit() - 1];
-                       // Take care of a just inserted para break:
-                       // FIXME: change tracking (MG)
-                       if (par.lookupChange(par.size()) != Change(Change::INSERTED)) {
-                               // FIXME: change tracking (MG)
-                               par.setChange(par.size(), Change(Change::DELETED));
-                               setCursorIntern(cur, cur.pit() - 1, par.size());
-                               return true;
-                       }
-               }
+               Paragraph & prev_par = pars_[cur.pit() - 1];
 
+               if (!prev_par.isMergedOnEndOfParDeletion(cur.buffer().params().trackChanges)) {
+                       prev_par.setChange(prev_par.size(), Change(Change::DELETED));
+                       setCursorIntern(cur, cur.pit() - 1, prev_par.size());
+                       return true;
+               }
+               // The cursor is at the beginning of a paragraph, so
+               // the backspace will collapse two paragraphs into one.
                needsUpdate = backspacePos0(cur);
 
        } else {
@@ -1786,14 +1177,15 @@ bool LyXText::backspace(LCursor & cur)
                // without the dreaded mechanism. (JMarc)
                setCursorIntern(cur, cur.pit(), cur.pos() - 1,
                                false, cur.boundary());
-               // FIXME: change tracking (MG)
                cur.paragraph().eraseChar(cur.pos(), cur.buffer().params().trackChanges);
        }
 
        if (cur.pos() == cur.lastpos())
                setCurrentFont(cur);
 
-       setCursor(cur, cur.pit(), cur.pos(), false, cur.boundary());
+       // A singlePar update is not enough in this case.
+//             cur.updateFlags(Update::Force);
+       setCursor(cur.top(), cur.pit(), cur.pos());
 
        return needsUpdate;
 }
@@ -1802,7 +1194,7 @@ bool LyXText::backspace(LCursor & cur)
 bool LyXText::dissolveInset(LCursor & cur) {
        BOOST_ASSERT(this == cur.text());
 
-       if (isMainText() || cur.inset().nargs() != 1)
+       if (isMainText(*cur.bv().buffer()) || cur.inset().nargs() != 1)
                return false;
 
        recordUndoInset(cur);
@@ -1819,9 +1211,18 @@ bool LyXText::dissolveInset(LCursor & cur) {
                spos += cur.pos();
        spit += cur.pit();
        Buffer & b = cur.buffer();
-       // FIXME: change tracking (MG)
        cur.paragraph().eraseChar(cur.pos(), b.params().trackChanges);
        if (!plist.empty()) {
+               // ERT paragraphs have the Language latex_language.
+               // This is invalid outside of ERT, so we need to
+               // change it to the buffer language.
+               ParagraphList::iterator it = plist.begin();
+               ParagraphList::iterator it_end = plist.end();
+               for (; it != it_end; it++) {
+                       it->changeLanguage(b.params(), latex_language,
+                                       b.getLanguage());
+               }
+
                pasteParagraphList(cur, plist, b.params().textclass,
                                   b.errorList("Paste"));
                // restore position
@@ -1834,123 +1235,6 @@ bool LyXText::dissolveInset(LCursor & cur) {
 }
 
 
-Row const & LyXText::firstRow() const
-{
-       return *paragraphs().front().rows().begin();
-}
-
-
-bool LyXText::redoParagraph(pit_type const pit)
-{
-       // remove rows of paragraph, keep track of height changes
-       Paragraph & par = pars_[pit];
-
-       // Add bibitem insets if necessary
-       if (par.layout()->labeltype == LABEL_BIBLIO) {
-               bool hasbibitem(false);
-               if (!par.insetlist.empty()
-                       // Insist on it being in pos 0
-                       && par.getChar(0) == Paragraph::META_INSET) {
-                       InsetBase * inset = par.insetlist.begin()->inset;
-                       if (inset->lyxCode() == InsetBase::BIBITEM_CODE)
-                               hasbibitem = true;
-               }
-               if (!hasbibitem) {
-                       InsetBibitem * inset(new
-                               InsetBibitem(InsetCommandParams("bibitem")));
-                       // FIXME: change tracking (MG)
-                       par.insertInset(0, static_cast<InsetBase *>(inset), Change(Change::INSERTED));
-                       bv()->cursor().posRight();
-               }
-       }
-
-       // redo insets
-       // FIXME: We should always use getFont(), see documentation of
-       // noFontChange() in insetbase.h.
-       LyXFont const bufferfont = bv()->buffer()->params().getFont();
-       InsetList::iterator ii = par.insetlist.begin();
-       InsetList::iterator iend = par.insetlist.end();
-       for (; ii != iend; ++ii) {
-               Dimension dim;
-               int const w = maxwidth_ - leftMargin(pit, ii->pos) - rightMargin(par);
-               LyXFont const & font = ii->inset->noFontChange() ?
-                       bufferfont :
-                       getFont(par, ii->pos);
-               MetricsInfo mi(bv(), font, w);
-               ii->inset->metrics(mi, dim);
-       }
-
-       // rebreak the paragraph
-       par.rows().clear();
-       Dimension dim;
-
-       par.setBeginOfBody();
-       pos_type z = 0;
-       do {
-               Row row(z);
-               rowBreakPoint(pit, row);
-               setRowWidth(pit, row);
-               setHeightOfRow(pit, row);
-               par.rows().push_back(row);
-               dim.wid = std::max(dim.wid, row.width());
-               dim.des += row.height();
-               z = row.endpos();
-       } while (z < par.size());
-
-       // Make sure that if a par ends in newline, there is one more row
-       // under it
-       // FIXME this is a dirty trick. Now the _same_ position in the
-       // paragraph occurs in _two_ different rows, and has two different
-       // display positions, leading to weird behaviour when moving up/down.
-       if (z > 0 && par.isNewline(z - 1)) {
-               Row row(z - 1);
-               row.endpos(z - 1);
-               setRowWidth(pit, row);
-               setHeightOfRow(pit, row);
-               par.rows().push_back(row);
-               dim.des += row.height();
-       }
-
-       dim.asc += par.rows()[0].ascent();
-       dim.des -= par.rows()[0].ascent();
-
-       bool const same = dim == par.dim();
-
-       par.dim() = dim;
-       //lyxerr << "redoParagraph: " << par.rows().size() << " rows\n";
-
-       return !same;
-}
-
-
-void LyXText::metrics(MetricsInfo & mi, Dimension & dim)
-{
-       //BOOST_ASSERT(mi.base.textwidth);
-       if (mi.base.textwidth)
-               maxwidth_ = mi.base.textwidth;
-       //lyxerr << "LyXText::metrics: width: " << mi.base.textwidth
-       //      << " maxWidth: " << maxwidth_ << "\nfont: " << mi.base.font << endl;
-       // save the caller's font locally:
-       font_ = mi.base.font;
-
-       unsigned int h = 0;
-       unsigned int w = 0;
-       for (pit_type pit = 0, n = paragraphs().size(); pit != n; ++pit) {
-               redoParagraph(pit);
-               Paragraph & par = paragraphs()[pit];
-               h += par.height();
-               if (w < par.width())
-                       w = par.width();
-       }
-
-       dim.wid = w;
-       dim.asc = pars_[0].ascent();
-       dim.des = h - dim.asc;
-
-       dim_ = dim;
-}
-
-
 // only used for inset right now. should also be used for main text
 void LyXText::draw(PainterInfo & pi, int x, int y) const
 {
@@ -1958,74 +1242,7 @@ void LyXText::draw(PainterInfo & pi, int x, int y) const
 }
 
 
-#if 0
 // only used for inset right now. should also be used for main text
-void LyXText::drawSelection(PainterInfo & pi, int x , int) const
-{
-       LCursor & cur = pi.base.bv->cursor();
-       if (!cur.selection())
-               return;
-       if (!ptr_cmp(cur.text(), this))
-               return;
-
-       lyxerr << "draw selection at " << x << endl;
-
-       DocIterator beg = cur.selectionBegin();
-       DocIterator end = cur.selectionEnd();
-
-       // the selection doesn't touch the visible screen
-       if (bv_funcs::status(pi.base.bv, beg) == bv_funcs::CUR_BELOW
-           || bv_funcs::status(pi.base.bv, end) == bv_funcs::CUR_ABOVE)
-               return;
-
-       Paragraph const & par1 = pars_[beg.pit()];
-       Paragraph const & par2 = pars_[end.pit()];
-
-       Row const & row1 = par1.getRow(beg.pos(), beg.boundary());
-       Row const & row2 = par2.getRow(end.pos(), end.boundary());
-
-       int y1,x1,x2;
-       if (bv_funcs::status(pi.base.bv, beg) == bv_funcs::CUR_ABOVE) {
-               y1 = 0;
-               x1 = 0;
-               x2 = 0;
-       } else {
-               y1 = bv_funcs::getPos(beg).y_ - row1.ascent();
-               int const startx = cursorX(beg.top(), begin.boundary());
-               x1 = isRTL(par1) ? startx : 0;
-               x2 = isRTL(par1) ? 0 + dim_.wid : startx;
-       }
-
-       int y2,X1,X2;
-       if (bv_funcs::status(pi.base.bv, end) == bv_funcs::CUR_BELOW) {
-               y2 = pi.base.bv->workHeight();
-               X1 = 0;
-               X2 = 0;
-       } else {
-               y2 = bv_funcs::getPos(end).y_ + row2.descent();
-               int const endx = cursorX(end.top(), end.boundary());
-               X1 = isRTL(par2) ? 0 : endx;
-               X2 = isRTL(par2) ? endx : 0 + dim_.wid;
-       }
-
-       lyxerr << " y1: " << y1 << " y2: " << y2
-               << " xo: " << xo_ << " wid: " << dim_.wid
-               << endl;
-
-       // paint big rectangle in one go
-       pi.pain.fillRectangle(x, y1, dim_.wid, y2 - y1, LColor::selection);
-
-       // reset background at begin of first selected line
-       pi.pain.fillRectangle(x + x1, y1, x2 - x1, row1.height(),
-               LColor::background);
-
-       // reset background at end of last selected line
-       pi.pain.fillRectangle(x + X1, y2  - row2.height(),
-               X2 - X1, row2.height(), LColor::background);
-}
-
-#else
-
 void LyXText::drawSelection(PainterInfo & pi, int x, int) const
 {
        LCursor & cur = pi.base.bv->cursor();
@@ -2034,71 +1251,91 @@ void LyXText::drawSelection(PainterInfo & pi, int x, int) const
        if (!ptr_cmp(cur.text(), this))
                return;
 
-       lyxerr[Debug::DEBUG]
-               << BOOST_CURRENT_FUNCTION
-               << "draw selection at " << x
-               << endl;
+       if (lyxerr.debugging(Debug::DEBUG)) {
+               lyxerr[Debug::DEBUG]
+                       << BOOST_CURRENT_FUNCTION
+                       << "draw selection at " << x
+                       << endl;
+       }
 
        DocIterator beg = cur.selectionBegin();
        DocIterator end = cur.selectionEnd();
 
-       BufferView * bv = pi.base.bv;
+       BufferView & bv = *pi.base.bv;
+       Buffer const & buffer = *bv.buffer();
 
        // the selection doesn't touch the visible screen
-       if (bv_funcs::status(bv, beg) == bv_funcs::CUR_BELOW
-           || bv_funcs::status(bv, end) == bv_funcs::CUR_ABOVE)
+       if (bv_funcs::status(&bv, beg) == bv_funcs::CUR_BELOW
+           || bv_funcs::status(&bv, end) == bv_funcs::CUR_ABOVE)
                return;
 
        Paragraph const & par1 = pars_[beg.pit()];
        Paragraph const & par2 = pars_[end.pit()];
+       TextMetrics const & tm = bv.textMetrics(this);
+       ParagraphMetrics const & pm1 = tm.parMetrics(beg.pit());
+       ParagraphMetrics const & pm2 = tm.parMetrics(end.pit());
 
-       bool const above = (bv_funcs::status(bv, beg)
+       bool const above = (bv_funcs::status(&bv, beg)
                            == bv_funcs::CUR_ABOVE);
-       bool const below = (bv_funcs::status(bv, end)
+       bool const below = (bv_funcs::status(&bv, end)
                            == bv_funcs::CUR_BELOW);
        int y1,y2,x1,x2;
        if (above) {
                y1 = 0;
                y2 = 0;
                x1 = 0;
-               x2 = dim_.wid;
+               x2 = tm.width();
        } else {
-               Row const & row1 = par1.getRow(beg.pos(), beg.boundary());
-               y1 = bv_funcs::getPos(*bv, beg, beg.boundary()).y_ - row1.ascent();
+               Row const & row1 = pm1.getRow(beg.pos(), beg.boundary());
+               y1 = bv_funcs::getPos(bv, beg, beg.boundary()).y_ - row1.ascent();
                y2 = y1 + row1.height();
-               int const startx = cursorX(beg.top(), beg.boundary());
-               x1 = !isRTL(par1) ? startx : 0;
-               x2 = !isRTL(par1) ? 0 + dim_.wid : startx;
+               int const startx = cursorX(bv, beg.top(), beg.boundary());
+               if (!isRTL(buffer, par1)) {
+                       x1 = startx;
+                       x2 = 0 + tm.width();
+               }
+               else {
+                       x1 = 0;
+                       x2 = startx;
+               }
        }
 
        int Y1,Y2,X1,X2;
        if (below) {
-               Y1 = bv->workHeight();
-               Y2 = bv->workHeight();
+               Y1 = bv.workHeight();
+               Y2 = bv.workHeight();
                X1 = 0;
-               X2 = dim_.wid;
+               X2 = tm.width();
        } else {
-               Row const & row2 = par2.getRow(end.pos(), end.boundary());
-               Y1 = bv_funcs::getPos(*bv, end, end.boundary()).y_ - row2.ascent();
+               Row const & row2 = pm2.getRow(end.pos(), end.boundary());
+               Y1 = bv_funcs::getPos(bv, end, end.boundary()).y_ - row2.ascent();
                Y2 = Y1 + row2.height();
-               int const endx = cursorX(end.top(), end.boundary());
-               X1 = !isRTL(par2) ? 0 : endx;
-               X2 = !isRTL(par2) ? endx : 0 + dim_.wid;
+               int const endx = cursorX(bv, end.top(), end.boundary());
+               if (!isRTL(buffer, par2)) {
+                       X1 = 0;
+                       X2 = endx;
+               }
+               else {
+                       X1 = endx;
+                       X2 = 0 + tm.width();
+               }
        }
 
-       if (!above && !below && &par1.getRow(beg.pos(), beg.boundary())
-           == &par2.getRow(end.pos(), end.boundary()))
+       if (!above && !below && &pm1.getRow(beg.pos(), beg.boundary())
+           == &pm2.getRow(end.pos(), end.boundary()))
        {
                // paint only one rectangle
-               int const b( !isRTL(par1) ? x + x1 : x + X1 );
-               int const w( !isRTL(par1) ? X2 - x1 : x2 - X1 );
+               int const b( !isRTL(*bv.buffer(), par1) ? x + x1 : x + X1 );
+               int const w( !isRTL(*bv.buffer(), par1) ? X2 - x1 : x2 - X1 );
                pi.pain.fillRectangle(b, y1, w, y2 - y1, LColor::selection);
                return;
        }
 
-       lyxerr[Debug::DEBUG] << " y1: " << y1 << " y2: " << y2
-              << "X1:" << X1 << " x2: " << X2 << " wid: " << dim_.wid
-               << endl;
+       if (lyxerr.debugging(Debug::DEBUG)) {
+               lyxerr[Debug::DEBUG] << " y1: " << y1 << " y2: " << y2
+                          << "X1:" << X1 << " x2: " << X2 << " wid: " << tm.width()
+                       << endl;
+       }
 
        // paint upper rectangle
        pi.pain.fillRectangle(x + x1, y1, x2 - x1, y2 - y1,
@@ -2107,10 +1344,10 @@ void LyXText::drawSelection(PainterInfo & pi, int x, int) const
        pi.pain.fillRectangle(x + X1, Y1, X2 - X1, Y2 - Y1,
                                      LColor::selection);
        // paint center rectangle
-       pi.pain.fillRectangle(x, y2, dim_.wid,
+       pi.pain.fillRectangle(x, y2, tm.width(),
                              Y1 - y2, LColor::selection);
 }
-#endif
+
 
 bool LyXText::isLastRow(pit_type pit, Row const & row) const
 {
@@ -2226,24 +1463,14 @@ bool LyXText::read(Buffer const & buf, LyXLex & lex, ErrorList & errorList)
        return true;
 }
 
-
-int LyXText::ascent() const
-{
-       return dim_.asc;
-}
-
-
-int LyXText::descent() const
-{
-       return dim_.des;
-}
-
-
-int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
+int LyXText::cursorX(BufferView const & bv, CursorSlice const & sl,
+               bool boundary) const
 {
+       TextMetrics const & tm = bv.textMetrics(sl.text());
        pit_type const pit = sl.pit();
        Paragraph const & par = pars_[pit];
-       if (par.rows().empty())
+       ParagraphMetrics const & pm = tm.parMetrics(pit);
+       if (pm.rows().empty())
                return 0;
 
        pos_type ppos = sl.pos();
@@ -2252,11 +1479,12 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
        if (boundary_correction)
                --ppos;
 
-       Row const & row = par.getRow(sl.pos(), boundary);
+       Row const & row = pm.getRow(sl.pos(), boundary);
 
        pos_type cursor_vpos = 0;
 
-       RowMetrics const m = computeRowMetrics(pit, row);
+       Buffer const & buffer = *bv.buffer();
+       RowMetrics const m = tm.computeRowMetrics(pit, row);
        double x = m.x;
 
        pos_type const row_pos  = row.pos();
@@ -2265,7 +1493,7 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
        if (end <= row_pos)
                cursor_vpos = row_pos;
        else if (ppos >= end)
-               cursor_vpos = isRTL(par) ? row_pos : end;
+               cursor_vpos = isRTL(buffer, par) ? row_pos : end;
        else if (ppos > row_pos && ppos >= end)
                // Place cursor after char at (logical) position pos - 1
                cursor_vpos = (bidi.level(ppos - 1) % 2 == 0)
@@ -2283,7 +1511,8 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
        // Use font span to speed things up, see below
        FontSpan font_span;
        LyXFont font;
-       FontMetrics const & labelfm = theFontMetrics(getLabelFont(par));
+       FontMetrics const & labelfm = theFontMetrics(
+               getLabelFont(buffer, par));
 
        for (pos_type vpos = row_pos; vpos < cursor_vpos; ++vpos) {
                pos_type pos = bidi.vis2log(vpos);
@@ -2292,13 +1521,13 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
                        docstring const lsep = from_utf8(par.layout()->labelsep);
                        x += m.label_hfill + labelfm.width(lsep);
                        if (par.isLineSeparator(body_pos - 1))
-                               x -= singleWidth(par, body_pos - 1);
+                               x -= singleWidth(buffer, par, body_pos - 1);
                }
 
                // Use font span to speed things up, see above
                if (pos < font_span.first || pos > font_span.last) {
                        font_span = par.fontSpan(pos);
-                       font = getFont(par, pos);
+                       font = getFont(buffer, par, pos);
                }
 
                x += singleWidth(par, pos, par.getChar(pos), font);
@@ -2311,15 +1540,15 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
 
        // see correction above
        if (boundary_correction)
-               if (getFont(par, ppos).isVisibleRightToLeft())
-                       x -= singleWidth(par, ppos);
+               if (getFont(buffer, par, ppos).isVisibleRightToLeft())
+                       x -= singleWidth(buffer, par, ppos);
                else
-                       x += singleWidth(par, ppos);
+                       x += singleWidth(buffer, par, ppos);
 
        // Make sure inside an inset we always count from the left
        // edge (bidi!) -- MV
        if (sl.pos() < par.size()) {
-               font = getFont(par, sl.pos());
+               font = getFont(buffer, par, sl.pos());
                if (!boundary && font.isVisibleRightToLeft()
                  && par.isInset(sl.pos()))
                        x -= par.getInset(sl.pos())->width();
@@ -2328,50 +1557,49 @@ int LyXText::cursorX(CursorSlice const & sl, bool boundary) const
 }
 
 
-int LyXText::cursorY(CursorSlice const & sl, bool boundary) const
+int LyXText::cursorY(BufferView const & bv, CursorSlice const & sl, bool boundary) const
 {
        //lyxerr << "LyXText::cursorY: boundary: " << boundary << std::endl;
-       Paragraph const & par = getPar(sl.pit());
-       if (par.rows().empty())
+       ParagraphMetrics const & pm = bv.parMetrics(this, sl.pit());
+       if (pm.rows().empty())
                return 0;
 
        int h = 0;
-       h -= pars_[0].rows()[0].ascent();
-       for (pit_type pit = 0; pit < sl.pit(); ++pit)
-               h += pars_[pit].height();
+       h -= bv.parMetrics(this, 0).rows()[0].ascent();
+       for (pit_type pit = 0; pit < sl.pit(); ++pit) {
+               h += bv.parMetrics(this, pit).height();
+       }
        int pos = sl.pos();
        if (pos && boundary)
                --pos;
-       size_t const rend = par.pos2row(pos);
+       size_t const rend = pm.pos2row(pos);
        for (size_t rit = 0; rit != rend; ++rit)
-               h += par.rows()[rit].height();
-       h += par.rows()[rend].ascent();
+               h += pm.rows()[rit].height();
+       h += pm.rows()[rend].ascent();
        return h;
 }
 
 
 // Returns the current font and depth as a message.
-string LyXText::currentState(LCursor & cur)
+docstring LyXText::currentState(LCursor & cur)
 {
        BOOST_ASSERT(this == cur.text());
        Buffer & buf = cur.buffer();
        Paragraph const & par = cur.paragraph();
-       std::ostringstream os;
-
-       // FIXME: change tracking (MG)
-       bool const show_change = par.lookupChange(cur.pos()) != Change(Change::UNCHANGED);
+       odocstringstream os;
 
        if (buf.params().trackChanges)
-               os << "[C] ";
+               os << _("[Change Tracking] ");
+
+       Change change = par.lookupChange(cur.pos());
 
-       if (show_change) {
-               Change change = par.lookupChange(cur.pos());
+       if (change.type != Change::UNCHANGED) {
                Author const & a = buf.params().authors().get(change.author);
-               os << to_utf8(_("Change: ")) << a.name();
+               os << _("Change: ") << a.name();
                if (!a.email().empty())
                        os << " (" << a.email() << ")";
-               if (change.changetime)
-                       os << to_utf8(_(" at ")) << ctime(&change.changetime);
+               // FIXME ctime is english, we should translate that
+               os << _(" at ") << ctime(&change.changetime);
                os << " : ";
        }
 
@@ -2381,34 +1609,30 @@ string LyXText::currentState(LCursor & cur)
        LyXFont font = real_current_font;
        font.reduce(buf.params().getFont());
 
-       // avoid to_utf8(_(...)) re-entrance problem
-       string const s = font.stateText(&buf.params());
-       os << to_utf8(bformat(_("Font: %1$s"), from_utf8(s)));
-
-       // os << to_utf8(bformat(_("Font: %1$s"), font.stateText(&buf.params)));
+       os << bformat(_("Font: %1$s"), font.stateText(&buf.params()));
 
        // The paragraph depth
        int depth = cur.paragraph().getDepth();
        if (depth > 0)
-               os << to_utf8(bformat(_(", Depth: %1$d"), depth));
+               os << bformat(_(", Depth: %1$d"), depth);
 
        // The paragraph spacing, but only if different from
        // buffer spacing.
        Spacing const & spacing = par.params().spacing();
        if (!spacing.isDefault()) {
-               os << to_utf8(_(", Spacing: "));
+               os << _(", Spacing: ");
                switch (spacing.getSpace()) {
                case Spacing::Single:
-                       os << to_utf8(_("Single"));
+                       os << _("Single");
                        break;
                case Spacing::Onehalf:
-                       os << to_utf8(_("OneHalf"));
+                       os << _("OneHalf");
                        break;
                case Spacing::Double:
-                       os << to_utf8(_("Double"));
+                       os << _("Double");
                        break;
                case Spacing::Other:
-                       os << to_utf8(_("Other (")) << spacing.getValueAsString() << ')';
+                       os << _("Other (") << from_ascii(spacing.getValueAsString()) << ')';
                        break;
                case Spacing::Default:
                        // should never happen, do nothing
@@ -2417,11 +1641,11 @@ string LyXText::currentState(LCursor & cur)
        }
 
 #ifdef DEVEL_VERSION
-       os << to_utf8(_(", Inset: ")) << &cur.inset();
-       os << to_utf8(_(", Paragraph: ")) << cur.pit();
-       os << to_utf8(_(", Id: ")) << par.id();
-       os << to_utf8(_(", Position: ")) << cur.pos();
-       os << to_utf8(_(", Boundary: ")) << cur.boundary();
+       os << _(", Inset: ") << &cur.inset();
+       os << _(", Paragraph: ") << cur.pit();
+       os << _(", Id: ") << par.id();
+       os << _(", Position: ") << cur.pos();
+       os << _(", Boundary: ") << cur.boundary();
 //     Row & row = cur.textRow();
 //     os << bformat(_(", Row b:%1$d e:%2$d"), row.pos(), row.endpos());
 #endif
@@ -2429,7 +1653,7 @@ string LyXText::currentState(LCursor & cur)
 }
 
 
-string LyXText::getPossibleLabel(LCursor & cur) const
+docstring LyXText::getPossibleLabel(LCursor & cur) const
 {
        pit_type pit = cur.pit();
 
@@ -2443,7 +1667,7 @@ string LyXText::getPossibleLabel(LCursor & cur) const
                }
        }
 
-       string name = layout->latexname();
+       docstring name = from_ascii(layout->latexname());
 
        // for captions, we want the abbreviation of the float type
        if (layout->labeltype == LABEL_SENSITIVE) {
@@ -2452,26 +1676,25 @@ string LyXText::getPossibleLabel(LCursor & cur) const
                        InsetBase * const in = &cur[i].inset();
                        if (in->lyxCode() == InsetBase::FLOAT_CODE
                            || in->lyxCode() == InsetBase::WRAP_CODE) {
-                               name = to_utf8(in->getInsetName());
+                               name = in->getInsetName();
                                break;
                        }
                }
        }
 
-       string text = name.substr(0, 3);
+       docstring text = name.substr(0, 3);
        if (name == "theorem")
-               text = "thm"; // Create a correct prefix for prettyref
+               text = from_ascii("thm"); // Create a correct prefix for prettyref
 
        text += ':';
        if (layout->latextype == LATEX_PARAGRAPH || lyxrc.label_init_length < 0)
                text.erase();
 
-       // FIXME UNICODE
-       string par_text = to_utf8(pars_[pit].asString(cur.buffer(), false));
+       docstring par_text = pars_[pit].asString(cur.buffer(), false);
        for (int i = 0; i < lyxrc.label_init_length; ++i) {
                if (par_text.empty())
                        break;
-               string head;
+               docstring head;
                par_text = split(par_text, head, ' ');
                // Is it legal to use spaces in labels ?
                if (i > 0)
@@ -2483,48 +1706,15 @@ string LyXText::getPossibleLabel(LCursor & cur) const
 }
 
 
-//pos_type LyXText::x2pos(pit_type pit, int row, int x) const
-//{
-//     int lastx = 0;
-//     int currx = 0;
-//     Paragraph const & par = pars_[pit];
-//     Row const & r = par.rows()[row];
-//     int pos = r.pos();
-//     for (; currx < x && pos < r.endpos(); ++pos) {
-//             lastx = currx;
-//             currx += singleWidth(par, pos);
-//     }
-//     if (abs(lastx - x) < abs(currx - x) && pos != r.pos())
-//             --pos;
-//     return pos;
-//}
-
-
-pos_type LyXText::x2pos(pit_type pit, int row, int x) const
+void LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
 {
-       BOOST_ASSERT(row < int(pars_[pit].rows().size()));
-       bool bound = false;
-       Row const & r = pars_[pit].rows()[row];
-       return r.pos() + getColumnNearX(pit, r, x, bound);
-}
-
-
-//int LyXText::pos2x(pit_type pit, pos_type pos) const
-//{
-//     Paragraph const & par = pars_[pit];
-//     Row const & r = par.rows()[row];
-//     int x = 0;
-//     pos -= r.pos();
-//}
+       BOOST_ASSERT(this == cur.text());
+       pit_type pit = getPitNearY(cur.bv(), y);
 
+       TextMetrics const & tm = cur.bv().textMetrics(this);
+       ParagraphMetrics const & pm = tm.parMetrics(pit);
 
-// x,y are screen coordinates
-// sets cursor only within this LyXText
-bool LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
-{
-       BOOST_ASSERT(this == cur.text());
-       pit_type pit = getPitNearY(y);
-       int yy = cur.bv().coordCache().get(this, pit).y_ - pars_[pit].ascent();
+       int yy = cur.bv().coordCache().get(this, pit).y_ - pm.ascent();
        lyxerr[Debug::DEBUG]
                << BOOST_CURRENT_FUNCTION
                << ": x: " << x
@@ -2532,17 +1722,16 @@ bool LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
                << " pit: " << pit
                << " yy: " << yy << endl;
 
-       Paragraph const & par = pars_[pit];
        int r = 0;
-       BOOST_ASSERT(par.rows().size());
-       for (; r < int(par.rows().size()) - 1; ++r) {
-               Row const & row = par.rows()[r];
+       BOOST_ASSERT(pm.rows().size());
+       for (; r < int(pm.rows().size()) - 1; ++r) {
+               Row const & row = pm.rows()[r];
                if (int(yy + row.height()) > y)
                        break;
                yy += row.height();
        }
 
-       Row const & row = par.rows()[r];
+       Row const & row = pm.rows()[r];
 
        lyxerr[Debug::DEBUG]
                << BOOST_CURRENT_FUNCTION
@@ -2552,7 +1741,8 @@ bool LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
 
        bool bound = false;
        int xx = x;
-       pos_type const pos = row.pos() + getColumnNearX(pit, row, xx, bound);
+       pos_type const pos = row.pos() 
+               + tm.getColumnNearX(pit, row, xx, bound);
 
        lyxerr[Debug::DEBUG]
                << BOOST_CURRENT_FUNCTION
@@ -2560,7 +1750,66 @@ bool LyXText::setCursorFromCoordinates(LCursor & cur, int const x, int const y)
                << " pos: " << pos
                << endl;
 
-       return setCursor(cur, pit, pos, true, bound);
+       setCursor(cur, pit, pos, true, bound);
+       // remember new position.
+       cur.setTargetX();
+}
+
+
+void LyXText::charsTranspose(LCursor & cur)
+{
+       BOOST_ASSERT(this == cur.text());
+
+       pos_type pos = cur.pos();
+
+       // If cursor is at beginning or end of paragraph, do nothing.
+       if (pos == cur.lastpos() || pos == 0)
+               return;
+
+       Paragraph & par = cur.paragraph();
+
+       // Get the positions of the characters to be transposed. 
+       pos_type pos1 = pos - 1;
+       pos_type pos2 = pos;
+
+       // In change tracking mode, ignore deleted characters.
+       while (pos2 < cur.lastpos() && par.isDeleted(pos2))
+               ++pos2;
+       if (pos2 == cur.lastpos())
+               return;
+
+       while (pos1 >= 0 && par.isDeleted(pos1))
+               --pos1;
+       if (pos1 < 0)
+               return;
+
+       // Don't do anything if one of the "characters" is not regular text.
+       if (par.isInset(pos1) || par.isInset(pos2))
+               return;
+
+       // Store the characters to be transposed (including font information).
+       char_type char1 = par.getChar(pos1);
+       LyXFont const font1 =
+               par.getFontSettings(cur.buffer().params(), pos1);
+       
+       char_type char2 = par.getChar(pos2);
+       LyXFont const font2 =
+               par.getFontSettings(cur.buffer().params(), pos2);
+
+       // And finally, we are ready to perform the transposition.
+       // Track the changes if Change Tracking is enabled.
+       bool const trackChanges = cur.buffer().params().trackChanges;
+
+       recordUndo(cur);
+
+       par.eraseChar(pos2, trackChanges);
+       par.eraseChar(pos1, trackChanges);
+       par.insertChar(pos1, char2, font2, trackChanges);
+       par.insertChar(pos2, char1, font1, trackChanges);
+
+       // After the transposition, move cursor to after the transposition.
+       setCursor(cur, cur.pit(), pos2);
+       cur.forwardPos();
 }