]> git.lyx.org Git - lyx.git/blobdiff - src/rowpainter.C
Point fix, earlier forgotten
[lyx.git] / src / rowpainter.C
index db2cc8135a203ae786e1a13691c4c0055c27ca50..a59e8385bbda8be267fc7b6cb4aaada930b43ce6 100644 (file)
@@ -6,34 +6,39 @@
  * \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 "paragraph_funcs.h"
 #include "support/textutils.h"
 
 #include "insets/insettext.h"
 #include "ParagraphParameters.h"
+#include "debug.h"
 #include "BufferView.h"
 #include "buffer.h"
 #include "gettext.h"
 #include "language.h"
 #include "encoding.h"
 #include "lyxtext.h"
-#include "lyxrow.h"
 #include "rowpainter.h"
 #include "lyxrc.h"
+#include "lyxrow_funcs.h"
+#include "metricsinfo.h"
+
+#include <algorithm>
 
+using namespace lyx::support;
 
 using std::max;
+using std::endl;
 using lyx::pos_type;
 
 extern int PAPER_MARGIN;
@@ -50,76 +55,136 @@ BufferView * perv(BufferView const & bv)
        return const_cast<BufferView *>(&bv);
 }
 
-} // namespace anon
-
-
-RowPainter::RowPainter(BufferView const & bv, LyXText const & text, Row const & row)
-       : bv_(bv), pain_(bv_.painter()), text_(text), row_(row), par_(*row.par())
-{
-}
+/**
+ * A class used for painting an individual row of text.
+ */
+class RowPainter {
+public:
+       /// initialise painter
+       RowPainter(BufferView const & bv, LyXText const & text,
+               ParagraphList::iterator pit,
+               RowList::iterator rit, int y_offset, int x_offset, int y);
+
+       /// do the painting
+       void paint();
+private:
+       // paint various parts
+       void paintBackground();
+       void paintSelection();
+       void paintAppendix();
+       void paintDepthBar();
+       void paintChangeBar();
+       void paintFirst();
+       void paintLast();
+       void paintForeignMark(double orig_x, LyXFont const & orig_font);
+       void paintHebrewComposeChar(lyx::pos_type & vpos);
+       void paintArabicComposeChar(lyx::pos_type & vpos);
+       void paintChars(lyx::pos_type & vpos, bool hebrew, bool arabic);
+       int paintPageBreak(string const & label, int y);
+       int paintAppendixStart(int y);
+       int paintLengthMarker(string const & prefix, VSpace const & vsp, int start);
+       void paintText();
+       void paintFromPos(lyx::pos_type & vpos);
+       void paintInset(lyx::pos_type const pos);
+
+       /// return left margin
+       int leftMargin() const;
+
+       /// return the font at the given pos
+       LyXFont const getFont(lyx::pos_type pos) const;
+
+       /// return the label font for this row
+       LyXFont const getLabelFont() const;
+
+       char const transformChar(char c, lyx::pos_type pos) const;
+
+       /// return pixel width for the given pos
+       int singleWidth(lyx::pos_type pos) const;
+       int singleWidth(lyx::pos_type pos, char c) const;
+
+       /// bufferview to paint on
+       BufferView const & bv_;
+
+       /// Painter to use
+       Painter & pain_;
+
+       /// LyXText for the row
+       LyXText const & text_;
+
+       /// The row to paint
+       RowList::iterator row_;
+
+       /// Row's paragraph
+       mutable ParagraphList::iterator  pit_;
+
+       // Looks ugly - is
+       double xo_;
+       int yo_;
+       double x_;
+       int y_;
+       int width_;
+       double separator_;
+       double hfill_;
+       double label_hfill_;
+};
+
+
+RowPainter::RowPainter(BufferView const & bv, LyXText const & text,
+     ParagraphList::iterator pit, RowList::iterator rit,
+     int y_offset, int x_offset, int y)
+       : bv_(bv), pain_(bv_.painter()), text_(text), row_(rit),
+         pit_(pit), xo_(x_offset), yo_(y_offset), y_(y)
+{}
 
 
 /// "temporary"
 LyXFont const RowPainter::getFont(pos_type pos) const
 {
-       Paragraph * perverted_par = const_cast<Paragraph *>(&par_);
-       return text_.getFont(bv_.buffer(), perverted_par, pos);
+       return text_.getFont(pit_, pos);
 }
 
 
 int RowPainter::singleWidth(lyx::pos_type pos) const
 {
-       BufferView * bv(perv(bv_));
-       Paragraph * par(const_cast<Paragraph*>(&par_));
-       return text_.singleWidth(bv, par, pos);
+       return text_.singleWidth(pit_, pos);
 }
 
 
 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);
