]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insettext.C
The markDirty() and fitCursor() changes
[lyx.git] / src / insets / insettext.C
index 03c2764251af6248e7c5ea7243f1646f562a618e..3cfc909e5593cd0e76d5f1b46979a48d062447a7 100644 (file)
@@ -1,18 +1,15 @@
-/* This file is part of
- * ======================================================
+/**
+ * \file insettext.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           LyX, The Document Processor
+ * \author Jürgen Vigna
  *
- *           Copyright 1998-2001 The LyX Team.
- *
- * ======================================================
+ * Full author contact details are available in file CREDITS
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
 
 #include "insettext.h"
 
 #include "trans_mgr.h"
 #include "undo_funcs.h"
 #include "WordLangTuple.h"
+#include "paragraph_funcs.h"
+#include "sgml.h"
+#include "rowpainter.h"
+#include "insetnewline.h"
 
 #include "frontends/Alert.h"
 #include "frontends/Dialogs.h"
@@ -50,6 +51,8 @@
 #include "support/lstrings.h"
 #include "support/lyxalgo.h" // lyx::count
 
+#include <boost/bind.hpp>
+
 #include <fstream>
 #include <algorithm>
 #include <cstdlib>
@@ -63,6 +66,7 @@ using std::max;
 using std::make_pair;
 using std::vector;
 using std::pair;
+using std::for_each;
 
 using lyx::pos_type;
 using lyx::textclass_type;
@@ -77,14 +81,14 @@ extern int greek_kb_flag;
 void InsetText::saveLyXTextState(LyXText * t) const
 {
        // check if my paragraphs are still valid
-       Paragraph * p = &*(paragraphs.begin());
-       while (p) {
-               if (p == t->cursor.par())
+       ParagraphList::iterator it = paragraphs.begin();
+       ParagraphList::iterator end = paragraphs.end();
+       for (; it != end; ++it) {
+               if (&*it == t->cursor.par())
                        break;
-               p = p->next();
        }
 
-       if (p && t->cursor.pos() <= p->size()) {
+       if (it != end && t->cursor.pos() <= it->size()) {
                sstate.lpar = t->cursor.par();
                sstate.pos = t->cursor.pos();
                sstate.boundary = t->cursor.boundary();
@@ -96,34 +100,32 @@ void InsetText::saveLyXTextState(LyXText * t) const
                sstate.selendboundary = t->selection.end.boundary();
                sstate.selection = t->selection.set();
                sstate.mark_set = t->selection.mark();
-               sstate.refresh = t->refresh_row != 0;
        } else {
                sstate.lpar = 0;
        }
 }
 
-void InsetText::restoreLyXTextState(BufferView * bv, LyXText * t) const
+
+void InsetText::restoreLyXTextState(LyXText * t) const
 {
-       if (sstate.lpar) {
-               t->selection.set(true);
-               /* at this point just to avoid the Delete-Empty-Paragraph
-                * Mechanism when setting the cursor */
-               t->selection.mark(sstate.mark_set);
-               if (sstate.selection) {
-                       t->setCursor(bv, sstate.selstartpar, sstate.selstartpos,
-                                    true, sstate.selstartboundary);
-                       t->selection.cursor = t->cursor;
-                       t->setCursor(bv, sstate.selendpar, sstate.selendpos,
-                                    true, sstate.selendboundary);
-                       t->setSelection(bv);
-                       t->setCursor(bv, sstate.lpar, sstate.pos);
-               } else {
-                       t->setCursor(bv, sstate.lpar, sstate.pos, true, sstate.boundary);
-                       t->selection.cursor = t->cursor;
-                       t->selection.set(false);
-               }
-               if (sstate.refresh) {
-               }
+       if (!sstate.lpar)
+               return;
+
+       t->selection.set(true);
+       /* at this point just to avoid the DEPM when setting the cursor */
+       t->selection.mark(sstate.mark_set);
+       if (sstate.selection) {
+               t->setCursor(sstate.selstartpar, sstate.selstartpos,
+                            true, sstate.selstartboundary);
+               t->selection.cursor = t->cursor;
+               t->setCursor(sstate.selendpar, sstate.selendpos,
+                            true, sstate.selendboundary);
+               t->setSelection();
+               t->setCursor(sstate.lpar, sstate.pos);
+       } else {
+               t->setCursor(sstate.lpar, sstate.pos, true, sstate.boundary);
+               t->selection.cursor = t->cursor;
+               t->selection.set(false);
        }
 }
 
@@ -141,6 +143,8 @@ InsetText::InsetText(BufferParams const & bp)
 {
        paragraphs.set(new Paragraph);
        paragraphs.begin()->layout(bp.getLyXTextClass().defaultLayout());
+       if (bp.tracking_changes)
+               paragraphs.begin()->trackChanges();
        init();
 }
 
