]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insettabular.C
The markDirty() and fitCursor() changes
[lyx.git] / src / insets / insettabular.C
index 5ed539c7899539b0d97a90b2ed91867c6d8c0813..c48a62b75f5a12c308ef24102b077e2fb447177d 100644 (file)
@@ -1,42 +1,41 @@
-/* This file is part of
- * ======================================================
- * 
- *           LyX, The Document Processor
+/**
+ * \file insettabular.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           Copyright 2001 The LyX Team.
+ * \author Jürgen Vigna
  *
- * ======================================================
+ * 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 "Painter.h"
-#include "font.h"
-#include "lyxtext.h"
-#include "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) {
@@ -437,12 +336,12 @@ void InsetTabular::draw(BufferView * bv, LyXFont const & font, int baseline,
 
 
 void InsetTabular::drawCellLines(Painter & pain, int x, int baseline,
-                                 int row, int cell) const
+                                int row, int cell) const
 {
        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),
@@ -450,11 +349,11 @@ void InsetTabular::drawCellLines(Painter & pain, int x, int baseline,
                          on_off ? Painter::line_onoffdash : Painter::line_solid);
        }
        on_off = !tabular->BottomLine(cell);
-       pain.line(x,baseline +  tabular->GetDescentOfRow(row),
-                 x2, baseline +  tabular->GetDescentOfRow(row),
+       pain.line(x, baseline + tabular->GetDescentOfRow(row),
+                 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),
@@ -483,16 +382,16 @@ void InsetTabular::drawCellSelection(Painter & pain, int x, int baseline,
        } else {
                ce = tabular->right_column_of_cell(sel_cell_end);
        }
-       
+
        int rs = tabular->row_of_cell(sel_cell_start);
        int re = tabular->row_of_cell(sel_cell_end);
        if (rs > re)
                swap(rs, re);
-       
+
        if ((column >= cs) && (column <= ce) && (row >= rs) && (row <= re)) {
                int w = tabular->GetWidthOfColumn(cell);
-               int h = tabular->GetAscentOfRow(row) + tabular->GetDescentOfRow(row);
-               pain.fillRectangle(x, baseline - tabular->GetAscentOfRow(row),
+               int h = tabular->GetAscentOfRow(row) + tabular->GetDescentOfRow(row)-1;
+               pain.fillRectangle(x, baseline - tabular->GetAscentOfRow(row) + 1,
                                   w, h, LColor::selection);
        }
 }
@@ -530,8 +429,10 @@ void InsetTabular::update(BufferView * bv, LyXFont const & font, bool reinit)
        case INIT:
        case FULL:
        case CELL:
-               if (calculate_dimensions_of_cells(bv, font, false))
+               if (calculate_dimensions_of_cells(bv, font, false)) {
                        need_update = INIT;
+                       resetPos(bv);
+               }
                break;
        case SELECTION:
                need_update = FULL;
@@ -549,10 +450,10 @@ string const InsetTabular::editMessage() const
 }
 
 
-void InsetTabular::edit(BufferView * bv, int x, int y, unsigned int button)
+void InsetTabular::edit(BufferView * bv, int x, int y, mouse_button::state button)
 {
        UpdatableInset::edit(bv, x, y, button);
-       
+
        if (!bv->lockInset(this)) {
                lyxerr[Debug::INSETTEXT] << "InsetTabular::Cannot lock inset" << endl;
                return;
@@ -564,7 +465,7 @@ void InsetTabular::edit(BufferView * bv, int x, int y, unsigned int button)
        setPos(bv, x, y);
        clearSelection();
        finishUndo();
-       if (insetHit(bv, x, y) && (button != 3)) {
+       if (insetHit(bv, x, y) && (button != mouse_button::button3)) {
                activateCellInsetAbs(bv, x, y, button);
        }
 }
@@ -573,22 +474,30 @@ void InsetTabular::edit(BufferView * bv, int x, int y, unsigned int button)
 void InsetTabular::edit(BufferView * bv, bool front)
 {
        UpdatableInset::edit(bv, front);
-       
+
        if (!bv->lockInset(this)) {
                lyxerr[Debug::INSETTEXT] << "InsetTabular::Cannot lock inset" << endl;
                return;
        }
+       finishUndo();
        locked = true;
        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);
-       finishUndo();
+       bv->fitCursor();
 }
 
 
@@ -596,9 +505,11 @@ void InsetTabular::insetUnlock(BufferView * bv)
 {
        if (the_locking_inset) {
                the_locking_inset->insetUnlock(bv);
+               updateLocal(bv, CELL, false);
                the_locking_inset = 0;
        }
        hideInsetCursor(bv);
+       actcell = 0;
        oldcell = -1;
        locked = false;
        if (scroll(false) || hasSelection()) {
@@ -612,7 +523,7 @@ void InsetTabular::insetUnlock(BufferView * bv)
 
 
 void InsetTabular::updateLocal(BufferView * bv, UpdateCodes what,
-                               bool mark_dirty) const
+                              bool mark_dirty) const
 {
        if (what == INIT) {
                LyXFont font;
@@ -622,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);
        }
 }
@@ -685,7 +591,6 @@ bool InsetTabular::unlockInsetInInset(BufferView * bv, UpdatableInset * inset,
                return false;
        if (the_locking_inset == inset) {
                the_locking_inset->insetUnlock(bv);
-               the_locking_inset = 0;
 #ifdef WITH_WARNINGS
 #warning fix scrolling when cellinset has requested a scroll (Jug)!!!
 #endif
@@ -694,13 +599,16 @@ bool InsetTabular::unlockInsetInInset(BufferView * bv, UpdatableInset * inset,
                        scroll(bv, 0.0F);
 #endif
                updateLocal(bv, CELL, false);
+               // this has to be here otherwise we don't redraw the cell!
+               the_locking_inset = 0;
 //             showInsetCursor(bv, false);
                return true;
        }
        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;
@@ -728,7 +636,7 @@ bool InsetTabular::updateInsetInInset(BufferView * bv, Inset * inset)
 }
 
 
-unsigned int InsetTabular::insetInInsetY()
+int InsetTabular::insetInInsetY() const
 {
        if (!the_locking_inset)
                return 0;
@@ -761,28 +669,36 @@ bool InsetTabular::insertInset(BufferView * bv, Inset * inset)
 }
 
 
-void InsetTabular::insetButtonPress(BufferView * bv, int x, int y, int button)
+void InsetTabular::lfunMousePress(FuncRequest const & cmd)
 {
-       if (hasSelection() && (button == 3))
+       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);
-       setPos(bv, x, y);
+       if (!locked) {
+               locked = true;
+               the_locking_inset = 0;
+               inset_x = 0;
+               inset_y = 0;
+       }
+       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);
                        the_locking_inset = 0;
                }
                showInsetCursor(bv);
@@ -790,99 +706,103 @@ void InsetTabular::insetButtonPress(BufferView * bv, int x, int y, int 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;
        }
-       the_locking_inset = 0;
-       if (button == 2) {
-               localDispatch(bv, LFUN_PASTESELECTION, "paragraph");
+
+       if (cmd.button() == mouse_button::button2) {
+               localDispatch(FuncRequest(bv, LFUN_PASTESELECTION, "paragraph"));
                return;
        }
+
        if (inset_hit && bv->theLockingInset()) {
-               // only activate the Inset so that no internal inset is hit
-               // by this call. It should be only hit by the insetButtonPress call.
-               if (activateCellInsetAbs(bv, 0, 0, 0))
-                       the_locking_inset->insetButtonPress(bv,
-                                                           x - inset_x,
-                                                           y - inset_y,
-                                                           button);
+               if (!bv->lockInset(static_cast<UpdatableInset*>
+                               (tabular->GetCellInset(actcell))))
+               {
+                       lyxerr[Debug::INSETS] << "Cannot lock inset" << endl;
+                       return;
+               }
+               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, int 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 == 3 && !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, int 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;
-       
-       setPos(bv, x, y);
+       int const old_cell = actcell;
+
+       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);
 }
 
 
-void InsetTabular::insetKeyPress(XKeyEvent * xke)
+Inset::RESULT InsetTabular::localDispatch(FuncRequest const & cmd)
 {
-       if (the_locking_inset) {
-               the_locking_inset->insetKeyPress(xke);
-               return;
-       }
-}
-
-
-UpdatableInset::RESULT
-InsetTabular::localDispatch(BufferView * bv, kb_action action,
-                            string const & arg)
-{
-       // We need to save the value of the_locking_inset as the call to 
-       // the_locking_inset->LocalDispatch might unlock it.
+       // We need to save the value of the_locking_inset as the call to
+       // 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();
@@ -891,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);
@@ -914,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);
@@ -949,7 +882,7 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                        setSelection(start, actcell);
                        break;
                }
-               
+
                int end = actcell;
                // if we are starting a selection, only select
                // the current cell at the beginning
@@ -973,11 +906,11 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                        setSelection(start, actcell);
                        break;
                }
+
                int end = actcell;
                // if we are starting a selection, only select
                // the current cell at the beginning
-               if (hasSelection()) { 
+               if (hasSelection()) {
                        moveLeft(bv, false);
                        end = actcell;
                }
@@ -1048,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->scrollCB(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 {
@@ -1070,7 +1003,7 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
                int column = actcol;
                unlockInsetInInset(bv, the_locking_inset);
                if (top_baseline < 0) {
-                       bv->scrollCB(bv->text->first_y - bv->painter().paperHeight());
+                       bv->scrollDocView(bv->text->top_y() - bv->painter().paperHeight());
                        code = FULL;
                        if (top_baseline > 0)
                                actcell = column;
@@ -1090,33 +1023,40 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        case LFUN_NEXTSEL:
        case LFUN_WORDLEFT:
        case LFUN_WORDLEFTSEL:
-       case LFUN_WORDRIGHT: 
+       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:
        case LFUN_ENDSEL:
        case LFUN_BEGINNINGBUF:
-       case LFUN_BEGINNINGBUFSEL: 
+       case LFUN_BEGINNINGBUFSEL:
        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;
                break;
                // insert file functions
        case LFUN_FILE_INSERT_ASCII_PARA:
-       case LFUN_FILE_INSERT_ASCII: 
+       case LFUN_FILE_INSERT_ASCII:
        {
                string tmpstr = getContentsOfAsciiFile(bv, arg, false);
                if (tmpstr.empty())
@@ -1131,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();
+                       bv->text->cursor.par(),
+                       bv->text->cursor.par()->next());
+               cutSelection(bv->buffer()->params);
                updateLocal(bv, INIT, true);
                break;
        case LFUN_COPY:
@@ -1217,8 +1159,8 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        case LFUN_PASTE:
                if (hasPasteBuffer()) {
                        setUndo(bv, Undo::INSERT,
-                               bv->text->cursor.par(),
-                               bv->text->cursor.par()->next());
+                               bv->text->cursor.par(),
+                               bv->text->cursor.par()->next());
                        pasteSelection(bv);
                        updateLocal(bv, INIT, true);
                        break;
@@ -1240,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:
@@ -1255,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);
@@ -1276,7 +1218,10 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
        }
        if (result < FINISHED) {
                if (!the_locking_inset) {
-//                     showInsetCursor(bv);
+                       if (bv->fitCursor())
+                               updateLocal(bv, FULL, false);
+                       if (locked)
+                               showInsetCursor(bv);
                }
        } else
                bv->unlockInset(this);
@@ -1285,7 +1230,7 @@ InsetTabular::localDispatch(BufferView * bv, kb_action action,
 
 
 int InsetTabular::latex(Buffer const * buf, ostream & os,
-                        bool fragile, bool fp) const
+                       bool fragile, bool fp) const
 {
        return tabular->latex(buf, os, fragile, fp);
 }
@@ -1295,7 +1240,7 @@ int InsetTabular::ascii(Buffer const * buf, ostream & os, int ll) const
 {
        if (ll > 0)
                return tabular->ascii(buf, os, (int)parOwner()->params().depth(),
-                                     false,0);
+                                     false,0);
        return tabular->ascii(buf, os, 0, false,0);
 }
 
@@ -1311,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;
@@ -1323,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;
@@ -1342,15 +1291,15 @@ void InsetTabular::validate(LaTeXFeatures & features) const
 
 
 bool InsetTabular::calculate_dimensions_of_cells(BufferView * bv,
-                                                 LyXFont const & font,
-                                                 bool reinit) const
+                                                LyXFont const & font,
+                                                bool reinit) const
 {
        int cell = -1;
        int maxAsc = 0;
        int maxDesc = 0;
        InsetText * inset;
        bool changed = false;
-       
+
        // if we have a locking_inset we should have to check only this cell for
        // change so I'll try this to have a boost, but who knows ;)
        if ((need_update != INIT) &&
@@ -1411,12 +1360,12 @@ void InsetTabular::toggleInsetCursor(BufferView * bv)
                the_locking_inset->toggleInsetCursor(bv);
                return;
        }
-       
+
        LyXFont font; // = the_locking_inset->GetFont(par, cursor.pos);
-       
-       int const asc = lyxfont::maxAscent(font);
-       int const desc = lyxfont::maxDescent(font);
-       
+
+       int const asc = font_metrics::maxAscent(font);
+       int const desc = font_metrics::maxDescent(font);
+
        if (isCursorVisible())
                bv->hideLockedInsetCursor();
        else
@@ -1431,9 +1380,9 @@ void InsetTabular::showInsetCursor(BufferView * bv, bool show)
                return;
        if (!isCursorVisible()) {
                LyXFont font; // = GetFont(par, cursor.pos);
-       
-               int const asc = lyxfont::maxAscent(font);
-               int const desc = lyxfont::maxDescent(font);
+
+               int const asc = font_metrics::maxAscent(font);
+               int const desc = font_metrics::maxDescent(font);
                bv->fitLockedInsetCursor(cursor_.x(), cursor_.y(), asc, desc);
                if (show)
                        bv->showLockedInsetCursor(cursor_.x(), cursor_.y(), asc, desc);
@@ -1454,21 +1403,27 @@ void InsetTabular::hideInsetCursor(BufferView * bv)
 void InsetTabular::fitInsetCursor(BufferView * bv) const
 {
        if (the_locking_inset) {
+               int old_top_y = bv->text->top_y();
                the_locking_inset->fitInsetCursor(bv);
+               if (old_top_y != bv->text->top_y())
+                       need_update = FULL;
                return;
        }
        LyXFont font;
-       
-       int const asc = lyxfont::maxAscent(font);
-       int const desc = lyxfont::maxDescent(font);
-       bv->fitLockedInsetCursor(cursor_.x(), cursor_.y(), asc, desc);
+
+       int const asc = font_metrics::maxAscent(font);
+       int const desc = font_metrics::maxDescent(font);
+       resetPos(bv);
+
+       if (bv->fitLockedInsetCursor(cursor_.x(), cursor_.y(), asc, desc))
+               need_update = FULL;
 }
 
 
 void InsetTabular::setPos(BufferView * bv, int x, int y) const
 {
        cursor_.y(0);
-       
+
        actcell = actrow = actcol = 0;
        int ly = tabular->GetDescentOfRow(actrow);
 
@@ -1497,7 +1452,7 @@ void InsetTabular::setPos(BufferView * bv, int x, int y) const
 int InsetTabular::getCellXPos(int cell) const
 {
        int c = cell;
-       
+
        for (; !tabular->IsFirstCellInRow(c); --c)
                ;
        int lx = tabular->GetWidthOfColumn(cell);
@@ -1514,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;
@@ -1535,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);
@@ -1550,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);
@@ -1570,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);
@@ -1583,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))
@@ -1608,13 +1564,13 @@ 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)
                return FINISHED;
        if (lock) {       // behind the inset
-               if (activateCellInset(bv, 0, 0, 0, true))
+               if (activateCellInset(bv, 0, 0, mouse_button::none, true))
                        return DISPATCHED;
        }
        resetPos(bv);
@@ -1622,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);
@@ -1643,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);
@@ -1686,7 +1642,7 @@ bool InsetTabular::moveNextCell(BufferView * bv, bool lock)
        if (lock) {
                bool rtl = tabular->GetCellInset(actcell)->paragraph()->
                        isRightToLeftPar(bv->buffer()->params);
-               activateCellInset(bv, 0, 0, 0, !rtl);
+               activateCellInset(bv, 0, 0, mouse_button::none, !rtl);
        }
        resetPos(bv);
        return true;
@@ -1715,30 +1671,24 @@ bool InsetTabular::movePrevCell(BufferView * bv, bool lock)
        if (lock) {
                bool rtl = tabular->GetCellInset(actcell)->paragraph()->
                        isRightToLeftPar(bv->buffer()->params);
-               activateCellInset(bv, 0, 0, 0, !rtl);
+               activateCellInset(bv, 0, 0, mouse_button::none, !rtl);
        }
        resetPos(bv);
        return true;
 }
 
 
-bool InsetTabular::deletable() const
-{
-       return true;
-}
-
-
 void InsetTabular::setFont(BufferView * bv, LyXFont const & font, bool tall,
-                           bool selectall)
+                          bool selectall)
 {
        if (selectall) {
                setSelection(0, tabular->GetNumberOfCells() - 1);
        }
        if (hasSelection()) {
                setUndo(bv, Undo::EDIT,
-                       bv->text->cursor.par(),
-                       bv->text->cursor.par()->next());
-               bool frozen = undo_frozen;
+                       bv->text->cursor.par(),
+                       bv->text->cursor.par()->next());
+               bool const frozen = undo_frozen;
                if (!frozen)
                        freezeUndo();
                // apply the fontchange on the whole selection
@@ -1754,6 +1704,8 @@ void InsetTabular::setFont(BufferView * bv, LyXFont const & font, bool tall,
                }
                if (!frozen)
                        unFreezeUndo();
+               if (selectall)
+                       clearSelection();
                updateLocal(bv, INIT, true);
        }
        if (the_locking_inset)
@@ -1764,11 +1716,11 @@ void InsetTabular::setFont(BufferView * bv, LyXFont const & font, bool tall,
 bool InsetTabular::tabularFeatures(BufferView * bv, string const & what)
 {
        LyXTabular::Feature action = LyXTabular::LAST_ACTION;
-       
+
        int i = 0;
        for (; tabularFeature[i].action != LyXTabular::LAST_ACTION; ++i) {
                string const tmp = tabularFeature[i].feature;
-               
+
                if (tmp == what.substr(0, tmp.length())) {
                        //if (!compare(tabularFeatures[i].feature.c_str(), what.c_str(),
                        //tabularFeatures[i].feature.length())) {
@@ -1780,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;
@@ -1803,10 +1757,12 @@ static void checkLongtableSpecial(LyXTabular::ltType & ltt,
        }
 }
 
+}
+
 
 void InsetTabular::tabularFeatures(BufferView * bv,
-                                   LyXTabular::Feature feature,
-                                   string const & value)
+                                  LyXTabular::Feature feature,
+                                  string const & value)
 {
        int sel_col_start;
        int sel_col_end;
@@ -1829,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;
@@ -1851,20 +1810,20 @@ void InsetTabular::tabularFeatures(BufferView * bv,
                sel_row_start = sel_row_end = tabular->row_of_cell(actcell);
        }
        setUndo(bv, Undo::FINISH,
-               bv->text->cursor.par(),
-               bv->text->cursor.par()->next());
+               bv->text->cursor.par(),
+               bv->text->cursor.par()->next());
 
        int row =  tabular->row_of_cell(actcell);
        int column = tabular->column_of_cell(actcell);
        bool flag = true;
        LyXTabular::ltType ltt;
-       
+
        switch (feature) {
        case LyXTabular::SET_PWIDTH:
        {
                LyXLength const vallen(value);
                LyXLength const & tmplen = tabular->GetColumnPWidth(actcell);
-               
+
                bool const update = (tmplen != vallen);
                tabular->SetColumnPWidth(actcell, vallen);
                if (update) {
@@ -1875,13 +1834,20 @@ 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:
        {
                LyXLength const vallen(value);
                LyXLength const & tmplen = tabular->GetPWidth(actcell);
-               
+
                bool const update = (tmplen != vallen);
                tabular->SetMColumnPWidth(actcell, vallen);
                if (update) {
@@ -1946,12 +1912,12 @@ void InsetTabular::tabularFeatures(BufferView * bv,
                updateLocal(bv, INIT, true);
                break;
        }
-       
+
        case LyXTabular::M_TOGGLE_LINE_BOTTOM:
                flag = false;
        case LyXTabular::TOGGLE_LINE_BOTTOM:
        {
-               bool lineSet = !tabular->BottomLine(actcell, flag); 
+               bool lineSet = !tabular->BottomLine(actcell, flag);
                for (int i = sel_row_start; i <= sel_row_end; ++i)
                        for (int j = sel_col_start; j <= sel_col_end; ++j)
                                tabular->SetBottomLine(
@@ -1961,7 +1927,7 @@ void InsetTabular::tabularFeatures(BufferView * bv,
                updateLocal(bv, INIT, true);
                break;
        }
-       
+
        case LyXTabular::M_TOGGLE_LINE_LEFT:
                flag = false;
        case LyXTabular::TOGGLE_LINE_LEFT:
@@ -1976,7 +1942,7 @@ void InsetTabular::tabularFeatures(BufferView * bv,
                updateLocal(bv, INIT, true);
                break;
        }
-       
+
        case LyXTabular::M_TOGGLE_LINE_RIGHT:
                flag = false;
        case LyXTabular::TOGGLE_LINE_RIGHT:
@@ -1991,7 +1957,7 @@ void InsetTabular::tabularFeatures(BufferView * bv,
                updateLocal(bv, INIT, true);
                break;
        }
-       
+
        case LyXTabular::M_ALIGN_LEFT:
        case LyXTabular::M_ALIGN_RIGHT:
        case LyXTabular::M_ALIGN_CENTER:
@@ -1999,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(
@@ -2024,8 +1991,8 @@ void InsetTabular::tabularFeatures(BufferView * bv,
        case LyXTabular::MULTICOLUMN:
        {
                if (sel_row_start != sel_row_end) {
-                       Alert::alert(_("Impossible Operation!"), 
-                                  _("Multicolumns can only be horizontally."), 
+                       Alert::alert(_("Impossible operation!"),
+                                  _("Multicolumns can only be horizontally."),
                                   _("Sorry."));
                        return;
                }
@@ -2147,7 +2114,7 @@ void InsetTabular::tabularFeatures(BufferView * bv,
 }
 
 
-bool InsetTabular::activateCellInset(BufferView * bv, int x, int y, int button,
+bool InsetTabular::activateCellInset(BufferView * bv, int x, int y, mouse_button::state button,
                                     bool behind)
 {
        UpdatableInset * inset =
@@ -2168,7 +2135,7 @@ bool InsetTabular::activateCellInset(BufferView * bv, int x, int y, int button,
 
 
 bool InsetTabular::activateCellInsetAbs(BufferView * bv, int x, int y,
-                                       int button)
+                                       mouse_button::state button)
 {
        inset_x = cursor_.x()
                - top_x + tabular->GetBeginningOfTextInCell(actcell);
@@ -2189,15 +2156,15 @@ bool InsetTabular::insetHit(BufferView *, int x, int) const
 int InsetTabular::getMaxWidthOfCell(BufferView * bv, int cell) const
 {
        LyXLength const len = tabular->GetPWidth(cell);
-       
+
        if (len.zero())
                return -1;
-       return len.inPixels(latexTextWidth(bv), bv->text->defaultHeight());
+       return len.inPixels(latexTextWidth(bv));
 }
 
 
 int InsetTabular::getMaxWidth(BufferView * bv,
-                              UpdatableInset const * inset) const
+                             UpdatableInset const * inset) const
 {
        int cell = tabular->GetCellFromInset(inset, actcell);
 
@@ -2206,13 +2173,13 @@ int InsetTabular::getMaxWidth(BufferView * bv,
                       << endl;
                return -1;
        }
-       
+
        int w = getMaxWidthOfCell(bv, cell);
        if (w > 0) {
                // because the inset then subtracts it's top_x and owner->x()
                w += (inset->x() - top_x);
        }
-       
+
        return w;
 }
 
@@ -2237,7 +2204,7 @@ void InsetTabular::resizeLyXText(BufferView * bv, bool force) const
 
 
 LyXText * InsetTabular::getLyXText(BufferView const * bv,
-                                   bool const recursive) const
+                                  bool const recursive) const
 {
        if (the_locking_inset)
                return the_locking_inset->getLyXText(bv, recursive);
@@ -2256,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;
 }
 
@@ -2273,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);
 }
 
 
@@ -2286,11 +2256,11 @@ FuncStatus InsetTabular::getStatus(string const & what) const
 {
        int action = LyXTabular::LAST_ACTION;
        FuncStatus status;
-       
+
        int i = 0;
        for (; tabularFeature[i].action != LyXTabular::LAST_ACTION; ++i) {
                string const tmp = tabularFeature[i].feature;
-               if (tmp == what.substr(0, tmp.length())) {                  
+               if (tmp == what.substr(0, tmp.length())) {
                        //if (!compare(tabularFeatures[i].feature.c_str(), what.c_str(),
                        //   tabularFeatures[i].feature.length())) {
                        action = tabularFeature[i].action;
@@ -2301,8 +2271,8 @@ FuncStatus InsetTabular::getStatus(string const & what) const
                status.clear();
                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;
@@ -2369,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:
@@ -2477,8 +2451,8 @@ 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());
+                            (int)parOwner()->params().depth(), true, '\t');
+       bv->stuffClipboard(STRCONV(sstr.str()));
        return true;
 }
 
@@ -2510,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;
@@ -2539,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;
@@ -2571,18 +2546,12 @@ 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
 {
        int const start = hasSelection() ? sel_cell_start : actcell;
        int const end = hasSelection() ? sel_cell_end : actcell;
+
        srow = tabular->row_of_cell(start);
        erow = tabular->row_of_cell(end);
        if (srow > erow) {
@@ -2599,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)
@@ -2652,49 +2608,49 @@ 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;
        }
        // otherwise we have to lock the next inset and ask for it's selecttion
        UpdatableInset * inset =
                static_cast<UpdatableInset*>(tabular->GetCellInset(actcell));
-       inset->edit(bv, 0,  0, 0);
-       string const str(selectNextWordInt(bv, value));
+       inset->edit(bv, 0,  0, mouse_button::none);
+       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
        UpdatableInset * inset =
                static_cast<UpdatableInset*>(tabular->GetCellInset(++actcell));
@@ -2721,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)
+                                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)
+                              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;
 }
 
 
@@ -2781,7 +2774,10 @@ bool InsetTabular::insetAllowed(Inset::Code code) const
 {
        if (the_locking_inset)
                return the_locking_inset->insetAllowed(code);
-       return false;
+       // we return true here because if the inset is not locked someone
+       // wants to insert something in one of our insettexts and we generally
+       // allow to do so.
+       return true;
 }
 
 
@@ -2800,11 +2796,11 @@ bool InsetTabular::forceDefaultParagraphs(Inset const * in) const
 }
 
 bool InsetTabular::insertAsciiString(BufferView * bv, string const & buf,
-                                     bool usePaste)
+                                    bool usePaste)
 {
-       if (buf.find('\t') == string::npos)
-               return false;
-       
+       if (buf.length() <= 0)
+               return true;
+
        int cols = 1;
        int rows = 1;
        int maxCols = 1;
@@ -2835,7 +2831,7 @@ bool InsetTabular::insertAsciiString(BufferView * bv, string const & buf,
        if (usePaste) {
                delete paste_tabular;
                paste_tabular = new LyXTabular(bv->buffer()->params,
-                                              this, rows, maxCols);
+                                              this, rows, maxCols);
                loctab = paste_tabular;
                cols = 0;
        } else {
@@ -2844,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)
        {
@@ -2859,15 +2857,22 @@ bool InsetTabular::insertAsciiString(BufferView * bv, string const & buf,
                case '\t':
                        // we can only set this if we are not too far right
                        if (cols < columns) {
-                               loctab->GetCellInset(cell)->setText(buf.substr(op, p-op));
+                               InsetText * ti = loctab->GetCellInset(cell);
+                               LyXFont const font = ti->getLyXText(bv)->
+                                       getFont(bv->buffer(), ti->paragraph(), 0);
+                               ti->setText(buf.substr(op, p-op), font);
                                ++cols;
                                ++cell;
                        }
                        break;
                case '\n':
                        // we can only set this if we are not too far right
-                       if (cols < columns)
-                               loctab->GetCellInset(cell)->setText(buf.substr(op, p-op));
+                       if (cols < columns) {
+                               InsetText * ti = loctab->GetCellInset(cell);
+                               LyXFont const font = ti->getLyXText(bv)->
+                                       getFont(bv->buffer(), ti->paragraph(), 0);
+                               ti->setText(buf.substr(op, p-op), font);
+                       }
                        cols = ocol;
                        ++row;
                        if (row < rows)
@@ -2878,8 +2883,100 @@ bool InsetTabular::insertAsciiString(BufferView * bv, string const & buf,
                op = p;
        }
        // check for the last cell if there is no trailing '\n'
-       if ((cell < cells) && (op < len))
-               loctab->GetCellInset(cell)->setText(buf.substr(op, len-op));
-       
+       if ((cell < cells) && (op < len)) {
+               InsetText * ti = loctab->GetCellInset(cell);
+               LyXFont const font = ti->getLyXText(bv)->
+                       getFont(bv->buffer(), ti->paragraph(), 0);
+               ti->setText(buf.substr(op, len-op), font);
+       }
+
        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();
+}