]> git.lyx.org Git - lyx.git/blobdiff - src/rowpainter.C
Add GTK bibitem dialog
[lyx.git] / src / rowpainter.C
index 31c3df5130dd2bc0ab165a100b0afc3f9c59edbc..b992b00143a71425bccc3eff85218e728ef80add 100644 (file)
  * \author various
  * \author John Levon
  *
- * Full author contact details are available in file CREDITS
+ * Full author contact details are available in file CREDITS.
  */
 
 #include <config.h>
 
-#include <algorithm>
-
-#include "frontends/Painter.h"
-#include "frontends/screen.h"
-#include "frontends/font_metrics.h"
-#include "support/LAssert.h"
-#include "paragraph.h"
-#include "support/textutils.h"
+#include "rowpainter.h"
 
-#include "insets/insettext.h"
-#include "ParagraphParameters.h"
-#include "BufferView.h"
 #include "buffer.h"
+#include "coordcache.h"
+#include "cursor.h"
+#include "debug.h"
+#include "bufferparams.h"
+#include "BufferView.h"
+#include "encoding.h"
 #include "gettext.h"
 #include "language.h"
-#include "encoding.h"
-#include "lyxtext.h"
-#include "lyxrow.h"
-#include "rowpainter.h"
+#include "LColor.h"
 #include "lyxrc.h"
+#include "lyxrow.h"
+#include "lyxrow_funcs.h"
+#include "metricsinfo.h"
+#include "paragraph.h"
+#include "paragraph_funcs.h"
+#include "ParagraphParameters.h"
+#include "vspace.h"
 
+#include "frontends/font_metrics.h"
+#include "frontends/nullpainter.h"
+#include "frontends/Painter.h"
 
-using std::max;
-using lyx::pos_type;
-
-extern int PAPER_MARGIN;
-extern int CHANGEBAR_MARGIN;
-extern int LEFT_MARGIN;
+#include "insets/insettext.h"
 
-namespace {
+#include "support/textutils.h"
 
-// "temporary". We'll never get to use more
-// references until we start adding hacks like
-// these until other places catch up.
-BufferView * perv(BufferView const & bv)
-{
-       return const_cast<BufferView *>(&bv);
-}
+#include <boost/crc.hpp>
 
-} // namespace anon
+using lyx::pos_type;
+using lyx::pit_type;
 
+using std::endl;
+using std::max;
+using std::min;
+using std::string;
 
-RowPainter::RowPainter(BufferView const & bv, LyXText const & text, Row const & row)
-       : bv_(bv), pain_(bv_.painter()), text_(text), row_(row), par_(*row.par())
-{
-}
 