@@ -170,11 +174,9 @@ void InsetText::init(InsetText const * ins, bool same_id)
                if (same_id)
                        id_ = ins->id_;
        } else {
-               Paragraph * p = &*(paragraphs.begin());
-               while (p) {
-                       p->setInsetOwner(this);
-                       p = p->next();
-               }
+               for_each(paragraphs.begin(), paragraphs.end(),
+                        boost::bind(&Paragraph::setInsetOwner, _1, this));
+
                the_locking_inset = 0;
                drawFrame_ = NEVER;
                frame_color = LColor::insetframe;
@@ -192,7 +194,6 @@ void InsetText::init(InsetText const * ins, bool same_id)
        locked = false;
        old_par = 0;
        last_drawn_width = -1;
-       frame_is_visible = false;
        cached_bview = 0;
        sstate.lpar = 0;
        in_insetAllowed = false;
@@ -201,16 +202,22 @@ void InsetText::init(InsetText const * ins, bool same_id)
 
 InsetText::~InsetText()
 {
-       cached_bview = 0;
-
-       // NOTE
-
        paragraphs.clear();
 }
 
 
-void InsetText::clear()
+void InsetText::clear(bool just_mark_erased)
 {
+       if (just_mark_erased) {
+               ParagraphList::iterator it = paragraphs.begin();
+               ParagraphList::iterator end = paragraphs.end();
+               for (; it != end; ++it) {
+                       it->markErased();
+               }
+               need_update = FULL;
+               return;
+       }
+
        // This is a gross hack...
        LyXLayout_ptr old_layout = paragraphs.begin()->layout();
 
@@ -239,19 +246,28 @@ void InsetText::write(Buffer const * buf, ostream & os) const
 
 void InsetText::writeParagraphData(Buffer const * buf, ostream & os) const
 {
-       paragraphs.begin()->writeFile(buf, os, buf->params, 0);
+       ParagraphList::iterator it = paragraphs.begin();
+       ParagraphList::iterator end = paragraphs.end();
+       Paragraph::depth_type dth = 0;
+       for (; it != end; ++it) {
+               it->write(buf, os, buf->params, dth);
+       }
 }
 
 
 void InsetText::read(Buffer const * buf, LyXLex & lex)
 {
        string token;
-       int pos = 0;
-       Paragraph * return_par = 0;
        Paragraph::depth_type depth = 0;
-       LyXFont font(LyXFont::ALL_INHERIT);
 
-       clear();
+       clear(false);
+
+       if (buf->params.tracking_changes)
+               paragraphs.begin()->trackChanges();
+
+       // delete the initial paragraph
+       paragraphs.clear();
+       ParagraphList::iterator pit = paragraphs.begin();
 
        while (lex.isOK()) {
                lex.nextToken();
@@ -259,30 +275,23 @@ void InsetText::read(Buffer const * buf, LyXLex & lex)
                if (token.empty())
                        continue;
                if (token == "\\end_inset") {
-#ifndef NO_COMPABILITY
-                       const_cast<Buffer*>(buf)->insertErtContents(par, pos, false);
-#endif
                        break;
                }
 
-               Paragraph * tmp = &*(paragraphs.begin());
-               
-               if (const_cast<Buffer*>(buf)->
-                       parseSingleLyXformat2Token(lex, tmp, return_par,
-                                                  token, pos, depth, font)) {
-                       // the_end read this should NEVER happen
+               if (token == "\\the_end") {
                        lex.printError("\\the_end read in inset! Error in document!");
                        return;
                }
-       }
-       if (!return_par)
-               return_par = &*(paragraphs.begin());
-       paragraphs.set(return_par);
-       while (return_par) {
-               return_par->setInsetOwner(this);
-               return_par = return_par->next();
+
+               // FIXME: ugly.
+               const_cast<Buffer*>(buf)->readParagraph(lex, token, paragraphs, pit, depth);
        }
 
+       pit = paragraphs.begin();
+       ParagraphList::iterator const end = paragraphs.end();
+       for (; pit != end; ++pit)
+               pit->setInsetOwner(this);
+
        if (token != "\\end_inset") {
                lex.printError("Missing \\end_inset at this point. "
                                           "Read: `$$Token'");
@@ -336,7 +345,7 @@ int InsetText::textWidth(BufferView * bv, bool fordraw) const
 
 
 void InsetText::draw(BufferView * bv, LyXFont const & f,
-                    int baseline, float & x, bool cleared) const
+                    int baseline, float & x) const
 {
        if (nodraw())
                return;
@@ -346,7 +355,6 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
        // this is the first thing we have to ask because if the x pos
        // changed we have to do a complete rebreak of the text as we
        // may have few space to draw in. Well we should check on this too
-       int old_x = top_x;
        if (top_x != int(x)) {
                top_x = int(x);
                topx_set = true;
@@ -354,7 +362,7 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
                if (nw > 0 && old_max_width != nw) {
                        need_update = INIT;
                        old_max_width = nw;
-                       bv->text->status(bv, LyXText::CHANGED_IN_DRAW);
+                       bv->text->postChangedInDraw();
                        return;
                }
        }
@@ -366,16 +374,13 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
        descent(bv, f);
 
        // repaint the background if needed
-       if (cleared && backgroundColor() != LColor::background) {
-               clearInset(bv, baseline, cleared);
-       }
+       if (backgroundColor() != LColor::background)
+               clearInset(bv, baseline);
 
        // no draw is necessary !!!
        if ((drawFrame_ == LOCKED) && !locked && paragraphs.begin()->empty()) {
                top_baseline = baseline;
                x += width(bv, f);
-               if (need_update & CLEAR_FRAME)
-                       clearFrame(pain, cleared);
                need_update = NONE;
                return;
        }
@@ -383,31 +388,10 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
        if (!owner())
                x += static_cast<float>(scroll());
 
-       // if top_x differs we did it already
-       if (!cleared && (old_x == int(x))
-           && ((need_update&(INIT|FULL)) || (top_baseline != baseline)
-               ||(last_drawn_width != insetWidth)))
-       {
-               // Condition necessary to eliminate bug 59 attachment 37
-               if (baseline > 0)
-                       clearInset(bv, baseline, cleared);
-       }
-
-       if (cleared)
-               frame_is_visible = false;
-
-       if (!cleared && (need_update == NONE)) {
-               if (locked)
-                       drawFrame(pain, cleared);
-               return;
-       }
-
        top_baseline = baseline;
        top_y = baseline - insetAscent;
 
        if (last_drawn_width != insetWidth) {
-               if (!cleared)
-                       clearInset(bv, baseline, cleared);
                need_update |= FULL;
                last_drawn_width = insetWidth;
        }
@@ -417,13 +401,7 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
                inset_x = cix(bv) - top_x + drawTextXOffset;
                inset_y = ciy(bv) + drawTextYOffset;
        }
-       if (!cleared && (need_update == CURSOR)
-           && !getLyXText(bv)->selection.set()) {
-               drawFrame(pain, cleared);
-               x += insetWidth;
-               need_update = NONE;
-               return;
-       }
+
        bool clear = false;
        if (!lt) {
                lt = getLyXText(bv);
@@ -442,67 +420,45 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
                row = row->next();
        }
        if (y_offset < 0) {
-               lt->first_y = -y_offset;
+               lt->top_y(-y_offset);
                first = y;
                y_offset = 0;
        } else {
-               lt->first_y = first;
+               lt->top_y(first);
                first = 0;
        }
-       if (cleared || (need_update&(INIT|FULL))) {
-               int yf = y_offset + first;
-               y = 0;
-               while ((row != 0) && (yf < ph)) {
-                       lt->getVisibleRow(bv, y + y_offset + first, int(x),
-                                         row, y + lt->first_y, cleared);
-                       if (bv->text->status() == LyXText::CHANGED_IN_DRAW) {
-                               lt->need_break_row = row;
-                               lt->fullRebreak(bv);
-                               lt->setCursor(bv, lt->cursor.par(),
-                                             lt->cursor.pos());
-                               if (lt->selection.set()) {
-                                       lt->setCursor(bv, lt->selection.start,
-                                                     lt->selection.start.par(),
-                                                     lt->selection.start.pos());
-                                       lt->setCursor(bv, lt->selection.end,
-                                                     lt->selection.end.par(),
-                                                     lt->selection.end.pos());
-                               }
-                               break;
+
+       int yf = y_offset + first;
+       y = 0;
+       while ((row != 0) && (yf < ph)) {
+               Row * prev = row->previous();
+               RowPainter rp(*bv, *lt, *row);
+               if (rp.paint(y + y_offset + first, int(x), y + lt->top_y()))
+                       lt->markChangeInDraw(row, prev);
+               if (bv->text->status() == LyXText::CHANGED_IN_DRAW) {
+                       lt->need_break_row = row;
+                       lt->fullRebreak();
+                       lt->setCursor(lt->cursor.par(),
+                                     lt->cursor.pos());
+                       if (lt->selection.set()) {
+                               lt->setCursor(lt->selection.start,
+                                             lt->selection.start.par(),
+                                             lt->selection.start.pos());
+                               lt->setCursor(lt->selection.end,
+                                             lt->selection.end.par(),
+                                             lt->selection.end.pos());
                        }
-                       y += row->height();
-                       yf += row->height();
-                       row = row->next();
-               }
-       } else if (!locked) {
-               if (need_update & CURSOR) {
-                       bv->screen().toggleSelection(lt, bv, true, y_offset,int(x));
-                       lt->clearSelection();
-                       lt->selection.cursor = lt->cursor;
-               }
-               bv->screen().update(lt, bv, y_offset, int(x));
-       } else {
-               locked = false;
-               if (need_update & SELECTION) {
-                       bv->screen().toggleToggle(lt, bv, y_offset, int(x));
-               } else if (need_update & CURSOR) {
-                       bv->screen().toggleSelection(lt, bv, true, y_offset,int(x));
-                       lt->clearSelection();
-                       lt->selection.cursor = lt->cursor;
+                       break;
                }
-               bv->screen().update(lt, bv, y_offset, int(x));
-               locked = true;
+               y += row->height();
+               yf += row->height();
+               row = row->next();
        }
 
-       lt->refresh_y = 0;
-       lt->status(bv, LyXText::UNCHANGED);
-       if ((drawFrame_ == ALWAYS) ||
-               ((cleared || (need_update != CURSOR_PAR)) &&
-                (drawFrame_ == LOCKED) && locked))
-       {
-               drawFrame(pain, cleared);
-       } else if (need_update & CLEAR_FRAME) {
-               clearFrame(pain, cleared);
+       lt->clearPaint();
+
+       if ((drawFrame_ == ALWAYS) || (drawFrame_ == LOCKED && locked)) {
+               drawFrame(pain);
        }
 
        x += insetWidth - TEXT_TO_INSET_OFFSET;
@@ -517,30 +473,15 @@ void InsetText::draw(BufferView * bv, LyXFont const & f,
 }
 
 
-void InsetText::drawFrame(Painter & pain, bool cleared) const
+void InsetText::drawFrame(Painter & pain) const
 {
        static int const ttoD2 = TEXT_TO_INSET_OFFSET / 2;
-       if (!frame_is_visible || cleared) {
-               frame_x = top_x + ttoD2;
-               frame_y = top_baseline - insetAscent + ttoD2;
-               frame_w = insetWidth - TEXT_TO_INSET_OFFSET;
-               frame_h = insetAscent + insetDescent - TEXT_TO_INSET_OFFSET;
-               pain.rectangle(frame_x, frame_y, frame_w, frame_h,
-                              frame_color);
-               frame_is_visible = true;
-       }
-}
-
-
-void InsetText::clearFrame(Painter & pain, bool cleared) const
-{
-       if (frame_is_visible) {
-               if (!cleared) {
-                       pain.rectangle(frame_x, frame_y, frame_w, frame_h,
-                                      backgroundColor());
-               }
-               frame_is_visible = false;
-       }
+       frame_x = top_x + ttoD2;
+       frame_y = top_baseline - insetAscent + ttoD2;
+       frame_w = insetWidth - TEXT_TO_INSET_OFFSET;
+       frame_h = insetAscent + insetDescent - TEXT_TO_INSET_OFFSET;
+       pain.rectangle(frame_x, frame_y, frame_w, frame_h,
+                      frame_color);
 }
 
 
@@ -581,7 +522,7 @@ void InsetText::update(BufferView * bv, LyXFont const & font, bool reinit)
        if ((need_update & CURSOR_PAR) && (lt->status() == LyXText::UNCHANGED) &&
                the_locking_inset)
        {
-               lt->updateInset(bv, the_locking_inset);
+               lt->updateInset(the_locking_inset);
        }
        if (lt->status() == LyXText::NEED_MORE_REFRESH)
                need_update |= FULL;
@@ -617,7 +558,7 @@ void InsetText::setUpdateStatus(BufferView * bv, int what) const
 }
 
 
-void InsetText::updateLocal(BufferView * bv, int what, bool mark_dirty) const
+void InsetText::updateLocal(BufferView * bv, int what, bool mark_dirty)
 {
        if (!autoBreakRows && paragraphs.begin()->next())
                collapseParagraphs(bv);
@@ -626,7 +567,7 @@ void InsetText::updateLocal(BufferView * bv, int what, bool mark_dirty) const
                lt = getLyXText(bv);
                clear = true;
        }
-       lt->fullRebreak(bv);
+       lt->fullRebreak();
        setUpdateStatus(bv, what);
        bool flag = mark_dirty ||
                (((need_update != CURSOR) && (need_update != NONE)) ||
@@ -645,7 +586,7 @@ void InsetText::updateLocal(BufferView * bv, int what, bool mark_dirty) const
        bv->fitCursor();
 #endif
        if (flag)
-               bv->updateInset(const_cast<InsetText *>(this), mark_dirty);
+               bv->updateInset(const_cast<InsetText *>(this));
 
        if (need_update == CURSOR)
                need_update = NONE;
@@ -690,7 +631,7 @@ void InsetText::edit(BufferView * bv, int x, int y, mouse_button::state button)
        // cycle hopefully (Jug 20020509)
        // FIXME: GUII I've changed this to none: probably WRONG
        if (!checkAndActivateInset(bv, x, tmp_y, mouse_button::none)) {
-               lt->setCursorFromCoordinates(bv, x - drawTextXOffset,
+               lt->setCursorFromCoordinates(x - drawTextXOffset,
                                            y + insetAscent);
                lt->cursor.x_fix(lt->cursor.x());
        }
@@ -715,7 +656,7 @@ void InsetText::edit(BufferView * bv, int x, int y, mouse_button::state button)
        showInsetCursor(bv);
 
        // Tell the paragraph dialog that we've entered an insettext.
-       bv->owner()->getDialogs().updateParagraph();
+       bv->dispatch(FuncRequest(LFUN_PARAGRAPH_UPDATE));
 }
 
 
@@ -739,13 +680,14 @@ void InsetText::edit(BufferView * bv, bool front)
                clear = true;
        }
        if (front)
-               lt->setCursor(bv, &*(paragraphs.begin()), 0);
+               lt->setCursor(&*(paragraphs.begin()), 0);
        else {
-               Paragraph * p = &*(paragraphs.begin());
-               while (p->next())
-                       p = p->next();
+               ParagraphList::iterator it = paragraphs.begin();
+               ParagraphList::iterator end = paragraphs.end();
+               while (boost::next(it) != end)
+                       ++it;
 //             int const pos = (p->size() ? p->size()-1 : p->size());
-               lt->setCursor(bv, p, p->size());
+               lt->setCursor(&*it, it->size());
        }
        lt->clearSelection();
        finishUndo();
@@ -778,16 +720,8 @@ void InsetText::insetUnlock(BufferView * bv)
        no_selection = true;
        locked = false;
        int code = NONE;
-#if 0
-       if (drawFrame_ == LOCKED)
-               code = CURSOR|CLEAR_FRAME;
-       else
-               code = CURSOR;
-#else
-       if (drawFrame_ == LOCKED)
-               code = CLEAR_FRAME;
-#endif
        bool clear = false;
+
        if (!lt) {
                lt = getLyXText(bv);
                clear = true;
@@ -802,9 +736,9 @@ void InsetText::insetUnlock(BufferView * bv)
                bv->owner()->setLayout(bv->text->cursor.par()->layout()->name());
        // hack for deleteEmptyParMech
        if (!paragraphs.begin()->empty()) {
-               lt->setCursor(bv, &*(paragraphs.begin()), 0);
+               lt->setCursor(&*(paragraphs.begin()), 0);
        } else if (paragraphs.begin()->next()) {
-               lt->setCursor(bv, paragraphs.begin()->next(), 0);
+               lt->setCursor(paragraphs.begin()->next(), 0);
        }
        if (clear)
                lt = 0;
@@ -830,7 +764,7 @@ void InsetText::lockInset(BufferView * bv)
                lt = getLyXText(bv);
                clear = true;
        }
-       lt->setCursor(bv, &*(paragraphs.begin()), 0);
+       lt->setCursor(&*(paragraphs.begin()), 0);
        lt->clearSelection();
        finishUndo();
        // If the inset is empty set the language of the current font to the
@@ -878,12 +812,12 @@ bool InsetText::lockInsetInInset(BufferView * bv, UpdatableInset * inset)
                                p->insetlist.end();
                        for (; it != end; ++it) {
                                if (it.getInset() == inset) {
-                                       getLyXText(bv)->setCursorIntern(bv, p, it.getPos());
+                                       getLyXText(bv)->setCursorIntern(p, it.getPos());
                                        lockInset(bv, inset);
                                        return true;
                                }
                                if (it.getInset()->getInsetFromID(id)) {
-                                       getLyXText(bv)->setCursorIntern(bv, p, it.getPos());
+                                       getLyXText(bv)->setCursorIntern(p, it.getPos());
                                        it.getInset()->edit(bv);
                                        return the_locking_inset->lockInsetInInset(bv, inset);
                                }
@@ -920,7 +854,7 @@ bool InsetText::unlockInsetInInset(BufferView * bv, UpdatableInset * inset,
                return false;
        if (the_locking_inset == inset) {
                the_locking_inset->insetUnlock(bv);
-               getLyXText(bv)->updateInset(bv, inset);
+               getLyXText(bv)->updateInset(inset);
                the_locking_inset = 0;
                if (lr)
                        moveRightIntern(bv, true, false);
@@ -962,14 +896,14 @@ bool InsetText::updateInsetInInset(BufferView * bv, Inset * inset)
                        ustat = FULL;
                }
                if (found)
-                       lt->updateInset(bv, tl_inset);
+                       lt->updateInset(tl_inset);
                if (clear)
                        lt = 0;
                if (found)
                        setUpdateStatus(bv, ustat);
                return found;
        }
-       bool found = lt->updateInset(bv, inset);
+       bool found = lt->updateInset(inset);
        if (clear)
                lt = 0;
        if (found) {
@@ -985,29 +919,29 @@ bool InsetText::updateInsetInInset(BufferView * bv, Inset * inset)
 }
 
 
-void InsetText::insetButtonPress(BufferView * bv,
-       int x, int y, mouse_button::state button)
+void InsetText::lfunMousePress(FuncRequest const & cmd)
 {
        no_selection = true;
 
        // use this to check mouse motion for selection!
-       mouse_x = x;
-       mouse_y = y;
+       mouse_x = cmd.x;
+       mouse_y = cmd.y;
 
+       BufferView * bv = cmd.view();
+       FuncRequest cmd1 = cmd;
+       cmd1.x -= inset_x;
+       cmd1.y -= inset_y;
        if (!locked)
                lockInset(bv);
 
-       int tmp_x = x - drawTextXOffset;
-       int tmp_y = y + insetAscent - getLyXText(bv)->first_y;
-       Inset * inset = bv->checkInsetHit(getLyXText(bv), tmp_x, tmp_y);
+       int tmp_x = cmd.x - drawTextXOffset;
+       int tmp_y = cmd.y + insetAscent - getLyXText(bv)->top_y();
+       Inset * inset = getLyXText(bv)->checkInsetHit(tmp_x, tmp_y);
 
        hideInsetCursor(bv);
        if (the_locking_inset) {
                if (the_locking_inset == inset) {
-                       the_locking_inset->insetButtonPress(bv,
-                                                           x - inset_x,
-                                                           y - inset_y,
-                                                           button);
+                       the_locking_inset->localDispatch(cmd1);
                        return;
                }
 #if 0
@@ -1017,8 +951,7 @@ void InsetText::insetButtonPress(BufferView * bv,
                        inset_x = cix(bv) - top_x + drawTextXOffset;
                        inset_y = ciy(bv) + drawTextYOffset;
                        the_locking_inset = 0;
-                       inset->insetButtonPress(bv, x - inset_x,
-                                               y - inset_y, button);
+                       inset->localDispatch(cmd1);
 //                     inset->edit(bv, x - inset_x, y - inset_y, button);
                        if (the_locking_inset)
                                updateLocal(bv, CURSOR, false);
@@ -1040,7 +973,7 @@ void InsetText::insetButtonPress(BufferView * bv,
                        if (!bv->lockInset(uinset)) {
                                lyxerr[Debug::INSETS] << "Cannot lock inset" << endl;
                        }
-                       inset->insetButtonPress(bv, x - inset_x, y - inset_y, button);
+                       inset->localDispatch(cmd1);
                        if (the_locking_inset)
                                updateLocal(bv, CURSOR, false);
                        return;
@@ -1048,7 +981,7 @@ void InsetText::insetButtonPress(BufferView * bv,
        }
        if (!inset) { // && (button == mouse_button::button2)) {
                bool paste_internally = false;
-               if ((button == mouse_button::button2) && getLyXText(bv)->selection.set()) {
+               if (cmd.button() == mouse_button::button2 && getLyXText(bv)->selection.set()) {
                        localDispatch(FuncRequest(bv, LFUN_COPY));
                        paste_internally = true;
                }
@@ -1057,10 +990,10 @@ void InsetText::insetButtonPress(BufferView * bv,
                        lt = getLyXText(bv);
                        clear = true;
                }
-               int old_first_y = lt->first_y;
+               int old_top_y = lt->top_y();
 
-               lt->setCursorFromCoordinates(bv, x - drawTextXOffset,
-                                            y + insetAscent);
+               lt->setCursorFromCoordinates(cmd.x - drawTextXOffset,
+                                            cmd.y + insetAscent);
                // set the selection cursor!
                lt->selection.cursor = lt->cursor;
                lt->cursor.x_fix(lt->cursor.x());
@@ -1080,13 +1013,13 @@ void InsetText::insetButtonPress(BufferView * bv,
                bv->owner()->setLayout(cpar(bv)->layout()->name());
 
                // we moved the view we cannot do mouse selection in this case!
-               if (getLyXText(bv)->first_y != old_first_y)
+               if (getLyXText(bv)->top_y() != old_top_y)
                        no_selection = true;
                old_par = cpar(bv);
                // Insert primary selection with middle mouse
                // if there is a local selection in the current buffer,
                // insert this
-               if (button == mouse_button::button2) {
+               if (cmd.button() == mouse_button::button2) {
                        if (paste_internally)
                                localDispatch(FuncRequest(bv, LFUN_PASTE));
                        else
@@ -1099,30 +1032,33 @@ void InsetText::insetButtonPress(BufferView * bv,
 }
 
 
-bool InsetText::insetButtonRelease(BufferView * bv,
-       int x, int y, mouse_button::state button)
+bool InsetText::lfunMouseRelease(FuncRequest const & cmd)
 {
+       BufferView * bv = cmd.view();
+       FuncRequest cmd1 = cmd;
+       cmd1.x -= inset_x;
+       cmd1.y -= inset_y;
+
        no_selection = true;
-       if (the_locking_inset) {
-               return the_locking_inset->insetButtonRelease(bv,
-                                                            x - inset_x, y - inset_y,
-                                                            button);
-       }
-       int tmp_x = x - drawTextXOffset;
-       int tmp_y = y + insetAscent - getLyXText(bv)->first_y;
-       Inset * inset = bv->checkInsetHit(getLyXText(bv), tmp_x, tmp_y);
+       if (the_locking_inset)
+               return the_locking_inset->localDispatch(cmd1);
+
+       int tmp_x = cmd.x - drawTextXOffset;
+       int tmp_y = cmd.y + insetAscent - getLyXText(bv)->top_y();
+       Inset * inset = getLyXText(bv)->checkInsetHit(tmp_x, tmp_y);
        bool ret = false;
        if (inset) {
-               if (isHighlyEditableInset(inset)) {
-                       ret = inset->insetButtonRelease(bv, x - inset_x,
-                                                       y - inset_y, button);
-               } else {
+               if (isHighlyEditableInset(inset))
+                       ret = inset->localDispatch(cmd1);
+               else {
                        inset_x = cix(bv) - top_x + drawTextXOffset;
                        inset_y = ciy(bv) + drawTextYOffset;
-                       ret = inset->insetButtonRelease(bv, x - inset_x,
-                                                       y - inset_y, button);
-                       inset->edit(bv, x - inset_x,
-                                   y - inset_y, button);
+                       cmd1.x = cmd.x - inset_x;
+                       cmd1.y = cmd.x - inset_y;
+// note that we should do ret = inset->localDispatch(cmd1)
+// and fix this instead (Alfredo);
+                       ret = true;
+                       inset->edit(bv, cmd1.x, cmd1.y, cmd.button());
                }
                updateLocal(bv, CURSOR_PAR, false);
        }
@@ -1130,17 +1066,21 @@ bool InsetText::insetButtonRelease(BufferView * bv,
 }
 
 
-void InsetText::insetMotionNotify(BufferView * bv, int x, int y, mouse_button::state state)
+void InsetText::lfunMouseMotion(FuncRequest const & cmd)
 {
+       FuncRequest cmd1 = cmd;
+       cmd1.x -= inset_x;
+       cmd1.y -= inset_y;
+
        if (the_locking_inset) {
-               the_locking_inset->insetMotionNotify(bv, x - inset_x,
-                                                    y - inset_y,state);
+               the_locking_inset->localDispatch(cmd1);
                return;
        }
 
-       if (no_selection || ((mouse_x == x) && (mouse_y == y)))
+       if (no_selection || (mouse_x == cmd.x && mouse_y == cmd.y))
                return;
 
+       BufferView * bv = cmd.view();
        bool clear = false;
        if (!lt) {
                lt = getLyXText(bv);
@@ -1148,14 +1088,15 @@ void InsetText::insetMotionNotify(BufferView * bv, int x, int y, mouse_button::s
        }
        hideInsetCursor(bv);
        LyXCursor cur = lt->cursor;
-       lt->setCursorFromCoordinates(bv, x - drawTextXOffset, y + insetAscent);
+       lt->setCursorFromCoordinates
+               (cmd.x - drawTextXOffset, cmd.y + insetAscent);
        lt->cursor.x_fix(lt->cursor.x());
        if (cur == lt->cursor) {
                if (clear)
                        lt = 0;
                return;
        }
-       lt->setSelection(bv);
+       lt->setSelection();
        bool flag = (lt->toggle_cursor.par() != lt->toggle_end_cursor.par() ||
                                 lt->toggle_cursor.pos() != lt->toggle_end_cursor.pos());
        if (clear)
@@ -1167,13 +1108,25 @@ void InsetText::insetMotionNotify(BufferView * bv, int x, int y, mouse_button::s
 }
 
 
-UpdatableInset::RESULT
-InsetText::localDispatch(FuncRequest const & ev)
+Inset::RESULT InsetText::localDispatch(FuncRequest const & ev)
 {
        BufferView * bv = ev.view();
+       switch (ev.action) {
+               case LFUN_MOUSE_PRESS:
+                       lfunMousePress(ev);
+                       return DISPATCHED;
+               case LFUN_MOUSE_MOTION:
+                       lfunMouseMotion(ev);
+                       return DISPATCHED;
+               case LFUN_MOUSE_RELEASE:
+                       return lfunMouseRelease(ev) ? DISPATCHED : UNDISPATCHED;
+               default:
+                       break;
+       }
+
        bool was_empty = (paragraphs.begin()->empty() && !paragraphs.begin()->next());
        no_selection = false;
-       RESULT result= UpdatableInset::localDispatch(ev);
+       RESULT result = UpdatableInset::localDispatch(ev);
        if (result != UNDISPATCHED)
                return DISPATCHED;
 
@@ -1201,20 +1154,10 @@ InsetText::localDispatch(FuncRequest const & ev)
                                }
                                break;
                        case FINISHED_DOWN:
-                       {
-                               LyXText *lt = getLyXText(bv);
-                               if (lt->cursor.irow()->next()) {
-                                       lt->setCursorFromCoordinates(
-                                               bv, lt->cursor.ix() + inset_x,
-                                               lt->cursor.iy() -
-                                               lt->cursor.irow()->baseline() +
-                                               lt->cursor.irow()->height() + 1);
-                                       lt->cursor.x_fix(lt->cursor.x());
+                               if ((result = moveDown(bv)) >= FINISHED) {
                                        updateLocal(bv, CURSOR, false);
-                               } else {
                                        bv->unlockInset(this);
                                }
-                       }
                                break;
                        default:
                                result = DISPATCHED;
@@ -1234,6 +1177,7 @@ InsetText::localDispatch(FuncRequest const & ev)
        int updwhat = 0;
        int updflag = false;
        switch (ev.action) {
+
        // Normal chars
        case LFUN_SELFINSERT:
                if (bv->buffer()->isReadonly()) {
@@ -1246,13 +1190,15 @@ InsetText::localDispatch(FuncRequest const & ev)
                         * typed in now. Depends on lyxrc settings
                         * "auto_region_delete", which defaults to
                         * true (on). */
-
+#if 0
+                       // This should not be needed here and is also WRONG!
                        setUndo(bv, Undo::INSERT,
                                lt->cursor.par(), lt->cursor.par()->next());
+#endif
                        bv->switchKeyMap();
                        if (lyxrc.auto_region_delete) {
                                if (lt->selection.set()) {
-                                       lt->cutSelection(bv, false, false);
+                                       lt->cutSelection(false, false);
                                }
                        }
                        lt->clearSelection();
@@ -1270,7 +1216,7 @@ InsetText::localDispatch(FuncRequest const & ev)
        case LFUN_RIGHTSEL:
                finishUndo();
                moveRight(bv, false, true);
-               lt->setSelection(bv);
+               lt->setSelection();
                updwhat = SELECTION;
                break;
        case LFUN_RIGHT:
@@ -1281,7 +1227,7 @@ InsetText::localDispatch(FuncRequest const & ev)
        case LFUN_LEFTSEL:
                finishUndo();
                moveLeft(bv, false, true);
-               lt->setSelection(bv);
+               lt->setSelection();
                updwhat = SELECTION;
                break;
        case LFUN_LEFT:
@@ -1292,7 +1238,7 @@ InsetText::localDispatch(FuncRequest const & ev)
        case LFUN_DOWNSEL:
                finishUndo();
                moveDown(bv);
-               lt->setSelection(bv);
+               lt->setSelection();
                updwhat = SELECTION;
                break;
        case LFUN_DOWN:
@@ -1303,7 +1249,7 @@ InsetText::localDispatch(FuncRequest const & ev)
        case LFUN_UPSEL:
                finishUndo();
                moveUp(bv);
-               lt->setSelection(bv);
+               lt->setSelection();
                updwhat = SELECTION;
                break;
        case LFUN_UP:
@@ -1313,32 +1259,46 @@ InsetText::localDispatch(FuncRequest const & ev)
                break;
        case LFUN_HOME:
                finishUndo();
-               lt->cursorHome(bv);
+               lt->cursorHome();
                updwhat = CURSOR;
                break;
        case LFUN_END:
-               lt->cursorEnd(bv);
+               lt->cursorEnd();
+               updwhat = CURSOR;
+               break;
+       case LFUN_PRIOR:
+               if (!crow(bv)->previous())
+                       result = FINISHED_UP;
+               else {
+                       lt->cursorPrevious();
+                       result = DISPATCHED_NOUPDATE;
+               }
+               updwhat = CURSOR;
+               break;
+       case LFUN_NEXT:
+               if (!crow(bv)->next())
+                       result = FINISHED_DOWN;
+               else {
+                       lt->cursorNext();
+                       result = DISPATCHED_NOUPDATE;
+               }
                updwhat = CURSOR;
                break;
        case LFUN_BACKSPACE: {
-               setUndo(bv, Undo::DELETE,
-                       lt->cursor.par(), lt->cursor.par()->next());
                if (lt->selection.set())
-                       lt->cutSelection(bv, true, false);
+                       lt->cutSelection(true, false);
                else
-                       lt->backspace(bv);
+                       lt->backspace();
                updwhat = CURSOR_PAR;
                updflag = true;
        }
        break;
 
        case LFUN_DELETE: {
-               setUndo(bv, Undo::DELETE,
-                       lt->cursor.par(), lt->cursor.par()->next());
                if (lt->selection.set()) {
-                       lt->cutSelection(bv, true, false);
+                       lt->cutSelection(true, false);
                } else {
-                       lt->Delete(bv);
+                       lt->Delete();
                }
                updwhat = CURSOR_PAR;
                updflag = true;
@@ -1346,8 +1306,6 @@ InsetText::localDispatch(FuncRequest const & ev)
        break;
 
        case LFUN_CUT: {
-               setUndo(bv, Undo::DELETE,
-                       lt->cursor.par(), lt->cursor.par()->next());
                lt->cutSelection(bv);
                updwhat = CURSOR_PAR;
                updflag = true;
@@ -1356,7 +1314,7 @@ InsetText::localDispatch(FuncRequest const & ev)
 
        case LFUN_COPY:
                finishUndo();
-               lt->copySelection(bv);
+               lt->copySelection();
                updwhat = CURSOR_PAR;
                break;
        case LFUN_PASTESELECTION:
@@ -1366,9 +1324,9 @@ InsetText::localDispatch(FuncRequest const & ev)
                if (clip.empty())
                        break;
                if (ev.argument == "paragraph") {
-                       lt->insertStringAsParagraphs(bv, clip);
+                       lt->insertStringAsParagraphs(clip);
                } else {
-                       lt->insertStringAsLines(bv, clip);
+                       lt->insertStringAsLines(clip);
                }
                // bug 393
                lt->clearSelection();
@@ -1381,15 +1339,18 @@ InsetText::localDispatch(FuncRequest const & ev)
                if (!autoBreakRows) {
 
                        if (CutAndPaste::nrOfParagraphs() > 1) {
-                               Alert::alert(_("Impossible operation"),
+                               Alert::alert(_("Impossible operation!"),
                                                   _("Cannot include more than one paragraph!"),
                                                   _("Sorry."));
                                break;
                        }
                }
+#if 0
+               // This should not be needed here and is also WRONG!
                setUndo(bv, Undo::INSERT,
                        lt->cursor.par(), lt->cursor.par()->next());
-               lt->pasteSelection(bv);
+#endif
+               lt->pasteSelection();
                // bug 393
                lt->clearSelection();
                updwhat = CURSOR_PAR;
@@ -1402,7 +1363,7 @@ InsetText::localDispatch(FuncRequest const & ev)
                        result = DISPATCHED;
                        break;
                }
-               lt->breakParagraph(bv, 0);
+               lt->breakParagraph(paragraphs, 0);
                updwhat = CURSOR | FULL;
                updflag = true;
                break;
@@ -1411,7 +1372,7 @@ InsetText::localDispatch(FuncRequest const & ev)
                        result = DISPATCHED;
                        break;
                }
-               lt->breakParagraph(bv, 1);
+               lt->breakParagraph(paragraphs, 1);
                updwhat = CURSOR | FULL;
                updflag = true;
                break;
@@ -1421,9 +1382,12 @@ InsetText::localDispatch(FuncRequest const & ev)
                        result = DISPATCHED;
                        break;
                }
+#if 0
+               // This should not be needed here and is also WRONG!
                setUndo(bv, Undo::INSERT,
                        lt->cursor.par(), lt->cursor.par()->next());
-               lt->insertChar(bv, Paragraph::META_NEWLINE);
+#endif
+               lt->insertInset(new InsetNewline);
                updwhat = CURSOR | CURSOR_PAR;
                updflag = true;
        }
@@ -1458,7 +1422,7 @@ InsetText::localDispatch(FuncRequest const & ev)
 
                        if (cur_layout != layout) {
                                cur_layout = layout;
-                               lt->setLayout(bv, layout);
+                               lt->setLayout(layout);
                                bv->owner()->setLayout(cpar(bv)->layout()->name());
                                updwhat = CURSOR_PAR;
                                updflag = true;
@@ -1542,24 +1506,28 @@ InsetText::localDispatch(FuncRequest const & ev)
 }
 
 
-int InsetText::latex(Buffer const * buf, ostream & os, bool moving_arg, bool) const
+int InsetText::latex(Buffer const * buf, ostream & os,
+                    bool moving_arg, bool) const
 {
        TexRow texrow;
-       buf->latexParagraphs(os, &*(paragraphs.begin()), 0, texrow, moving_arg);
+       latexParagraphs(buf, paragraphs,
+                       paragraphs.begin(), paragraphs.end(),
+                       os, texrow, moving_arg);
        return texrow.rows();
 }
 
 
 int InsetText::ascii(Buffer const * buf, ostream & os, int linelen) const
 {
-       Paragraph * p = &*(paragraphs.begin());
        unsigned int lines = 0;
 
-       while (p) {
-               string const tmp = buf->asciiParagraph(p, linelen, p->previous()==0);
+       ParagraphList::iterator beg = paragraphs.begin();
+       ParagraphList::iterator end = paragraphs.end();
+       ParagraphList::iterator it = beg;
+       for (; it != end; ++it) {
+               string const tmp = buf->asciiParagraph(*it, linelen, it == beg);
                lines += lyx::count(tmp.begin(), tmp.end(), '\n');
                os << tmp;
-               p = p->next();
        }
        return lines;
 }
@@ -1588,11 +1556,11 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                for (; depth > p->params().depth(); --depth) {
                        if (environment_inner[depth] != "!-- --") {
                                item_name = "listitem";
-                               lines += buf->sgmlCloseTag(os, command_depth + depth, mixcont, item_name);
+                               lines += sgml::closeTag(os, command_depth + depth, mixcont, item_name);
                                if (environment_inner[depth] == "varlistentry")
-                                       lines += buf->sgmlCloseTag(os, depth+command_depth, mixcont, environment_inner[depth]);
+                                       lines += sgml::closeTag(os, depth+command_depth, mixcont, environment_inner[depth]);
                        }
-                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, environment_stack[depth]);
+                       lines += sgml::closeTag(os, depth + command_depth, mixcont, environment_stack[depth]);
                        environment_stack[depth].erase();
                        environment_inner[depth].erase();
                }
@@ -1602,12 +1570,12 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                   && !environment_stack[depth].empty()) {
                        if (environment_inner[depth] != "!-- --") {
                                item_name= "listitem";
-                               lines += buf->sgmlCloseTag(os, command_depth+depth, mixcont, item_name);
+                               lines += sgml::closeTag(os, command_depth+depth, mixcont, item_name);
                                if (environment_inner[depth] == "varlistentry")
-                                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, environment_inner[depth]);
+                                       lines += sgml::closeTag(os, depth + command_depth, mixcont, environment_inner[depth]);
                        }
 
-                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, environment_stack[depth]);
+                       lines += sgml::closeTag(os, depth + command_depth, mixcont, environment_stack[depth]);
 
                        environment_stack[depth].erase();
                        environment_inner[depth].erase();
@@ -1616,11 +1584,11 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                // Write opening SGML tags.
                switch (style->latextype) {
                case LATEX_PARAGRAPH:
-                       lines += buf->sgmlOpenTag(os, depth + command_depth, mixcont, style->latexname());
+                       lines += sgml::openTag(os, depth + command_depth, mixcont, style->latexname());
                        break;
 
                case LATEX_COMMAND:
-                       buf->sgmlError(p, 0,  _("Error : LatexType Command not allowed here.\n"));
+                       buf->sgmlError(p, 0,  _("Error: LatexType Command not allowed here.\n"));
                        return -1;
                        break;
 
@@ -1638,13 +1606,13 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                                }
                                environment_stack[depth] = style->latexname();
                                environment_inner[depth] = "!-- --";
-                               lines += buf->sgmlOpenTag(os, depth + command_depth, mixcont, environment_stack[depth]);
+                               lines += sgml::openTag(os, depth + command_depth, mixcont, environment_stack[depth]);
                        } else {
                                if (environment_inner[depth] != "!-- --") {
                                        item_name= "listitem";
-                                       lines += buf->sgmlCloseTag(os, command_depth + depth, mixcont, item_name);
+                                       lines += sgml::closeTag(os, command_depth + depth, mixcont, item_name);
                                        if (environment_inner[depth] == "varlistentry")
-                                               lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, environment_inner[depth]);
+                                               lines += sgml::closeTag(os, depth + command_depth, mixcont, environment_inner[depth]);
                                }
                        }
 
@@ -1653,22 +1621,22 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                                        if (style->latexparam() == "CDATA")
                                                os << "<![CDATA[";
                                        else
-                                         lines += buf->sgmlOpenTag(os, depth + command_depth, mixcont, style->latexparam());
+                                         lines += sgml::openTag(os, depth + command_depth, mixcont, style->latexparam());
                                }
                                break;
                        }
 
                        desc_on = (style->labeltype == LABEL_MANUAL);
 
-                       environment_inner[depth] = desc_on?"varlistentry":"listitem";
-                       lines += buf->sgmlOpenTag(os, depth + 1 + command_depth, mixcont, environment_inner[depth]);
+                       environment_inner[depth] = desc_on ? "varlistentry" : "listitem";
+                       lines += sgml::openTag(os, depth + 1 + command_depth, mixcont, environment_inner[depth]);
 
-                       item_name = desc_on?"term":"para";
-                       lines += buf->sgmlOpenTag(os, depth + 1 + command_depth, mixcont, item_name);
+                       item_name = desc_on ? "term" : "para";
+                       lines += sgml::openTag(os, depth + 1 + command_depth, mixcont, item_name);
 
                        break;
                default:
-                       lines += buf->sgmlOpenTag(os, depth + command_depth, mixcont, style->latexname());
+                       lines += sgml::openTag(os, depth + command_depth, mixcont, style->latexname());
                        break;
                }
 
@@ -1683,19 +1651,19 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                                if (style->latexparam() == "CDATA")
                                        os << "]]>";
                                else
-                                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, style->latexparam());
+                                       lines += sgml::closeTag(os, depth + command_depth, mixcont, style->latexparam());
                        }
                        break;
                case LATEX_ITEM_ENVIRONMENT:
                        if (desc_on == 1) break;
                        end_tag= "para";
-                       lines += buf->sgmlCloseTag(os, depth + 1 + command_depth, mixcont, end_tag);
+                       lines += sgml::closeTag(os, depth + 1 + command_depth, mixcont, end_tag);
                        break;
                case LATEX_PARAGRAPH:
-                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, style->latexname());
+                       lines += sgml::closeTag(os, depth + command_depth, mixcont, style->latexname());
                        break;
                default:
-                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, style->latexname());
+                       lines += sgml::closeTag(os, depth + command_depth, mixcont, style->latexname());
                        break;
                }
        }
@@ -1705,12 +1673,12 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
                if (!environment_stack[depth].empty()) {
                        if (environment_inner[depth] != "!-- --") {
                                item_name = "listitem";
-                               lines += buf->sgmlCloseTag(os, command_depth + depth, mixcont, item_name);
+                               lines += sgml::closeTag(os, command_depth + depth, mixcont, item_name);
                               if (environment_inner[depth] == "varlistentry")
-                                      lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, environment_inner[depth]);
+                                      lines += sgml::closeTag(os, depth + command_depth, mixcont, environment_inner[depth]);
                        }
 
-                       lines += buf->sgmlCloseTag(os, depth + command_depth, mixcont, environment_stack[depth]);
+                       lines += sgml::closeTag(os, depth + command_depth, mixcont, environment_stack[depth]);
                }
        }
 
@@ -1720,20 +1688,8 @@ int InsetText::docbook(Buffer const * buf, ostream & os, bool mixcont) const
 
 void InsetText::validate(LaTeXFeatures & features) const
 {
-       Paragraph * p = &*(paragraphs.begin());
-       while (p) {
-               p->validate(features);
-               p = p->next();
-       }
-}
-
-
-int InsetText::beginningOfMainBody(Paragraph * p) const
-{
-       if (p->layout()->labeltype != LABEL_MANUAL)
-               return 0;
-       else
-               return p->beginningOfMainBody();
+       for_each(paragraphs.begin(), paragraphs.end(),
+                boost::bind(&Paragraph::validate, _1, boost::ref(features)));
 }
 
 
@@ -1826,7 +1782,7 @@ void InsetText::fitInsetCursor(BufferView * bv) const
 }
 
 
-UpdatableInset::RESULT
+Inset::RESULT
 InsetText::moveRight(BufferView * bv, bool activate_inset, bool selecting)
 {
        if (getLyXText(bv)->cursor.par()->isRightToLeftPar(bv->buffer()->params))
@@ -1836,7 +1792,7 @@ InsetText::moveRight(BufferView * bv, bool activate_inset, bool selecting)
 }
 
 
-UpdatableInset::RESULT
+Inset::RESULT
 InsetText::moveLeft(BufferView * bv, bool activate_inset, bool selecting)
 {
        if (getLyXText(bv)->cursor.par()->isRightToLeftPar(bv->buffer()->params))
@@ -1846,7 +1802,7 @@ InsetText::moveLeft(BufferView * bv, bool activate_inset, bool selecting)
 }
 
 
-UpdatableInset::RESULT
+Inset::RESULT
 InsetText::moveRightIntern(BufferView * bv, bool front,
                           bool activate_inset, bool selecting)
 {
@@ -1861,7 +1817,7 @@ InsetText::moveRightIntern(BufferView * bv, bool front,
 }
 
 
-UpdatableInset::RESULT
+Inset::RESULT
 InsetText::moveLeftIntern(BufferView * bv, bool front,
                          bool activate_inset, bool selecting)
 {
@@ -1876,8 +1832,7 @@ InsetText::moveLeftIntern(BufferView * bv, bool front,
 }
 
 
-UpdatableInset::RESULT
-InsetText::moveUp(BufferView * bv)
+Inset::RESULT InsetText::moveUp(BufferView * bv)
 {
        if (!crow(bv)->previous())
                return FINISHED_UP;
@@ -1886,8 +1841,7 @@ InsetText::moveUp(BufferView * bv)
 }
 
 
-UpdatableInset::RESULT
-InsetText::moveDown(BufferView * bv)
+Inset::RESULT InsetText::moveDown(BufferView * bv)
 {
        if (!crow(bv)->next())
                return FINISHED_DOWN;
@@ -1905,7 +1859,7 @@ bool InsetText::insertInset(BufferView * bv, Inset * inset)
        }
        inset->setOwner(this);
        hideInsetCursor(bv);
-       getLyXText(bv)->insertInset(bv, inset);
+       getLyXText(bv)->insertInset(inset);
        bv->fitCursor();
        updateLocal(bv, CURSOR_PAR|CURSOR, true);
        return true;
@@ -1960,15 +1914,15 @@ vector<string> const InsetText::getLabelList() const
 {
        vector<string> label_list;
 
-       Paragraph * tpar = &*(paragraphs.begin());
-       while (tpar) {
-               InsetList::iterator beg = tpar->insetlist.begin();
-               InsetList::iterator end = tpar->insetlist.end();
+       ParagraphList::iterator pit = paragraphs.begin();
+       ParagraphList::iterator pend = paragraphs.end();
+       for (; pit != pend; ++pit) {
+               InsetList::iterator beg = pit->insetlist.begin();
+               InsetList::iterator end = pit->insetlist.end();
                for (; beg != end; ++beg) {
                        vector<string> const l = beg.getInset()->getLabelList();
                        label_list.insert(label_list.end(), l.begin(), l.end());
                }
-               tpar = tpar->next();
        }
        return label_list;
 }
@@ -1982,7 +1936,7 @@ void InsetText::setFont(BufferView * bv, LyXFont const & font, bool toggleall,
                return;
        }
        if ((!paragraphs.begin()->next() && paragraphs.begin()->empty()) || cpar(bv)->empty()) {
-               getLyXText(bv)->setFont(bv, font, toggleall);
+               getLyXText(bv)->setFont(font, toggleall);
                return;
        }
        bool clear = false;
@@ -1995,7 +1949,7 @@ void InsetText::setFont(BufferView * bv, LyXFont const & font, bool toggleall,
        }
        if (selectall)
                selectAll(bv);
-       lt->toggleFree(bv, font, toggleall);
+       lt->toggleFree(font, toggleall);
        if (selectall)
                lt->clearSelection();
        bv->fitCursor();
@@ -2032,7 +1986,7 @@ bool InsetText::checkAndActivateInset(BufferView * bv, int x, int y,
        x -= drawTextXOffset;
        int dummyx = x;
        int dummyy = y + insetAscent;
-       Inset * inset = bv->checkInsetHit(getLyXText(bv), dummyx, dummyy);
+       Inset * inset = getLyXText(bv)->checkInsetHit(dummyx, dummyy);
        // we only do the edit() call if the inset was hit by the mouse
        // or if it is a highly editable inset. So we should call this
        // function from our own edit with button < 0.
@@ -2098,9 +2052,24 @@ void InsetText::setParagraphData(Paragraph * p, bool same_id)
 }
 
 
+void InsetText::markNew(bool track_changes)
+{
+       ParagraphList::iterator pit = paragraphs.begin();
+       ParagraphList::iterator pend = paragraphs.end();
+       for (; pit != pend; ++pit) {
+               if (track_changes) {
+                       pit->trackChanges();
+               } else {
+                       // no-op when not tracking
+                       pit->cleanChanges();
+               }
+       }
+}
+
+
 void InsetText::setText(string const & data, LyXFont const & font)
 {
-       clear();
+       clear(false);
        for (unsigned int i = 0; i < data.length(); ++i)
                paragraphs.begin()->insertChar(i, data[i], font);
        reinitLyXText();
@@ -2246,7 +2215,7 @@ LyXText * InsetText::getLyXText(BufferView const * lbv,
                        // when we have to reinit the existing LyXText!
                        //
                        it->second.text->init(bv);
-                       restoreLyXTextState(bv, it->second.text.get());
+                       restoreLyXTextState(it->second.text.get());
                        it->second.remove = false;
                }
                cached_text = it->second.text;
@@ -2258,9 +2227,9 @@ LyXText * InsetText::getLyXText(BufferView const * lbv,
        ///
        // we are here only if we don't have a BufferView * in the cache!!!
        ///
-       cached_text.reset(new LyXText(const_cast<InsetText *>(this)));
+       cached_text.reset(new LyXText(bv, const_cast<InsetText *>(this)));
        cached_text->init(bv);
-       restoreLyXTextState(bv, cached_text.get());
+       restoreLyXTextState(cached_text.get());
 
        cache.insert(make_pair(bv, cached_text));
 
@@ -2286,10 +2255,8 @@ void InsetText::deleteLyXText(BufferView * bv, bool recursive) const
        it->second.remove = true;
        if (recursive) {
                /// then remove all LyXText in text-insets
-               Paragraph * p = &*(paragraphs.begin());
-               for (; p; p = p->next()) {
-                       p->deleteInsetsLyXText(bv);
-               }
+               for_each(paragraphs.begin(), paragraphs.end(),
+                        boost::bind(&Paragraph::deleteInsetsLyXText, _1, bv));
        }
 }
 
@@ -2309,7 +2276,7 @@ void InsetText::resizeLyXText(BufferView * bv, bool force) const
                LyXText * t = getLyXText(bv);
                saveLyXTextState(t);
                t->init(bv, true);
-               restoreLyXTextState(bv, t);
+               restoreLyXTextState(t);
                return;
        }
        // one endless line, resize normally not necessary
@@ -2324,19 +2291,20 @@ void InsetText::resizeLyXText(BufferView * bv, bool force) const
 
        LyXText * t = it->second.text.get();
        saveLyXTextState(t);
-       for (Paragraph * p = &*(paragraphs.begin()); p; p = p->next()) {
-               p->resizeInsetsLyXText(bv);
-       }
+
+       for_each(paragraphs.begin(), paragraphs.end(),
+                boost::bind(&Paragraph::resizeInsetsLyXText, _1, bv));
+
        t->init(bv, true);
-       restoreLyXTextState(bv, t);
+       restoreLyXTextState(t);
        if (the_locking_inset) {
                inset_x = cix(bv) - top_x + drawTextXOffset;
                inset_y = ciy(bv) + drawTextYOffset;
        }
 
-       t->first_y = bv->screen().topCursorVisible(t->cursor, t->first_y);
+       t->top_y(bv->screen().topCursorVisible(t->cursor, t->top_y()));
        if (!owner()) {
-               updateLocal(bv, FULL, false);
+               const_cast<InsetText*>(this)->updateLocal(bv, FULL, false);
                // this will scroll the screen such that the cursor becomes visible
                bv->updateScrollbar();
        } else {
@@ -2356,26 +2324,26 @@ void InsetText::reinitLyXText() const
        do_reinit = false;
        do_resize = 0;
 //     lyxerr << "InsetText::reinitLyXText\n";
-       for(Cache::iterator it = cache.begin(); it != cache.end(); ++it) {
+       for (Cache::iterator it = cache.begin(); it != cache.end(); ++it) {
                lyx::Assert(it->second.text.get());
 
                LyXText * t = it->second.text.get();
                BufferView * bv = it->first;
 
                saveLyXTextState(t);
-               for (Paragraph * p = &*(paragraphs.begin());
-                    p; p = p->next()) {
-                       p->resizeInsetsLyXText(bv);
-               }
+
+               for_each(paragraphs.begin(), paragraphs.end(),
+                        boost::bind(&Paragraph::resizeInsetsLyXText, _1, bv));
+
                t->init(bv, true);
-               restoreLyXTextState(bv, t);
+               restoreLyXTextState(t);
                if (the_locking_inset) {
                        inset_x = cix(bv) - top_x + drawTextXOffset;
                        inset_y = ciy(bv) + drawTextYOffset;
                }
-               t->first_y = bv->screen().topCursorVisible(t->cursor, t->first_y);
+               t->top_y(bv->screen().topCursorVisible(t->cursor, t->top_y()));
                if (!owner()) {
-                       updateLocal(bv, FULL, false);
+                       const_cast<InsetText*>(this)->updateLocal(bv, FULL, false);
                        // this will scroll the screen such that the cursor becomes visible
                        bv->updateScrollbar();
                } else {
@@ -2389,11 +2357,13 @@ void InsetText::removeNewlines()
 {
        bool changed = false;
 
-       for (Paragraph * p = &*(paragraphs.begin()); p; p = p->next()) {
-               for (int i = 0; i < p->size(); ++i) {
-                       if (p->getChar(i) == Paragraph::META_NEWLINE) {
+       ParagraphList::iterator it = paragraphs.begin();
+       ParagraphList::iterator end = paragraphs.end();
+       for (; it != end; ++it) {
+               for (int i = 0; i < it->size(); ++i) {
+                       if (it->isNewline(i)) {
                                changed = true;
-                               p->erase(i);
+                               it->erase(i);
                        }
                }
        }
@@ -2421,18 +2391,12 @@ int InsetText::scroll(bool recursive) const
 }
 
 
-bool InsetText::doClearArea() const
-{
-       return !locked || (need_update & (FULL|INIT));
-}
-
-
 void InsetText::selectAll(BufferView * bv)
 {
-       getLyXText(bv)->cursorTop(bv);
+       getLyXText(bv)->cursorTop();
        getLyXText(bv)->selection.cursor = getLyXText(bv)->cursor;
-       getLyXText(bv)->cursorBottom(bv);
-       getLyXText(bv)->setSelection(bv);
+       getLyXText(bv)->cursorBottom();
+       getLyXText(bv)->setSelection();
 }
 
 
@@ -2442,7 +2406,7 @@ void InsetText::clearSelection(BufferView * bv)
 }
 
 
-void InsetText::clearInset(BufferView * bv, int baseline, bool & cleared) const
+void InsetText::clearInset(BufferView * bv, int baseline) const
 {
        Painter & pain = bv->painter();
        int w = insetWidth;
@@ -2459,37 +2423,7 @@ void InsetText::clearInset(BufferView * bv, int baseline, bool & cleared) const
                w = pain.paperWidth();
 //     w -= TEXT_TO_INSET_OFFSET;
        pain.fillRectangle(top_x + 1, ty + 1, w - 1, h - 1, backgroundColor());
-       cleared = true;
        need_update = FULL;
-       frame_is_visible = false;
-}
-
-
-Paragraph * InsetText::getParFromID(int id) const
-{
-#if 0
-       Paragraph * result = par;
-       Paragraph * ires = 0;
-       while (result && result->id() != id) {
-               if ((ires = result->getParFromID(id)))
-                       return ires;
-               result = result->next();
-       }
-       return result;
-#else
-       Paragraph * tmp = &*(paragraphs.begin());
-       while (tmp) {
-               if (tmp->id() == id) {
-                       return tmp;
-               }
-               Paragraph * tmp2 = tmp->getParFromID(id);
-               if (tmp2 != 0) {
-                       return tmp2;
-               }
-               tmp = tmp->next();
-       }
-       return 0;
-#endif
 }
 
 
@@ -2511,9 +2445,9 @@ Paragraph * InsetText::getFirstParagraph(int i) const
 
 LyXCursor const & InsetText::cursor(BufferView * bv) const
 {
-               if (the_locking_inset)
-                               return the_locking_inset->cursor(bv);
-               return getLyXText(bv)->cursor;
+       if (the_locking_inset)
+               return the_locking_inset->cursor(bv);
+       return getLyXText(bv)->cursor;
 }
 
 
@@ -2532,11 +2466,9 @@ void InsetText::paragraph(Paragraph * p)
        // and are done!
        paragraphs.set(p);
        // set ourself as owner for all the paragraphs inserted!
-       Paragraph * np = &*(paragraphs.begin());
-       while (np) {
-               np->setInsetOwner(this);
-               np = np->next();
-       }
+       for_each(paragraphs.begin(), paragraphs.end(),
+                boost::bind(&Paragraph::setInsetOwner, _1, this));
+
        reinitLyXText();
        // redraw myself when asked for
        need_update = INIT;
@@ -2548,20 +2480,18 @@ Inset * InsetText::getInsetFromID(int id_arg) const
        if (id_arg == id())
                return const_cast<InsetText *>(this);
 
-       Paragraph * lp = &*(paragraphs.begin());
-
-       while (lp) {
-               for (InsetList::iterator it = lp->insetlist.begin(),
-                        en = lp->insetlist.end();
-                        it != en; ++it)
-               {
+       ParagraphList::iterator pit = paragraphs.begin();
+       ParagraphList::iterator pend = paragraphs.end();
+       for (; pit != pend; ++pit) {
+               InsetList::iterator it = pit->insetlist.begin();
+               InsetList::iterator end = pit->insetlist.end();
+               for (; it != end; ++it) {
                        if (it.getInset()->id() == id_arg)
                                return it.getInset();
                        Inset * in = it.getInset()->getInsetFromID(id_arg);
                        if (in)
                                return in;
                }
-               lp = lp->next();
        }
        return 0;
 }
@@ -2589,7 +2519,7 @@ InsetText::selectNextWordToSpellcheck(BufferView * bv,
                // we have to go on checking so move cursor to the next char
                lt->cursor.pos(lt->cursor.pos() + 1);
        }
-       word = lt->selectNextWordToSpellcheck(bv, value);
+       word = lt->selectNextWordToSpellcheck(value);
        if (word.word().empty())
                bv->unlockInset(const_cast<InsetText *>(this));
        else
@@ -2606,7 +2536,7 @@ void InsetText::selectSelectedWord(BufferView * bv)
                the_locking_inset->selectSelectedWord(bv);
                return;
        }
-       getLyXText(bv)->selectSelectedWord(bv);
+       getLyXText(bv)->selectSelectedWord();
        updateLocal(bv, SELECTION, false);
 }
 
@@ -2642,6 +2572,36 @@ void InsetText::toggleSelection(BufferView * bv, bool kill_selection)
 }
 
 
+bool InsetText::nextChange(BufferView * bv, lyx::pos_type & length)
+{
+       bool clear = false;
+       if (!lt) {
+               lt = getLyXText(bv);
+               clear = true;
+       }
+       if (the_locking_inset) {
+               if (the_locking_inset->nextChange(bv, length))
+                       return true;
+               lt->cursorRight(true);
+       }
+       lyxfind::SearchResult result =
+               lyxfind::findNextChange(bv, lt, length);
+
+       if (result == lyxfind::SR_FOUND) {
+               LyXCursor cur = lt->cursor;
+               bv->unlockInset(bv->theLockingInset());
+               if (bv->lockInset(this))
+                       locked = true;
+               lt->cursor = cur;
+               lt->setSelectionRange(length);
+               updateLocal(bv, SELECTION, false);
+       }
+       if (clear)
+               lt = 0;
+       return result != lyxfind::SR_NOT_FOUND;
+}
+
+
 bool InsetText::searchForward(BufferView * bv, string const & str,
                              bool cs, bool mw)
 {
@@ -2653,7 +2613,7 @@ bool InsetText::searchForward(BufferView * bv, string const & str,
        if (the_locking_inset) {
                if (the_locking_inset->searchForward(bv, str, cs, mw))
                        return true;
-               lt->cursorRight(bv, true);
+               lt->cursorRight(true);
        }
        lyxfind::SearchResult result =
                lyxfind::LyXFind(bv, lt, str, true, cs, mw);
@@ -2664,7 +2624,7 @@ bool InsetText::searchForward(BufferView * bv, string const & str,
                if (bv->lockInset(this))
                        locked = true;
                lt->cursor = cur;
-               lt->setSelectionOverString(bv, str);
+               lt->setSelectionRange(str.length());
                updateLocal(bv, SELECTION, false);
        }
        if (clear)
@@ -2688,7 +2648,7 @@ bool InsetText::searchBackward(BufferView * bv, string const & str,
                Paragraph * p = &*(paragraphs.begin());
                while (p->next())
                        p = p->next();
-               lt->setCursor(bv, p, p->size());
+               lt->setCursor(p, p->size());
        }
        lyxfind::SearchResult result =
                lyxfind::LyXFind(bv, lt, str, false, cs, mw);
@@ -2699,7 +2659,7 @@ bool InsetText::searchBackward(BufferView * bv, string const & str,
                if (bv->lockInset(this))
                        locked = true;
                lt->cursor = cur;
-               lt->setSelectionOverString(bv, str);
+               lt->setSelectionRange(str.length());
                updateLocal(bv, SELECTION, false);
        }
        if (clear)
@@ -2716,9 +2676,8 @@ bool InsetText::checkInsertChar(LyXFont & font)
 }
 
 
-void InsetText::collapseParagraphs(BufferView * bv) const
+void InsetText::collapseParagraphs(BufferView * bv)
 {
-       BufferParams const & bparams = bv->buffer()->params;
        LyXText * llt = getLyXText(bv);
 
        while (paragraphs.begin()->next()) {
@@ -2739,7 +2698,7 @@ void InsetText::collapseParagraphs(BufferView * bv) const
                                        llt->selection.end.pos() + paragraphs.begin()->size());
                        }
                }
-               paragraphs.begin()->pasteParagraph(bparams);
+               mergeParagraph(bv->buffer()->params, paragraphs, paragraphs.begin());
        }
        reinitLyXText();
 }
@@ -2753,18 +2712,23 @@ void InsetText::getDrawFont(LyXFont & font) const
 }
 
 
-void InsetText::appendParagraphs(BufferParams const & bparams,
+void InsetText::appendParagraphs(Buffer * buffer,
                                 Paragraph * newpar)
 {
+       BufferParams const & bparams = buffer->params;
        Paragraph * buf;
        Paragraph * tmpbuf = newpar;
        Paragraph * lastbuffer = buf = new Paragraph(*tmpbuf, false);
+       if (bparams.tracking_changes)
+               buf->cleanChanges();
 
        while (tmpbuf->next()) {
                tmpbuf = tmpbuf->next();
                lastbuffer->next(new Paragraph(*tmpbuf, false));
                lastbuffer->next()->previous(lastbuffer);
                lastbuffer = lastbuffer->next();
+               if (bparams.tracking_changes)
+                       lastbuffer->cleanChanges();
        }
        lastbuffer = &*(paragraphs.begin());
        while (lastbuffer->next())
@@ -2781,7 +2745,7 @@ void InsetText::appendParagraphs(BufferParams const & bparams,
        // paste it!
        lastbuffer->next(buf);
        buf->previous(lastbuffer);
-       lastbuffer->pasteParagraph(bparams);
+       mergeParagraph(buffer->params, paragraphs, lastbuffer);
 
        reinitLyXText();
 }