]> git.lyx.org Git - lyx.git/blobdiff - src/insets/InsetTabular.cpp
Fix GRAPHICS_EDIT of InsetGraphics
[lyx.git] / src / insets / InsetTabular.cpp
index 09f0422ed254a06b83b8651c58f05d57c29589aa..aeb3e7eb4c371be8785c5dec033969c791114e07 100644 (file)
@@ -3,7 +3,6 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author Jürgen Vigna
  * \author Lars Gullik Bjønnes
  * \author Matthias Ettrich
  * \author José Matos
 #include "Counters.h"
 #include "Cursor.h"
 #include "CutAndPaste.h"
-#include "support/debug.h"
 #include "DispatchResult.h"
 #include "FuncRequest.h"
 #include "FuncStatus.h"
-#include "support/gettext.h"
 #include "Language.h"
 #include "LaTeXFeatures.h"
 #include "Lexer.h"
 #include "LyXFunc.h"
+#include "LyXRC.h"
 #include "MetricsInfo.h"
 #include "OutputParams.h"
 #include "paragraph_funcs.h"
 #include "TextClass.h"
 #include "TextMetrics.h"
 
-#include "support/convert.h"
-#include "support/docstream.h"
-#include "support/FileName.h"
-#include "support/lstrings.h"
-
 #include "frontends/alert.h"
 #include "frontends/Clipboard.h"
 #include "frontends/Painter.h"
 #include "frontends/Selection.h"
 
+
+#include "support/convert.h"
+#include "support/debug.h"
+#include "support/docstream.h"
+#include "support/FileName.h"
+#include "support/gettext.h"
+#include "support/lstrings.h"
+
 #include <boost/scoped_ptr.hpp>
 
 #include <sstream>
@@ -84,10 +85,10 @@ namespace Alert = frontend::Alert;
 
 namespace {
 
-int const ADD_TO_HEIGHT = 2;
-int const ADD_TO_TABULAR_WIDTH = 2;
-int const default_line_space = 10;
-int const WIDTH_OF_LINE = 5;
+int const ADD_TO_HEIGHT = 2; // in cell
+int const ADD_TO_TABULAR_WIDTH = 6; // horiz space before and after the table
+int const default_line_space = 10; // ?
+int const WIDTH_OF_LINE = 5; // space between double lines
 
 
 ///
@@ -119,10 +120,6 @@ TabularFeature tabularFeature[] =
        { Tabular::VALIGN_TOP, "valign-top" },
        { Tabular::VALIGN_BOTTOM, "valign-bottom" },
        { Tabular::VALIGN_MIDDLE, "valign-middle" },
-       { Tabular::M_TOGGLE_LINE_TOP, "m-toggle-line-top" },
-       { Tabular::M_TOGGLE_LINE_BOTTOM, "m-toggle-line-bottom" },
-       { Tabular::M_TOGGLE_LINE_LEFT, "m-toggle-line-left" },
-       { Tabular::M_TOGGLE_LINE_RIGHT, "m-toggle-line-right" },
        { Tabular::M_ALIGN_LEFT, "m-align-left" },
        { Tabular::M_ALIGN_RIGHT, "m-align-right" },
        { Tabular::M_ALIGN_CENTER, "m-align-center" },
@@ -159,6 +156,7 @@ TabularFeature tabularFeature[] =
        { Tabular::SET_TOP_SPACE, "set-top-space" },
        { Tabular::SET_BOTTOM_SPACE, "set-bottom-space" },
        { Tabular::SET_INTERLINE_SPACE, "set-interline-space" },
+       { Tabular::SET_BORDER_LINES, "set-border-lines" },
        { Tabular::LAST_ACTION, "" }
 };
 
@@ -344,7 +342,7 @@ bool getTokenValue(string const & str, char const * token, string & ret)
 {
        ret.erase();
        size_t token_length = strlen(token);
-       string::size_type pos = str.find(token);
+       size_t pos = str.find(token);
 
        if (pos == string::npos || pos + token_length + 1 >= str.length()
                || str[pos + token_length] != '=')
@@ -471,25 +469,26 @@ string const featureAsString(Tabular::Feature feature)
 /////////////////////////////////////////////////////////////////////
 
 
-Tabular::cellstruct::cellstruct(BufferParams const & bp)
+Tabular::CellData::CellData(Buffer const & buf, Tabular const & table)
        : cellno(0),
          width(0),
          multicolumn(Tabular::CELL_NORMAL),
          alignment(LYX_ALIGN_CENTER),
          valignment(LYX_VALIGN_TOP),
-         top_line(true),
+         top_line(false),
          bottom_line(false),
-         left_line(true),
+         left_line(false),
          right_line(false),
          usebox(BOX_NONE),
          rotate(false),
-         inset(new InsetText(bp))
+         inset(new InsetTableCell(buf, this, &table))
 {
-       inset->paragraphs().back().layout(bp.getTextClass().emptyLayout());
+       inset->setBuffer(const_cast<Buffer &>(buf));
+       inset->paragraphs().back().setLayout(buf.params().documentClass().emptyLayout());
 }
 
 
-Tabular::cellstruct::cellstruct(cellstruct const & cs)
+Tabular::CellData::CellData(CellData const & cs)
        : cellno(cs.cellno),
          width(cs.width),
          multicolumn(cs.multicolumn),
@@ -503,18 +502,20 @@ Tabular::cellstruct::cellstruct(cellstruct const & cs)
          rotate(cs.rotate),
          align_special(cs.align_special),
          p_width(cs.p_width),
-         inset(dynamic_cast<InsetText*>(cs.inset->clone()))
-{}
+         inset(dynamic_cast<InsetTableCell *>(cs.inset->clone()))
+{
+       inset->setCellData(this);
+}
 
 
-Tabular::cellstruct & Tabular::cellstruct::operator=(cellstruct cs)
+Tabular::CellData & Tabular::CellData::operator=(CellData cs)
 {
        swap(cs);
        return *this;
 }
 
 
-void Tabular::cellstruct::swap(cellstruct & rhs)
+void Tabular::CellData::swap(CellData & rhs)
 {
        std::swap(cellno, rhs.cellno);
        std::swap(width, rhs.width);
@@ -533,11 +534,9 @@ void Tabular::cellstruct::swap(cellstruct & rhs)
 }
 
 
-Tabular::rowstruct::rowstruct()
+Tabular::RowData::RowData()
        : ascent(0),
          descent(0),
-         top_line(true),
-         bottom_line(false),
          top_space_default(false),
          bottom_space_default(false),
          interline_space_default(false),
@@ -549,11 +548,9 @@ Tabular::rowstruct::rowstruct()
 {}
 
 
-Tabular::columnstruct::columnstruct()
+Tabular::ColumnData::ColumnData()
        : alignment(LYX_ALIGN_CENTER),
          valignment(LYX_VALIGN_TOP),
-         left_line(true),
-         right_line(false),
          width(0)
 {
 }
@@ -572,58 +569,56 @@ Tabular::Tabular()
 }
 
 
-Tabular::Tabular(BufferParams const & bp, row_type rows_arg,
-                      col_type columns_arg)
+Tabular::Tabular(Buffer const & buffer, row_type rows_arg, col_type columns_arg)
 {
-       init(bp, rows_arg, columns_arg);
+       init(buffer, rows_arg, columns_arg);
 }
 
 
 // activates all lines and sets all widths to 0