+       LyXFont const & font = text_.getFont(pit_, pos);
+       return text_.singleWidth(pit_, pos, c, font);
 }
 
 
 LyXFont const RowPainter::getLabelFont() const
 {
-       Paragraph * par(const_cast<Paragraph*>(&par_));
-       return text_.getLabelFont(bv_.buffer(), par);
+       return text_.getLabelFont(pit_);
 }
 
 
 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_.transformChar(c, *pit_, pos);
 }
 
 
 int RowPainter::leftMargin() const
 {
-       BufferView * bv(perv(bv_));
-       Row * row(const_cast<Row *>(&row_));
-       return text_.leftMargin(bv, row);
+       return text_.leftMargin(pit_, *row_);
 }
 
 
-bool RowPainter::paintInset(pos_type const pos)
+void RowPainter::paintInset(pos_type const pos)
 {
-       Inset * inset = const_cast<Inset*>(par_.getInset(pos));
-
-       lyx::Assert(inset);
+       InsetOld * inset = const_cast<InsetOld*>(pit_->getInset(pos));
 
-       LyXFont const & font = getFont(pos);
-
-       inset->update(perv(bv_), font, false);
-       inset->draw(perv(bv_), font, yo_ + row_.baseline(), x_);
-
-       // return true if something changed when we drew an inset
+       Assert(inset);
 
-       return (!text_.need_break_row && !text_.isInInset()
-           && bv_.text->status() == LyXText::CHANGED_IN_DRAW);
+       PainterInfo pi(perv(bv_));
+       pi.base.font = getFont(pos);
+       inset->draw(pi, int(x_), yo_ + row_->baseline());
+       x_ += inset->width();
 }
 
 
@@ -130,7 +195,7 @@ void RowPainter::paintHebrewComposeChar(pos_type & vpos)
        string str;
 
        // first char
-       char c = par_.getChar(pos);
+       char c = pit_->getChar(pos);
        str += c;
        ++vpos;
 