+namespace {
 
-/// "temporary"
-LyXFont const RowPainter::getFont(pos_type pos) const
+/**
+ * A class used for painting an individual row of text.
+ */
+class RowPainter {
+public:
+       /// initialise and run painter
+       RowPainter(PainterInfo & pi, LyXText const & text,
+               pit_type pit, Row const & row, int x, int y);
+
+       // paint various parts
+       void paintAppendix();
+       void paintDepthBar();
+       void paintChangeBar();
+       void paintFirst();
+       void paintLast();
+       void paintText();
+
+private:
+       void paintForeignMark(double orig_x, LyXFont const & font);
+       void paintHebrewComposeChar(lyx::pos_type & vpos, LyXFont const & font);
+       void paintArabicComposeChar(lyx::pos_type & vpos, LyXFont const & font);
+       void paintChars(lyx::pos_type & vpos, LyXFont font, 
+                       bool hebrew, bool arabic);
+       int paintAppendixStart(int y);
+       void paintFromPos(lyx::pos_type & vpos);
+       void paintInset(lyx::pos_type const pos, LyXFont const & font);
+
+       /// return left margin
+       int leftMargin() const;
+
+       /// return the label font for this row
+       LyXFont const getLabelFont() const;
+
+       /// bufferview to paint on
+       BufferView const & bv_;
+
+       /// Painter to use
+       Painter & pain_;
+
+       /// LyXText for the row
+       LyXText const & text_;
+       ParagraphList const & pars_;
+
+       /// The row to paint
+       Row const & row_;
+
+       /// Row's paragraph
+       pit_type const pit_;
+       Paragraph const & par_;
+
+       /// is row erased? (change tracking)
+       bool erased_;
+
+       // Looks ugly - is
+       double const xo_;
+       int const yo_;    // current baseline
+       double x_;
+       int width_;
+       double separator_;
+       double hfill_;
+       double label_hfill_;
+};
+
+
+RowPainter::RowPainter(PainterInfo & pi,
+       LyXText const & text, pit_type pit, Row const & row, int x, int y)
+       : bv_(*pi.base.bv), pain_(pi.pain), text_(text), pars_(text.paragraphs()),
+         row_(row), pit_(pit), par_(text.paragraphs()[pit]),
+         erased_(pi.erased_),
+         xo_(x), yo_(y), width_(text_.width())
 {
-       Paragraph * perverted_par = const_cast<Paragraph *>(&par_);
-       return text_.getFont(bv_.buffer(), perverted_par, pos);
-}
-
+       RowMetrics m = text_.computeRowMetrics(pit, row_);
+       x_ = m.x + xo_;
 
-int RowPainter::singleWidth(lyx::pos_type pos) const
-{
-       BufferView * bv(perv(bv_));
-       Paragraph * par(const_cast<Paragraph*>(&par_));
-       return text_.singleWidth(bv, par, pos);
-}
+       //lyxerr << "RowPainter: x: " << x_ << " xo: " << xo_ << " yo: " << yo_ << endl;
+       //row_.dump();
 
+       separator_ = m.separator;
+       hfill_ = m.hfill;
+       label_hfill_ = m.label_hfill;
 
-int RowPainter::singleWidth(lyx::pos_type pos, char c) const
-{
-       BufferView * bv(perv(bv_));
-       Paragraph * par(const_cast<Paragraph*>(&par_));
-       return text_.singleWidth(bv, par, pos, c);
+       BOOST_ASSERT(pit >= 0);
+       BOOST_ASSERT(pit < int(text.paragraphs().size()));
 }
 
 
 LyXFont const RowPainter::getLabelFont() const
 {
-       Paragraph * par(const_cast<Paragraph*>(&par_));
-       return text_.getLabelFont(bv_.buffer(), par);
-}
-
-
-char const RowPainter::transformChar(char c, lyx::pos_type pos) const
-{
-       Paragraph * par(const_cast<Paragraph*>(&par_));
-       return text_.transformChar(c, par, pos);
+       return text_.getLabelFont(par_);
 }
 
 
 int RowPainter::leftMargin() const
 {
-       BufferView * bv(perv(bv_));
-       Row * row(const_cast<Row *>(&row_));
-       return text_.leftMargin(bv, row);
-}
-
-
-void RowPainter::paintNewline(pos_type const pos)
-{
-       LyXFont const font = getFont(pos);
-       int const wid = font_metrics::width('n', font);
-       int const asc = font_metrics::maxAscent(font);
-       int const y = yo_ + row_.baseline();
-       // FIXME: rtl_pos, or ltr_pos ?
-       bool const rtl_pos = (text_.bidi_level(pos) % 2 == 0);
-       int xp[3];
-       int yp[3];
-
-       yp[0] = int(y - 0.875 * asc * 0.75);
-       yp[1] = int(y - 0.500 * asc * 0.75);
-       yp[2] = int(y - 0.125 * asc * 0.75);
-
-       if (rtl_pos) {
-               xp[0] = int(x_ + wid * 0.375);
-               xp[1] = int(x_);
-               xp[2] = int(x_ + wid * 0.375);
-       } else {
-               xp[0] = int(x_ + wid * 0.625);
-               xp[1] = int(x_ + wid);
-               xp[2] = int(x_ + wid * 0.625);
-       }
-
-       pain_.lines(xp, yp, 3, LColor::eolmarker);
-
-       yp[0] = int(y - 0.500 * asc * 0.75);
-       yp[1] = int(y - 0.500 * asc * 0.75);
-       yp[2] = int(y - asc * 0.75);
-
-       if (rtl_pos) {
-               xp[0] = int(x_);
-               xp[1] = int(x_ + wid);
-               xp[2] = int(x_ + wid);
-       } else {
-               xp[0] = int(x_ + wid);
-               xp[1] = int(x_);
-               xp[2] = int(x_);
-       }
-
-       pain_.lines(xp, yp, 3, LColor::eolmarker);
-
-       x_ += wid;
+       return text_.leftMargin(pit_, row_.pos());
 }
 
 
-bool RowPainter::paintInset(pos_type const pos)
+void RowPainter::paintInset(pos_type const pos, LyXFont const & font)
 {
-       Inset * inset = const_cast<Inset*>(par_.getInset(pos));
-
-       lyx::Assert(inset);
-
-       LyXFont const & font = getFont(pos);
-
-       inset->update(perv(bv_), font, false);
-       inset->draw(perv(bv_), font, yo_ + row_.baseline(), x_, cleared_);
-
-       // return true if something changed when we drew an inset
-
-       return (!text_.need_break_row && !text_.isInInset()
-           && bv_.text->status() == LyXText::CHANGED_IN_DRAW);
+       InsetBase const * inset = par_.getInset(pos);
+       BOOST_ASSERT(inset);
+       PainterInfo pi(const_cast<BufferView *>(&bv_), pain_);
+       // FIXME: We should always use font, see documentation of
+       // noFontChange() in insetbase.h.
+       pi.base.font = inset->noFontChange() ?
+               bv_.buffer()->params().getLyXTextClass().defaultfont() :
+               font;
+       pi.ltr_pos = (text_.bidi.level(pos) % 2 == 0);
+       pi.erased_ = erased_ || isDeletedText(par_, pos);
+       theCoords.insets().add(inset, int(x_), yo_);
+       inset->drawSelection(pi, int(x_), yo_);
+       inset->draw(pi, int(x_), yo_);
+       x_ += inset->width();
 }
 
 
-void RowPainter::paintHebrewComposeChar(pos_type & vpos)
+void RowPainter::paintHebrewComposeChar(pos_type & vpos, LyXFont const & font)
 {
-       pos_type pos = text_.vis2log(vpos);
+       pos_type pos = text_.bidi.vis2log(vpos);
 
        string str;
 
@@ -181,7 +181,6 @@ void RowPainter::paintHebrewComposeChar(pos_type & vpos)
        str += c;
        ++vpos;
 
-       LyXFont const & font = getFont(pos);
        int const width = font_metrics::width(c, font);
        int dx = 0;
 
@@ -190,7 +189,7 @@ void RowPainter::paintHebrewComposeChar(pos_type & vpos)
                if (!Encodings::IsComposeChar_hebrew(c)) {
                        if (IsPrintableNonspace(c)) {
                                int const width2 =
-                                       singleWidth(i, c);
+                                       text_.singleWidth(par_, i, c, text_.getFont(par_, i));
                                // dalet / resh
                                dx = (c == 'ø' || c == 'ã')
                                        ? width2 - width
@@ -201,141 +200,130 @@ void RowPainter::paintHebrewComposeChar(pos_type & vpos)
        }
 
        // Draw nikud
-       pain_.text(int(x_) + dx, yo_ + row_.baseline(), str, font);
+       pain_.text(int(x_) + dx, yo_, str, font);
 }
 
 
-void RowPainter::paintArabicComposeChar(pos_type & vpos)
+void RowPainter::paintArabicComposeChar(pos_type & vpos, LyXFont const & font)
 {
-       pos_type pos = text_.vis2log(vpos);
+       pos_type pos = text_.bidi.vis2log(vpos);
        string str;
 
        // first char
        char c = par_.getChar(pos);
-       c = transformChar(c, pos);
-       str +=c;
+       c = par_.transformChar(c, pos);
+       str += c;
        ++vpos;
 
-       LyXFont const & font = getFont(pos);
        int const width = font_metrics::width(c, font);
        int dx = 0;
 
-       for (pos_type i = pos-1; i >= 0; --i) {
+       for (pos_type i = pos - 1; i >= 0; --i) {
                c = par_.getChar(i);
                if (!Encodings::IsComposeChar_arabic(c)) {
                        if (IsPrintableNonspace(c)) {
-                               int const width2 =
-                                       singleWidth(i, c);
+                               int const width2 = 
+                                       text_.singleWidth(par_, i, c, text_.getFont(par_, i));
                                dx = (width2 - width) / 2;
                        }
                        break;
                }
        }
        // Draw nikud
-       pain_.text(int(x_) + dx, yo_ + row_.baseline(), str, font);
+       pain_.text(int(x_) + dx, yo_, str, font);
 }
 
 
-void RowPainter::paintChars(pos_type & vpos, bool hebrew, bool arabic)
+void RowPainter::paintChars(pos_type & vpos, LyXFont font, 
+                           bool hebrew, bool arabic)
 {
-       pos_type pos = text_.vis2log(vpos);
-       pos_type const last = row_.lastPrintablePos();
-       LyXFont orig_font(getFont(pos));
+       pos_type pos = text_.bidi.vis2log(vpos);
+       pos_type const end = row_.endpos();
+       FontSpan const font_span = par_.fontSpan(pos);
+       Change::Type const prev_change = par_.lookupChange(pos);
 
        // first character
        string str;
        str += par_.getChar(pos);
        if (arabic) {
                unsigned char c = str[0];
-               str[0] = transformChar(c, pos);
+               str[0] = par_.transformChar(c, pos);
        }
-       bool prev_struckout(isDeletedText(par_, pos));
-       bool prev_newtext(isInsertedText(par_, pos));
-       ++vpos;
 
        // collect as much similar chars as we can
-       while (vpos <= last && (pos = text_.vis2log(vpos)) >= 0) {
+       for (++vpos ; vpos < end ; ++vpos) {
+               pos = text_.bidi.vis2log(vpos);
+               if (pos < font_span.first || pos > font_span.last)
+                       break;
+
+               if (prev_change != par_.lookupChange(pos))
+                       break;
+
                char c = par_.getChar(pos);
 
                if (!IsPrintableNonspace(c))
                        break;
 
-               if (prev_struckout != isDeletedText(par_, pos))
-                       break;
-               if (prev_newtext != isInsertedText(par_, pos))
-                       break;
                if (arabic && Encodings::IsComposeChar_arabic(c))
                        break;
-               if (hebrew && Encodings::IsComposeChar_hebrew(c))
-                       break;
 
-               if (orig_font != getFont(pos))
+               if (hebrew && Encodings::IsComposeChar_hebrew(c))
                        break;
 
                if (arabic)
-                       c = transformChar(c, pos);
+                       c = par_.transformChar(c, pos);
+
                str += c;
-               ++vpos;
        }
 
-       if (prev_struckout) {
-               orig_font.setColor(LColor::strikeout);
-       } else if (prev_newtext) {
-               orig_font.setColor(LColor::newtext);
-       }
+       if (prev_change == Change::DELETED)
+               font.setColor(LColor::strikeout);
+       else if (prev_change == Change::INSERTED)
+               font.setColor(LColor::newtext);
 
        // Draw text and set the new x position
-       pain_.text(int(x_), yo_ + row_.baseline(), str, orig_font);
-       x_ += font_metrics::width(str, orig_font);
+       //lyxerr << "paint row: yo_ " << yo_ << "\n";
+       pain_.text(int(x_), yo_, str, font);
+       x_ += font_metrics::width(str, font);
 }
 
 
-void RowPainter::paintForeignMark(float const orig_x, LyXFont const & orig_font)
+void RowPainter::paintForeignMark(double orig_x, LyXFont const & font)
 {
        if (!lyxrc.mark_foreign_language)
                return;
-       if (orig_font.language() == latex_language)
+       if (font.language() == latex_language)
                return;
-       if (orig_font.language() == bv_.buffer()->params.language)
+       if (font.language() == bv_.buffer()->params().language)
                return;
 
-       int const y = yo_ + row_.baseline() + 1;
+       int const y = yo_ + 1;
        pain_.line(int(orig_x), y, int(x_), y, LColor::language);
 }
 
 
-bool RowPainter::paintFromPos(pos_type & vpos)
+void RowPainter::paintFromPos(pos_type & vpos)
 {
-       pos_type const pos = text_.vis2log(vpos);
-
-       LyXFont const & orig_font = getFont(pos);
-
-       float const orig_x = x_;
+       pos_type const pos = text_.bidi.vis2log(vpos);
+       LyXFont orig_font = text_.getFont(par_, pos);
+       text_.applyOuterFont(orig_font);
 
-       char const c = par_.getChar(pos);
+       double const orig_x = x_;
 
-       if (IsNewlineChar(c)) {
-               ++vpos;
-               paintNewline(pos);
-               return true;
-       } else if (IsInsetChar(c)) {
-               if (paintInset(pos))
-                       return true;
+       if (par_.isInset(pos)) {
+               paintInset(pos, orig_font);
                ++vpos;
                paintForeignMark(orig_x, orig_font);
-               return false;
+               return;
        }
 
        // usual characters, no insets
+       char const c = par_.getChar(pos);
 
        // special case languages
-       bool const hebrew = (orig_font.language()->lang() == "hebrew");
-       bool const arabic =
-               orig_font.language()->lang() == "arabic" &&
+       std::string const & lang = orig_font.language()->lang();
+       bool const hebrew = lang == "hebrew";
+       bool const arabic = lang == "arabic" &&
                (lyxrc.font_norm_type == LyXRC::ISO_8859_6_8 ||
                lyxrc.font_norm_type == LyXRC::ISO_10646_1);
 
@@ -343,208 +331,45 @@ bool RowPainter::paintFromPos(pos_type & vpos)
        if ((!hebrew && !arabic)
                || (hebrew && !Encodings::IsComposeChar_hebrew(c))
                || (arabic && !Encodings::IsComposeChar_arabic(c))) {
-               paintChars(vpos, hebrew, arabic);
+               paintChars(vpos, orig_font, hebrew, arabic);
        } else if (hebrew) {
-               paintHebrewComposeChar(vpos);
+               paintHebrewComposeChar(vpos, orig_font);
        } else if (arabic) {
-               paintArabicComposeChar(vpos);
+               paintArabicComposeChar(vpos, orig_font);
        }
 
        paintForeignMark(orig_x, orig_font);
-
-       return false;
 }
 
 
-bool RowPainter::paintBackground()
+void RowPainter::paintChangeBar()
 {
-       pos_type const last = row_.lastPrintablePos();
-       bool clear_area = true;
-       Inset const * inset = 0;
-
-       if (!bv_.screen().forceClear() && last == row_.pos()
-               && par_.isInset(row_.pos())) {
-               inset = par_.getInset(row_.pos());
-               clear_area = inset->doClearArea();
-       }
-
-       if (cleared_) {
-               return true;
-       }
-
-       if (clear_area) {
-               int const x = xo_;
-               int const y = yo_ < 0 ? 0 : yo_;
-               int const h = yo_ < 0 ? row_.height() + yo_ : row_.height();
-               pain_.fillRectangle(x, y, width_, h, text_.backgroundColor());
-               return true;
-       }
-
-       if (!inset)
-               return false;
-
-       LyXFont font(LyXFont::ALL_SANE);
-
-       // FIXME
-       BufferView * bv = perv(bv_);
-
-       int h = row_.baseline() - inset->ascent(bv, font);
-
-       // first clear the whole row above the inset!
-       if (h > 0) {
-               pain_.fillRectangle(xo_, yo_, width_, h, text_.backgroundColor());
-       }
-
-       // clear the space below the inset!
-       h += inset->ascent(bv, font) + inset->descent(bv, font);
-       if ((row_.height() - h) > 0) {
-               pain_.fillRectangle(xo_, yo_ + h,
-                       width_, row_.height() - h, text_.backgroundColor());
-       }
-
-       // clear the space behind the inset, if needed
-       if (!inset->display() && !inset->needFullRow()) {
-               int const xp = int(x_) + inset->width(bv, font);
-               if (width_ - xp > 0) {
-                       pain_.fillRectangle(xp, yo_, width_ - xp,
-                               row_.height(), text_.backgroundColor());
-               }
-       }
-
-       return false;
-}
-
+       pos_type const start = row_.pos();
+       pos_type const end = row_.endpos();
 
-void RowPainter::paintSelection()
-{
-       bool const is_rtl = par_.isRightToLeftPar(bv_.buffer()->params);
-
-       // the current selection
-       int const startx = text_.selection.start.x();
-       int const endx = text_.selection.end.x();
-       int const starty = text_.selection.start.y();
-       int const endy = text_.selection.end.y();
-       Row const * startrow = text_.selection.start.row();
-       Row const * endrow = text_.selection.end.row();
-
-       // Bleh.
-       Row const * row = &row_;
-
-       if (text_.bidi_same_direction) {
-               int x;
-               int y = yo_;
-               int w;
-               int h = row_.height();
-
-               if (startrow == row && endrow == row) {
-                       if (startx < endx) {
-                               x = xo_ + startx;
-                               w = endx - startx;
-                               pain_.fillRectangle(x, y, w, h, LColor::selection);
-                       } else {
-                               x = xo_ + endx;
-                               w = startx - endx;
-                               pain_.fillRectangle(x, y, w, h, LColor::selection);
-                       }
-               } else if (startrow == row) {
-                       int const x = (is_rtl) ? xo_ : (xo_ + startx);
-                       int const w = (is_rtl) ? startx : (width_ - startx);
-                       pain_.fillRectangle(x, y, w, h, LColor::selection);
-               } else if (endrow == row) {
-                       int const x = (is_rtl) ? (xo_ + endx) : xo_;
-                       int const w = (is_rtl) ? (width_ - endx) : endx;
-                       pain_.fillRectangle(x, y, w, h, LColor::selection);
-               } else if (y_ > starty && y_ < endy) {
-                       pain_.fillRectangle(xo_, y, width_, h, LColor::selection);
-               }
-               return;
-       } else if (startrow != row && endrow != row) {
-               if (y_ > starty && y_ < endy) {
-                       int w = width_;
-                       int h = row_.height();
-                       pain_.fillRectangle(xo_, yo_, w, h, LColor::selection);
-               }
+       if (start == end || !par_.isChanged(start, end - 1))
                return;
-       }
 
-       if ((startrow != row && !is_rtl) || (endrow != row && is_rtl))
-               pain_.fillRectangle(xo_, yo_, int(x_), row_.height(), LColor::selection);
+       int const height = text_.isLastRow(pit_, row_)
+               ? row_.ascent()
+               : row_.height();
 
-       pos_type const main_body = par_.beginningOfMainBody();
-       pos_type const last = row_.lastPrintablePos();
-       float tmpx = x_;
-
-       for (pos_type vpos = row_.pos(); vpos <= last; ++vpos)  {
-               pos_type pos = text_.vis2log(vpos);
-               float const old_tmpx = tmpx;
-               if (main_body > 0 && pos == main_body - 1) {
-                       LyXLayout_ptr const & layout = par_.layout();
-                       LyXFont const lfont = getLabelFont();
-
-                       tmpx += label_hfill_ + font_metrics::width(layout->labelsep, lfont);
-
-                       if (par_.isLineSeparator(main_body - 1))
-                               tmpx -= singleWidth(main_body - 1);
-               }
-
-               if (row_.hfillExpansion(pos)) {
-                       tmpx += singleWidth(pos);
-                       if (pos >= main_body)
-                               tmpx += hfill_;
-                       else
-                               tmpx += label_hfill_;
-               }
-
-               else if (par_.isSeparator(pos)) {
-                       tmpx += singleWidth(pos);
-                       if (pos >= main_body)
-                               tmpx += separator_;
-               } else {
-                       tmpx += singleWidth(pos);
-               }
-
-               if ((startrow != row || text_.selection.start.pos() <= pos) &&
-                       (endrow != row || pos < text_.selection.end.pos())) {
-                       // Here we do not use x_ as xo_ was added to x_.
-                       pain_.fillRectangle(int(old_tmpx), yo_,
-                               int(tmpx - old_tmpx + 1),
-                               row_.height(), LColor::selection);
-               }
-       }
-
-       if ((startrow != row && is_rtl) || (endrow != row && !is_rtl)) {
-               pain_.fillRectangle(xo_ + int(tmpx),
-                                     yo_, int(bv_.workWidth() - tmpx),
-                                     row_.height(), LColor::selection);
-       }
+       pain_.fillRectangle(4, yo_ - row_.ascent(), 5, height, LColor::changebar);
 }
 
 
-void RowPainter::paintChangeBar()
+void RowPainter::paintAppendix()
 {
-       pos_type const start = row_.pos();
-       pos_type const end = row_.lastPrintablePos();
-
-       if (!par_.isChanged(start, end)) 
+       if (!par_.params().appendix())
                return;
-       int const height = (row_.next()
-               ? row_.height() + row_.next()->top_of_text() 
-               : row_.baseline());
-       pain_.fillRectangle(4, yo_, 5, height, LColor::changebar);
-}
 
-void RowPainter::paintAppendix()
-{
-       // FIXME: can be just width_ ?
-       int const ww = bv_.workWidth();
+       int y = yo_ - row_.ascent();
 
-       if (par_.params().appendix()) {
-               pain_.line(1, yo_, 1, yo_ + row_.height(), LColor::appendixline);
-               pain_.line(ww - 2, yo_, ww - 2, yo_ + row_.height(), LColor::appendixline);
-       }
+       if (par_.params().startOfAppendix())
+               y += 2 * defaultRowHeight();
+
+       pain_.line(1, y, 1, yo_ + row_.height(), LColor::appendix);
+       pain_.line(width_ - 2, y, width_ - 2, yo_ + row_.height(), LColor::appendix);
 }
 
 
@@ -556,136 +381,60 @@ void RowPainter::paintDepthBar()
                return;
 
        Paragraph::depth_type prev_depth = 0;
-       if (row_.previous())
-               prev_depth = row_.previous()->par()->getDepth();
-       Paragraph::depth_type next_depth = 0;
-       if (row_.next())
-               next_depth = row_.next()->par()->getDepth();
-
-       for (Paragraph::depth_type i = 1; i <= depth; ++i) {
-               int x = (PAPER_MARGIN / 5) * i + xo_;
-               // only consider the changebar space if we're drawing outer left
-               if (!xo_)
-                       x += CHANGEBAR_MARGIN;
-               int const h = yo_ + row_.height() - 1 - (i - next_depth - 1) * 3;
-
-               pain_.line(x, yo_, x, h, LColor::depthbar);
-
-               int const w = PAPER_MARGIN / 5;
-
-               if (i > prev_depth) {
-                       pain_.fillRectangle(x, yo_, w, 2, LColor::depthbar);
-               }
-               if (i > next_depth) {
-                       pain_.fillRectangle(x, h, w, 2, LColor::depthbar);
-               }
+       if (!text_.isFirstRow(pit_, row_)) {
+               pit_type pit2 = pit_;
+               if (row_.pos() == 0)
+                       --pit2;
+               prev_depth = pars_[pit2].getDepth();
        }
-}
 
-
-int getLengthMarkerHeight(BufferView const & bv, VSpace const & vsp)
-{
-       if (vsp.kind() == VSpace::NONE)
-               return 0;
-
-       int const arrow_size = 4;
-       int const space_size = int(vsp.inPixels(bv));
-
-       LyXFont font;
-       font.decSize();
-       int const min_size = max(3 * arrow_size,
-               font_metrics::maxAscent(font)
-               + font_metrics::maxDescent(font));
-
-       if (vsp.length().len().value() < 0.0)
-               return min_size;
-       else
-               return max(min_size, space_size);
-}
-
-
-int RowPainter::paintLengthMarker(string const & prefix, VSpace const & vsp, int start)
-{
-       if (vsp.kind() == VSpace::NONE)
-               return 0;
-
-       int const arrow_size = 4;
-       int const size = getLengthMarkerHeight(bv_, vsp);
-       int const end = start + size;
-
-       // the label to display (if any)
-       string str;
-       // y-values for top arrow
-       int ty1, ty2;
-       // y-values for bottom arrow
-       int by1, by2;
-
-       str = prefix + " (" + vsp.asLyXCommand() + ")";
-
-       if (vsp.kind() == VSpace::VFILL) {
-               ty1 = ty2 = start;
-               by1 = by2 = end;
-       } else {
-               // adding or removing space
-               bool const added = vsp.kind() != VSpace::LENGTH ||
-                                  vsp.length().len().value() > 0.0;
-               ty1 = added ? (start + arrow_size) : start;
-               ty2 = added ? start : (start + arrow_size);
-               by1 = added ? (end - arrow_size) : end;
-               by2 = added ? end : (end - arrow_size);
+       Paragraph::depth_type next_depth = 0;
+       if (!text_.isLastRow(pit_, row_)) {
+               pit_type pit2 = pit_;
+               if (row_.endpos() >= pars_[pit2].size())
+                       ++pit2;
+               next_depth = pars_[pit2].getDepth();
        }
 
-       int const leftx = xo_ + leftMargin();
-       int const midx = leftx + arrow_size;
-       int const rightx = midx + arrow_size;
-
-       // first the string
-       int w = 0;
-       int a = 0;
-       int d = 0;
-
-       LyXFont font;
-       font.setColor(LColor::added_space).decSize().decSize();
-       font_metrics::rectText(str, font, w, a, d);
-
-       pain_.rectText(leftx + 2 * arrow_size + 5,
-                        start + ((end - start) / 2) + d,
-                        str, font);
-
-       // top arrow
-       pain_.line(leftx, ty1, midx, ty2, LColor::added_space);
-       pain_.line(midx, ty2, rightx, ty1, LColor::added_space);
+       for (Paragraph::depth_type i = 1; i <= depth; ++i) {
+               int const w = nestMargin() / 5;
+               int x = int(xo_) + w * i;
+               // only consider the changebar space if we're drawing outermost text
+               if (text_.isMainText())
+                       x += changebarMargin();
 
-       // bottom arrow
-       pain_.line(leftx, by1, midx, by2, LColor::added_space);
-       pain_.line(midx, by2, rightx, by1, LColor::added_space);
+               int const starty = yo_ - row_.ascent();
+               int const h =  row_.height() - 1 - (i - next_depth - 1) * 3;
 
-       // joining line
-       pain_.line(midx, ty2, midx, by2, LColor::added_space);
+               pain_.line(x, starty, x, starty + h, LColor::depthbar);
 
-       return size;
+               if (i > prev_depth)
+                       pain_.fillRectangle(x, starty, w, 2, LColor::depthbar);
+               if (i > next_depth)
+                       pain_.fillRectangle(x, starty + h, w, 2, LColor::depthbar);
+       }
 }
 
 
-int RowPainter::paintPageBreak(string const & label, int y)
+int RowPainter::paintAppendixStart(int y)
 {
        LyXFont pb_font;
-       pb_font.setColor(LColor::pagebreak).decSize();
+       pb_font.setColor(LColor::appendix);
+       pb_font.decSize();
 
+       string const label = _("Appendix");
        int w = 0;
        int a = 0;
        int d = 0;
        font_metrics::rectText(label, pb_font, w, a, d);
 
-       int const text_start = xo_ + ((width_ - w) / 2);
+       int const text_start = int(xo_ + (width_ - w) / 2);
        int const text_end = text_start + w;
 
-       pain_.rectText(text_start, y + d, label, pb_font);
+       pain_.rectText(text_start, y + d, label, pb_font, LColor::none, LColor::none);
 
-       pain_.line(xo_, y, text_start, y,
-               LColor::pagebreak, Painter::line_onoffdash);
-       pain_.line(text_end, y, xo_ + width_, y,
-               LColor::pagebreak, Painter::line_onoffdash);
+       pain_.line(int(xo_ + 1), y, text_start, y, LColor::appendix);
+       pain_.line(text_end, y, int(xo_ + width_ - 2), y, LColor::appendix);
 
        return 3 * defaultRowHeight();
 }
@@ -695,150 +444,114 @@ void RowPainter::paintFirst()
 {
        ParagraphParameters const & parparams = par_.params();
 
-       // start of appendix?
-       if (parparams.startOfAppendix()) {
-               pain_.line(1, yo_, width_ - 2, yo_, LColor::appendixline);
-       }
-
        int y_top = 0;
 
-       // the top margin
-       if (!row_.previous() && text_.isTopLevel())
-               y_top += PAPER_MARGIN;
-
-       // draw a top pagebreak
-       if (parparams.pagebreakTop()) {
-               y_top += paintPageBreak(_("Page Break (top)"),
-                       yo_ + y_top + 2 * defaultRowHeight());
-       }
-
-       // draw the additional space if needed:
-       y_top += paintLengthMarker(_("Space above"), parparams.spaceTop(),
-                       yo_ + y_top);
+       // start of appendix?
+       if (parparams.startOfAppendix())
+               y_top += paintAppendixStart(yo_ - row_.ascent() + 2 * defaultRowHeight());
 
-       Buffer const * buffer = bv_.buffer();
+       Buffer const & buffer = *bv_.buffer();
 
        LyXLayout_ptr const & layout = par_.layout();
 
-       if (buffer->params.paragraph_separation == BufferParams::PARSEP_SKIP) {
-               if (par_.previous()) {
+       if (buffer.params().paragraph_separation == BufferParams::PARSEP_SKIP) {
+               if (pit_ != 0) {
                        if (layout->latextype == LATEX_PARAGRAPH
                                && !par_.getDepth()) {
-                               y_top += buffer->params.getDefSkip().inPixels(bv_);
+                               y_top += buffer.params().getDefSkip().inPixels(bv_);
                        } else {
-                               LyXLayout_ptr const & playout =
-                                       par_.previous()->layout();
+                               LyXLayout_ptr const & playout = pars_[pit_ - 1].layout();
                                if (playout->latextype == LATEX_PARAGRAPH
-                                       && !par_.previous()->getDepth()) {
+                                       && !pars_[pit_ - 1].getDepth()) {
                                        // is it right to use defskip here, too? (AS)
-                                       y_top += buffer->params.getDefSkip().inPixels(bv_);
+                                       y_top += buffer.params().getDefSkip().inPixels(bv_);
                                }
                        }
                }
        }
 
-       int const ww = bv_.workWidth();
-
-       // draw a top line
-       if (parparams.lineTop()) {
-               LyXFont font(LyXFont::ALL_SANE);
-               int const asc = font_metrics::ascent('x', getFont(0));
-
-               y_top += asc;
-
-               int const w = (text_.isInInset() ? text_.inset_owner->width(perv(bv_), font) : ww);
-               int const xp = static_cast<int>(text_.isInInset() ? xo_ : 0);
-               pain_.line(xp, yo_ + y_top, xp + w, yo_ + y_top,
-                       LColor::topline, Painter::line_solid,
-                       Painter::line_thick);
-
-               y_top += asc;
-       }
-
-       bool const is_rtl = par_.isRightToLeftPar(bv_.buffer()->params);
+       bool const is_rtl = text_.isRTL(par_);
+       bool const is_seq = isFirstInSequence(pit_, text_.paragraphs());
+       //lyxerr << "paintFirst: " << par_.id() << " is_seq: " << is_seq << std::endl;
 
        // should we print a label?
        if (layout->labeltype >= LABEL_STATIC
            && (layout->labeltype != LABEL_STATIC
-               || layout->latextype != LATEX_ENVIRONMENT
-               || par_.isFirstInSequence())) {
+                     || layout->latextype != LATEX_ENVIRONMENT
+                     || is_seq)) {
 
-               LyXFont font = getLabelFont();
-               if (!par_.getLabelstring().empty()) {
-                       float x = x_;
-                       string const str = par_.getLabelstring();
+               LyXFont const font = getLabelFont();
+               string const str = par_.getLabelstring();
+               if (!str.empty()) {
+                       double x = x_;
 
                        // this is special code for the chapter layout. This is
                        // printed in an extra row and has a pagebreak at
                        // the top.
-                       if (layout->labeltype == LABEL_COUNTER_CHAPTER) {
-                               if (buffer->params.secnumdepth >= 0) {
-                                       float spacing_val = 1.0;
-                                       if (!parparams.spacing().isDefault()) {
-                                               spacing_val = parparams.spacing().getValue();
-                                       } else {
-                                               spacing_val = buffer->params.spacing.getValue();
-                                       }
-
-                                       int const maxdesc =
-                                               int(font_metrics::maxDescent(font) * layout->spacing.getValue() * spacing_val)
-                                               + int(layout->parsep) * defaultRowHeight();
-
-                                       if (is_rtl) {
-                                               x = ww - leftMargin() -
-                                                       font_metrics::width(str, font);
-                                       }
-
-                                       pain_.text(int(x),
-                                               yo_ + row_.baseline() -
-                                               row_.ascent_of_text() - maxdesc,
-                                               str, font);
+                       if (layout->counter == "chapter") {
+                               double spacing_val = 1.0;
+                               if (!parparams.spacing().isDefault()) {
+                                       spacing_val = parparams.spacing().getValue();
+                               } else {
+                                       spacing_val = buffer.params().spacing().getValue();
+                               }
+
+                               int const labeladdon = int(font_metrics::maxHeight(font) * layout->spacing.getValue() * spacing_val);
+
+                               int const maxdesc = int(font_metrics::maxDescent(font) * layout->spacing.getValue() * spacing_val)
+                                       + int(layout->parsep) * defaultRowHeight();
+
+                               if (is_rtl) {
+                                       x = width_ - leftMargin() -
+                                               font_metrics::width(str, font);
                                }
+
+                               pain_.text(int(x), yo_ - maxdesc - labeladdon, str, font);
                        } else {
                                if (is_rtl) {
-                                       x = ww - leftMargin()
+                                       x = width_ - leftMargin()
                                                + font_metrics::width(layout->labelsep, font);
                                } else {
                                        x = x_ - font_metrics::width(layout->labelsep, font)
                                                - font_metrics::width(str, font);
                                }
 
-                               pain_.text(int(x), yo_ + row_.baseline(), str, font);
+                               pain_.text(int(x), yo_, str, font);
                        }
                }
 
        // the labels at the top of an environment.
        // More or less for bibliography
-       } else if (par_.isFirstInSequence() &&
+       } else if (is_seq &&
                (layout->labeltype == LABEL_TOP_ENVIRONMENT ||
                layout->labeltype == LABEL_BIBLIO ||
                layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT)) {
                LyXFont font = getLabelFont();
                if (!par_.getLabelstring().empty()) {
                        string const str = par_.getLabelstring();
-                       float spacing_val = 1.0;
-                       if (!parparams.spacing().isDefault()) {
+                       double spacing_val = 1.0;
+                       if (!parparams.spacing().isDefault())
                                spacing_val = parparams.spacing().getValue();
-                       } else {
-                               spacing_val = buffer->params.spacing.getValue();
-                       }
+                       else
+                               spacing_val = buffer.params().spacing().getValue();
+
+                       int const labeladdon = int(font_metrics::maxHeight(font) * layout->spacing.getValue() * spacing_val);
 
                        int maxdesc =
                                int(font_metrics::maxDescent(font) * layout->spacing.getValue() * spacing_val
                                + (layout->labelbottomsep * defaultRowHeight()));
 
-                       float x = x_;
+                       double x = x_;
                        if (layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT) {
-                               x = ((is_rtl ? leftMargin() : x_)
-                                        + ww - text_.rightMargin(*bv_.buffer(), row_)) / 2;
+                               if (is_rtl)
+                                       x = leftMargin();
+                               x += (width_ - text_.rightMargin(par_) - leftMargin()) / 2;
                                x -= font_metrics::width(str, font) / 2;
                        } else if (is_rtl) {
-                               x = ww - leftMargin() -
+                               x = width_ - leftMargin() -
                                        font_metrics::width(str, font);
                        }
-                       pain_.text(int(x), yo_ + row_.baseline()
-                                 - row_.ascent_of_text() - maxdesc,
-                                 str, font);
+                       pain_.text(int(x), yo_ - maxdesc - labeladdon, str, font);
                }
        }
 }
@@ -846,242 +559,313 @@ void RowPainter::paintFirst()
 
 void RowPainter::paintLast()
 {
-       ParagraphParameters const & parparams = par_.params();
-       int y_bottom = row_.height() - 1;
-
-       // the bottom margin
-       if (!row_.next() && text_.isTopLevel())
-               y_bottom -= PAPER_MARGIN;
-
-       int const ww = bv_.workWidth();
-
-       // draw a bottom pagebreak
-       if (parparams.pagebreakBottom()) {
-               y_bottom -= paintPageBreak(_("Page Break (bottom)"),
-                       yo_ + y_bottom - 2 * defaultRowHeight());
-       }
-
-       // draw the additional space if needed:
-       int const height = getLengthMarkerHeight(bv_, parparams.spaceBottom());
-       y_bottom -= paintLengthMarker(_("Space below"), parparams.spaceBottom(),
-                            yo_ + y_bottom - height);
-
-       // draw a bottom line
-       if (parparams.lineBottom()) {
-               LyXFont font(LyXFont::ALL_SANE);
-               int const asc = font_metrics::ascent('x',
-                       getFont(max(pos_type(0), par_.size() - 1)));
-
-               y_bottom -= asc;
-
-               int const w = (text_.isInInset() ? text_.inset_owner->width(perv(bv_), font) : ww);
-               int const xp = static_cast<int>(text_.isInInset() ? xo_ : 0);
-               int const y = yo_ + y_bottom;
-               pain_.line(xp, y, xp + w, y, LColor::topline, Painter::line_solid,
-                         Painter::line_thick);
-
-               y_bottom -= asc;
-       }
-
-       bool const is_rtl = par_.isRightToLeftPar(bv_.buffer()->params);
-       int const endlabel = par_.getEndLabel();
+       bool const is_rtl = text_.isRTL(par_);
+       int const endlabel = getEndLabel(pit_, text_.paragraphs());
 
        // draw an endlabel
        switch (endlabel) {
        case END_LABEL_BOX:
-       case END_LABEL_FILLED_BOX:
-       {
+       case END_LABEL_FILLED_BOX: {
                LyXFont const font = getLabelFont();
                int const size = int(0.75 * font_metrics::maxAscent(font));
-               int const y = (yo_ + row_.baseline()) - size;
-               int x = is_rtl ? LEFT_MARGIN : ww - PAPER_MARGIN - size;
+               int const y = yo_ - size;
+               int x = is_rtl ? nestMargin() + changebarMargin() : width_ - size;
 
-               if (row_.fill() <= size)
-                       x += (size - row_.fill() + 1) * (is_rtl ? -1 : 1);
+               if (width_ - int(row_.width()) <= size)
+                       x += (size - width_ + row_.width() + 1) * (is_rtl ? -1 : 1);
 
-               if (endlabel == END_LABEL_BOX) {
+               if (endlabel == END_LABEL_BOX)
                        pain_.rectangle(x, y, size, size, LColor::eolmarker);
-               } else {
+               else
                        pain_.fillRectangle(x, y, size, size, LColor::eolmarker);
-               }
                break;
        }
-       case END_LABEL_STATIC:
-       {
-#if 0
-               LyXFont font(LyXFont::ALL_SANE);
-               font = getLabelFont();
-#else
+
+       case END_LABEL_STATIC: {
                LyXFont font = getLabelFont();
-#endif
                string const & str = par_.layout()->endlabelstring();
-               int const x = is_rtl ?
-                       int(x_) - font_metrics::width(str, font)
-                       : ww - text_.rightMargin(*bv_.buffer(), row_) - row_.fill();
-               pain_.text(x, yo_ + row_.baseline(), str, font);
+               double const x = is_rtl ?
+                       x_ - font_metrics::width(str, font)
+                       : - text_.rightMargin(par_) - row_.width();
+               pain_.text(int(x), yo_, str, font);
                break;
        }
+
        case END_LABEL_NO_LABEL:
                break;
        }
 }
 
 
-bool RowPainter::paintText()
+void RowPainter::paintText()
 {
-       pos_type const last = row_.lastPrintablePos();
-       pos_type main_body = par_.beginningOfMainBody();
-       if (main_body > 0 &&
-               (main_body - 1 > last ||
-               !par_.isLineSeparator(main_body - 1))) {
-               main_body = 0;
+       pos_type const end = row_.endpos();
+       pos_type body_pos = par_.beginOfBody();
+       if (body_pos > 0 &&
+               (body_pos > end || !par_.isLineSeparator(body_pos - 1))) {
+               body_pos = 0;
        }
 
        LyXLayout_ptr const & layout = par_.layout();
 
        bool running_strikeout = false;
        bool is_struckout = false;
-       float last_strikeout_x = 0.0;
-       pos_type vpos = row_.pos();
-       while (vpos <= last) {
+       int last_strikeout_x = 0;
+
+       // Use font span to speed things up, see below 
+       FontSpan font_span;
+       LyXFont font;
+
+       for (pos_type vpos = row_.pos(); vpos < end; ) {
                if (x_ > bv_.workWidth())
                        break;
-               pos_type pos = text_.vis2log(vpos);
-               if (x_ + singleWidth(pos) < 0) {
-                       x_ += singleWidth(pos);
+
+               pos_type const pos = text_.bidi.vis2log(vpos);
+
+               if (pos >= par_.size()) {
                        ++vpos;
                        continue;
                }
+
+               // Use font span to speed things up, see above
+               if (vpos < font_span.first || vpos > font_span.last) {
+                       font_span = par_.fontSpan(vpos);
+                       font = text_.getFont(par_, vpos);
+               }
+
+               const int width_pos =
+                       text_.singleWidth(par_, pos, par_.getChar(pos), font);
+
+               if (x_ + width_pos < 0) {
+                       x_ += width_pos;
+                       ++vpos;
+                       continue;
+               }
+
                is_struckout = isDeletedText(par_, pos);
 
                if (is_struckout && !running_strikeout) {
                        running_strikeout = true;
-                       last_strikeout_x = x_;
+                       last_strikeout_x = int(x_);
                }
+
                bool const highly_editable_inset = par_.isInset(pos)
                        && isHighlyEditableInset(par_.getInset(pos));
 
-               // if we reach the end of a struck out range, paint it
-               // we also don't paint across things like tables
+               // If we reach the end of a struck out range, paint it.
+               // We also don't paint across things like tables
                if (running_strikeout && (highly_editable_inset || !is_struckout)) {
-                       int const middle = yo_ + row_.top_of_text()
-                               + ((row_.baseline() - row_.top_of_text()) / 2);
-                       pain_.line(int(last_strikeout_x), middle, int(x_), middle,
+                       // Calculate 1/3 height of the buffer's default font
+                       int const middle =
+                               yo_ - font_metrics::maxAscent(text_.defaultfont_) / 3;
+                       pain_.line(last_strikeout_x, middle, int(x_), middle,
                                LColor::strikeout, Painter::line_solid, Painter::line_thin);
                        running_strikeout = false;
                }
-               if (main_body > 0 && pos == main_body - 1) {
+
+               if (body_pos > 0 && pos == body_pos - 1) {
                        int const lwidth = font_metrics::width(layout->labelsep,
                                getLabelFont());
 
-                       x_ += label_hfill_ + lwidth
-                               - singleWidth(main_body - 1);
+                       x_ += label_hfill_ + lwidth - width_pos;
                }
 
                if (par_.isHfill(pos)) {
                        x_ += 1;
 
-                       int const y0 = yo_ + row_.baseline();
+                       int const y0 = yo_;
                        int const y1 = y0 - defaultRowHeight() / 2;
 
-                       pain_.line(int(x_), y1, int(x_), y0,
-                                    LColor::added_space);
+                       pain_.line(int(x_), y1, int(x_), y0, LColor::added_space);
 
-                       if (row_.hfillExpansion(pos)) {
+                       if (hfillExpansion(par_, row_, pos)) {
                                int const y2 = (y0 + y1) / 2;
 
-                               if (pos >= main_body) {
-                                       pain_.line(int(x_), y2,
-                                                 int(x_ + hfill_), y2,
+                               if (pos >= body_pos) {
+                                       pain_.line(int(x_), y2, int(x_ + hfill_), y2,
                                                  LColor::added_space,
                                                  Painter::line_onoffdash);
                                        x_ += hfill_;
                                } else {
-                                       pain_.line(int(x_), y2,
-                                                 int(x_ + label_hfill_), y2,
+                                       pain_.line(int(x_), y2, int(x_ + label_hfill_), y2,
                                                  LColor::added_space,
                                                  Painter::line_onoffdash);
                                        x_ += label_hfill_;
                                }
-                               pain_.line(int(x_), y1,
-                                            int(x_), y0,
-                                            LColor::added_space);
+                               pain_.line(int(x_), y1, int(x_), y0, LColor::added_space);
                        }
                        x_ += 2;
                        ++vpos;
                } else if (par_.isSeparator(pos)) {
-                       x_ += singleWidth(pos);
-                       if (pos >= main_body)
+                       x_ += width_pos;
+                       if (pos >= body_pos)
                                x_ += separator_;
                        ++vpos;
                } else {
-                       if (paintFromPos(vpos))
-                               return true;    
+                       paintFromPos(vpos);
                }
        }
+
        // if we reach the end of a struck out range, paint it
        if (running_strikeout) {
-               int const middle = yo_ + row_.top_of_text()
-                       + ((row_.baseline() - row_.top_of_text()) / 2);
-               pain_.line(int(last_strikeout_x), middle, int(x_), middle,
+               // calculate 1/3 height of the buffer's default font
+               int const middle =
+                       yo_ - font_metrics::maxAscent(text_.defaultfont_) / 3;
+               pain_.line(last_strikeout_x, middle, int(x_), middle,
                        LColor::strikeout, Painter::line_solid, Painter::line_thin);
                running_strikeout = false;
        }
+}
+
+
+lyx::size_type calculateRowSignature(Row const & row, Paragraph const & par)
+{
+       boost::crc_32_type crc;
+       for (lyx::pos_type i = row.pos(); i < row.endpos(); ++i) {
+               const unsigned char b[] = { par.getChar(i) };
+               crc.process_bytes(b, 1);
+       }
+       return crc.checksum();
+}
+
+
+bool isCursorOnRow(PainterInfo & pi, pit_type pit, RowList::const_iterator rit)
+{
+       LCursor & cur = pi.base.bv->cursor();
+       for (lyx::size_type d = 0; d < cur.depth(); d++)
+               if (cur[d].pit() == pit
+                   && cur[d].pos() >= rit->pos()
+                   && cur[d].pos() <= rit->endpos())
+                       return true;
        return false;
 }
 
 
-bool RowPainter::paint(int y_offset, int x_offset, int y, bool cleared)
+void paintPar
+       (PainterInfo & pi, LyXText const & text, pit_type pit, int x, int y,
+        bool repaintAll)
+{
+//     lyxerr << "  paintPar: pit: " << pit << " at y: " << y << endl;
+       static NullPainter nop;
+       static PainterInfo nullpi(pi.base.bv, nop);
+       int const ww = pi.base.bv->workHeight();
+
+       Paragraph const & par = text.paragraphs()[pit];
+
+       RowList::const_iterator const rb = par.rows().begin();
+       RowList::const_iterator const re = par.rows().end();
+       theCoords.parPos()[&text][pit] = Point(x, y);
+
+       y -= rb->ascent();
+       lyx::size_type rowno(0);
+       for (RowList::const_iterator rit = rb; rit != re; ++rit, ++rowno) {
+               y += rit->ascent();
+
+               // Row signature; has row changed since last paint?
+               lyx::size_type const row_sig = calculateRowSignature(*rit, par);
+
+               bool cursor_on_row = isCursorOnRow(pi, pit, rit);
+               
+               // If selection is on, the current row signature differs from
+               // from cache, or cursor is inside an inset _on this row_, 
+               // then paint the row
+               if (repaintAll || par.rowSignature()[rowno] != row_sig
+                           || cursor_on_row) {
+                       // Add to row signature cache
+                       par.rowSignature()[rowno] = row_sig;
+
+                       bool const inside = (y + rit->descent() >= 0
+                                      && y - rit->ascent() < ww);
+                       RowPainter rp(inside ? pi : nullpi, text, pit, *rit, x, y);
+                       // Clear background of this row 
+                       // (if paragraph background was not cleared)
+                       if (!repaintAll) {
+                               pi.pain.fillRectangle(x, y - rit->ascent(), 
+                                   pi.base.bv->workWidth(), rit->height(),
+                                   text.backgroundColor());
+                       }
+                       
+                       // Instrumentation for testing row cache (see also
+                       // 12 lines lower):
+                       //lyxerr << "#";
+                       rp.paintAppendix();
+                       rp.paintDepthBar();
+                       rp.paintChangeBar();
+                       if (rit == rb)
+                               rp.paintFirst();
+                       if (rit + 1 == re)
+                               rp.paintLast();
+                       rp.paintText();
+               }
+               y += rit->descent();
+       }
+       //lyxerr << "." << endl;
+}
+
+} // namespace anon
+
+
+void paintText(BufferView const & bv, ViewMetricsInfo const & vi)
 {
-       xo_ = x_offset;
-       yo_ = y_offset;
-       y_ = y;
-       cleared_ = cleared;
-       width_ = text_.isInInset()
-               ? text_.inset_owner->textWidth(perv(bv_), true) : bv_.workWidth();
-
-       // FIXME: must be a cleaner way here. Aren't these calculations
-       // belonging to row metrics ?
-       BufferView * bv(const_cast<BufferView *>(&bv_));
-       Row * row(const_cast<Row *>(&row_));
-       text_.prepareToPrint(bv, row, x_, separator_, hfill_, label_hfill_);
-
-       // FIXME: what is this fixing ?
-       if (text_.isInInset() && (x_ < 0))
-               x_ = 0;
-       x_ += xo_;
-
-       // clear to background if necessary
-       cleared_ = paintBackground();
-
-       // paint the selection background
-       if (text_.selection.set()) {
-               paintSelection();
+       Painter & pain = bv.painter();
+       LyXText * const text = bv.text();
+       bool const select = bv.cursor().selection();
+
+       PainterInfo pi(const_cast<BufferView *>(&bv), pain);
+       if (select || !vi.singlepar) {
+               // Clear background (Delegated to rows if no selection)
+               pain.fillRectangle(0, vi.y1, bv.workWidth(), vi.y2 - vi.y1,
+                       text->backgroundColor());
+       }
+       if (select) {
+               text->drawSelection(pi, 0, 0);
        }
 
-       // vertical lines for appendix
-       paintAppendix();
+       int yy = vi.y1;
+       // draw contents
+       for (pit_type pit = vi.p1; pit <= vi.p2; ++pit) {
+               Paragraph const & par = text->getPar(pit);
+               yy += par.ascent();
+               paintPar(pi, *bv.text(), pit, 0, yy, select || !vi.singlepar);
+               yy += par.descent();
+       }
 
-       // environment depth brackets
-       paintDepthBar();
+       // Cache one paragraph above and one below
+       // Note MV: this cannot be suppressed even for singlepar.
+       // Try viewing the User Guide Mobius figure
 
-       // changebar
-       paintChangeBar();
-       if (row_.isParStart()) {
-               paintFirst();
+       if (vi.p1 > 0) {
+               text->redoParagraph(vi.p1 - 1);
+               theCoords.parPos()[bv.text()][vi.p1 - 1] = 
+                       Point(0, vi.y1 - text->getPar(vi.p1 - 1).descent());
        }
 
-       if (row_.isParEnd()) {
-               paintLast();
+       if (vi.p2 < lyx::pit_type(text->paragraphs().size()) - 1) {
+               text->redoParagraph(vi.p2 + 1);
+               theCoords.parPos()[bv.text()][vi.p2 + 1] = 
+                       Point(0, vi.y2 + text->getPar(vi.p2 + 1).ascent());
        }
 
-       // paint text
-       return paintText();
+       // and grey out above (should not happen later)
+//     lyxerr << "par ascent: " << text->getPar(vi.p1).ascent() << endl;
+       if (vi.y1 > 0 && !vi.singlepar)
+               pain.fillRectangle(0, 0, bv.workWidth(), vi.y1, LColor::bottomarea);
+
+       // and possibly grey out below
+//     lyxerr << "par descent: " << text->getPar(vi.p1).ascent() << endl;
+       if (vi.y2 < bv.workHeight() && !vi.singlepar)
+               pain.fillRectangle(0, vi.y2, bv.workWidth(), bv.workHeight() - vi.y2, LColor::bottomarea);
+}
+
+
+void paintTextInset(LyXText const & text, PainterInfo & pi, int x, int y)
+{
+//     lyxerr << "  paintTextInset: y: " << y << endl;
+
+       y -= text.getPar(0).ascent();
+       for (int pit = 0; pit < int(text.paragraphs().size()); ++pit) {
+               y += text.getPar(pit).ascent();
+               paintPar(pi, text, pit, x, y, true);
+               y += text.getPar(pit).descent();
+       }
 }