-void Tabular::init(BufferParams const & bp, row_type rows_arg,
+void Tabular::init(Buffer const & buf, row_type rows_arg,
                      col_type columns_arg)
 {
+       buffer_ = &buf;
        row_info = row_vector(rows_arg);
        column_info = column_vector(columns_arg);
-       cell_info = cell_vvector(rows_arg, cell_vector(columns_arg, cellstruct(bp)));
+       cell_info = cell_vvector(rows_arg, cell_vector(columns_arg, CellData(buf, *this)));
        row_info.reserve(10);
        column_info.reserve(10);
        cell_info.reserve(100);
        fixCellNums();
-       for (row_type i = 0; i < rows_arg; ++i)
-               cell_info[i].back().right_line = true;
-       row_info.back().bottom_line = true;
-       row_info.front().bottom_line = true;
-       column_info.back().right_line = true;
        is_long_tabular = false;
        rotate = false;
        use_booktabs = false;
+       // set silly default lines
+       for (row_type i = 0; i < rowCount(); ++i)
+               for (col_type j = 0; j < columnCount(); ++j) {
+                       cell_info[i][j].top_line = true;
+                       cell_info[i][j].left_line = true;
+                       cell_info[i][j].bottom_line = i == 0 || i == rowCount() - 1;
+                       cell_info[i][j].right_line = j == columnCount() - 1;
+               }
 }
 
 
 void Tabular::fixCellNums()
 {
        idx_type cellno = 0;
-       for (row_type i = 0; i < rowCount(); ++i) {
-               for (col_type j = 0; j < columnCount(); ++j) {
-                       // When debugging it can be nice to set
-                       // this to true.
-                       cell_info[i][j].inset->setDrawFrame(false);
+       for (row_type i = 0; i < rowCount(); ++i)
+               for (col_type j = 0; j < columnCount(); ++j)
                        cell_info[i][j].cellno = cellno++;
-               }
-               cell_info[i].back().right_line = true;
-       }
 
        updateIndexes();
 }
 
 
-void Tabular::appendRow(BufferParams const & bp, idx_type const cell)
+void Tabular::appendRow(idx_type const cell)
 {
+       BufferParams const & bp = buffer().params();
        row_type const row = cellRow(cell);
 
        row_vector::iterator rit = row_info.begin() + row;
-       row_info.insert(rit, rowstruct());
+       row_info.insert(rit, RowData());
        // now set the values of the row before
        row_info[row] = row_info[row + 1];
 
@@ -634,7 +629,7 @@ void Tabular::appendRow(BufferParams const & bp, idx_type const cell)
        for (row_type i = 0; i < nrows - 1; ++i)
                swap(cell_info[i], old[i]);
 
-       cell_info = cell_vvector(nrows, cell_vector(ncols, cellstruct(bp)));
+       cell_info = cell_vvector(nrows, cell_vector(ncols, CellData(buffer(), *this)));
 
        for (row_type i = 0; i <= row; ++i)
                swap(cell_info[i], old[i]);
@@ -661,12 +656,12 @@ void Tabular::deleteRow(row_type const row)
 }
 
 
-void Tabular::copyRow(BufferParams const & bp, row_type const row)
+void Tabular::copyRow(row_type const row)
 {
        row_info.insert(row_info.begin() + row, row_info[row]);
        cell_info.insert(cell_info.begin() + row, cell_info[row]);
 
-       if (bp.trackChanges)
+       if (buffer().params().trackChanges)
                for (col_type j = 0; j < columnCount(); ++j)
                        cell_info[row + 1][j].inset->setChange(Change(Change::INSERTED));
 
@@ -674,30 +669,28 @@ void Tabular::copyRow(BufferParams const & bp, row_type const row)
 }
 
 
-void Tabular::appendColumn(BufferParams const & bp, idx_type const cell)
+void Tabular::appendColumn(idx_type const cell)
 {
        col_type const column = cellColumn(cell);
-       col_type const ncols = columnCount();
        column_vector::iterator cit = column_info.begin() + column + 1;
-       column_info.insert(cit, columnstruct());
+       column_info.insert(cit, ColumnData());
+       col_type const ncols = columnCount();
        // set the column values of the column before
        column_info[column + 1] = column_info[column];
 
        for (row_type i = 0; i < rowCount(); ++i) {
-               cell_info[i].insert(cell_info[i].begin() + column + 1, cellstruct(bp));
-
-               // care about multicolumns
-               if (cell_info[i][column + 1].multicolumn == CELL_BEGIN_OF_MULTICOLUMN)
-                       cell_info[i][column + 1].multicolumn = CELL_PART_OF_MULTICOLUMN;
-
-               if (column + 2 >= ncols
-                   || cell_info[i][column + 2].multicolumn != CELL_PART_OF_MULTICOLUMN)
-                       cell_info[i][column + 1].multicolumn = Tabular::CELL_NORMAL;
+               cell_info[i].insert(cell_info[i].begin() + column + 1, CellData(buffer(), *this));
+               col_type c = column + 2;
+               while (c < ncols 
+                       && cell_info[i][c].multicolumn == CELL_PART_OF_MULTICOLUMN) {
+                       cell_info[i][c].multicolumn = CELL_NORMAL;
+                       ++c;
+               }
        }
        //++column;
        for (row_type i = 0; i < rowCount(); ++i) {
                cell_info[i][column + 1].inset->clear();
-               if (bp.trackChanges)
+               if (buffer().params().trackChanges)
                        cell_info[i][column + 1].inset->setChange(Change(Change::INSERTED));
        }
        fixCellNums();
@@ -724,8 +717,9 @@ void Tabular::deleteColumn(col_type const column)
 }
 
 
-void Tabular::copyColumn(BufferParams const & bp, col_type const column)
+void Tabular::copyColumn(col_type const column)
 {
+       BufferParams const & bp = buffer().params();
        column_info.insert(column_info.begin() + column, column_info[column]);
 
        for (row_type i = 0; i < rowCount(); ++i)
@@ -746,7 +740,7 @@ void Tabular::updateIndexes()
        for (row_type row = 0; row < rowCount(); ++row) {
                for (col_type column = 0; column < columnCount(); ++column) {
                        if (cell_info[row][column].multicolumn
-                               != Tabular::CELL_PART_OF_MULTICOLUMN)
+                               != CELL_PART_OF_MULTICOLUMN)
                                ++numberofcells;
                        BOOST_ASSERT(numberofcells != 0);
                        cell_info[row][column].cellno =
@@ -767,8 +761,8 @@ void Tabular::updateIndexes()
                do {
                        ++column;
                } while (column < columnCount() &&
-                                cell_info[row][column].multicolumn
-                                == Tabular::CELL_PART_OF_MULTICOLUMN);
+                               cell_info[row][column].multicolumn
+                                               == Tabular::CELL_PART_OF_MULTICOLUMN);
 
                if (column == columnCount()) {
                        column = 0;
@@ -804,78 +798,51 @@ Tabular::idx_type Tabular::numberOfCellsInRow(idx_type const cell) const
 }
 
 
-bool Tabular::topLine(idx_type const cell, bool const wholerow) const
+bool Tabular::topLine(idx_type const cell) const
 {
-       if (!wholerow && isMultiColumn(cell) &&
-           !(use_booktabs && cellRow(cell) == 0))
-               return cellinfo_of_cell(cell).top_line;
-       return row_info[cellRow(cell)].top_line;
+       return cellinfo_of_cell(cell).top_line;
 }
 
 
-bool Tabular::bottomLine(idx_type const cell, bool wholerow) const
+bool Tabular::bottomLine(idx_type const cell) const
 {
-       if (!wholerow && isMultiColumn(cell) &&
-           !(use_booktabs && isLastRow(cell)))
-               return cellinfo_of_cell(cell).bottom_line;
-       return row_info[cellRow(cell)].bottom_line;
+       return cellinfo_of_cell(cell).bottom_line;
 }
 
 
-bool Tabular::leftLine(idx_type cell, bool wholecolumn) const
+bool Tabular::leftLine(idx_type cell) const
 {
        if (use_booktabs)
                return false;
-       if (!wholecolumn && isMultiColumn(cell) &&
-               (isFirstCellInRow(cell) || isMultiColumn(cell-1)))
-               return cellinfo_of_cell(cell).left_line;
-       return column_info[cellColumn(cell)].left_line;
+       return cellinfo_of_cell(cell).left_line;
 }
 
 
-bool Tabular::rightLine(idx_type cell, bool wholecolumn) const
+bool Tabular::rightLine(idx_type cell) const
 {
        if (use_booktabs)
                return false;
-       if (!wholecolumn && isMultiColumn(cell) &&
-               (isLastCellInRow(cell) || isMultiColumn(cell + 1)))
-               return cellinfo_of_cell(cell).right_line;
-       return column_info[cellRightColumn(cell)].right_line;
+       return cellinfo_of_cell(cell).right_line;
 }
 
 
 bool Tabular::topAlreadyDrawn(idx_type cell) const
 {
        row_type row = cellRow(cell);
-       if (row > 0 && !getAdditionalHeight(row)) {
-               col_type column = cellColumn(cell);
-               --row;
-               while (column
-                          && cell_info[row][column].multicolumn
-                          == Tabular::CELL_PART_OF_MULTICOLUMN)
-                       --column;
-               if (cell_info[row][column].multicolumn == Tabular::CELL_NORMAL)
-                       return row_info[row].bottom_line;
-               else
-                       return cell_info[row][column].bottom_line;
-       }
-       return false;
+       if (row == 0)
+               return false;
+       idx_type i = cellIndex(row - 1, cellColumn(cell));
+       return !rowBottomLine(row - 1) && bottomLine(i);
 }
 
 
 bool Tabular::leftAlreadyDrawn(idx_type cell) const
 {
-       col_type column = cellColumn(cell);
-       if (column > 0) {
-               row_type row = cellRow(cell);
-               while (--column &&
-                          (cell_info[row][column].multicolumn ==
-                               Tabular::CELL_PART_OF_MULTICOLUMN)) { }
-               if (getAdditionalWidth(cell_info[row][column].cellno))
-                       return false;
-               return rightLine(cell_info[row][column].cellno);
-       }
-       return false;
+       col_type col = cellColumn(cell);
+       if (col == 0)
+               return false;
+       idx_type i = cellIndex(cellRow(cell), col - 1);
+       return !columnRightLine(col - 1) && rightLine(i);
 }
 
 
@@ -890,31 +857,10 @@ int Tabular::getAdditionalHeight(row_type row) const
        if (!row || row >= rowCount())
                return 0;
 
-       bool top = true;
-       bool bottom = true;
-
-       for (col_type column = 0; column < columnCount() && bottom; ++column) {
-               switch (cell_info[row - 1][column].multicolumn) {
-               case Tabular::CELL_BEGIN_OF_MULTICOLUMN:
-                       bottom = cell_info[row - 1][column].bottom_line;
-                       break;
-               case Tabular::CELL_NORMAL:
-                       bottom = row_info[row - 1].bottom_line;
-               }
-       }
-       for (col_type column = 0; column < columnCount() && top; ++column) {
-               switch (cell_info[row][column].multicolumn) {
-               case Tabular::CELL_BEGIN_OF_MULTICOLUMN:
-                       top = cell_info[row][column].top_line;
-                       break;
-               case Tabular::CELL_NORMAL:
-                       top = row_info[row].top_line;
-               }
-       }
        int const interline_space = row_info[row - 1].interline_space_default ?
                default_line_space :
                row_info[row - 1].interline_space.inPixels(width());
-       if (top && bottom)
+       if (rowTopLine(row) && rowBottomLine(row - 1))
                return interline_space + WIDTH_OF_LINE;
        return interline_space;
 }
@@ -924,13 +870,11 @@ int Tabular::getAdditionalWidth(idx_type cell) const
 {
        // internally already set in setCellWidth
        // used to get it back in text.cpp
-       col_type const col = cellRightColumn(cell);
-       row_type const row = cellRow(cell);
-       if (col < columnCount() - 1 && rightLine(cell) &&
-               leftLine(cell_info[row][col+1].cellno)) // column_info[col+1].left_line)
-       {
+       col_type c = cellColumn(cell);
+       if (c < columnCount() - 1 
+               && columnRightLine(c) && columnLeftLine(c + 1)
+               && cellinfo_of_cell(cell).multicolumn == CELL_NORMAL)
                return WIDTH_OF_LINE;
-       }
        return 0;
 }
 
@@ -1019,15 +963,13 @@ void Tabular::recalculateMulticolumnsOfColumn(col_type column)
 void Tabular::setCellWidth(idx_type cell, int new_width)
 {
        row_type const row = cellRow(cell);
-       col_type const column1 = cellColumn(cell);
+       col_type const col = cellColumn(cell);
        bool tmp = false;
        int width = 0;
        int add_width = 0;
 
-       if (rightLine(cell_info[row][column1].cellno, true) &&
-               column1 < columnCount() - 1 &&
-               leftLine(cell_info[row][column1+1].cellno, true))
-       {
+       if (col < columnCount() - 1 && columnRightLine(col) &&
+               columnLeftLine(col + 1)) {
                add_width = WIDTH_OF_LINE;
        }
 
@@ -1038,10 +980,10 @@ void Tabular::setCellWidth(idx_type cell, int new_width)
                tmp = setWidthOfMulticolCell(cell, new_width);
        } else {
                width = new_width + 2 * WIDTH_OF_LINE + add_width;
-               cell_info[row][column1].width = width;
-               tmp = calculate_width_of_column_NMC(column1);
+               cell_info[row][col].width = width;
+               tmp = calculate_width_of_column_NMC(col);
                if (tmp)
-                       recalculateMulticolumnsOfColumn(column1);
+                       recalculateMulticolumnsOfColumn(col);
        }
        if (tmp) {
                for (col_type i = 0; i < columnCount(); ++i)
@@ -1077,7 +1019,7 @@ namespace {
  * merge cell paragraphs and reset layout to standard for variable width
  * cells.
  */
-void toggleFixedWidth(Cursor & cur, InsetText * inset, bool fixedWidth)
+void toggleFixedWidth(Cursor & cur, InsetTableCell * inset, bool fixedWidth)
 {
        inset->setAutoBreakRows(fixedWidth);
        if (fixedWidth)
@@ -1092,7 +1034,7 @@ void toggleFixedWidth(Cursor & cur, InsetText * inset, bool fixedWidth)
        cur.push(*inset);
        // undo information has already been recorded
        inset->getText(0)->setLayout(cur.bv().buffer(), 0, cur.lastpit() + 1,
-                       bp.getTextClass().emptyLayoutName());
+                       bp.documentClass().emptyLayoutName());
        cur.pop();
 }
 
@@ -1157,40 +1099,74 @@ void Tabular::setAllLines(idx_type cell, bool line)
 }
 
 
-void Tabular::setTopLine(idx_type cell, bool line, bool wholerow)
+void Tabular::setTopLine(idx_type i, bool line)
 {
-       row_type const row = cellRow(cell);
-       if (wholerow || !isMultiColumn(cell))
-               row_info[row].top_line = line;
-       else
-               cellinfo_of_cell(cell).top_line = line;
+       cellinfo_of_cell(i).top_line = line;
 }
 
 
-void Tabular::setBottomLine(idx_type cell, bool line, bool wholerow)
+void Tabular::setBottomLine(idx_type i, bool line)
 {
-       if (wholerow || !isMultiColumn(cell))
-               row_info[cellRow(cell)].bottom_line = line;
-       else
-               cellinfo_of_cell(cell).bottom_line = line;
+       cellinfo_of_cell(i).bottom_line = line;
 }
 
 
-void Tabular::setLeftLine(idx_type cell, bool line, bool wholecolumn)
+void Tabular::setLeftLine(idx_type cell, bool line)
 {
-       if (wholecolumn || !isMultiColumn(cell))
-               column_info[cellColumn(cell)].left_line = line;
-       else
-               cellinfo_of_cell(cell).left_line = line;
+       cellinfo_of_cell(cell).left_line = line;
 }
 
 
-void Tabular::setRightLine(idx_type cell, bool line, bool wholecolumn)
+void Tabular::setRightLine(idx_type cell, bool line)
 {
-       if (wholecolumn || !isMultiColumn(cell))
-               column_info[cellRightColumn(cell)].right_line = line;
-       else
-               cellinfo_of_cell(cell).right_line = line;
+       cellinfo_of_cell(cell).right_line = line;
+}
+
+bool Tabular::rowTopLine(row_type r) const
+{
+       idx_type i0 = getFirstCellInRow(r);
+       idx_type i1 = getLastCellInRow(r);
+       bool all_rows_set = true;
+       for (idx_type j = i0; all_rows_set && j <= i1; ++j)
+               all_rows_set = cellinfo_of_cell(j).top_line;
+       return all_rows_set;
+}
+
+
+bool Tabular::rowBottomLine(row_type r) const
+{
+       idx_type i0 = getFirstCellInRow(r);
+       idx_type i1 = getLastCellInRow(r);
+       bool all_rows_set = true;
+       for (idx_type j = i0; all_rows_set && j <= i1; ++j)
+               all_rows_set = cellinfo_of_cell(j).bottom_line;
+       return all_rows_set;
+}
+
+
+bool Tabular::columnLeftLine(col_type c) const
+{
+       bool all_cols_set = true;
+       row_type nrows = rowCount();
+       for (row_type r = 0; all_cols_set && r < nrows; ++r) {
+               idx_type i = cellIndex(r, c);
+               if (columnSpan(i) == 1)
+                       all_cols_set = cellinfo_of_cell(i).left_line;
+       }
+       return all_cols_set;
+}
+
+
+bool Tabular::columnRightLine(col_type c) const
+{
+       bool all_cols_set = true;
+       row_type nrows = rowCount();
+       for (row_type r = 0; all_cols_set && r < nrows; ++r) {
+               idx_type i = cellIndex(r, c);
+               if (c == cellColumn(i) + columnSpan(i) - 1)
+                       all_cols_set = cellinfo_of_cell(i).right_line;
+       }
+       return all_cols_set;
 }
 
 
@@ -1362,13 +1338,13 @@ Tabular::col_type Tabular::cellRightColumn(idx_type cell) const
        row_type const row = cellRow(cell);
        col_type column = cellColumn(cell);
        while (column < columnCount() - 1 &&
-                  cell_info[row][column + 1].multicolumn == Tabular::CELL_PART_OF_MULTICOLUMN)
+                  cell_info[row][column + 1].multicolumn == CELL_PART_OF_MULTICOLUMN)
                ++column;
        return column;
 }
 
 