@@ -139,7 +204,7 @@ void RowPainter::paintHebrewComposeChar(pos_type & vpos)
        int dx = 0;
 
        for (pos_type i = pos - 1; i >= 0; --i) {
-               c = par_.getChar(i);
+               c = pit_->getChar(i);
                if (!Encodings::IsComposeChar_hebrew(c)) {
                        if (IsPrintableNonspace(c)) {
                                int const width2 =
@@ -154,7 +219,7 @@ void RowPainter::paintHebrewComposeChar(pos_type & vpos)
        }
 
        // Draw nikud
-       pain_.text(int(x_) + dx, yo_ + row_.baseline(), str, font);
+       pain_.text(int(x_) + dx, yo_ + row_->baseline(), str, font);
 }
 
 
@@ -164,7 +229,7 @@ void RowPainter::paintArabicComposeChar(pos_type & vpos)
        string str;
 
        // first char
-       char c = par_.getChar(pos);
+       char c = pit_->getChar(pos);
        c = transformChar(c, pos);
        str +=c;
        ++vpos;
@@ -173,8 +238,8 @@ void RowPainter::paintArabicComposeChar(pos_type & vpos)
        int const width = font_metrics::width(c, font);
        int dx = 0;
 
-       for (pos_type i = pos-1; i >= 0; --i) {
-               c = par_.getChar(i);
+       for (pos_type i = pos - 1; i >= 0; --i) {
+               c = pit_->getChar(i);
                if (!Encodings::IsComposeChar_arabic(c)) {
                        if (IsPrintableNonspace(c)) {
                                int const width2 =
@@ -185,40 +250,40 @@ void RowPainter::paintArabicComposeChar(pos_type & vpos)
                }
        }
        // Draw nikud
-       pain_.text(int(x_) + dx, yo_ + row_.baseline(), str, font);
+       pain_.text(int(x_) + dx, yo_ + row_->baseline(), str, font);
 }
 
 
 void RowPainter::paintChars(pos_type & vpos, bool hebrew, bool arabic)
 {
        pos_type pos = text_.vis2log(vpos);
-       pos_type const last = row_.lastPrintablePos();
-       LyXFont orig_font(getFont(pos));
+       pos_type const last = lastPrintablePos(*pit_, row_);
+       LyXFont orig_font = getFont(pos);
 
        // first character
        string str;
-       str += par_.getChar(pos);
+       str += pit_->getChar(pos);
        if (arabic) {
                unsigned char c = str[0];
                str[0] = transformChar(c, pos);
        }
 
-       bool prev_struckout(isDeletedText(par_, pos));
-       bool prev_newtext(isInsertedText(par_, pos));
+       bool prev_struckout(isDeletedText(*pit_, pos));
+       bool prev_newtext(isInsertedText(*pit_, pos));
 
        ++vpos;
 
        // collect as much similar chars as we can
        while (vpos <= last && (pos = text_.vis2log(vpos)) >= 0) {
-               char c = par_.getChar(pos);
+               char c = pit_->getChar(pos);
 
                if (!IsPrintableNonspace(c))
                        break;
 
-               if (prev_struckout != isDeletedText(par_, pos))
+               if (prev_struckout != isDeletedText(*pit_, pos))
                        break;
 
-               if (prev_newtext != isInsertedText(par_, pos))
+               if (prev_newtext != isInsertedText(*pit_, pos))
                        break;
 
                if (arabic && Encodings::IsComposeChar_arabic(c))
@@ -242,12 +307,14 @@ void RowPainter::paintChars(pos_type & vpos, bool hebrew, bool arabic)
        }
 
        // Draw text and set the new x position
-       pain_.text(int(x_), yo_ + row_.baseline(), str, orig_font);
+       //lyxerr << "paint row: yo_ " << yo_ << " baseline: " << row_->baseline()
+       //      << "\n";
+       pain_.text(int(x_), yo_ + row_->baseline(), str, orig_font);
        x_ += font_metrics::width(str, orig_font);
 }
 
 
-void RowPainter::paintForeignMark(float const orig_x, LyXFont const & orig_font)
+void RowPainter::paintForeignMark(double orig_x, LyXFont const & orig_font)
 {
        if (!lyxrc.mark_foreign_language)
                return;
@@ -256,27 +323,26 @@ void RowPainter::paintForeignMark(float const orig_x, LyXFont const & orig_font)
        if (orig_font.language() == bv_.buffer()->params.language)
                return;
 
-       int const y = yo_ + row_.baseline() + 1;
+       int const y = yo_ + row_->baseline() + 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_;
+       double const orig_x = x_;
 
-       char const c = par_.getChar(pos);
+       char const c = pit_->getChar(pos);
 
        if (IsInsetChar(c)) {
-               if (paintInset(pos))
-                       return true;
+               paintInset(pos);
                ++vpos;
                paintForeignMark(orig_x, orig_font);
-               return false;
+               return;
        }
 
        // usual characters, no insets
@@ -301,92 +367,90 @@ bool RowPainter::paintFromPos(pos_type & vpos)
 
        paintForeignMark(orig_x, orig_font);
 
-       return false;
+       return;
 }
 
 
 void RowPainter::paintBackground()
 {
-       int const x = xo_;
+       int const x = int(xo_);
        int const y = yo_ < 0 ? 0 : yo_;
-       int const h = yo_ < 0 ? row_.height() + yo_ : row_.height();
+       int const h = yo_ < 0 ? row_->height() + yo_ : row_->height();
        pain_.fillRectangle(x, y, width_, h, text_.backgroundColor());
 }
 
 
 void RowPainter::paintSelection()
 {
-       bool const is_rtl = par_.isRightToLeftPar(bv_.buffer()->params);
+       bool const is_rtl = pit_->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_;
+       RowList::iterator startrow = text_.getRow(text_.selection.start);
+       RowList::iterator endrow = text_.getRow(text_.selection.end);
 
        if (text_.bidi_same_direction) {
                int x;
                int y = yo_;
                int w;
-               int h = row_.height();
+               int h = row_->height();
 
-               if (startrow == row && endrow == row) {
+               if (startrow == row_ && endrow == row_) {
                        if (startx < endx) {
-                               x = xo_ + startx;
+                               x = int(xo_) + startx;
                                w = endx - startx;
                                pain_.fillRectangle(x, y, w, h, LColor::selection);
                        } else {
-                               x = xo_ + endx;
+                               x = int(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);
+               } else if (startrow == row_) {
+                       int const x = is_rtl ? int(xo_) : int(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;
+               } else if (endrow == row_) {
+                       int const x = is_rtl ? int(xo_ + endx) : int(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);
+                       pain_.fillRectangle(int(xo_), y, width_, h, LColor::selection);
                }
                return;
-       } else if (startrow != row && endrow != row) {
+       } 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);
+                       int h = row_->height();
+                       pain_.fillRectangle(int(xo_), yo_, w, h, LColor::selection);
                }
                return;
        }
 
-       if ((startrow != row && !is_rtl) || (endrow != row && is_rtl))
-               pain_.fillRectangle(xo_, yo_, int(x_), row_.height(), LColor::selection);
+       if ((startrow != row_ && !is_rtl) || (endrow != row_ && is_rtl))
+               pain_.fillRectangle(int(xo_), yo_,
+                       int(x_), row_->height(), LColor::selection);
 
-       pos_type const body_pos = par_.beginningOfBody();
-       pos_type const last = row_.lastPrintablePos();
-       float tmpx = x_;
+       pos_type const body_pos = pit_->beginningOfBody();
+       pos_type const last = lastPrintablePos(*pit_, row_);
+       double tmpx = x_;
 
-       for (pos_type vpos = row_.pos(); vpos <= last; ++vpos)  {
+       for (pos_type vpos = row_->pos(); vpos <= last; ++vpos)  {
                pos_type pos = text_.vis2log(vpos);
-               float const old_tmpx = tmpx;
+               double const old_tmpx = tmpx;
                if (body_pos > 0 && pos == body_pos - 1) {
-                       LyXLayout_ptr const & layout = par_.layout();
+                       LyXLayout_ptr const & layout = pit_->layout();
                        LyXFont const lfont = getLabelFont();
 
                        tmpx += label_hfill_ + font_metrics::width(layout->labelsep, lfont);
 
-                       if (par_.isLineSeparator(body_pos - 1))
+                       if (pit_->isLineSeparator(body_pos - 1))
                                tmpx -= singleWidth(body_pos - 1);
                }
 
-               if (row_.hfillExpansion(pos)) {
+               if (hfillExpansion(*pit_, row_, pos)) {
                        tmpx += singleWidth(pos);
                        if (pos >= body_pos)
                                tmpx += hfill_;
@@ -394,7 +458,7 @@ void RowPainter::paintSelection()
                                tmpx += label_hfill_;
                }
 
-               else if (par_.isSeparator(pos)) {
+               else if (pit_->isSeparator(pos)) {
                        tmpx += singleWidth(pos);
                        if (pos >= body_pos)
                                tmpx += separator_;
@@ -402,34 +466,34 @@ void RowPainter::paintSelection()
                        tmpx += singleWidth(pos);
                }
 
-               if ((startrow != row || text_.selection.start.pos() <= pos) &&
-                       (endrow != row || pos < text_.selection.end.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);
+                               row_->height(), LColor::selection);
                }
        }
 
-       if ((startrow != row && is_rtl) || (endrow != row && !is_rtl)) {
-               pain_.fillRectangle(xo_ + int(tmpx),
+       if ((startrow != row_ && is_rtl) || (endrow != row_ && !is_rtl)) {
+               pain_.fillRectangle(int(xo_ + tmpx),
                                      yo_, int(bv_.workWidth() - tmpx),
-                                     row_.height(), LColor::selection);
+                                     row_->height(), LColor::selection);
        }
 }
 
 
 void RowPainter::paintChangeBar()
 {
-       pos_type const start = row_.pos();
-       pos_type const end = row_.lastPrintablePos();
+       pos_type const start = row_->pos();
+       pos_type const end = lastPrintablePos(*pit_, row_);
 
-       if (!par_.isChanged(start, end))
+       if (!pit_->isChanged(start, end))
                return;
 
-       int const height = (row_.next()
-               ? row_.height() + row_.next()->top_of_text()
-               : row_.baseline());
+       int const height = (row_ == text_.lastRow())
+               ? row_->baseline()
+               : row_->height() + boost::next(row_)->top_of_text();
 
        pain_.fillRectangle(4, yo_, 5, height, LColor::changebar);
 }
@@ -437,7 +501,7 @@ void RowPainter::paintChangeBar()
 
 void RowPainter::paintAppendix()
 {
-       if (!par_.params().appendix())
+       if (!pit_->params().appendix())
                return;
 
        // FIXME: can be just width_ ?
@@ -445,70 +509,55 @@ void RowPainter::paintAppendix()
 
        int y = yo_;
 
-       if (par_.params().startOfAppendix())
+       if (pit_->params().startOfAppendix())
                y += 2 * defaultRowHeight();
 
-       pain_.line(1, y, 1, yo_ + row_.height(), LColor::appendix);
-       pain_.line(ww - 2, y, ww - 2, yo_ + row_.height(), LColor::appendix);
+       pain_.line(1, y, 1, yo_ + row_->height(), LColor::appendix);
+       pain_.line(ww - 2, y, ww - 2, yo_ + row_->height(), LColor::appendix);
 }
 
 
 void RowPainter::paintDepthBar()
 {
-       Paragraph::depth_type const depth = par_.getDepth();
+       Paragraph::depth_type const depth = pit_->getDepth();
 
        if (depth <= 0)
                return;
 
        Paragraph::depth_type prev_depth = 0;
-       if (row_.previous())
-               prev_depth = row_.previous()->par()->getDepth();
+       if (row_ != text_.firstRow()) {
+               ParagraphList::iterator pit2 = pit_;
+               if (row_ == pit2->rows.begin())
+                       --pit2;
+               prev_depth = pit2->getDepth();
+       }
+
        Paragraph::depth_type next_depth = 0;
-       if (row_.next())
-               next_depth = row_.next()->par()->getDepth();
+       if (row_ != text_.lastRow()) {
+               ParagraphList::iterator pit2 = pit_;
+               if (boost::next(row_) == pit2->rows.end())
+                       ++pit2;
+               next_depth = pit2->getDepth();
+       }
 
        for (Paragraph::depth_type i = 1; i <= depth; ++i) {
-               int x = (PAPER_MARGIN / 5) * i + xo_;
+               int const w = PAPER_MARGIN / 5;
+               int x = int(w * 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;
+               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) {
+               if (i > prev_depth)
                        pain_.fillRectangle(x, yo_, w, 2, LColor::depthbar);
-               }
-               if (i > next_depth) {
+               if (i > next_depth)
                        pain_.fillRectangle(x, h, w, 2, LColor::depthbar);
-               }
        }
 }
 
 
-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)
@@ -540,7 +589,7 @@ int RowPainter::paintLengthMarker(string const & prefix, VSpace const & vsp, int
                by2 = added ? end : (end - arrow_size);
        }
 
-       int const leftx = xo_ + leftMargin();
+       int const leftx = int(xo_) + leftMargin();
        int const midx = leftx + arrow_size;
        int const rightx = midx + arrow_size;
 
@@ -582,14 +631,14 @@ int RowPainter::paintPageBreak(string const & label, int y)
        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_.line(xo_, y, text_start, y,
+       pain_.line(int(xo_), y, text_start, y,
                LColor::pagebreak, Painter::line_onoffdash);
-       pain_.line(text_end, y, xo_ + width_, y,
+       pain_.line(text_end, y, int(xo_ + width_), y,
                LColor::pagebreak, Painter::line_onoffdash);
 
        return 3 * defaultRowHeight();
@@ -607,13 +656,13 @@ int RowPainter::paintAppendixStart(int y)
        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_.line(xo_ + 1, y, text_start, y, LColor::appendix);
-       pain_.line(text_end, y, xo_ + width_ - 2, y, LColor::appendix);
+       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();
 }
@@ -621,45 +670,43 @@ int RowPainter::paintAppendixStart(int y)
 
 void RowPainter::paintFirst()
 {
-       ParagraphParameters const & parparams = par_.params();
+       ParagraphParameters const & parparams = pit_->params();
 
        int y_top = 0;
 
        // start of appendix?
-       if (parparams.startOfAppendix()) {
+       if (parparams.startOfAppendix())
                y_top += paintAppendixStart(yo_ + y_top + 2 * defaultRowHeight());
-       }
 
        // the top margin
-       if (!row_.previous() && text_.isTopLevel())
+       if (row_ == text_.firstRow() && !text_.isInInset())
                y_top += PAPER_MARGIN;
 
        // draw a top pagebreak
-       if (parparams.pagebreakTop()) {
+       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);
 
-       Buffer const * buffer = bv_.buffer();
+       Buffer const & buffer = *bv_.buffer();
 
-       LyXLayout_ptr const & layout = par_.layout();
+       LyXLayout_ptr const & layout = pit_->layout();
 
-       if (buffer->params.paragraph_separation == BufferParams::PARSEP_SKIP) {
-               if (par_.previous()) {
+       if (buffer.params.paragraph_separation == BufferParams::PARSEP_SKIP) {
+               if (pit_ != text_.ownerParagraphs().begin()) {
                        if (layout->latextype == LATEX_PARAGRAPH
-                               && !par_.getDepth()) {
-                               y_top += buffer->params.getDefSkip().inPixels(bv_);
+                               && !pit_->getDepth()) {
+                               y_top += buffer.params.getDefSkip().inPixels(bv_);
                        } else {
                                LyXLayout_ptr const & playout =
-                                       par_.previous()->layout();
+                                       boost::prior(pit_)->layout();
                                if (playout->latextype == LATEX_PARAGRAPH
-                                       && !par_.previous()->getDepth()) {
+                                       && !boost::prior(pit_)->getDepth()) {
                                        // is it right to use defskip here, too? (AS)
-                                       y_top += buffer->params.getDefSkip().inPixels(bv_);
+                                       y_top += buffer.params.getDefSkip().inPixels(bv_);
                                }
                        }
                }
@@ -669,12 +716,11 @@ void RowPainter::paintFirst()
 
        // 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 w = (text_.isInInset() ? text_.inset_owner->width() : 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,
@@ -683,29 +729,29 @@ void RowPainter::paintFirst()
                y_top += asc;
        }
 
-       bool const is_rtl = par_.isRightToLeftPar(bv_.buffer()->params);
+       bool const is_rtl = pit_->isRightToLeftPar(bv_.buffer()->params);
 
        // should we print a label?
        if (layout->labeltype >= LABEL_STATIC
            && (layout->labeltype != LABEL_STATIC
                || layout->latextype != LATEX_ENVIRONMENT
-               || par_.isFirstInSequence())) {
+               || isFirstInSequence(pit_, text_.ownerParagraphs()))) {
 
                LyXFont font = getLabelFont();
-               if (!par_.getLabelstring().empty()) {
-                       float x = x_;
-                       string const str = par_.getLabelstring();
+               if (!pit_->getLabelstring().empty()) {
+                       double x = x_;
+                       string const str = pit_->getLabelstring();
 
                        // 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) {
+                               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();
+                                               spacing_val = buffer.params.spacing.getValue();
                                        }
 
                                        int const maxdesc =
@@ -718,8 +764,8 @@ void RowPainter::paintFirst()
                                        }
 
                                        pain_.text(int(x),
-                                               yo_ + row_.baseline() -
-                                               row_.ascent_of_text() - maxdesc,
+                                               yo_ + row_->baseline() -
+                                               row_->ascent_of_text() - maxdesc,
                                                str, font);
                                }
                        } else {
@@ -731,41 +777,41 @@ void RowPainter::paintFirst()
                                                - font_metrics::width(str, font);
                                }
 
-                               pain_.text(int(x), yo_ + row_.baseline(), str, font);
+                               pain_.text(int(x), yo_ + row_->baseline(), str, font);
                        }
                }
 
        // the labels at the top of an environment.
        // More or less for bibliography
-       } else if (par_.isFirstInSequence() &&
+       } else if (isFirstInSequence(pit_, text_.ownerParagraphs()) &&
                (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();
+               if (!pit_->getLabelstring().empty()) {
+                       string const str = pit_->getLabelstring();
                        float spacing_val = 1.0;
                        if (!parparams.spacing().isDefault()) {
                                spacing_val = parparams.spacing().getValue();
                        } else {
-                               spacing_val = buffer->params.spacing.getValue();
+                               spacing_val = buffer.params.spacing.getValue();
                        }
 
                        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;
+                                        + ww - text_.rightMargin(pit_, *bv_.buffer(), *row_)) / 2;
                                x -= font_metrics::width(str, font) / 2;
                        } else if (is_rtl) {
                                x = ww - leftMargin() -
                                        font_metrics::width(str, font);
                        }
-                       pain_.text(int(x), yo_ + row_.baseline()
-                                 - row_.ascent_of_text() - maxdesc,
+                       pain_.text(int(x),
+                           yo_ + row_->baseline() - row_->ascent_of_text() - maxdesc,
                                  str, font);
                }
        }
@@ -774,11 +820,11 @@ void RowPainter::paintFirst()
 
 void RowPainter::paintLast()
 {
-       ParagraphParameters const & parparams = par_.params();
-       int y_bottom = row_.height() - 1;
+       ParagraphParameters const & parparams = pit_->params();
+       int y_bottom = row_->height() - 1;
 
        // the bottom margin
-       if (!row_.next() && text_.isTopLevel())
+       if (row_ == text_.lastRow() && !text_.isInInset())
                y_bottom -= PAPER_MARGIN;
 
        int const ww = bv_.workWidth();
@@ -796,14 +842,13 @@ void RowPainter::paintLast()
 
        // 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)));
+                       getFont(max(pos_type(0), pit_->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 w = text_.isInInset() ? text_.inset_owner->width() : ww;
+               int const xp = 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);
@@ -811,8 +856,8 @@ void RowPainter::paintLast()
                y_bottom -= asc;
        }
 
-       bool const is_rtl = par_.isRightToLeftPar(bv_.buffer()->params);
-       int const endlabel = par_.getEndLabel();
+       bool const is_rtl = pit_->isRightToLeftPar(bv_.buffer()->params);
+       int const endlabel = getEndLabel(pit_, text_.ownerParagraphs());
 
        // draw an endlabel
        switch (endlabel) {
@@ -821,32 +866,26 @@ void RowPainter::paintLast()
        {
                LyXFont const font = getLabelFont();
                int const size = int(0.75 * font_metrics::maxAscent(font));
-               int const y = (yo_ + row_.baseline()) - size;
+               int const y = (yo_ + row_->baseline()) - size;
                int x = is_rtl ? LEFT_MARGIN : ww - PAPER_MARGIN - size;
 
-               if (row_.fill() <= size)
-                       x += (size - row_.fill() + 1) * (is_rtl ? -1 : 1);
+               if (row_->fill() <= size)
+                       x += (size - row_->fill() + 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
                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);
+               string const & str = pit_->layout()->endlabelstring();
+               double const x = is_rtl ?
+                       x_ - font_metrics::width(str, font)
+                       : ww - text_.rightMargin(pit_, *bv_.buffer(), *row_) - row_->fill();
+               pain_.text(int(x), yo_ + row_->baseline(), str, font);
                break;
        }
        case END_LABEL_NO_LABEL:
@@ -855,29 +894,28 @@ void RowPainter::paintLast()
 }
 
 
-bool RowPainter::paintText()
+void RowPainter::paintText()
 {
-       pos_type const last = row_.lastPrintablePos();
-       pos_type body_pos = par_.beginningOfBody();
+       pos_type const last = lastPrintablePos(*pit_, row_);
+       pos_type body_pos = pit_->beginningOfBody();
        if (body_pos > 0 &&
-               (body_pos - 1 > last ||
-               !par_.isLineSeparator(body_pos - 1))) {
+               (body_pos - 1 > last || !pit_->isLineSeparator(body_pos - 1))) {
                body_pos = 0;
        }
 
-       LyXLayout_ptr const & layout = par_.layout();
+       LyXLayout_ptr const & layout = pit_->layout();
 
        bool running_strikeout = false;
        bool is_struckout = false;
-       float last_strikeout_x = 0.0;
+       int last_strikeout_x = 0;
 
-       pos_type vpos = row_.pos();
+       pos_type vpos = row_->pos();
        while (vpos <= last) {
                if (x_ > bv_.workWidth())
                        break;
                pos_type pos = text_.vis2log(vpos);
 
-               if (pos >= par_.size()) {
+               if (pos >= pit_->size()) {
                        ++vpos;
                        continue;
                }
@@ -888,22 +926,22 @@ bool RowPainter::paintText()
                        continue;
                }
 
-               is_struckout = isDeletedText(par_, pos);
+               is_struckout = isDeletedText(*pit_, 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));
+               bool const highly_editable_inset = pit_->isInset(pos)
+                       && isHighlyEditableInset(pit_->getInset(pos));
 
                // 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,
+                       int const middle = yo_ + row_->top_of_text()
+                               + (row_->baseline() - row_->top_of_text()) / 2;
+                       pain_.line(last_strikeout_x, middle, int(x_), middle,
                                LColor::strikeout, Painter::line_solid, Painter::line_thin);
                        running_strikeout = false;
                }
@@ -912,80 +950,66 @@ bool RowPainter::paintText()
                        int const lwidth = font_metrics::width(layout->labelsep,
                                getLabelFont());
 
-                       x_ += label_hfill_ + lwidth
-                               - singleWidth(body_pos - 1);
+                       x_ += label_hfill_ + lwidth - singleWidth(body_pos - 1);
                }
 
-               if (par_.isHfill(pos)) {
+               if (pit_->isHfill(pos)) {
                        x_ += 1;
 
-                       int const y0 = yo_ + row_.baseline();
+                       int const y0 = yo_ + row_->baseline();
                        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(*pit_, row_, pos)) {
                                int const y2 = (y0 + y1) / 2;
 
                                if (pos >= body_pos) {
-                                       pain_.line(int(x_), y2,
-                                                 int(x_ + hfill_), y2,
+                                       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)) {
+               } else if (pit_->isSeparator(pos)) {
                        x_ += singleWidth(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,
+               int const middle = yo_ + row_->top_of_text()
+                       + ((row_->baseline() - row_->top_of_text()) / 2);
+               pain_.line(last_strikeout_x, middle, int(x_), middle,
                        LColor::strikeout, Painter::line_solid, Painter::line_thin);
                running_strikeout = false;
        }
-       return false;
 }
 
 
-bool RowPainter::paint(int y_offset, int x_offset, int y)
+void RowPainter::paint()
 {
-       xo_ = x_offset;
-       yo_ = y_offset;
-       y_ = y;
-       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_);
+       width_       = text_.workWidth();
+       x_           = row_->x();
+       separator_   = row_->fill_separator();
+       hfill_       = row_->fill_hfill();
+       label_hfill_ = row_->fill_label_hfill();
 
        // FIXME: what is this fixing ?
-       if (text_.isInInset() && (x_ < 0))
+       if (text_.isInInset() && x_ < 0)
                x_ = 0;
        x_ += xo_;
 
@@ -995,9 +1019,8 @@ bool RowPainter::paint(int y_offset, int x_offset, int y)
                paintBackground();
 
        // paint the selection background
-       if (text_.selection.set()) {
+       if (text_.selection.set())
                paintSelection();
-       }
 
        // vertical lines for appendix
        paintAppendix();
@@ -1008,14 +1031,99 @@ bool RowPainter::paint(int y_offset, int x_offset, int y)
        // changebar
        paintChangeBar();
 
-       if (row_.isParStart()) {
+       if (row_->isParStart())
                paintFirst();
-       }
 
-       if (row_.isParEnd()) {
+       if (isParEnd(*pit_, row_))
                paintLast();
-       }
 
        // paint text
-       return paintText();
+       paintText();
+}
+
+
+int paintRows(BufferView const & bv, LyXText const & text,
+       ParagraphList::iterator pit, RowList::iterator rit,
+       int xo, int y, int yf, int yo)
+{
+       //lyxerr << "  paintRows: rit: " << &*rit << endl;
+       //const_cast<LyXText&>(text).updateRowPositions();
+       int const yy = yf - y;
+       int const y2 = bv.painter().paperHeight();
+
+       ParagraphList::iterator end = text.ownerParagraphs().end();
+       bool active = false;
+
+       for ( ; pit != end; ++pit) {
+               RowList::iterator row = pit->rows.begin();
+               RowList::iterator rend = pit->rows.end();
+
+               for ( ; row != rend; ++row) {
+                       if (row == rit)
+                               active = true;
+                       if (active) {
+                               RowPainter painter(bv, text, pit, row, y + yo, xo, y + bv.top_y());
+                               painter.paint();
+                               y += row->height();
+                               if (yy + y >= y2)
+                                       return y;
+                       } else {
+                               //lyxerr << "   paintRows: row: " << &*row << " ignored" << endl;
+                       }
+               }
+       }
+
+       return y;
+}
+
+} // namespace anon
+
+
+int paintText(BufferView & bv)
+{
+       int const topy = bv.top_y();
+       ParagraphList::iterator pit;
+       RowList::iterator rit = bv.text->getRowNearY(topy, pit);
+       int y = rit->y() - topy;
+       return paintRows(bv, *bv.text, pit, rit, 0, y, y, 0);
+}
+
+
+void paintTextInset(BufferView & bv, LyXText & text, int x, int baseline)
+{
+       RowList::iterator rit = text.firstRow();
+       RowList::iterator end = text.endRow();
+       ParagraphList::iterator pit = text.ownerParagraphs().begin();
+
+       int y_offset = baseline - rit->ascent_of_text();
+       int y = y_offset;
+       while (rit != end && y + rit->height() <= 0) {
+               y += rit->height();
+               text.nextRow(pit, rit);
+       }
+       if (y_offset < 0)
+               paintRows(bv, text, pit, rit, x, 0, y, y);
+       else
+               paintRows(bv, text, pit, rit, x, 0, y_offset, y_offset);
+}
+
+
+int getLengthMarkerHeight(BufferView const & bv, VSpace const & vsp)
+{
+       if (vsp.kind() == VSpace::NONE)
+               return 0;
+
+       int const arrow_size = 4;
+       int const space_size = 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);
 }