]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insettabular.C
The markDirty() and fitCursor() changes
[lyx.git] / src / insets / insettabular.C
index 194b18401af0f938909d0a865cad3994eaac9853..c48a62b75f5a12c308ef24102b077e2fb447177d 100644 (file)
@@ -1,42 +1,41 @@
-/* This file is part of
- * ======================================================
+/**
+ * \file insettabular.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 2001 The LyX Team.
- *
- * ======================================================
+ * Full author contact details are available in file CREDITS
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
 
 #include "insettabular.h"
+#include "insettext.h"
 
-#include "lyx_cb.h"
 #include "buffer.h"
+#include "BufferView.h"
 #include "commandtags.h"
-#include "lyxfunc.h"
-#include "debug.h"
-#include "LaTeXFeatures.h"
-#include "frontends/Painter.h"
-#include "frontends/font_metrics.h"
-#include "lyxtext.h"
-#include "frontends/LyXView.h"
-#include "insets/insettext.h"
 #include "debug.h"
+#include "funcrequest.h"
 #include "gettext.h"
 #include "language.h"
-#include "BufferView.h"
-#include "undo_funcs.h"
+#include "LaTeXFeatures.h"
+#include "lyx_cb.h"
+#include "lyxfunc.h"
 #include "lyxlength.h"
+#include "lyxlex.h"
+#include "lyxtext.h"
 #include "ParagraphParameters.h"
+#include "undo_funcs.h"
+#include "WordLangTuple.h"
 
-#include "frontends/Dialogs.h"
 #include "frontends/Alert.h"
+#include "frontends/Dialogs.h"
+#include "frontends/font_metrics.h"
+#include "frontends/LyXView.h"
+#include "frontends/Painter.h"
 
 #include "support/LAssert.h"
 #include "support/lstrings.h"
@@ -84,6 +83,7 @@ TabularFeature tabularFeature[] =
        { LyXTabular::ALIGN_LEFT, "align-left" },
        { LyXTabular::ALIGN_RIGHT, "align-right" },
        { LyXTabular::ALIGN_CENTER, "align-center" },
+       { LyXTabular::ALIGN_BLOCK, "align-block" },
        { LyXTabular::VALIGN_TOP, "valign-top" },
        { LyXTabular::VALIGN_BOTTOM, "valign-bottom" },
        { LyXTabular::VALIGN_CENTER, "valign-center" },
@@ -119,9 +119,29 @@ TabularFeature tabularFeature[] =
        { LyXTabular::LAST_ACTION, "" }
 };
 
+struct FindFeature {
+       FindFeature(LyXTabular::Feature feature) : feature_(feature) {}
+       bool operator()(TabularFeature & tf)
+       {
+               return tf.action == feature_;
+       }
+private:
+       LyXTabular::Feature feature_;
+};
+
 } // namespace anon
 
 
+string const featureAsString(LyXTabular::Feature feature)
+{
+       TabularFeature * it  = tabularFeature;
+       TabularFeature * end = it +
+               sizeof(tabularFeature) / sizeof(TabularFeature);
+       it = std::find_if(it, end, FindFeature(feature));
+       return (it == end) ? string() : it->feature;
+}
+
+
 bool InsetTabular::hasPasteBuffer() const
 {
        return (paste_tabular != 0);
@@ -146,7 +166,7 @@ InsetTabular::InsetTabular(Buffer const & buf, int rows, int columns)
        clearSelection();
        need_update = INIT;
        in_update = false;
-       in_reset_pos = false;
+       in_reset_pos = 0;
        inset_x = 0;
        inset_y = 0;
 }
@@ -166,7 +186,7 @@ InsetTabular::InsetTabular(InsetTabular const & tab, Buffer const & buf,
        clearSelection();
        need_update = INIT;
        in_update = false;
-       in_reset_pos = false;
+       in_reset_pos = 0;
        inset_x = 0;
        inset_y = 0;
 }
@@ -174,7 +194,8 @@ InsetTabular::InsetTabular(InsetTabular const & tab, Buffer const & buf,
 
 InsetTabular::~InsetTabular()
 {
-       hideDialog();
+       InsetTabularMailer mailer(*this);
+       mailer.hideDialog();
 }
 
 
@@ -184,6 +205,12 @@ Inset * InsetTabular::clone(Buffer const & buf, bool same_id) const
 }
 
 
+BufferView * InsetTabular::view() const
+{
+       return buffer->getUser();
+}
+
+
 void InsetTabular::write(Buffer const * buf, ostream & os) const
 {
        os << " Tabular" << endl;
@@ -234,20 +261,12 @@ int InsetTabular::width(BufferView *, LyXFont const &) const
 
 
 void InsetTabular::draw(BufferView * bv, LyXFont const & font, int baseline,
-                       float & x, bool cleared) const
+                       float & x) const
 {
        if (nodraw()) {
-               if (cleared)
-                       need_update = FULL;
-               return;
-       }
-#if 0
-       if (need_update == INIT) {
-               if (calculate_dimensions_of_cells(bv, font, true))
-                       bv->text->status = LyXText::CHANGED_IN_DRAW;
                need_update = FULL;
+               return;
        }
-#endif
 
        Painter & pain = bv->painter();
        int i;
@@ -255,173 +274,53 @@ void InsetTabular::draw(BufferView * bv, LyXFont const & font, int baseline,
        int nx;
 
 #if 0
-       UpdatableInset::draw(bv, font, baseline, x, cleared);
+       UpdatableInset::draw(bv, font, baseline, x);
 #else
        if (!owner())
                x += static_cast<float>(scroll());
 #endif
-       if (!cleared && ((need_update == INIT) || (need_update == FULL) ||
-                        (top_x != int(x)) || (top_baseline != baseline)))
-       {
-               int h = ascent(bv, font) + descent(bv, font);
-               int const tx = display() || !owner() ? 0 : top_x;
-               int w =  tx ? width(bv, font) : pain.paperWidth();
-               int ty = baseline - ascent(bv, font);
-
-               if (ty < 0)
-                       ty = 0;
-               if ((ty + h) > pain.paperHeight())
-                       h = pain.paperHeight();
-               if ((top_x + w) > pain.paperWidth())
-                       w = pain.paperWidth();
-               pain.fillRectangle(tx, ty, w, h, backgroundColor());
-               need_update = FULL;
-               cleared = true;
-       }
+
        top_x = int(x);
        topx_set = true;
        top_baseline = baseline;
        x += ADD_TO_TABULAR_WIDTH;
-       if (cleared) {
-               int cell = 0;
-               float cx;
-               first_visible_cell = -1;
-               for (i = 0; i < tabular->rows(); ++i) {
-                       nx = int(x);
-                       cell = tabular->GetCellNumber(i, 0);
-                       if (!((baseline + tabular->GetDescentOfRow(i)) > 0) &&
-                               (baseline - tabular->GetAscentOfRow(i))<pain.paperHeight())
-                       {
-                               baseline += tabular->GetDescentOfRow(i) +
-                                       tabular->GetAscentOfRow(i + 1) +
-                                       tabular->GetAdditionalHeight(i + 1);
-                               continue;
-                       }
-                       for (j = 0; j < tabular->columns(); ++j) {
-                               if (nx > bv->workWidth())
-                                       break;
-                               if (tabular->IsPartOfMultiColumn(i, j))
-                                       continue;
-                               cx = nx + tabular->GetBeginningOfTextInCell(cell);
-                               if (first_visible_cell < 0)
-                                       first_visible_cell = cell;
-                               if (hasSelection()) {
-                                       drawCellSelection(pain, nx, baseline, i, j, cell);
-                               }
 
-                               tabular->GetCellInset(cell)->draw(bv, font, baseline, cx, cleared);
-                               drawCellLines(pain, nx, baseline, i, cell);
-                               nx += tabular->GetWidthOfColumn(cell);
-                               ++cell;
-                       }
-                       baseline += tabular->GetDescentOfRow(i) +
+       int cell = 0;
+       float cx;
+       first_visible_cell = -1;
+       for (i = 0; i < tabular->rows(); ++i) {
+               nx = int(x);
+               cell = tabular->GetCellNumber(i, 0);
+               if (!((baseline + tabular->GetDescentOfRow(i)) > 0) &&
+                       (baseline - tabular->GetAscentOfRow(i))<pain.paperHeight())
+               {
+               baseline += tabular->GetDescentOfRow(i) +
                                tabular->GetAscentOfRow(i + 1) +
                                tabular->GetAdditionalHeight(i + 1);
+                       continue;
                }
-       } else if (need_update == CELL) {
-               int cell = 0;
-               nx = int(x);
-               if (the_locking_inset &&
-                       tabular->GetCellInset(actcell) != the_locking_inset)
-               {
-                       Inset * inset = tabular->GetCellInset(cell);
-                       for (i = 0;
-                            inset != the_locking_inset && i < tabular->rows();
-                            ++i)
-                       {
-                               for (j = 0;
-                                    inset != the_locking_inset && j < tabular->columns();
-                                    ++j)
-                               {
-                                       if (tabular->IsPartOfMultiColumn(i, j))
-                                               continue;
-                                       nx += tabular->GetWidthOfColumn(cell);
-                                       ++cell;
-                                       inset = tabular->GetCellInset(cell);
-                               }
-                               if (tabular->row_of_cell(cell) > i) {
-                                       nx = int(x);
-                                       baseline += tabular->GetDescentOfRow(i) +
-                                               tabular->GetAscentOfRow(i + 1) +
-                                               tabular->GetAdditionalHeight(i + 1);
-                               }
-                       }
-               } else {
-                       // compute baseline for actual row
-                       for (i = 0; i < actrow; ++i) {
-                               baseline += tabular->GetDescentOfRow(i) +
-                                       tabular->GetAscentOfRow(i + 1) +
-                                       tabular->GetAdditionalHeight(i + 1);
-                       }
-                       // now compute the right x position
-                       cell = tabular->GetCellNumber(actrow, 0);
-                       for (j = 0; (cell < actcell) && (j < tabular->columns()); ++j) {
-                                       if (tabular->IsPartOfMultiColumn(actrow, j))
-                                               continue;
-                                       nx += tabular->GetWidthOfColumn(cell);
-                                       ++cell;
+               for (j = 0; j < tabular->columns(); ++j) {
+                       if (nx > bv->workWidth())
+                               break;
+                       if (tabular->IsPartOfMultiColumn(i, j))
+                               continue;
+                       cx = nx + tabular->GetBeginningOfTextInCell(cell);
+                       if (first_visible_cell < 0)
+                               first_visible_cell = cell;
+                       if (hasSelection()) {
+                               drawCellSelection(pain, nx, baseline, i, j, cell);
                        }
+
+                       tabular->GetCellInset(cell)->draw(bv, font, baseline, cx);
+                       drawCellLines(pain, nx, baseline, i, cell);
+                       nx += tabular->GetWidthOfColumn(cell);
+                       ++cell;
                }
-               i = tabular->row_of_cell(cell);
-               if (the_locking_inset != tabular->GetCellInset(cell)) {
-                       lyxerr[Debug::INSETTEXT] << "ERROR this shouldn't happen\n";
-                       return;
-               }
-               float dx = nx + tabular->GetBeginningOfTextInCell(cell);
-               float cx = dx;
-               tabular->GetCellInset(cell)->draw(bv, font, baseline, dx, false);
-               //
-               // Here we use rectangular backgroundColor patches to clean up
-               // within a cell around the cell's red inset box. As follows:
-               //
-               //  +--------------------+
-               //  |         C          |   The rectangles are A, B and C
-               //  | A |------------| B |   below, origin top left (tx, ty),
-               //  |   |  inset box |   |   dimensions w(idth), h(eight).
-               //  +---+------------+---+   x grows rightward, y downward
-               //  |         D          |
-               //  +--------------------+
-               //
-#if 0
-               // clear only if we didn't have a change
-               if (bv->text->status() != LyXText::CHANGED_IN_DRAW) {
-#endif
-                       // clear before the inset
-                       int tx, ty, w, h;
-                       tx = nx + 1;
-                       ty = baseline - tabular->GetAscentOfRow(i) + 1;
-                       w = int(cx - nx - 1);
-                       h = tabular->GetAscentOfRow(i) +
-                               tabular->GetDescentOfRow(i) - 1;
-                       pain.fillRectangle(tx, ty, w, h, backgroundColor());
-                       // clear behind the inset
-                       tx = int(cx + the_locking_inset->width(bv,font) + 1);
-                       ty = baseline - tabular->GetAscentOfRow(i) + 1;
-                       w = tabular->GetWidthOfColumn(cell) -
-                               tabular->GetBeginningOfTextInCell(cell) -
-                               the_locking_inset->width(bv,font) -
-                               tabular->GetAdditionalWidth(cell) - 1;
-                       h = tabular->GetAscentOfRow(i) + tabular->GetDescentOfRow(i) - 1;
-                       pain.fillRectangle(tx, ty, w, h, backgroundColor());
-                       // clear below the inset
-                       tx = nx + 1;
-                       ty = baseline + the_locking_inset->descent(bv, font) + 1;
-                       w = tabular->GetWidthOfColumn(cell) -
-                               tabular->GetAdditionalWidth(cell) - 1;
-                       h = tabular->GetDescentOfRow(i) -
-                               the_locking_inset->descent(bv, font) - 1;
-                       pain.fillRectangle(tx, ty, w, h, backgroundColor());
-                       // clear above the inset
-                       tx = nx + 1;
-                       ty = baseline - tabular->GetAscentOfRow(i) + 1;
-                       w = tabular->GetWidthOfColumn(cell) -
-                               tabular->GetAdditionalWidth(cell) - 1;
-                       h = tabular->GetAscentOfRow(i) - the_locking_inset->ascent(bv, font);
-                       pain.fillRectangle(tx, ty, w, h, backgroundColor());
-#if 0
-               }
-#endif
+               baseline += tabular->GetDescentOfRow(i) +
+                       tabular->GetAscentOfRow(i + 1) +
+                       tabular->GetAdditionalHeight(i + 1);
        }
+
        x -= ADD_TO_TABULAR_WIDTH;
        x += width(bv, font);
        if (bv->text->status() == LyXText::CHANGED_IN_DRAW) {
@@ -442,7 +341,7 @@ void InsetTabular::drawCellLines(Painter & pain, int x, int baseline,
        int x2 = x + tabular->GetWidthOfColumn(cell);
        bool on_off;
 
-       if (!tabular->TopAlreadyDrawed(cell)) {
+       if (!tabular->topAlreadyDrawn(cell)) {
                on_off = !tabular->TopLine(cell);
                pain.line(x, baseline - tabular->GetAscentOfRow(row),
                          x2, baseline -  tabular->GetAscentOfRow(row),
@@ -454,7 +353,7 @@ void InsetTabular::drawCellLines(Painter & pain, int x, int baseline,
                  x2, baseline + tabular->GetDescentOfRow(row),
                  on_off ? LColor::tabularonoffline : LColor::tabularline,
                  on_off ? Painter::line_onoffdash : Painter::line_solid);
-       if (!tabular->LeftAlreadyDrawed(cell)) {
+       if (!tabular->leftAlreadyDrawn(cell)) {
                on_off = !tabular->LeftLine(cell);
                pain.line(x, baseline -  tabular->GetAscentOfRow(row),
                          x, baseline +  tabular->GetDescentOfRow(row),
@@ -585,10 +484,17 @@ void InsetTabular::edit(BufferView * bv, bool front)
        the_locking_inset = 0;
        inset_x = 0;
        inset_y = 0;
-       if (front)
-               actcell = 0;
-       else
-               actcell = tabular->GetNumberOfCells() - 1;
+       if (front) {
+               if (isRightToLeft(bv))
+                       actcell = tabular->GetLastCellInRow(0);
+               else
+                       actcell = 0;
+       } else {
+               if (isRightToLeft(bv))
+                       actcell = tabular->GetFirstCellInRow(tabular->rows()-1);
+               else
+                       actcell = tabular->GetNumberOfCells() - 1;
+       }
        clearSelection();
        resetPos(bv);
        bv->fitCursor();
@@ -603,6 +509,7 @@ void InsetTabular::insetUnlock(BufferView * bv)
                the_locking_inset = 0;
        }
        hideInsetCursor(bv);
+       actcell = 0;
        oldcell = -1;
        locked = false;
        if (scroll(false) || hasSelection()) {
@@ -626,15 +533,10 @@ void InsetTabular::updateLocal(BufferView * bv, UpdateCodes what,
                what = FULL;
        if (need_update < what) // only set this if it has greater update
                need_update = what;
-#if 0 // maybe this should not be done!
-       if ((what == INIT) && hasSelection()) {
-               clearSelection();
-       }
-#endif
        // Dirty Cast! (Lgb)
        if (need_update != NONE) {
-               bv->updateInset(const_cast<InsetTabular *>(this), mark_dirty);
-               if (locked) // && (what != NONE))
+               bv->updateInset(const_cast<InsetTabular *>(this));
+               if (locked)
                        resetPos(bv);
        }
 }
@@ -705,7 +607,8 @@ bool InsetTabular::unlockInsetInInset(BufferView * bv, UpdatableInset * inset,
        if (the_locking_inset->unlockInsetInInset(bv, inset, lr)) {
                if (inset->lyxCode() == TABULAR_CODE &&
                    !the_locking_inset->getFirstLockingInsetOfType(TABULAR_CODE)) {
-                       bv->owner()->getDialogs()->updateTabular(this);
+                       InsetTabularMailer mailer(*this);
+                       mailer.updateDialog(bv);
                        oldcell = actcell;
                }
                return true;
@@ -766,18 +669,19 @@ bool InsetTabular::insertInset(BufferView * bv, Inset * inset)
 }
 
 
-void InsetTabular::insetButtonPress(BufferView * bv, int x, int y, mouse_button::state button)
+void InsetTabular::lfunMousePress(FuncRequest const & cmd)
 {
-       if (hasSelection() && (button == mouse_button::button3))
+       if (hasSelection() && cmd.button() == mouse_button::button3)
                return;
 
        if (hasSelection()) {
                clearSelection();
-               updateLocal(bv, SELECTION, false);
+               updateLocal(cmd.view(), SELECTION, false);
        }
 
        int const ocell = actcell;
        int const orow = actrow;
+       BufferView * bv = cmd.view();
 
        hideInsetCursor(bv);
        if (!locked) {
@@ -786,12 +690,12 @@ void InsetTabular::insetButtonPress(BufferView * bv, int x, int y, mouse_button:
                inset_x = 0;
                inset_y = 0;
        }
-       setPos(bv, x, y);
+       setPos(bv, cmd.x, cmd.y);
        if (actrow != orow)
                updateLocal(bv, NONE, false);
        clearSelection();
 #if 0
-       if (button == 3) {
+       if (cmd.button() == mouse_button::button3) {
                if ((ocell != actcell) && the_locking_inset) {
                        the_locking_inset->insetUnlock(bv);
                        updateLocal(bv, CELL, false);
@@ -802,90 +706,103 @@ void InsetTabular::insetButtonPress(BufferView * bv, int x, int y, mouse_button:
        }
 #endif
 
-       bool const inset_hit = insetHit(bv, x, y);
+       bool const inset_hit = insetHit(bv, cmd.x, cmd.y);
 
        if ((ocell == actcell) && the_locking_inset && inset_hit) {
                resetPos(bv);
-               the_locking_inset->insetButtonPress(bv,
-                                                   x - inset_x, y - inset_y,
-                                                   button);
+               FuncRequest cmd1 = cmd;
+               cmd1.x -= inset_x;
+               cmd1.y -= inset_y;
+               the_locking_inset->localDispatch(cmd1);
                return;
-       } else if (the_locking_inset) {
+       }
+
+       if (the_locking_inset) {
                the_locking_inset->insetUnlock(bv);
                updateLocal(bv, CELL, false);
                the_locking_inset = 0;
        }
-       if (button == mouse_button::button2) {
-               localDispatch(bv, LFUN_PASTESELECTION, "paragraph");
+
+       if (cmd.button() == mouse_button::button2) {
+               localDispatch(FuncRequest(bv, LFUN_PASTESELECTION, "paragraph"));
                return;
        }
+
        if (inset_hit && bv->theLockingInset()) {
-               if (!bv->lockInset(static_cast<UpdatableInset*>(tabular->GetCellInset(actcell)))) {
+               if (!bv->lockInset(static_cast<UpdatableInset*>
+                               (tabular->GetCellInset(actcell))))
+               {
                        lyxerr[Debug::INSETS] << "Cannot lock inset" << endl;
                        return;
                }
-               the_locking_inset->insetButtonPress(
-                       bv, x - inset_x, y - inset_y, button);
+               FuncRequest cmd1 = cmd;
+               cmd1.x -= inset_x;
+               cmd1.y -= inset_y;
+               the_locking_inset->localDispatch(cmd1);
                return;
        }
        showInsetCursor(bv);
 }
 
 
-bool InsetTabular::insetButtonRelease(BufferView * bv,
-                                     int x, int y, mouse_button::state button)
+bool InsetTabular::lfunMouseRelease(FuncRequest const & cmd)
 {
        bool ret = false;
-       if (the_locking_inset)
-               ret = the_locking_inset->insetButtonRelease(bv, x - inset_x,
-                                                                                                       y - inset_y, button);
-       if (button == mouse_button::button3 && !ret) {
-               bv->owner()->getDialogs()->showTabular(this);
+       if (the_locking_inset) {
+               FuncRequest cmd1 = cmd;
+               cmd1.x -= inset_x;
+               cmd1.y -= inset_y;
+               ret = the_locking_inset->localDispatch(cmd1);
+       }
+       if (cmd.button() == mouse_button::button3 && !ret) {
+               InsetTabularMailer mailer(*this);
+               mailer.showDialog(cmd.view());
                return true;
        }
        return ret;
 }
 
 
-void InsetTabular::insetMotionNotify(BufferView * bv, int x, int y, mouse_button::state button)
+void InsetTabular::lfunMouseMotion(FuncRequest const & cmd)
 {
        if (the_locking_inset) {
-               the_locking_inset->insetMotionNotify(bv,
-                                                    x - inset_x,
-                                                    y - inset_y,
-                                                    button);
+               FuncRequest cmd1 = cmd;
+               cmd1.x -= inset_x;
+               cmd1.y -= inset_y;
+               the_locking_inset->localDispatch(cmd1);
                return;
        }
 
+       BufferView * bv = cmd.view();
        hideInsetCursor(bv);
-//     int const old_cell = actcell;
+       int const old_cell = actcell;
 
-       setPos(bv, x, y);
+       setPos(bv, cmd.x, cmd.y);
        if (!hasSelection()) {
                setSelection(actcell, actcell);
-       } else {
+               updateLocal(bv, SELECTION, false);
+       } else if (old_cell != actcell) {
                setSelection(sel_cell_start, actcell);
+               updateLocal(bv, SELECTION, false);
        }
-       updateLocal(bv, SELECTION, false);
        showInsetCursor(bv);
 }
 
 
-UpdatableInset::RESULT
-InsetTabular::localDispatch(BufferView * bv, kb_action action,
-                           string const & arg)
+Inset::RESULT InsetTabular::localDispatch(FuncRequest const & cmd)
 {
        // We need to save the value of the_locking_inset as the call to
-       // the_locking_inset->LocalDispatch might unlock it.
+       // the_locking_inset->localDispatch might unlock it.
        old_locking_inset = the_locking_inset;
-       UpdatableInset::RESULT result =
-               UpdatableInset::localDispatch(bv, action, arg);
+       RESULT result = UpdatableInset::localDispatch(cmd);
+
+       BufferView * bv = cmd.view();
        if (result == DISPATCHED || result == DISPATCHED_NOUPDATE) {
                resetPos(bv);
                return result;
        }
 
-       if ((action < 0) && arg.empty())
+       if (cmd.action < 0 && cmd.argument.empty())
                return FINISHED;
 
        bool hs = hasSelection();
@@ -894,12 +811,23 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        // this one have priority over the locked InsetText, if we're not already
        // inside another tabular then that one get's priority!
        if (getFirstLockingInsetOfType(Inset::TABULAR_CODE) == this) {
-               switch (action) {
+               switch (cmd.action) {
+               case LFUN_MOUSE_PRESS:
+                       lfunMousePress(cmd);
+                       return DISPATCHED;
+
+               case LFUN_MOUSE_MOTION:
+                       lfunMouseMotion(cmd);
+                       return DISPATCHED;
+
+               case LFUN_MOUSE_RELEASE:
+                       return lfunMouseRelease(cmd) ? DISPATCHED : UNDISPATCHED;
+
                case LFUN_SHIFT_TAB:
                case LFUN_TAB:
                        hideInsetCursor(bv);
                        unlockInsetInInset(bv, the_locking_inset);
-                       if (action == LFUN_TAB)
+                       if (cmd.action == LFUN_TAB)
                                moveNextCell(bv, old_locking_inset != 0);
                        else
                                movePrevCell(bv, old_locking_inset != 0);
@@ -917,8 +845,10 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                }
        }
 
+       kb_action action = cmd.action;
+       string    arg    = cmd.argument;
        if (the_locking_inset) {
-               result = the_locking_inset->localDispatch(bv, action, arg);
+               result = the_locking_inset->localDispatch(cmd);
                if (result == DISPATCHED_NOUPDATE) {
                        int sc = scroll();
                        resetPos(bv);
@@ -1051,10 +981,10 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                }
                int column = actcol;
                unlockInsetInInset(bv, the_locking_inset);
-               if (bv->text->first_y + bv->painter().paperHeight() <
+               if (bv->text->top_y() + bv->painter().paperHeight() <
                    (top_baseline + tabular->GetHeightOfTabular()))
                        {
-                               bv->scrollDocView(bv->text->first_y + bv->painter().paperHeight());
+                               bv->scrollDocView(bv->text->top_y() + bv->painter().paperHeight());
                                code = FULL;
                                actcell = tabular->GetCellBelow(first_visible_cell) + column;
                        } else {
@@ -1073,7 +1003,7 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                int column = actcol;
                unlockInsetInInset(bv, the_locking_inset);
                if (top_baseline < 0) {
-                       bv->scrollDocView(bv->text->first_y - bv->painter().paperHeight());
+                       bv->scrollDocView(bv->text->top_y() - bv->painter().paperHeight());
                        code = FULL;
                        if (top_baseline > 0)
                                actcell = column;
@@ -1095,12 +1025,12 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        case LFUN_WORDLEFTSEL:
        case LFUN_WORDRIGHT:
        case LFUN_WORDRIGHTSEL:
+       case LFUN_WORDSEL:
        case LFUN_DOWN_PARAGRAPH:
        case LFUN_DOWN_PARAGRAPHSEL:
        case LFUN_UP_PARAGRAPH:
        case LFUN_UP_PARAGRAPHSEL:
        case LFUN_BACKSPACE:
-       case LFUN_DELETE:
        case LFUN_HOME:
        case LFUN_HOMESEL:
        case LFUN_END:
@@ -1110,9 +1040,16 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        case LFUN_ENDBUF:
        case LFUN_ENDBUFSEL:
                break;
-       case LFUN_LAYOUT_TABULAR:
-               bv->owner()->getDialogs()->showTabular(this);
+       case LFUN_LAYOUT_TABULAR: {
+               InsetTabularMailer mailer(*this);
+               mailer.showDialog(bv);
                break;
+       }
+       case LFUN_INSET_DIALOG_UPDATE: {
+               InsetTabularMailer mailer(*this);
+               mailer.updateDialog(bv);
+               break;
+       }
        case LFUN_TABULAR_FEATURE:
                if (!tabularFeatures(bv, arg))
                        result = UNDISPATCHED;
@@ -1134,10 +1071,12 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        case LFUN_CUT:
                if (!copySelection(bv))
                        break;
+               // no break here!
+       case LFUN_DELETE:
                setUndo(bv, Undo::DELETE,
                        bv->text->cursor.par(),
                        bv->text->cursor.par()->next());
-               cutSelection();
+               cutSelection(bv->buffer()->params);
                updateLocal(bv, INIT, true);
                break;
        case LFUN_COPY:
@@ -1243,7 +1182,7 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                        case LFUN_DEFAULT:
                        case LFUN_UNDERLINE:
                        case LFUN_FONT_SIZE:
-                               if (bv->Dispatch(action, arg))
+                               if (bv->dispatch(FuncRequest(bv, action, arg)))
                                        result = DISPATCHED;
                                break;
                        default:
@@ -1258,7 +1197,7 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                if (activateCellInset(bv)) {
                        // reset need_update setted in above function!
                        need_update = NONE;
-                       result = the_locking_inset->localDispatch(bv, action, arg);
+                       result = the_locking_inset->localDispatch(FuncRequest(bv, action, arg));
                        if ((result == UNDISPATCHED) || (result >= FINISHED)) {
                                unlockInsetInInset(bv, the_locking_inset);
                                nodraw(false);
@@ -1317,7 +1256,7 @@ int InsetTabular::linuxdoc(Buffer const * buf, ostream & os) const
 }
 
 
-int InsetTabular::docbook(Buffer const * buf, ostream & os) const
+int InsetTabular::docbook(Buffer const * buf, ostream & os, bool mixcont) const
 {
        int ret = 0;
        Inset * master;
@@ -1329,12 +1268,16 @@ int InsetTabular::docbook(Buffer const * buf, ostream & os) const
            master = master->owner());
 
        if (!master) {
-               os << "<informaltable>\n";
+               os << "<informaltable>";
+               if (mixcont)
+                       os << endl;
                ret++;
        }
-       ret+= tabular->docBook(buf,os);
+       ret+= tabular->docbook(buf, os, mixcont);
        if (!master) {
-               os << "</informaltable>\n";
+               os << "</informaltable>";
+               if (mixcont)
+                       os << endl;
                ret++;
        }
        return ret;
@@ -1460,9 +1403,9 @@ void InsetTabular::hideInsetCursor(BufferView * bv)
 void InsetTabular::fitInsetCursor(BufferView * bv) const
 {
        if (the_locking_inset) {
-               int old_first_y = bv->text->first_y;
+               int old_top_y = bv->text->top_y();
                the_locking_inset->fitInsetCursor(bv);
-               if (old_first_y != bv->text->first_y)
+               if (old_top_y != bv->text->top_y())
                        need_update = FULL;
                return;
        }
@@ -1526,7 +1469,7 @@ void InsetTabular::resetPos(BufferView * bv) const
 #warning This should be fixed in the right manner (20011128 Jug)
 #endif
        // fast hack to fix infinite repaintings!
-       if (in_reset_pos)
+       if (in_reset_pos > 0)
                return;
 
        int cell = 0;
@@ -1547,14 +1490,14 @@ void InsetTabular::resetPos(BufferView * bv) const
                return;
        }
        // we need this only from here on!!!
-       in_reset_pos = true;
+       ++in_reset_pos;
        static int const offset = ADD_TO_TABULAR_WIDTH + 2;
        int new_x = getCellXPos(actcell);
        int old_x = cursor_.x();
        new_x += offset;
        cursor_.x(new_x);
 //    cursor.x(getCellXPos(actcell) + offset);
-       if ((actcol < tabular->columns()-1) && scroll(false) &&
+       if ((actcol < tabular->columns() - 1) && scroll(false) &&
                (tabular->GetWidthOfTabular() < bv->workWidth()-20))
        {
                scroll(bv, 0.0F);
@@ -1562,7 +1505,7 @@ void InsetTabular::resetPos(BufferView * bv) const
        } else if (the_locking_inset &&
                 (tabular->GetWidthOfColumn(actcell) > bv->workWidth()-20))
        {
-               int xx = cursor_.x() - offset + bv->text->getRealCursorX(bv);
+               int xx = cursor_.x() - offset + bv->text->getRealCursorX();
                if (xx > (bv->workWidth()-20)) {
                        scroll(bv, -(xx - bv->workWidth() + 60));
                        updateLocal(bv, FULL, false);
@@ -1582,7 +1525,7 @@ void InsetTabular::resetPos(BufferView * bv) const
        } else if ((cursor_.x() - offset) < 20) {
                scroll(bv, 20 - cursor_.x() + offset);
                updateLocal(bv, FULL, false);
-       } else if (scroll(false) && top_x > 20 &&
+       } else if (scroll() && top_x > 20 &&
                   (top_x + tabular->GetWidthOfTabular()) > (bv->workWidth() - 20)) {
                scroll(bv, old_x - cursor_.x());
                updateLocal(bv, FULL, false);
@@ -1595,14 +1538,15 @@ void InsetTabular::resetPos(BufferView * bv) const
             !the_locking_inset->getFirstLockingInsetOfType(TABULAR_CODE)) &&
            actcell != oldcell) {
                InsetTabular * inset = const_cast<InsetTabular *>(this);
-               bv->owner()->getDialogs()->updateTabular(inset);
+               InsetTabularMailer mailer(*inset);
+               mailer.updateDialog(bv);
                oldcell = actcell;
        }
-       in_reset_pos = false;
+       in_reset_pos = 0;
 }
 
 
-UpdatableInset::RESULT InsetTabular::moveRight(BufferView * bv, bool lock)
+Inset::RESULT InsetTabular::moveRight(BufferView * bv, bool lock)
 {
        if (lock && !old_locking_inset) {
                if (activateCellInset(bv))
@@ -1620,7 +1564,7 @@ UpdatableInset::RESULT InsetTabular::moveRight(BufferView * bv, bool lock)
 }
 
 
-UpdatableInset::RESULT InsetTabular::moveLeft(BufferView * bv, bool lock)
+Inset::RESULT InsetTabular::moveLeft(BufferView * bv, bool lock)
 {
        bool moved = isRightToLeft(bv) ? moveNextCell(bv) : movePrevCell(bv);
        if (!moved)
@@ -1634,7 +1578,7 @@ UpdatableInset::RESULT InsetTabular::moveLeft(BufferView * bv, bool lock)
 }
 
 
-UpdatableInset::RESULT InsetTabular::moveUp(BufferView * bv, bool lock)
+Inset::RESULT InsetTabular::moveUp(BufferView * bv, bool lock)
 {
        int const ocell = actcell;
        actcell = tabular->GetCellAbove(actcell);
@@ -1655,7 +1599,7 @@ UpdatableInset::RESULT InsetTabular::moveUp(BufferView * bv, bool lock)
 }
 
 
-UpdatableInset::RESULT InsetTabular::moveDown(BufferView * bv, bool lock)
+Inset::RESULT InsetTabular::moveDown(BufferView * bv, bool lock)
 {
        int const ocell = actcell;
        actcell = tabular->GetCellBelow(actcell);
@@ -1734,12 +1678,6 @@ bool InsetTabular::movePrevCell(BufferView * bv, bool lock)
 }
 
 
-bool InsetTabular::deletable() const
-{
-       return true;
-}
-
-
 void InsetTabular::setFont(BufferView * bv, LyXFont const & font, bool tall,
                           bool selectall)
 {
@@ -1794,13 +1732,15 @@ bool InsetTabular::tabularFeatures(BufferView * bv, string const & what)
                return false;
 
        string const val =
-               frontStrip(what.substr(tabularFeature[i].feature.length()));
+               ltrim(what.substr(tabularFeature[i].feature.length()));
        tabularFeatures(bv, action, val);
        return true;
 }
 
-static void checkLongtableSpecial(LyXTabular::ltType & ltt,
-                                 string const & special, bool & flag)
+namespace {
+
+void checkLongtableSpecial(LyXTabular::ltType & ltt,
+                         string const & special, bool & flag)
 {
        if (special == "dl_above") {
                ltt.topDL = flag;
@@ -1817,6 +1757,8 @@ static void checkLongtableSpecial(LyXTabular::ltType & ltt,
        }
 }
 
+}
+
 
 void InsetTabular::tabularFeatures(BufferView * bv,
                                   LyXTabular::Feature feature,
@@ -1843,6 +1785,9 @@ void InsetTabular::tabularFeatures(BufferView * bv,
        case LyXTabular::ALIGN_CENTER:
                setAlign = LYX_ALIGN_CENTER;
                break;
+       case LyXTabular::ALIGN_BLOCK:
+               setAlign = LYX_ALIGN_BLOCK;
+               break;
        case LyXTabular::M_VALIGN_TOP:
        case LyXTabular::VALIGN_TOP:
                setVAlign = LyXTabular::LYX_VALIGN_TOP;
@@ -1889,6 +1834,13 @@ void InsetTabular::tabularFeatures(BufferView * bv,
                        }
                        updateLocal(bv, INIT, true);
                }
+
+               if (vallen.zero()
+                   && tabular->GetAlignment(actcell, true) == LYX_ALIGN_BLOCK)
+                       tabularFeatures(bv, LyXTabular::ALIGN_CENTER, string());
+               else if (!vallen.zero()
+                        && tabular->GetAlignment(actcell, true) != LYX_ALIGN_BLOCK)
+                       tabularFeatures(bv, LyXTabular::ALIGN_BLOCK, string());
        }
        break;
        case LyXTabular::SET_MPWIDTH:
@@ -2013,6 +1965,7 @@ void InsetTabular::tabularFeatures(BufferView * bv,
        case LyXTabular::ALIGN_LEFT:
        case LyXTabular::ALIGN_RIGHT:
        case LyXTabular::ALIGN_CENTER:
+       case LyXTabular::ALIGN_BLOCK:
                for (int i = sel_row_start; i <= sel_row_end; ++i)
                        for (int j = sel_col_start; j <= sel_col_end; ++j)
                                tabular->SetAlignment(
@@ -2038,7 +1991,7 @@ void InsetTabular::tabularFeatures(BufferView * bv,
        case LyXTabular::MULTICOLUMN:
        {
                if (sel_row_start != sel_row_end) {
-                       Alert::alert(_("Impossible Operation!"),
+                       Alert::alert(_("Impossible operation!"),
                                   _("Multicolumns can only be horizontally."),
                                   _("Sorry."));
                        return;
@@ -2206,7 +2159,7 @@ int InsetTabular::getMaxWidthOfCell(BufferView * bv, int cell) const
 
        if (len.zero())
                return -1;
-       return len.inPixels(latexTextWidth(bv), bv->text->defaultHeight());
+       return len.inPixels(latexTextWidth(bv));
 }
 
 
@@ -2270,9 +2223,11 @@ LyXText * InsetTabular::getLyXText(BufferView const * bv,
 
 bool InsetTabular::showInsetDialog(BufferView * bv) const
 {
-       if (!the_locking_inset || !the_locking_inset->showInsetDialog(bv))
-               bv->owner()->getDialogs()
-                       ->showTabular(const_cast<InsetTabular *>(this));
+       if (!the_locking_inset || !the_locking_inset->showInsetDialog(bv)) {
+               InsetTabular * tmp = const_cast<InsetTabular *>(this);
+               InsetTabularMailer mailer(*tmp);
+               mailer.showDialog(bv);
+       }
        return true;
 }
 
@@ -2287,8 +2242,9 @@ void InsetTabular::openLayoutDialog(BufferView * bv) const
                        return;
                }
        }
-       bv->owner()->getDialogs()->showTabular(
-               const_cast<InsetTabular *>(this));
+       InsetTabular * tmp = const_cast<InsetTabular *>(this);
+       InsetTabularMailer mailer(*tmp);
+       mailer.showDialog(bv);
 }
 
 
@@ -2316,7 +2272,7 @@ FuncStatus InsetTabular::getStatus(string const & what) const
                return status.unknown(true);
        }
 
-       string const argument = frontStrip(what.substr(tabularFeature[i].feature.length()));
+       string const argument = ltrim(what.substr(tabularFeature[i].feature.length()));
 
        int sel_row_start;
        int sel_row_end;
@@ -2383,6 +2339,10 @@ FuncStatus InsetTabular::getStatus(string const & what) const
        case LyXTabular::ALIGN_CENTER:
                status.setOnOff(tabular->GetAlignment(actcell, flag) == LYX_ALIGN_CENTER);
                break;
+       case LyXTabular::ALIGN_BLOCK:
+               status.disabled(tabular->GetPWidth(actcell).zero());
+               status.setOnOff(tabular->GetAlignment(actcell, flag) == LYX_ALIGN_BLOCK);
+               break;
        case LyXTabular::M_VALIGN_TOP:
                flag = false;
        case LyXTabular::VALIGN_TOP:
@@ -2492,7 +2452,7 @@ bool InsetTabular::copySelection(BufferView * bv)
        ostringstream sstr;
        paste_tabular->ascii(bv->buffer(), sstr,
                             (int)parOwner()->params().depth(), true, '\t');
-       bv->stuffClipboard(sstr.str().c_str());
+       bv->stuffClipboard(STRCONV(sstr.str()));
        return true;
 }
 
@@ -2524,13 +2484,14 @@ bool InsetTabular::pasteSelection(BufferView * bv)
                        *(tabular->GetCellInset(n2)) = *(paste_tabular->GetCellInset(n1));
                        tabular->GetCellInset(n2)->setOwner(this);
                        tabular->GetCellInset(n2)->deleteLyXText(bv);
+                       tabular->GetCellInset(n2)->markNew();
                }
        }
        return true;
 }
 
 
-bool InsetTabular::cutSelection()
+bool InsetTabular::cutSelection(BufferParams const & bp)
 {
        if (!hasSelection())
                return false;
@@ -2553,7 +2514,7 @@ bool InsetTabular::cutSelection()
        }
        for (int i = sel_row_start; i <= sel_row_end; ++i) {
                for (int j = sel_col_start; j <= sel_col_end; ++j) {
-                       tabular->GetCellInset(tabular->GetCellNumber(i, j))->clear();
+                       tabular->GetCellInset(tabular->GetCellNumber(i, j))->clear(bp.tracking_changes);
                }
        }
        return true;
@@ -2585,12 +2546,6 @@ int InsetTabular::scroll(bool recursive) const
 }
 
 
-bool InsetTabular::doClearArea() const
-{
-       return !locked || (need_update & (FULL|INIT));
-}
-
-
 void InsetTabular::getSelection(int & srow, int & erow,
                                int & scol, int & ecol) const
 {
@@ -2613,19 +2568,6 @@ void InsetTabular::getSelection(int & srow, int & erow,
 }
 
 
-Paragraph * InsetTabular::getParFromID(int id) const
-{
-       Paragraph * result;
-       for(int i = 0; i < tabular->rows(); ++i) {
-               for(int j = 0; j < tabular->columns(); ++j) {
-                       if ((result = tabular->GetCellInset(i, j)->getParFromID(id)))
-                               return result;
-               }
-       }
-       return 0;
-}
-
-
 Paragraph * InsetTabular::firstParagraph() const
 {
        if (the_locking_inset)
@@ -2666,20 +2608,20 @@ Inset * InsetTabular::getInsetFromID(int id_arg) const
 }
 
 
-string const
+WordLangTuple const
 InsetTabular::selectNextWordToSpellcheck(BufferView * bv, float & value) const
 {
        nodraw(true);
        if (the_locking_inset) {
-               string const str(the_locking_inset->selectNextWordToSpellcheck(bv, value));
-               if (!str.empty()) {
+               WordLangTuple word(the_locking_inset->selectNextWordToSpellcheck(bv, value));
+               if (!word.word().empty()) {
                        nodraw(false);
-                       return str;
+                       return word;
                }
                if (tabular->IsLastCell(actcell)) {
                        bv->unlockInset(const_cast<InsetTabular *>(this));
                        nodraw(false);
-                       return string();
+                       return WordLangTuple();
                }
                ++actcell;
        }
@@ -2687,26 +2629,26 @@ InsetTabular::selectNextWordToSpellcheck(BufferView * bv, float & value) const
        UpdatableInset * inset =
                static_cast<UpdatableInset*>(tabular->GetCellInset(actcell));
        inset->edit(bv, 0,  0, mouse_button::none);
-       string const str(selectNextWordInt(bv, value));
+       WordLangTuple word(selectNextWordInt(bv, value));
        nodraw(false);
-       if (!str.empty())
+       if (!word.word().empty())
                resetPos(bv);
-       return str;
+       return word;
 }
 
 
-string InsetTabular::selectNextWordInt(BufferView * bv, float & value) const
+WordLangTuple InsetTabular::selectNextWordInt(BufferView * bv, float & value) const
 {
        // when entering this function the inset should be ALWAYS locked!
        lyx::Assert(the_locking_inset);
 
-       string const str(the_locking_inset->selectNextWordToSpellcheck(bv, value));
-       if (!str.empty())
-               return str;
+       WordLangTuple word(the_locking_inset->selectNextWordToSpellcheck(bv, value));
+       if (!word.word().empty())
+               return word;
 
        if (tabular->IsLastCell(actcell)) {
                bv->unlockInset(const_cast<InsetTabular *>(this));
-               return string();
+               return WordLangTuple();
        }
 
        // otherwise we have to lock the next inset and ask for it's selecttion
@@ -2735,59 +2677,96 @@ void InsetTabular::toggleSelection(BufferView * bv, bool kill_selection)
 }
 
 
+void InsetTabular::markErased()
+{
+       int cell = 0;
+
+       while (!tabular->IsLastCell(cell)) {
+               ++cell;
+               InsetText * inset = tabular->GetCellInset(cell);
+               inset->markErased();
+       }
+}
+
+
+bool InsetTabular::nextChange(BufferView * bv, lyx::pos_type & length)
+{
+       if (the_locking_inset) {
+               if (the_locking_inset->nextChange(bv, length)) {
+                       updateLocal(bv, CELL, false);
+                       return true;
+               }
+               if (tabular->IsLastCell(actcell))
+                       return false;
+               ++actcell;
+       }
+       InsetText * inset = tabular->GetCellInset(actcell);
+       if (inset->nextChange(bv, length)) {
+               updateLocal(bv, FULL, false);
+               return true;
+       }
+       while (!tabular->IsLastCell(actcell)) {
+               ++actcell;
+               inset = tabular->GetCellInset(actcell);
+               if (inset->nextChange(bv, length)) {
+                       updateLocal(bv, FULL, false);
+                       return true;
+               }
+       }
+       return false;
+}
+
+
 bool InsetTabular::searchForward(BufferView * bv, string const & str,
                                 bool cs, bool mw)
 {
-       nodraw(true);
        if (the_locking_inset) {
                if (the_locking_inset->searchForward(bv, str, cs, mw)) {
-                       nodraw(false);
                        updateLocal(bv, CELL, false);
                        return true;
                }
-               if (tabular->IsLastCell(actcell)) {
-                       nodraw(false);
-                       bv->unlockInset(const_cast<InsetTabular *>(this));
+               if (tabular->IsLastCell(actcell))
                        return false;
-               }
                ++actcell;
        }
-       // otherwise we have to lock the next inset and search there
-       UpdatableInset * inset =
-               static_cast<UpdatableInset*>(tabular->GetCellInset(actcell));
-       inset->edit(bv);
-       bool const ret = searchForward(bv, str, cs, mw);
-       nodraw(false);
-       updateLocal(bv, CELL, false);
-       return ret;
+       InsetText * inset = tabular->GetCellInset(actcell);
+       if (inset->searchForward(bv, str, cs, mw)) {
+               updateLocal(bv, FULL, false);
+               return true;
+       }
+       while (!tabular->IsLastCell(actcell)) {
+               ++actcell;
+               inset = tabular->GetCellInset(actcell);
+               if (inset->searchForward(bv, str, cs, mw)) {
+                       updateLocal(bv, FULL, false);
+                       return true;
+               }
+       }
+       return false;
 }
 
 
 bool InsetTabular::searchBackward(BufferView * bv, string const & str,
                               bool cs, bool mw)
 {
-       nodraw(true);
        if (the_locking_inset) {
                if (the_locking_inset->searchBackward(bv, str, cs, mw)) {
-                       nodraw(false);
                        updateLocal(bv, CELL, false);
                        return true;
                }
-               if (!actcell) { // we are already in the first cell
-                       nodraw(false);
-                       bv->unlockInset(const_cast<InsetTabular *>(this));
-                       return false;
-               }
+       }
+       if (!locked)
+               actcell = tabular->GetNumberOfCells();
+
+       while (actcell) {
                --actcell;
+               InsetText * inset = tabular->GetCellInset(actcell);
+               if (inset->searchBackward(bv, str, cs, mw)) {
+                       updateLocal(bv, CELL, false);
+                       return true;
+               }
        }
-       // otherwise we have to lock the next inset and search there
-       UpdatableInset * inset =
-               static_cast<UpdatableInset*>(tabular->GetCellInset(actcell));
-       inset->edit(bv, false);
-       bool const ret = searchBackward(bv, str, cs, mw);
-       nodraw(false);
-       updateLocal(bv, CELL, false);
-       return ret;
+       return false;
 }
 
 
@@ -2861,12 +2840,14 @@ bool InsetTabular::insertAsciiString(BufferView * bv, string const & buf,
                ocol = actcol;
                row = actrow;
        }
+
        string::size_type op = 0;
        int cells = loctab->GetNumberOfCells();
        p = 0;
        cols = ocol;
        rows = loctab->rows();
        int const columns = loctab->columns();
+
        while ((cell < cells) && (p < len) && (row < rows) &&
               (p = buf.find_first_of("\t\n", p)) != string::npos)
        {
@@ -2911,3 +2892,91 @@ bool InsetTabular::insertAsciiString(BufferView * bv, string const & buf,
 
        return true;
 }
+
+
+void InsetTabular::addPreview(grfx::PreviewLoader & loader) const
+{
+       int const rows = tabular->rows();
+       int const columns = tabular->columns();
+       for (int i = 0; i < rows; ++i) {
+               for (int j = 0; j < columns; ++j) {
+                       tabular->GetCellInset(i,j)->addPreview(loader);
+               }
+       }
+}
+
+
+string const InsetTabularMailer:: name_("tabular");
+
+InsetTabularMailer::InsetTabularMailer(InsetTabular & inset)
+       : inset_(inset)
+{}
+
+
+string const InsetTabularMailer::inset2string() const
+{
+       return params2string(inset_);
+}
+
+
+int InsetTabularMailer::string2params(string const & in, InsetTabular & inset)
+{
+       istringstream data(in);
+       LyXLex lex(0,0);
+       lex.setStream(data);
+
+       if (lex.isOK()) {
+               lex.next();
+               string const token = lex.getString();
+               if (token != name_)
+                       return -1;
+       }
+
+       int cell = -1;
+       if (lex.isOK()) {
+               lex.next();
+               string const token = lex.getString();
+               if (token != "\\active_cell")
+                       return -1;
+               lex.next();
+               cell = lex.getInteger();
+       }
+
+       // This is part of the inset proper that is usually swallowed
+       // by Buffer::readInset
+       if (lex.isOK()) {
+               lex.next();
+               string const token = lex.getString();
+               if (token != "Tabular")
+                       return -1;
+       }
+
+       if (!lex.isOK())
+               return -1;
+
+       BufferView * const bv = inset.view();
+       Buffer const * const buffer = bv ? bv->buffer() : 0;
+       if (buffer)
+               inset.read(buffer, lex);
+
+       // We can't set the active cell, but we can tell the frontend
+       // what it is.
+       return cell;
+}
+
+
+string const
+InsetTabularMailer::params2string(InsetTabular const & inset)
+{
+       BufferView * const bv = inset.view();
+       Buffer const * const buffer = bv ? bv->buffer() : 0;
+       if (!buffer)
+               return string();
+
+       ostringstream data;
+       data << name_ << " \\active_cell " << inset.getActCell() << '\n';
+       inset.write(buffer, data);
+       data << "\\end_inset\n";
+
+       return data.str();
+}