-void Tabular::write(Buffer const & buf, ostream & os) const
+void Tabular::write(ostream & os) const
 {
        // header line
        os << "<lyxtabular"
@@ -1396,17 +1372,13 @@ void Tabular::write(Buffer const & buf, ostream & os) const
                os << "<column"
                   << write_attribute("alignment", column_info[j].alignment)
                   << write_attribute("valignment", column_info[j].valignment)
-                  << write_attribute("leftline", column_info[j].left_line)
-                  << write_attribute("rightline", column_info[j].right_line)
                   << write_attribute("width", column_info[j].p_width.asString())
                   << write_attribute("special", column_info[j].align_special)
                   << ">\n";
        }
        for (row_type i = 0; i < rowCount(); ++i) {
-               os << "<row"
-                  << write_attribute("topline", row_info[i].top_line)
-                  << write_attribute("bottomline", row_info[i].bottom_line);
                static const string def("default");
+               os << "<row";
                if (row_info[i].top_space_default)
                        os << write_attribute("topspace", def);
                else
@@ -1440,7 +1412,7 @@ void Tabular::write(Buffer const & buf, ostream & os) const
                           << write_attribute("special", cell_info[i][j].align_special)
                           << ">\n";
                        os << "\\begin_inset ";
-                       cell_info[i][j].inset->write(buf, os);
+                       cell_info[i][j].inset->write(os);
                        os << "\n\\end_inset\n"
                           << "</cell>\n";
                }
@@ -1450,14 +1422,13 @@ void Tabular::write(Buffer const & buf, ostream & os) const
 }
 
 
-void Tabular::read(Buffer const & buf, Lexer & lex)
+void Tabular::read(Lexer & lex)
 {
        string line;
        istream & is = lex.getStream();
 
        l_getline(is, line);
-       if (!prefixIs(line, "<lyxtabular ")
-               && !prefixIs(line, "<Tabular ")) {
+       if (!prefixIs(line, "<lyxtabular ") && !prefixIs(line, "<Tabular ")) {
                BOOST_ASSERT(false);
                return;
        }
@@ -1473,7 +1444,7 @@ void Tabular::read(Buffer const & buf, Lexer & lex)
        int columns_arg;
        if (!getTokenValue(line, "columns", columns_arg))
                return;
-       init(buf.params(), rows_arg, columns_arg);
+       init(buffer(), rows_arg, columns_arg);
        l_getline(is, line);
        if (!prefixIs(line, "<features")) {
                lyxerr << "Wrong tabular format (expected <features ...> got"
@@ -1503,8 +1474,6 @@ void Tabular::read(Buffer const & buf, Lexer & lex)
                }
                getTokenValue(line, "alignment", column_info[j].alignment);
                getTokenValue(line, "valignment", column_info[j].valignment);
-               getTokenValue(line, "leftline", column_info[j].left_line);
-               getTokenValue(line, "rightline", column_info[j].right_line);
                getTokenValue(line, "width", column_info[j].p_width);
                getTokenValue(line, "special", column_info[j].align_special);
        }
@@ -1516,8 +1485,6 @@ void Tabular::read(Buffer const & buf, Lexer & lex)
                               << line << ')' << endl;
                        return;
                }
-               getTokenValue(line, "topline", row_info[i].top_line);
-               getTokenValue(line, "bottomline", row_info[i].bottom_line);
                getTokenValue(line, "topspace", row_info[i].top_space,
                              row_info[i].top_space_default);
                getTokenValue(line, "bottomspace", row_info[i].bottom_space,
@@ -1549,7 +1516,7 @@ void Tabular::read(Buffer const & buf, Lexer & lex)
                        getTokenValue(line, "special", cell_info[i][j].align_special);
                        l_getline(is, line);
                        if (prefixIs(line, "\\begin_inset")) {
-                               cell_info[i][j].inset->read(buf, lex);
+                               cell_info[i][j].inset->read(lex);
                                l_getline(is, line);
                        }
                        if (!prefixIs(line, "</cell>")) {
@@ -1574,44 +1541,40 @@ void Tabular::read(Buffer const & buf, Lexer & lex)
 
 bool Tabular::isMultiColumn(idx_type cell) const
 {
-       return cellinfo_of_cell(cell).multicolumn != Tabular::CELL_NORMAL;
+       return cellinfo_of_cell(cell).multicolumn != CELL_NORMAL;
 }
 
 
 bool Tabular::isMultiColumnReal(idx_type cell) const
 {
        return cellColumn(cell) != cellRightColumn(cell) &&
-                       cellinfo_of_cell(cell).multicolumn != Tabular::CELL_NORMAL;
+                       cellinfo_of_cell(cell).multicolumn != CELL_NORMAL;
 }
 
 
-Tabular::cellstruct & Tabular::cellinfo_of_cell(idx_type cell) const
+Tabular::CellData & Tabular::cellinfo_of_cell(idx_type cell) const
 {
        return cell_info[cellRow(cell)][cellColumn(cell)];
 }
 
 
-void Tabular::setMultiColumn(Buffer * buffer, idx_type cell,
-                               idx_type number)
+void Tabular::setMultiColumn(idx_type cell, idx_type number)
 {
-       cellstruct & cs = cellinfo_of_cell(cell);
+       CellData & cs = cellinfo_of_cell(cell);
        cs.multicolumn = CELL_BEGIN_OF_MULTICOLUMN;
        cs.alignment = column_info[cellColumn(cell)].alignment;
-       cs.top_line = row_info[cellRow(cell)].top_line;
-       cs.bottom_line = row_info[cellRow(cell)].bottom_line;
-       cs.left_line = column_info[cellColumn(cell)].left_line;
-       cs.right_line = column_info[cellColumn(cell+number-1)].right_line;
        for (idx_type i = 1; i < number; ++i) {
-               cellstruct & cs1 = cellinfo_of_cell(cell + i);
+               CellData & cs1 = cellinfo_of_cell(cell + i);
                cs1.multicolumn = CELL_PART_OF_MULTICOLUMN;
-               cs.inset->appendParagraphs(buffer, cs1.inset->paragraphs());
+               cs.inset->appendParagraphs(cs1.inset->paragraphs());
                cs1.inset->clear();
        }
+       setRightLine(cell, rightLine(cell + number - 1));
        updateIndexes();
 }
 
 
-Tabular::idx_type Tabular::cells_in_multicolumn(idx_type cell) const
+Tabular::idx_type Tabular::columnSpan(idx_type cell) const
 {
        row_type const row = cellRow(cell);
        col_type column = cellColumn(cell);
@@ -1734,26 +1697,6 @@ Tabular::idx_type Tabular::getCellBelow(idx_type cell) const
 }
 
 
-Tabular::idx_type Tabular::getLastCellAbove(idx_type cell) const
-{
-       if (cellRow(cell) == 0)
-               return cell;
-       if (!isMultiColumn(cell))
-               return getCellAbove(cell);
-       return cell_info[cellRow(cell) - 1][cellRightColumn(cell)].cellno;
-}
-
-
-Tabular::idx_type Tabular::getLastCellBelow(idx_type cell) const
-{
-       if (cellRow(cell) + 1 >= rowCount())
-               return cell;
-       if (!isMultiColumn(cell))
-               return getCellBelow(cell);
-       return cell_info[cellRow(cell) + 1][cellRightColumn(cell)].cellno;
-}
-
-
 Tabular::idx_type Tabular::cellIndex(row_type row,
                                               col_type column) const
 {
@@ -1963,7 +1906,8 @@ int Tabular::TeXTopHLine(odocstream & os, row_type row) const
                        ++tmp;
        }
        if (use_booktabs && row == 0) {
-               os << "\\toprule ";
+               if (topLine(fcell))
+                       os << "\\toprule ";
        } else if (tmp == n - fcell) {
                os << (use_booktabs ? "\\midrule " : "\\hline ");
        } else if (tmp) {
@@ -1999,7 +1943,8 @@ int Tabular::TeXBottomHLine(odocstream & os, row_type row) const
                        ++tmp;
        }
        if (use_booktabs && row == rowCount() - 1) {
-               os << "\\bottomrule";
+               if (bottomLine(fcell))
+                       os << "\\bottomrule";
        } else if (tmp == n - fcell) {
                os << (use_booktabs ? "\\midrule" : "\\hline");
        } else if (tmp) {
@@ -2028,18 +1973,20 @@ int Tabular::TeXCellPreamble(odocstream & os, idx_type cell) const
                os << "\\begin{sideways}\n";
                ++ret;
        }
-       if (isMultiColumn(cell)) {
-               os << "\\multicolumn{" << cells_in_multicolumn(cell) << "}{";
-               if (leftLine(cell) &&
-                       (isFirstCellInRow(cell) ||
-                        (!isMultiColumn(cell - 1) && !leftLine(cell, true) &&
-                         !rightLine(cell - 1, true))))
+       Tabular::VAlignment valign =  getVAlignment(cell, !isMultiColumn(cell));
+       LyXAlignment align = getAlignment(cell, !isMultiColumn(cell));
+       col_type c = cellColumn(cell);
+       if (isMultiColumn(cell)
+               || (leftLine(cell) && !columnLeftLine(c))
+               || (rightLine(cell) && !columnRightLine(c))) {
+               os << "\\multicolumn{" << columnSpan(cell) << "}{";
+               if (leftLine(cell))
                        os << '|';
                if (!cellinfo_of_cell(cell).align_special.empty()) {
                        os << cellinfo_of_cell(cell).align_special;
                } else {
                        if (!getPWidth(cell).zero()) {
-                               switch (getVAlignment(cell)) {
+                               switch (valign) {
                                case LYX_VALIGN_TOP:
                                        os << 'p';
                                        break;
@@ -2054,7 +2001,7 @@ int Tabular::TeXCellPreamble(odocstream & os, idx_type cell) const
                                   << from_ascii(getPWidth(cell).asLatexString())
                                   << '}';
                        } else {
-                               switch (getAlignment(cell)) {
+                               switch (align) {
                                case LYX_ALIGN_LEFT:
                                        os << 'l';
                                        break;
@@ -2069,14 +2016,11 @@ int Tabular::TeXCellPreamble(odocstream & os, idx_type cell) const
                } // end if else !cellinfo_of_cell
                if (rightLine(cell))
                        os << '|';
-               if (((cell + 1) < cellCount()) && !isFirstCellInRow(cell+1) &&
-                       leftLine(cell+1))
-                       os << '|';
                os << "}{";
                }
        if (getUsebox(cell) == BOX_PARBOX) {
                os << "\\parbox[";
-               switch (getVAlignment(cell)) {
+               switch (valign) {
                case LYX_VALIGN_TOP:
                        os << 't';
                        break;
@@ -2091,7 +2035,7 @@ int Tabular::TeXCellPreamble(odocstream & os, idx_type cell) const
                   << "}{";
        } else if (getUsebox(cell) == BOX_MINIPAGE) {
                os << "\\begin{minipage}[";
-               switch (getVAlignment(cell)) {
+               switch (valign) {
                case LYX_VALIGN_TOP:
                        os << 't';
                        break;
@@ -2121,7 +2065,10 @@ int Tabular::TeXCellPostamble(odocstream & os, idx_type cell) const
                os << "%\n\\end{minipage}";
                ret += 2;
        }
-       if (isMultiColumn(cell)) {
+       col_type c = cellColumn(cell);
+       if (isMultiColumn(cell)
+               || (leftLine(cell) && !columnLeftLine(c))
+               || (rightLine(cell) && !columnRightLine(c))) {
                os << '}';
        }
        if (getRotateCell(cell)) {
@@ -2132,7 +2079,7 @@ int Tabular::TeXCellPostamble(odocstream & os, idx_type cell) const
 }
 
 
-int Tabular::TeXLongtableHeaderFooter(odocstream & os, Buffer const & buf,
+int Tabular::TeXLongtableHeaderFooter(odocstream & os,
                                         OutputParams const & runparams) const
 {
        if (!is_long_tabular)
@@ -2147,7 +2094,7 @@ int Tabular::TeXLongtableHeaderFooter(odocstream & os, Buffer const & buf,
                }
                for (row_type i = 0; i < rowCount(); ++i) {
                        if (row_info[i].endhead) {
-                               ret += TeXRow(os, i, buf, runparams);
+                               ret += TeXRow(os, i, runparams);
                        }
                }
                if (endhead.bottomDL) {
@@ -2169,7 +2116,7 @@ int Tabular::TeXLongtableHeaderFooter(odocstream & os, Buffer const & buf,
                }
                for (row_type i = 0; i < rowCount(); ++i) {
                        if (row_info[i].endfirsthead) {
-                               ret += TeXRow(os, i, buf, runparams);
+                               ret += TeXRow(os, i, runparams);
                        }
                }
                if (endfirsthead.bottomDL) {
@@ -2187,7 +2134,7 @@ int Tabular::TeXLongtableHeaderFooter(odocstream & os, Buffer const & buf,
                }
                for (row_type i = 0; i < rowCount(); ++i) {
                        if (row_info[i].endfoot) {
-                               ret += TeXRow(os, i, buf, runparams);
+                               ret += TeXRow(os, i, runparams);
                        }
                }
                if (endfoot.bottomDL) {
@@ -2209,7 +2156,7 @@ int Tabular::TeXLongtableHeaderFooter(odocstream & os, Buffer const & buf,
                }
                for (row_type i = 0; i < rowCount(); ++i) {
                        if (row_info[i].endlastfoot) {
-                               ret += TeXRow(os, i, buf, runparams);
+                               ret += TeXRow(os, i, runparams);
                        }
                }
                if (endlastfoot.bottomDL) {
@@ -2232,7 +2179,7 @@ bool Tabular::isValidRow(row_type row) const
 }
 
 
-int Tabular::TeXRow(odocstream & os, row_type i, Buffer const & buf,
+int Tabular::TeXRow(odocstream & os, row_type i,
                       OutputParams const & runparams) const
 {
        idx_type cell = cellIndex(i, 0);
@@ -2260,24 +2207,24 @@ int Tabular::TeXRow(odocstream & os, row_type i, Buffer const & buf,
                if (isPartOfMultiColumn(i, j))
                        continue;
                ret += TeXCellPreamble(os, cell);
-               shared_ptr<InsetText> inset = getCellInset(cell);
+               shared_ptr<InsetTableCell> inset = getCellInset(cell);
 
                Paragraph const & par = inset->paragraphs().front();
-               bool rtl = par.isRTL(buf.params())
+               bool rtl = par.isRTL(buffer().params())
                        && !par.empty()
                        && getPWidth(cell).zero();
 
                if (rtl) {
-                       if (par.getParLanguage(buf.params())->lang() ==
+                       if (par.getParLanguage(buffer().params())->lang() ==
                        "farsi")
                                os << "\\textFR{";
-                       else if (par.getParLanguage(buf.params())->lang() == "arabic_arabi")
+                       else if (par.getParLanguage(buffer().params())->lang() == "arabic_arabi")
                                os << "\\textAR{";
                        // currently, remaning RTL languages are arabic_arabtex and hebrew
                        else
                                os << "\\R{";
                }
-               ret += inset->latex(buf, os, runparams);
+               ret += inset->latex(os, runparams);
                if (rtl)
                        os << '}';
 
@@ -2324,8 +2271,7 @@ int Tabular::TeXRow(odocstream & os, row_type i, Buffer const & buf,
 }
 
 
-int Tabular::latex(Buffer const & buf, odocstream & os,
-                     OutputParams const & runparams) const
+int Tabular::latex(odocstream & os, OutputParams const & runparams) const
 {
        int ret = 0;
 
@@ -2341,8 +2287,9 @@ int Tabular::latex(Buffer const & buf, odocstream & os,
                os << "\\begin{longtable}{";
        else
                os << "\\begin{tabular}{";
+
        for (col_type i = 0; i < columnCount(); ++i) {
-               if (!use_booktabs && column_info[i].left_line)
+               if (!use_booktabs && columnLeftLine(i))
                        os << '|';
                if (!column_info[i].align_special.empty()) {
                        os << column_info[i].align_special;
@@ -2393,13 +2340,13 @@ int Tabular::latex(Buffer const & buf, odocstream & os,
                                }
                        } // end if else !column_info[i].p_width
                } // end if else !column_info[i].align_special
-               if (!use_booktabs && column_info[i].right_line)
+               if (!use_booktabs && columnRightLine(i))
                        os << '|';
        }
        os << "}\n";
        ++ret;
 
-       ret += TeXLongtableHeaderFooter(os, buf, runparams);
+       ret += TeXLongtableHeaderFooter(os, runparams);
 
        //+---------------------------------------------------------------------
        //+                      the single row and columns (cells)            +
@@ -2407,7 +2354,7 @@ int Tabular::latex(Buffer const & buf, odocstream & os,
 
        for (row_type i = 0; i < rowCount(); ++i) {
                if (isValidRow(i)) {
-                       ret += TeXRow(os, i, buf, runparams);
+                       ret += TeXRow(os, i, runparams);
                        if (is_long_tabular && row_info[i].newpage) {
                                os << "\\newpage\n";
                                ++ret;
@@ -2432,7 +2379,7 @@ int Tabular::latex(Buffer const & buf, odocstream & os,
 }
 
 
-int Tabular::docbookRow(Buffer const & buf, odocstream & os, row_type row,
+int Tabular::docbookRow(odocstream & os, row_type row,
                           OutputParams const & runparams) const
 {
        int ret = 0;
@@ -2471,11 +2418,11 @@ int Tabular::docbookRow(Buffer const & buf, odocstream & os, row_type row,
 
                if (isMultiColumn(cell)) {
                        os << " namest=\"col" << j << "\" ";
-                       os << "nameend=\"col" << j + cells_in_multicolumn(cell) - 1<< '"';
+                       os << "nameend=\"col" << j + columnSpan(cell) - 1<< '"';
                }
 
                os << '>';
-               ret += getCellInset(cell)->docbook(buf, os, runparams);
+               ret += getCellInset(cell)->docbook(os, runparams);
                os << "</entry>\n";
                ++cell;
        }
@@ -2484,8 +2431,7 @@ int Tabular::docbookRow(Buffer const & buf, odocstream & os, row_type row,
 }
 
 
-int Tabular::docbook(Buffer const & buf, odocstream & os,
-                       OutputParams const & runparams) const
+int Tabular::docbook(odocstream & os, OutputParams const & runparams) const
 {
        int ret = 0;
 
@@ -2526,7 +2472,7 @@ int Tabular::docbook(Buffer const & buf, odocstream & os,
                ++ret;
                for (row_type i = 0; i < rowCount(); ++i) {
                        if (row_info[i].endhead || row_info[i].endfirsthead) {
-                               ret += docbookRow(buf, os, i, runparams);
+                               ret += docbookRow(os, i, runparams);
                        }
                }
                os << "</thead>\n";
@@ -2538,7 +2484,7 @@ int Tabular::docbook(Buffer const & buf, odocstream & os,
                ++ret;
                for (row_type i = 0; i < rowCount(); ++i) {
                        if (row_info[i].endfoot || row_info[i].endlastfoot) {
-                               ret += docbookRow(buf, os, i, runparams);
+                               ret += docbookRow(os, i, runparams);
                        }
                }
                os << "</tfoot>\n";
@@ -2553,7 +2499,7 @@ int Tabular::docbook(Buffer const & buf, odocstream & os,
        ++ret;
        for (row_type i = 0; i < rowCount(); ++i) {
                if (isValidRow(i)) {
-                       ret += docbookRow(buf, os, i, runparams);
+                       ret += docbookRow(os, i, runparams);
                }
        }
        os << "</tbody>\n";
@@ -2665,14 +2611,14 @@ bool Tabular::plaintextBottomHLine(odocstream & os, row_type row,
 }
 
 
-void Tabular::plaintextPrintCell(Buffer const & buf, odocstream & os,
+void Tabular::plaintextPrintCell(odocstream & os,
                               OutputParams const & runparams,
                               idx_type cell, row_type row, col_type column,
                               vector<unsigned int> const & clen,
                               bool onlydata) const
 {
        odocstringstream sstr;
-       getCellInset(cell)->plaintext(buf, sstr, runparams);
+       getCellInset(cell)->plaintext(sstr, runparams);
 
        if (onlydata) {
                os << sstr.str();
@@ -2716,7 +2662,7 @@ void Tabular::plaintextPrintCell(Buffer const & buf, odocstream & os,
 }
 
 
-void Tabular::plaintext(Buffer const & buf, odocstream & os,
+void Tabular::plaintext(odocstream & os,
                           OutputParams const & runparams, int const depth,
                           bool onlydata, char_type delim) const
 {
@@ -2732,7 +2678,7 @@ void Tabular::plaintext(Buffer const & buf, odocstream & os,
                                if (isMultiColumnReal(cell))
                                        continue;
                                odocstringstream sstr;
-                               getCellInset(cell)->plaintext(buf, sstr, runparams);
+                               getCellInset(cell)->plaintext(sstr, runparams);
                                if (clen[j] < sstr.str().length())
                                        clen[j] = sstr.str().length();
                        }
@@ -2744,9 +2690,9 @@ void Tabular::plaintext(Buffer const & buf, odocstream & os,
                                if (!isMultiColumnReal(cell) || isPartOfMultiColumn(i, j))
                                        continue;
                                odocstringstream sstr;
-                               getCellInset(cell)->plaintext(buf, sstr, runparams);
+                               getCellInset(cell)->plaintext(sstr, runparams);
                                int len = int(sstr.str().length());
-                               idx_type const n = cells_in_multicolumn(cell);
+                               idx_type const n = columnSpan(cell);
                                for (col_type k = j; len > 0 && k < j + n - 1; ++k)
                                        len -= clen[k];
                                if (len > int(clen[j + n - 1]))
@@ -2765,8 +2711,7 @@ void Tabular::plaintext(Buffer const & buf, odocstream & os,
                                // we don't use operator<< for single UCS4 character.
                                // see explanation in docstream.h
                                os.put(delim);
-                       plaintextPrintCell(buf, os, runparams,
-                                          cell, i, j, clen, onlydata);
+                       plaintextPrintCell(os, runparams, cell, i, j, clen, onlydata);
                        ++cell;
                }
                os << endl;
@@ -2779,13 +2724,13 @@ void Tabular::plaintext(Buffer const & buf, odocstream & os,
 }
 
 
-shared_ptr<InsetText> Tabular::getCellInset(idx_type cell) const
+shared_ptr<InsetTableCell> Tabular::getCellInset(idx_type cell) const
 {
        return cell_info[cellRow(cell)][cellColumn(cell)].inset;
 }
 
 
-shared_ptr<InsetText> Tabular::getCellInset(row_type row,
+shared_ptr<InsetTableCell> Tabular::getCellInset(row_type row,
                                               col_type column) const
 {
        return cell_info[row][column].inset;
@@ -2793,9 +2738,13 @@ shared_ptr<InsetText> Tabular::getCellInset(row_type row,
 
 
 void Tabular::setCellInset(row_type row, col_type column,
-                             shared_ptr<InsetText> ins) const
+                             shared_ptr<InsetTableCell> ins) const
 {
-       cell_info[row][column].inset = ins;
+       CellData & cd = cell_info[row][column];
+       cd.inset = ins;
+       // reset the InsetTableCell's pointers
+       ins->setCellData(&cd);
+       ins->setTabular(this);
 }
 
 
@@ -2859,44 +2808,72 @@ Tabular::BoxType Tabular::useParbox(idx_type cell) const
 
 /////////////////////////////////////////////////////////////////////
 //
-// InsetTabular
+// InsetTableCell
 //
 /////////////////////////////////////////////////////////////////////
 
-InsetTabular::InsetTabular(Buffer const & buf, row_type rows,
-                          col_type columns)
-       : tabular(buf.params(), max(rows, row_type(1)),
-         max(columns, col_type(1))), buffer_(&buf), scx_(0)
+InsetTableCell::InsetTableCell(Buffer const & buf,
+       Tabular::CellData const * cell, Tabular const * table)
+       : InsetText(buf), cell_data_(cell), table_(table)
 {}
 
 
-InsetTabular::InsetTabular(InsetTabular const & tab)
-       : Inset(tab), tabular(tab.tabular),
-               buffer_(tab.buffer_), scx_(0)
-{}
+bool InsetTableCell::forceEmptyLayout(idx_type) const
+{
+       BOOST_ASSERT(table_);
+       BOOST_ASSERT(cell_data_);
+       return table_->getPWidth(cell_data_->cellno).zero();
+}
 
+bool InsetTableCell::allowParagraphCustomization(idx_type) const
+{
+       BOOST_ASSERT(table_);
+       BOOST_ASSERT(cell_data_);
+       return !table_->getPWidth(cell_data_->cellno).zero();
+}
 
-InsetTabular::~InsetTabular()
+bool InsetTableCell::getStatus(Cursor & cur, FuncRequest const & cmd,
+       FuncStatus & status) const
 {
-       InsetTabularMailer(*this).hideDialog();
+       bool enabled;
+       switch (cmd.action) {
+       case LFUN_LAYOUT:
+               enabled = !forceEmptyLayout();
+               break;
+       case LFUN_LAYOUT_PARAGRAPH:
+               enabled = allowParagraphCustomization();
+               break;
+       default:
+               return InsetText::getStatus(cur, cmd, status);
+       }
+       status.enabled(enabled);
+       return true;
 }
 
+/////////////////////////////////////////////////////////////////////
+//
+// InsetTabular
+//
+/////////////////////////////////////////////////////////////////////
 
-Inset * InsetTabular::clone() const
+InsetTabular::InsetTabular(Buffer const & buf, row_type rows,
+                          col_type columns)
+       : tabular(buf, max(rows, row_type(1)), max(columns, col_type(1))), scx_(0)
 {
-       return new InsetTabular(*this);
+       setBuffer(const_cast<Buffer &>(buf)); // FIXME: remove later
 }
 
 
-Buffer const & InsetTabular::buffer() const
+InsetTabular::InsetTabular(InsetTabular const & tab)
+       : Inset(tab), tabular(tab.tabular),  scx_(0)
 {
-       return *buffer_;
+       setBuffer(const_cast<Buffer &>(tab.buffer())); // FIXME: remove later
 }
 
 
-void InsetTabular::buffer(Buffer const * b)
+InsetTabular::~InsetTabular()
 {
-       buffer_ = b;
+       InsetTabularMailer(*this).hideDialog();
 }
 
 
@@ -2909,18 +2886,18 @@ bool InsetTabular::insetAllowed(InsetCode code) const
 }
 
 
-void InsetTabular::write(Buffer const & buf, ostream & os) const
+void InsetTabular::write(ostream & os) const
 {
        os << "Tabular" << endl;
-       tabular.write(buf, os);
+       tabular.write(os);
 }
 
 
-void InsetTabular::read(Buffer const & buf, Lexer & lex)
+void InsetTabular::read(Lexer & lex)
 {
        bool const old_format = (lex.getString() == "\\LyXTable");
 
-       tabular.read(buf, lex);
+       tabular.read(lex);
 
        if (old_format)
                return;
@@ -2938,6 +2915,34 @@ void InsetTabular::read(Buffer const & buf, Lexer & lex)
 }
 
 
+int InsetTabular::rowFromY(Cursor & cur, int y) const
+{
+       // top y coordinate of tabular
+       int h = yo(cur.bv()) - tabular.rowAscent(0);
+       size_t nrows = tabular.rowCount();
+       row_type r = 0;
+       for (;r < nrows && y > h; ++r) {
+               h += tabular.rowAscent(r);
+               h += tabular.rowDescent(r);
+               h += tabular.getAdditionalHeight(r);
+       }
+       return r - 1;
+}
+
+
+int InsetTabular::columnFromX(Cursor & cur, int x) const
+{
+       // left x coordinate of tabular
+       int w = xo(cur.bv()) + ADD_TO_TABULAR_WIDTH;
+       size_t ncols = tabular.columnCount();
+       col_type c = 0;
+       for (;c < ncols && x > w; ++c) {
+               w += tabular.columnWidth(c);
+       }
+       return c - 1;
+}
+
+
 void InsetTabular::metrics(MetricsInfo & mi, Dimension & dim) const
 {
        //lyxerr << "InsetTabular::metrics: " << mi.base.bv << " width: " <<
@@ -2959,7 +2964,7 @@ void InsetTabular::metrics(MetricsInfo & mi, Dimension & dim) const
                        MetricsInfo m = mi;
                        Length p_width;
                        if (tabular.cell_info[i][j].multicolumn ==
-                           Tabular::CELL_BEGIN_OF_MULTICOLUMN)
+                               Tabular::CELL_BEGIN_OF_MULTICOLUMN)
                                p_width = tabular.cellinfo_of_cell(cell).p_width;
                        else
                                p_width = tabular.column_info[j].p_width;
@@ -3147,19 +3152,19 @@ void InsetTabular::drawCellLines(Painter & pain, int x, int y,
 }
 
 
-docstring const InsetTabular::editMessage() const
+docstring InsetTabular::editMessage() const
 {
        return _("Opened table");
 }
 
 
-void InsetTabular::edit(Cursor & cur, bool, EntryDirection direction)
+void InsetTabular::edit(Cursor & cur, bool front, EntryDirection)
 {
        //lyxerr << "InsetTabular::edit: " << this << endl;
        cur.finishUndo();
        cur.selection() = false;
        cur.push(*this);
-       if (direction == ENTRY_DIRECTION_LEFT) {
+       if (front) {
                if (isRightToLeft(cur))
                        cur.idx() = tabular.getLastCellInRow(0);
                else
@@ -3180,10 +3185,10 @@ void InsetTabular::edit(Cursor & cur, bool, EntryDirection direction)
 }
 
 
-void InsetTabular::updateLabels(Buffer const & buf, ParIterator const & it)
+void InsetTabular::updateLabels(ParIterator const & it)
 {
        // In a longtable, tell captions what the current float is
-       Counters & cnts = buf.params().getTextClass().counters();
+       Counters & cnts = buffer().params().documentClass().counters();
        string const saveflt = cnts.current_float();
        if (tabular.isLongTabular())
                cnts.current_float("table");
@@ -3192,7 +3197,7 @@ void InsetTabular::updateLabels(Buffer const & buf, ParIterator const & it)
        it2.forwardPos();
        size_t const end = it2.nargs();
        for ( ; it2.idx() < end; it2.top().forwardIdx())
-               lyx::updateLabels(buf, it2);
+               lyx::updateLabels(buffer(), it2);
 
        //reset afterwards
        if (tabular.isLongTabular())
@@ -3209,9 +3214,36 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
 
        switch (cmd.action) {
 
-       case LFUN_MOUSE_PRESS:
+       case LFUN_MOUSE_PRESS: {
                //lyxerr << "# InsetTabular::MousePress\n" << cur.bv().cursor() << endl;
 
+               // select row
+               if (cmd.x < xo(cur.bv()) + ADD_TO_TABULAR_WIDTH
+                       || cmd.x > xo(cur.bv()) + tabular.width()) {
+                       row_type r = rowFromY(cur, cmd.y);
+                       cur.idx() = tabular.getFirstCellInRow(r);
+                       cur.pos() = 0;
+                       cur.resetAnchor();
+                       cur.idx() = tabular.getLastCellInRow(r);
+                       cur.pos() = cur.lastpos();
+                       cur.selection() = true;
+                       bvcur = cur; 
+                       break;
+               }
+               // select column
+               int const y0 = yo(cur.bv()) - tabular.rowAscent(0);
+               if (cmd.y < y0 + ADD_TO_TABULAR_WIDTH 
+                       || cmd.y > y0 + tabular.height()) {
+                       col_type c = columnFromX(cur, cmd.x);
+                       cur.idx() = tabular.cellIndex(0, c);
+                       cur.pos() = 0;
+                       cur.resetAnchor();
+                       cur.idx() = tabular.cellIndex(tabular.rowCount() - 1, c);
+                       cur.pos() = cur.lastpos();
+                       cur.selection() = true;
+                       bvcur = cur; 
+                       break;
+               }
                // do not reset cursor/selection if we have selected
                // some cells (bug 2715).
                if (cmd.button() == mouse_button::button3
@@ -3219,23 +3251,44 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                    && tablemode(bvcur)) 
                        ;
                else
-                       // Let InsetText do it
+                       // Let InsetTableCell do it
                        cell(cur.idx())->dispatch(cur, cmd);
                break;
+       }
        case LFUN_MOUSE_MOTION:
                //lyxerr << "# InsetTabular::MouseMotion\n" << bvcur << endl;
                if (cmd.button() == mouse_button::button1) {
                        // only accept motions to places not deeper nested than the real anchor
-                       if (bvcur.anchor_.hasPart(cur)) {
-                               // only update if selection changes
-                               if (bvcur.idx() == cur.idx() &&
-                                       !(bvcur.anchor_.idx() == cur.idx() && bvcur.pos() != cur.pos()))
-                                       cur.noUpdate();
-                               setCursorFromCoordinates(cur, cmd.x, cmd.y);
+                       if (!bvcur.anchor_.hasPart(cur)) {
+                               cur.undispatched();
+                               break;
+                       }
+                       // select (additional) row
+                       if (cmd.x < xo(cur.bv()) + ADD_TO_TABULAR_WIDTH
+                               || cmd.x > xo(cur.bv()) + tabular.width()) {
+                               row_type r = rowFromY(cur, cmd.y);
+                               cur.idx() = tabular.getLastCellInRow(r);
                                bvcur.setCursor(cur);
                                bvcur.selection() = true;
-                       } else
-                               cur.undispatched();
+                               break;
+                       }
+                       // select (additional) column
+                       int const y0 = yo(cur.bv()) - tabular.rowAscent(0);
+                       if (cmd.y < y0 + ADD_TO_TABULAR_WIDTH 
+                               || cmd.y > y0 + tabular.height()) {
+                               col_type c = columnFromX(cur, cmd.x);
+                               cur.idx() = tabular.cellIndex(tabular.rowCount() - 1, c);
+                               bvcur.setCursor(cur);
+                               bvcur.selection() = true;
+                               break;
+                       }
+                       // only update if selection changes
+                       if (bvcur.idx() == cur.idx() &&
+                               !(bvcur.anchor_.idx() == cur.idx() && bvcur.pos() != cur.pos()))
+                               cur.noUpdate();
+                       setCursorFromCoordinates(cur, cmd.x, cmd.y);
+                       bvcur.setCursor(cur);
+                       bvcur.selection() = true;
                }
                break;
 
@@ -3254,7 +3307,6 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                moveNextCell(cur);
                cur.selection() = false;
                break;
-
        case LFUN_CHAR_FORWARD_SELECT:
        case LFUN_CHAR_FORWARD:
                cell(cur.idx())->dispatch(cur, cmd);
@@ -3585,6 +3637,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                case Tabular::SET_TOP_SPACE:
                case Tabular::SET_BOTTOM_SPACE:
                case Tabular::SET_INTERLINE_SPACE:
+               case Tabular::SET_BORDER_LINES:
                        status.clear();
                        return true;
 
@@ -3592,28 +3645,20 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        status.setOnOff(tabular.isMultiColumn(cur.idx()));
                        break;
 
-               case Tabular::M_TOGGLE_LINE_TOP:
-                       flag = false;
                case Tabular::TOGGLE_LINE_TOP:
-                       status.setOnOff(tabular.topLine(cur.idx(), flag));
+                       status.setOnOff(tabular.topLine(cur.idx()));
                        break;
 
-               case Tabular::M_TOGGLE_LINE_BOTTOM:
-                       flag = false;
                case Tabular::TOGGLE_LINE_BOTTOM:
-                       status.setOnOff(tabular.bottomLine(cur.idx(), flag));
+                       status.setOnOff(tabular.bottomLine(cur.idx()));
                        break;
 
-               case Tabular::M_TOGGLE_LINE_LEFT:
-                       flag = false;
                case Tabular::TOGGLE_LINE_LEFT:
-                       status.setOnOff(tabular.leftLine(cur.idx(), flag));
+                       status.setOnOff(tabular.leftLine(cur.idx()));
                        break;
 
-               case Tabular::M_TOGGLE_LINE_RIGHT:
-                       flag = false;
                case Tabular::TOGGLE_LINE_RIGHT:
-                       status.setOnOff(tabular.rightLine(cur.idx(), flag));
+                       status.setOnOff(tabular.rightLine(cur.idx()));
                        break;
 
                case Tabular::M_ALIGN_LEFT:
@@ -3807,25 +3852,22 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
 }
 
 
-int InsetTabular::latex(Buffer const & buf, odocstream & os,
-                       OutputParams const & runparams) const
+int InsetTabular::latex(odocstream & os, OutputParams const & runparams) const
 {
-       return tabular.latex(buf, os, runparams);
+       return tabular.latex(os, runparams);
 }
 
 
-int InsetTabular::plaintext(Buffer const & buf, odocstream & os,
-                           OutputParams const & runparams) const
+int InsetTabular::plaintext(odocstream & os, OutputParams const & runparams) const
 {
        os << '\n'; // output table on a new line
        int const dp = runparams.linelen > 0 ? runparams.depth : 0;
-       tabular.plaintext(buf, os, runparams, dp, false, 0);
+       tabular.plaintext(os, runparams, dp, false, 0);
        return PLAINTEXT_NEWLINE;
 }
 
 
-int InsetTabular::docbook(Buffer const & buf, odocstream & os,
-                         OutputParams const & runparams) const
+int InsetTabular::docbook(odocstream & os, OutputParams const & runparams) const
 {
        int ret = 0;
        Inset * master = 0;
@@ -3843,7 +3885,7 @@ int InsetTabular::docbook(Buffer const & buf, odocstream & os,
                os << "<informaltable>";
                ++ret;
        }
-       ret += tabular.docbook(buf, os, runparams);
+       ret += tabular.docbook(os, runparams);
        if (!master) {
                os << "</informaltable>";
                ++ret;
@@ -3858,13 +3900,13 @@ void InsetTabular::validate(LaTeXFeatures & features) const
 }
 
 
-shared_ptr<InsetText const> InsetTabular::cell(idx_type idx) const
+shared_ptr<InsetTableCell const> InsetTabular::cell(idx_type idx) const
 {
        return tabular.getCellInset(idx);
 }
 
 
-shared_ptr<InsetText> InsetTabular::cell(idx_type idx)
+shared_ptr<InsetTableCell> InsetTabular::cell(idx_type idx)
 {
        return tabular.getCellInset(idx);
 }
@@ -4119,7 +4161,6 @@ bool InsetTabular::oneCellHasRotationState(bool rotated,
 void InsetTabular::tabularFeatures(Cursor & cur,
        Tabular::Feature feature, string const & value)
 {
-       BufferView & bv = cur.bv();
        col_type sel_col_start;
        col_type sel_col_end;
        row_type sel_row_start;
@@ -4198,12 +4239,12 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::APPEND_ROW:
                // append the row into the tabular
-               tabular.appendRow(bv.buffer().params(), cur.idx());
+               tabular.appendRow(cur.idx());
                break;
 
        case Tabular::APPEND_COLUMN:
                // append the column into the tabular
-               tabular.appendColumn(bv.buffer().params(), cur.idx());
+               tabular.appendColumn(cur.idx());
                cur.idx() = tabular.cellIndex(row, column);
                break;
 
@@ -4230,62 +4271,43 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                break;
 
        case Tabular::COPY_ROW:
-               tabular.copyRow(bv.buffer().params(), row);
+               tabular.copyRow(row);
                break;
 
        case Tabular::COPY_COLUMN:
-               tabular.copyColumn(bv.buffer().params(), column);
+               tabular.copyColumn(column);
                cur.idx() = tabular.cellIndex(row, column);
                break;
 
-       case Tabular::M_TOGGLE_LINE_TOP:
-               flag = false;
        case Tabular::TOGGLE_LINE_TOP: {
-               bool lineSet = !tabular.topLine(cur.idx(), flag);
+               bool lineSet = !tabular.topLine(cur.idx());
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setTopLine(
-                                       tabular.cellIndex(i, j),
-                                       lineSet, flag);
+                               tabular.setTopLine(tabular.cellIndex(i, j), lineSet);
                break;
        }
 
-       case Tabular::M_TOGGLE_LINE_BOTTOM:
-               flag = false;
        case Tabular::TOGGLE_LINE_BOTTOM: {
-               bool lineSet = !tabular.bottomLine(cur.idx(), flag);
+               bool lineSet = !tabular.bottomLine(cur.idx());
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setBottomLine(
-                                       tabular.cellIndex(i, j),
-                                       lineSet,
-                                       flag);
+                               tabular.setBottomLine(tabular.cellIndex(i, j), lineSet);
                break;
        }
 
-       case Tabular::M_TOGGLE_LINE_LEFT:
-               flag = false;
        case Tabular::TOGGLE_LINE_LEFT: {
-               bool lineSet = !tabular.leftLine(cur.idx(), flag);
+               bool lineSet = !tabular.leftLine(cur.idx());
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setLeftLine(
-                                       tabular.cellIndex(i,j),
-                                       lineSet,
-                                       flag);
+                               tabular.setLeftLine(tabular.cellIndex(i, j), lineSet);
                break;
        }
 
-       case Tabular::M_TOGGLE_LINE_RIGHT:
-               flag = false;
        case Tabular::TOGGLE_LINE_RIGHT: {
-               bool lineSet = !tabular.rightLine(cur.idx(), flag);
+               bool lineSet = !tabular.rightLine(cur.idx());
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setRightLine(
-                                       tabular.cellIndex(i,j),
-                                       lineSet,
-                                       flag);
+                               tabular.setRightLine(tabular.cellIndex(i, j), lineSet);
                break;
        }
 
@@ -4299,10 +4321,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
        case Tabular::ALIGN_BLOCK:
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setAlignment(
-                                       tabular.cellIndex(i, j),
-                                       setAlign,
-                                       flag);
+                               tabular.setAlignment(tabular.cellIndex(i, j), setAlign, flag);
                break;
 
        case Tabular::M_VALIGN_TOP:
@@ -4314,9 +4333,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
        case Tabular::VALIGN_MIDDLE:
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setVAlignment(
-                                       tabular.cellIndex(i, j),
-                                       setVAlign, flag);
+                               tabular.setVAlignment(tabular.cellIndex(i, j), setVAlign, flag);
                break;
 
        case Tabular::MULTICOLUMN: {
@@ -4333,14 +4350,14 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                        if (tabular.isMultiColumn(cur.idx()))
                                tabular.unsetMultiColumn(cur.idx());
                        else
-                               tabular.setMultiColumn(&bv.buffer(), cur.idx(), 1);
+                               tabular.setMultiColumn(cur.idx(), 1);
                        break;
                }
                // we have a selection so this means we just add all this
                // cells to form a multicolumn cell
                idx_type const s_start = cur.selBegin().idx();
                idx_type const s_end = cur.selEnd().idx();
-               tabular.setMultiColumn(&bv.buffer(), s_start, s_end - s_start + 1);
+               tabular.setMultiColumn(s_start, s_end - s_start + 1);
                cur.idx() = s_start;
                cur.pit() = 0;
                cur.pos() = 0;
@@ -4357,6 +4374,17 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                                        tabular.cellIndex(i,j), setLines);
                break;
 
+       case Tabular::SET_BORDER_LINES:
+               for (row_type i = sel_row_start; i <= sel_row_end; ++i) {
+                       tabular.setLeftLine(tabular.cellIndex(i, sel_col_start), true);
+                       tabular.setRightLine(tabular.cellIndex(i, sel_col_end), true);
+               }
+               for (col_type j = sel_col_start; j <= sel_col_end; ++j) {
+                       tabular.setTopLine(tabular.cellIndex(sel_row_start, j), true);
+                       tabular.setBottomLine(tabular.cellIndex(sel_row_end, j), true);
+               }
+               break;
+
        case Tabular::SET_LONGTABULAR:
                tabular.setLongTabular(true);
                break;
@@ -4387,8 +4415,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
        case Tabular::UNSET_ROTATE_CELL:
                for (row_type i = sel_row_start; i <= sel_row_end; ++i)
                        for (col_type j = sel_col_start; j <= sel_col_end; ++j)
-                               tabular.setRotateCell(
-                                       tabular.cellIndex(i, j), false);
+                               tabular.setRotateCell(tabular.cellIndex(i, j), false);
                break;
 
        case Tabular::TOGGLE_ROTATE_CELL:
@@ -4549,10 +4576,6 @@ bool InsetTabular::copySelection(Cursor & cur)
        while (paste_tabular->rowCount() > rows)
                paste_tabular->deleteRow(rows);
 
-       paste_tabular->setTopLine(0, true, true);
-       paste_tabular->setBottomLine(paste_tabular->getFirstCellInRow(rows - 1),
-                                    true, true);
-
        for (col_type i = 0; i < cs; ++i)
                paste_tabular->deleteColumn(0);
 
@@ -4560,13 +4583,18 @@ bool InsetTabular::copySelection(Cursor & cur)
        while (paste_tabular->columnCount() > columns)
                paste_tabular->deleteColumn(columns);
 
-       paste_tabular->setLeftLine(0, true, true);
-       paste_tabular->setRightLine(paste_tabular->getLastCellInRow(0),
-                                   true, true);
+       // We clear all the InsetTableCell pointers, since they
+       // might now become invalid and there is no point in having
+       // them point to temporary things in paste_tabular.
+       for (row_type i = 0; i < paste_tabular->rowCount(); ++i)
+               for (col_type j = 0; j < paste_tabular->columnCount(); ++j) {
+                       paste_tabular->getCellInset(i,j)->setCellData(0);
+                       paste_tabular->getCellInset(i,j)->setTabular(0);
+               }
 
        odocstringstream os;
        OutputParams const runparams(0);
-       paste_tabular->plaintext(cur.buffer(), os, runparams, 0, true, '\t');
+       paste_tabular->plaintext(os, runparams, 0, true, '\t');
        // Needed for the "Edit->Paste recent" menu and the system clipboard.
        cap::copySelection(cur, os.str());
 
@@ -4603,8 +4631,10 @@ bool InsetTabular::pasteClipboard(Cursor & cur)
                                --c1;
                                continue;
                        }
-                       shared_ptr<InsetText> inset(
-                               new InsetText(*paste_tabular->getCellInset(r1, c1)));
+                       shared_ptr<InsetTableCell> inset(
+                               new InsetTableCell(*paste_tabular->getCellInset(r1, c1)));
+                       // note that setCellInset will call InsetTableCell::setCellData()
+                       // and InsetTableCell::setTabular()
                        tabular.setCellInset(r2, c2, inset);
                        // FIXME: change tracking (MG)
                        inset->setChange(Change(cur.buffer().params().trackChanges ?
@@ -4626,7 +4656,7 @@ void InsetTabular::cutSelection(Cursor & cur)
        getSelection(cur, rs, re, cs, ce);
        for (row_type i = rs; i <= re; ++i) {
                for (col_type j = cs; j <= ce; ++j) {
-                       shared_ptr<InsetText> t
+                       shared_ptr<InsetTableCell> t
                                = cell(tabular.cellIndex(i, j));
                        if (cur.buffer().params().trackChanges)
                                // FIXME: Change tracking (MG)
@@ -4709,19 +4739,23 @@ bool InsetTabular::allowParagraphCustomization(idx_type cell) const
 }
 
 
+bool InsetTabular::forceEmptyLayout(idx_type cell) const
+{
+       return !tabular.getPWidth(cell).zero();
+}
+
+
 bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
                                     bool usePaste)
 {
        if (buf.length() <= 0)
                return true;
 
-       Buffer const & buffer = bv.buffer();
-
        col_type cols = 1;
        row_type rows = 1;
        col_type maxCols = 1;
-       docstring::size_type const len = buf.length();
-       docstring::size_type p = 0;
+       size_t const len = buf.length();
+       size_t p = 0;
 
        while (p < len &&
               (p = buf.find_first_of(from_ascii("\t\n"), p)) != docstring::npos) {
@@ -4744,8 +4778,7 @@ bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
        col_type ocol = 0;
        row_type row = 0;
        if (usePaste) {
-               paste_tabular.reset(
-                       new Tabular(buffer.params(), rows, maxCols));
+               paste_tabular.reset(new Tabular(buffer(), rows, maxCols));
                loctab = paste_tabular.get();
                cols = 0;
                dirtyTabularStack(true);
@@ -4756,7 +4789,7 @@ bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
                row = tabular.cellRow(cell);
        }
 
-       docstring::size_type op = 0;
+       size_t op = 0;
        idx_type const cells = loctab->cellCount();
        p = 0;
        cols = ocol;
@@ -4772,11 +4805,11 @@ bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
                case '\t':
                        // we can only set this if we are not too far right
                        if (cols < columns) {
-                               shared_ptr<InsetText> inset = loctab->getCellInset(cell);
+                               shared_ptr<InsetTableCell> inset = loctab->getCellInset(cell);
                                Font const font = bv.textMetrics(&inset->text_).
-                                       getDisplayFont(0, 0);
+                                       displayFont(0, 0);
                                inset->setText(buf.substr(op, p - op), font,
-                                              buffer.params().trackChanges);
+                                              buffer().params().trackChanges);
                                ++cols;
                                ++cell;
                        }
@@ -4784,11 +4817,11 @@ bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
                case '\n':
                        // we can only set this if we are not too far right
                        if (cols < columns) {
-                               shared_ptr<InsetText> inset = tabular.getCellInset(cell);
+                               shared_ptr<InsetTableCell> inset = tabular.getCellInset(cell);
                                Font const font = bv.textMetrics(&inset->text_).
-                                       getDisplayFont(0, 0);
+                                       displayFont(0, 0);
                                inset->setText(buf.substr(op, p - op), font,
-                                              buffer.params().trackChanges);
+                                              buffer().params().trackChanges);
                        }
                        cols = ocol;
                        ++row;
@@ -4801,10 +4834,10 @@ bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
        }
        // check for the last cell if there is no trailing '\n'
        if (cell < cells && op < len) {
-               shared_ptr<InsetText> inset = loctab->getCellInset(cell);
-               Font const font = bv.textMetrics(&inset->text_).getDisplayFont(0, 0);
+               shared_ptr<InsetTableCell> inset = loctab->getCellInset(cell);
+               Font const font = bv.textMetrics(&inset->text_).displayFont(0, 0);
                inset->setText(buf.substr(op, len - op), font,
-                       buffer.params().trackChanges);
+                       buffer().params().trackChanges);
        }
        return true;
 }
@@ -4827,6 +4860,68 @@ bool InsetTabular::tablemode(Cursor & cur) const
 }
 
 
+bool InsetTabular::completionSupported(Cursor const & cur) const
+{
+       Cursor const & bvCur = cur.bv().cursor();
+       if (&bvCur.inset() != this)
+               return false;
+       return cur.text()->completionSupported(cur);
+}
+
+
+bool InsetTabular::inlineCompletionSupported(Cursor const & cur) const
+{
+       return completionSupported(cur);
+}
+
+
+bool InsetTabular::automaticInlineCompletion() const
+{
+       return lyxrc.completion_inline_text;
+}
+
+
+bool InsetTabular::automaticPopupCompletion() const
+{
+       return lyxrc.completion_popup_text;
+}
+
+
+bool InsetTabular::showCompletionCursor() const
+{
+       return lyxrc.completion_cursor_text;
+}
+
+
+CompletionList const * InsetTabular::createCompletionList(Cursor const & cur) const
+{
+       return completionSupported(cur) ? cur.text()->createCompletionList(cur) : 0;
+}
+
+
+docstring InsetTabular::completionPrefix(Cursor const & cur) const
+{
+       if (!completionSupported(cur))
+               return docstring();
+       return cur.text()->completionPrefix(cur);
+}
+
+
+bool InsetTabular::insertCompletion(Cursor & cur, docstring const & s, bool finished)
+{
+       if (!completionSupported(cur))
+               return false;
+
+       return cur.text()->insertCompletion(cur, s, finished);
+}
+
+
+void InsetTabular::completionPosAndDim(Cursor const & cur, int & x, int & y, 
+                                   Dimension & dim) const
+{
+       TextMetrics const & tm = cur.bv().textMetrics(cur.text());
+       tm.completionPosAndDim(cur, x, y, dim);
+}
 
 
 string const InsetTabularMailer::name_("tabular");
@@ -4861,11 +4956,9 @@ void InsetTabularMailer::string2params(string const & in, InsetTabular & inset)
        // by Buffer::readInset
        lex >> token;
        if (!lex || token != "Tabular")
-               return print_mailer_error("InsetTabularMailer", in, 2,
-                                         "Tabular");
+               return print_mailer_error("InsetTabularMailer", in, 2, "Tabular");
 
-       Buffer const & buffer = inset.buffer();
-       inset.read(buffer, lex);
+       inset.read(lex);
 }
 
 
@@ -4873,7 +4966,7 @@ string const InsetTabularMailer::params2string(InsetTabular const & inset)
 {
        ostringstream data;
        data << name_ << ' ';
-       inset.write(inset.buffer(), data);
+       inset.write(data);
        data << "\\end_inset\n";
        return data.str();
 }