]> git.lyx.org Git - lyx.git/blobdiff - src/insets/InsetTabular.cpp
InsetInfo: enable inset dissolve
[lyx.git] / src / insets / InsetTabular.cpp
index 113715a60c433967e5b2e630f7b1c404376a1ca4..324987c8a445e6474f0b61d8a370b5e59e3c4e15 100644 (file)
@@ -33,6 +33,7 @@
 #include "DispatchResult.h"
 #include "FuncRequest.h"
 #include "FuncStatus.h"
+#include "InsetIterator.h"
 #include "InsetList.h"
 #include "Language.h"
 #include "LaTeXFeatures.h"
@@ -45,6 +46,8 @@
 #include "Paragraph.h"
 #include "ParagraphParameters.h"
 #include "ParIterator.h"
+#include "TexRow.h"
+#include "texstream.h"
 #include "TextClass.h"
 #include "TextMetrics.h"
 
 #include "support/gettext.h"
 #include "support/lassert.h"
 #include "support/lstrings.h"
-
-#include <boost/scoped_ptr.hpp>
+#include "support/unique_ptr.h"
 
 #include <cstring>
+#include <iomanip>
 #include <iostream>
 #include <limits>
 #include <sstream>
@@ -96,7 +99,7 @@ int const WIDTH_OF_LINE = 5; // space between double lines
 
 
 ///
-boost::scoped_ptr<Tabular> paste_tabular;
+unique_ptr<Tabular> paste_tabular;
 
 
 struct TabularFeature {
@@ -110,6 +113,8 @@ TabularFeature tabularFeature[] =
 {
        // the SET/UNSET actions are used by the table dialog,
        // the TOGGLE actions by the table toolbar buttons
+       // FIXME: these values have been hardcoded in InsetMathGrid and other
+       // math insets.
        { Tabular::APPEND_ROW, "append-row", false },
        { Tabular::APPEND_COLUMN, "append-column", false },
        { Tabular::DELETE_ROW, "delete-row", false },
@@ -151,10 +156,12 @@ TabularFeature tabularFeature[] =
        { Tabular::SET_MROFFSET, "set-mroffset", true },
        { Tabular::SET_ALL_LINES, "set-all-lines", false },
        { Tabular::UNSET_ALL_LINES, "unset-all-lines", false },
+       { Tabular::TOGGLE_LONGTABULAR, "toggle-longtabular", false },
        { Tabular::SET_LONGTABULAR, "set-longtabular", false },
        { Tabular::UNSET_LONGTABULAR, "unset-longtabular", false },
        { Tabular::SET_PWIDTH, "set-pwidth", true },
        { Tabular::SET_MPWIDTH, "set-mpwidth", true },
+       { Tabular::TOGGLE_VARWIDTH_COLUMN, "toggle-varwidth-column", true },
        { Tabular::SET_ROTATE_TABULAR, "set-rotate-tabular", true },
        { Tabular::UNSET_ROTATE_TABULAR, "unset-rotate-tabular", true },
        { Tabular::TOGGLE_ROTATE_TABULAR, "toggle-rotate-tabular", true },
@@ -177,6 +184,7 @@ TabularFeature tabularFeature[] =
        { Tabular::UNSET_LTCAPTION, "unset-ltcaption", false },
        { Tabular::SET_SPECIAL_COLUMN, "set-special-column", true },
        { Tabular::SET_SPECIAL_MULTICOLUMN, "set-special-multicolumn", true },
+       { Tabular::TOGGLE_BOOKTABS, "toggle-booktabs", false },
        { Tabular::SET_BOOKTABS, "set-booktabs", false },
        { Tabular::UNSET_BOOKTABS, "unset-booktabs", false },
        { Tabular::SET_TOP_SPACE, "set-top-space", true },
@@ -256,13 +264,15 @@ string const tostr(Tabular::BoxType const & num)
                return "parbox";
        case Tabular::BOX_MINIPAGE:
                return "minipage";
+       case Tabular::BOX_VARWIDTH:
+               return "varwidth";
        }
        return string();
 }
 
 
 // I would have liked a fromstr template a lot better. (Lgb)
-bool string2type(string const str, LyXAlignment & num)
+bool string2type(string const str, LyXAlignment & num)
 {
        if (str == "none")
                num = LYX_ALIGN_NONE;
@@ -282,7 +292,7 @@ bool string2type(string const str, LyXAlignment & num)
 }
 
 
-bool string2type(string const str, Tabular::HAlignment & num)
+bool string2type(string const str, Tabular::HAlignment & num)
 {
        if (str == "left")
                num = Tabular::LYX_LONGTABULAR_ALIGN_LEFT;
@@ -296,7 +306,7 @@ bool string2type(string const str, Tabular::HAlignment & num)
 }
 
 
-bool string2type(string const str, Tabular::VAlignment & num)
+bool string2type(string const str, Tabular::VAlignment & num)
 {
        if (str == "top")
                num = Tabular::LYX_VALIGN_TOP;
@@ -310,7 +320,7 @@ bool string2type(string const str, Tabular::VAlignment & num)
 }
 
 
-bool string2type(string const str, Tabular::BoxType & num)
+bool string2type(string const str, Tabular::BoxType & num)
 {
        if (str == "none")
                num = Tabular::BOX_NONE;
@@ -318,13 +328,15 @@ bool string2type(string const str, Tabular::BoxType & num)
                num = Tabular::BOX_PARBOX;
        else if (str == "minipage")
                num = Tabular::BOX_MINIPAGE;
+       else if (str == "varwidth")
+               num = Tabular::BOX_VARWIDTH;
        else
                return false;
        return true;
 }
 
 
-bool string2type(string const str, bool & num)
+bool string2type(string const str, bool & num)
 {
        if (str == "true")
                num = true;
@@ -522,7 +534,7 @@ string const featureAsString(Tabular::Feature action)
 }
 
 
-DocIterator separatorPos(InsetTableCell * cell, docstring const & align_d)
+DocIterator separatorPos(InsetTableCell const * cell, docstring const & align_d)
 {
        DocIterator dit = doc_iterator_begin(&(cell->buffer()), cell);
        for (; dit; dit.forwardChar())
@@ -534,15 +546,15 @@ DocIterator separatorPos(InsetTableCell * cell, docstring const & align_d)
 }
 
 
-InsetTableCell splitCell(InsetTableCell & head, docstring const align_d, bool & hassep)
+InsetTableCell splitCell(InsetTableCell & head, docstring const align_d, bool & hassep)
 {
        InsetTableCell tail = InsetTableCell(head);
        DocIterator const dit = separatorPos(&head, align_d);
-       hassep = dit;
+       hassep = (bool)dit;
        if (hassep) {
                pit_type const psize = head.paragraphs().front().size();
                head.paragraphs().front().eraseChars(dit.pos(), psize, false);
-               tail.paragraphs().front().eraseChars(0, 
+               tail.paragraphs().front().eraseChars(0,
                        dit.pos() < psize ? dit.pos() + 1 : psize, false);
        }
 
@@ -602,36 +614,32 @@ Tabular::CellData::CellData(CellData const & cs)
 {
 }
 
-Tabular::CellData & Tabular::CellData::operator=(CellData cs)
-{
-       swap(cs);
+Tabular::CellData & Tabular::CellData::operator=(CellData const & cs)
+{
+       if (&cs == this)
+               return *this;
+       cellno = cs.cellno;
+       width = cs.width;
+       multicolumn = cs.multicolumn;
+       multirow = cs.multirow;
+       mroffset = cs.mroffset;
+       alignment = cs.alignment;
+       valignment = cs.valignment;
+       decimal_hoffset = cs.decimal_hoffset;
+       decimal_width = cs.decimal_width;
+       voffset = cs.voffset;
+       top_line = cs.top_line;
+       bottom_line = cs.bottom_line;
+       left_line = cs.left_line;
+       right_line = cs.right_line;
+       usebox = cs.usebox;
+       rotate = cs.rotate;
+       align_special = cs.align_special;
+       p_width = cs.p_width;
+       inset.reset(static_cast<InsetTableCell *>(cs.inset->clone()));
        return *this;
 }
 
-void Tabular::CellData::swap(CellData & rhs)
-{
-       std::swap(cellno, rhs.cellno);
-       std::swap(width, rhs.width);
-       std::swap(multicolumn, rhs.multicolumn);
-       std::swap(multirow, rhs.multirow);
-       std::swap(mroffset, rhs.mroffset);
-       std::swap(alignment, rhs.alignment);
-       std::swap(valignment, rhs.valignment);
-       std::swap(decimal_hoffset, rhs.decimal_hoffset);
-       std::swap(decimal_width, rhs.decimal_width);
-       std::swap(voffset, rhs.voffset);
-       std::swap(top_line, rhs.top_line);
-       std::swap(bottom_line, rhs.bottom_line);
-       std::swap(left_line, rhs.left_line);
-       std::swap(right_line, rhs.right_line);
-       std::swap(usebox, rhs.usebox);
-       std::swap(rotate, rhs.rotate);
-       std::swap(align_special, rhs.align_special);
-       p_width.swap(rhs.p_width);
-       inset.swap(rhs.inset);
-}
-
-
 Tabular::RowData::RowData()
        : ascent(0),
          descent(0),
@@ -650,13 +658,14 @@ Tabular::RowData::RowData()
 Tabular::ColumnData::ColumnData()
        : alignment(LYX_ALIGN_CENTER),
          valignment(LYX_VALIGN_TOP),
-         width(0)
+         width(0),
+         varwidth(false)
 {
 }
 
 
 Tabular::ltType::ltType()
-       : topDL(false),
+       : set(false), topDL(false),
          bottomDL(false),
          empty(false)
 {}
@@ -742,9 +751,9 @@ void Tabular::appendRow(row_type row)
 void Tabular::insertRow(row_type const row, bool copy)
 {
        row_info.insert(row_info.begin() + row + 1, RowData(row_info[row]));
-       cell_info.insert(cell_info.begin() + row + 1, 
+       cell_info.insert(cell_info.begin() + row + 1,
                cell_vector(0, CellData(buffer_)));
-       
+
        for (col_type c = 0; c < ncols(); ++c) {
                cell_info[row + 1].insert(cell_info[row + 1].begin() + c,
                        copy ? CellData(cell_info[row][c]) : CellData(buffer_));
@@ -753,7 +762,7 @@ void Tabular::insertRow(row_type const row, bool copy)
                if (cell_info[row][c].multirow == CELL_BEGIN_OF_MULTIROW)
                        cell_info[row + 1][c].multirow = CELL_PART_OF_MULTIROW;
        }
-       
+
        updateIndexes();
        for (col_type c = 0; c < ncols(); ++c) {
                if (isPartOfMultiRow(row, c))
@@ -850,7 +859,7 @@ void Tabular::copyColumn(col_type const col)
 
 
 void Tabular::appendColumn(col_type col)
-{      
+{
        insertColumn(col, false);
 }
 
@@ -876,8 +885,8 @@ void Tabular::insertColumn(col_type const col, bool copy)
                setBottomLine(i, bottomLine(j));
                setTopLine(i, topLine(j));
                setLeftLine(i, leftLine(j));
+               setRightLine(i, rightLine(j));
                if (rightLine(i) && rightLine(j)) {
-                       setRightLine(i, true);
                        setRightLine(j, false);
                }
                if (buffer().params().track_changes)
@@ -908,8 +917,11 @@ void Tabular::updateIndexes()
        // reset column and row of cells and update their width and alignment
        for (row_type row = 0; row < nrows(); ++row)
                for (col_type column = 0; column < ncols(); ++column) {
-                       if (isPartOfMultiColumn(row, column))
+                       if (isPartOfMultiColumn(row, column)) {
+                               cell_info[row][column].inset->toggleMultiCol(true);
                                continue;
+                       }
+                       cell_info[row][column].inset->toggleMultiCol(false);
                        // columnofcell needs to be called before setting width and aligment
                        // multirow cells inherit the width from the column width
                        if (!isPartOfMultiRow(row, column)) {
@@ -917,8 +929,11 @@ void Tabular::updateIndexes()
                                rowofcell[i] = row;
                        }
                        setFixedWidth(row, column);
-                       if (isPartOfMultiRow(row, column))
+                       if (isPartOfMultiRow(row, column)) {
+                               cell_info[row][column].inset->toggleMultiRow(true);
                                continue;
+                       }
+                       cell_info[row][column].inset->toggleMultiRow(false);
                        cell_info[row][column].inset->setContentAlignment(
                                getAlignment(cellIndex(row, column)));
                        ++i;
@@ -1014,15 +1029,60 @@ int Tabular::cellHeight(idx_type cell) const
 }
 
 
-bool Tabular::updateColumnWidths()
+bool Tabular::updateColumnWidths(MetricsInfo & mi)
 {
        vector<int> max_dwidth(ncols(), 0);
+       // collect max. fixed width of column
+       map<col_type, int> max_pwidth;
+       // collect max. variable width of column
+       map<col_type, int> max_width;
+
        for(col_type c = 0; c < ncols(); ++c)
                for(row_type r = 0; r < nrows(); ++r) {
                        idx_type const i = cellIndex(r, c);
                        if (getAlignment(i) == LYX_ALIGN_DECIMAL)
                                max_dwidth[c] = max(max_dwidth[c], cell_info[r][c].decimal_width);
+                       if (!getPWidth(i).zero())
+                               max_pwidth[c] = max(max_pwidth[c], cell_info[r][c].width);
+                       else if (!column_info[c].varwidth)
+                               max_width[c] = max(max_width[c], cell_info[r][c].width);
+               }
+
+       // If we have a fixed tabular width, we take this into account
+       Length tab_width = tabular_width;
+       bool const tabularx = hasVarwidthColumn();
+       if (tabularx && tab_width.zero())
+               // If no tabular width is specified with X columns,
+               // we use 100% colwidth
+               tab_width = Length(100, Length::PCW);
+       int restwidth = -1;
+       if (!tab_width.zero()) {
+               restwidth = mi.base.inPixels(tab_width);
+               // Substract the fixed widths from the table width
+               for (auto const w : max_pwidth)
+                       restwidth -= w.second;
+       }
+
+       // If we have a fixed width, distribute the available table width
+       // (minus the fixed widths) to the variable-width columns
+       int vcolwidth = -1;
+       int restcols = ncols() - max_pwidth.size();
+       if (restwidth > 0)
+               vcolwidth = restwidth / restcols;
+
+       // Now consider that some variable width columns exceed the vcolwidth
+       if (vcolwidth > 0) {
+               bool changed = false;
+               for (auto const w : max_width) {
+                       if (tabularx || w.second > vcolwidth) {
+                               --restcols;
+                               restwidth -= w.second;
+                               changed = true;
+                       }
                }
+               if (changed && restwidth > 0)
+                       vcolwidth = restwidth / restcols;
+       }
 
        bool update = false;
        // for each col get max of single col cells
@@ -1032,17 +1092,26 @@ bool Tabular::updateColumnWidths()
                        idx_type const i = cellIndex(r, c);
                        if (columnSpan(i) == 1) {
                                if (getAlignment(i) == LYX_ALIGN_DECIMAL
-                                       && cell_info[r][c].decimal_width!=0)
-                                       new_width = max(new_width, cellInfo(i).width 
+                                       && cell_info[r][c].decimal_width != 0)
+                                       new_width = max(new_width, cellInfo(i).width
                                                + max_dwidth[c] - cellInfo(i).decimal_width);
-                               else
+                               else if (getPWidth(i).zero() && vcolwidth > 0) {
+                                       if (tabularx && !column_info[c].varwidth)
+                                               new_width = max(new_width, cellInfo(i).width);
+                                       else if (tabularx)
+                                               new_width = vcolwidth;
+                                       else
+                                               new_width = max(vcolwidth, max(new_width, cellInfo(i).width));
+                               } else
                                        new_width = max(new_width, cellInfo(i).width);
                        }
                }
 
                if (column_info[c].width != new_width) {
                        column_info[c].width = new_width;
-                       update = true;
+                       // Do not trigger update when no space is left for variable
+                       // columns, as this will loop
+                       update = tab_width.zero() || restwidth > 0;
                }
        }
        // update col widths to fit merged cells
@@ -1059,7 +1128,9 @@ bool Tabular::updateColumnWidths()
 
                        if (cellInfo(i).width > old_width) {
                                column_info[c + span - 1].width += cellInfo(i).width - old_width;
-                               update = true;
+                               // Do not trigger update when no space is left for variable
+                               // columns, as this will loop
+                               update = tab_width.zero() || restwidth > 0;
                        }
                }
 
@@ -1102,8 +1173,8 @@ void Tabular::setAlignment(idx_type cell, LyXAlignment align,
                if (align == LYX_ALIGN_DECIMAL && dpoint.empty())
                        dpoint = from_utf8(lyxrc.default_decimal_point);
        } else {
-               cellInfo(cell).alignment = align; 
-               cellInset(cell).get()->setContentAlignment(align); 
+               cellInfo(cell).alignment = align;
+               cellInset(cell)->setContentAlignment(align);
        }
 }
 
@@ -1121,15 +1192,16 @@ void Tabular::setVAlignment(idx_type cell, VAlignment align,
 namespace {
 
 /**
- * Allow line and paragraph breaks for fixed width cells or disallow them,
- * merge cell paragraphs and reset layout to standard for variable width
- * cells.
+ * Allow line and paragraph breaks for fixed width multicol/multirow cells
+ * or disallow them, merge cell paragraphs and reset layout to standard
+ * for variable width multicol cells.
  */
-void toggleFixedWidth(Cursor & cur, InsetTableCell * inset, bool fixedWidth)
+void toggleFixedWidth(Cursor & cur, InsetTableCell * inset,
+                     bool const fixedWidth, bool const multicol,
+                     bool const multirow)
 {
-       inset->setAutoBreakRows(fixedWidth);
        inset->toggleFixedWidth(fixedWidth);
-       if (fixedWidth)
+       if (!multirow && (fixedWidth || !multicol))
                return;
 
        // merge all paragraphs to one
@@ -1137,6 +1209,19 @@ void toggleFixedWidth(Cursor & cur, InsetTableCell * inset, bool fixedWidth)
        while (inset->paragraphs().size() > 1)
                mergeParagraph(bp, inset->paragraphs(), 0);
 
+       // This is relevant for multirows
+       if (fixedWidth)
+               return;
+
+       // remove newlines
+       ParagraphList::iterator pit = inset->paragraphs().begin();
+       for (; pit != inset->paragraphs().end(); ++pit) {
+               for (pos_type j = 0; j != pit->size(); ++j) {
+                       if (pit->isNewline(j))
+                               pit->eraseChar(j, bp.track_changes);
+               }
+       }
+
        // reset layout
        cur.push(*inset);
        // undo information has already been recorded
@@ -1145,7 +1230,7 @@ void toggleFixedWidth(Cursor & cur, InsetTableCell * inset, bool fixedWidth)
        cur.pop();
 }
 
-}
+} // namespace
 
 
 void Tabular::setColumnPWidth(Cursor & cur, idx_type cell,
@@ -1154,23 +1239,22 @@ void Tabular::setColumnPWidth(Cursor & cur, idx_type cell,
        col_type const c = cellColumn(cell);
 
        column_info[c].p_width = width;
-       // reset the vertical alignment to top if the fixed with
+       // reset the vertical alignment to top if the fixed width
        // is removed or zero because only fixed width columns can
        // have a vertical alignment
        if (column_info[c].p_width.zero())
                column_info[c].valignment = LYX_VALIGN_TOP;
        for (row_type r = 0; r < nrows(); ++r) {
-               idx_type const cell = cellIndex(r, c);
+               idx_type const cidx = cellIndex(r, c);
                // because of multicolumns
-               toggleFixedWidth(cur, cellInset(cell).get(),
-                                !getPWidth(cell).zero());
+               toggleFixedWidth(cur, cellInset(cidx).get(),
+                                !getPWidth(cidx).zero(), isMultiColumn(cidx),
+                                isMultiRow(cidx));
+               if (isMultiRow(cidx))
+                       setAlignment(cidx, LYX_ALIGN_LEFT, false);
        }
-       // cur paragraph can become invalid after paragraphs were merged
-       if (cur.pit() > cur.lastpit())
-               cur.pit() = cur.lastpit();
-       // cur position can become invalid after newlines were removed
-       if (cur.pos() > cur.lastpos())
-               cur.pos() = cur.lastpos();
+       // cur can become invalid after paragraphs were merged
+       cur.fixIfBroken();
 }
 
 
@@ -1191,13 +1275,17 @@ bool Tabular::setMColumnPWidth(Cursor & cur, idx_type cell,
                return false;
 
        cellInfo(cell).p_width = width;
-       toggleFixedWidth(cur, cellInset(cell).get(), !width.zero());
-       // cur paragraph can become invalid after paragraphs were merged
-       if (cur.pit() > cur.lastpit())
-               cur.pit() = cur.lastpit();
-       // cur position can become invalid after newlines were removed
-       if (cur.pos() > cur.lastpos())
-               cur.pos() = cur.lastpos();
+       toggleFixedWidth(cur, cellInset(cell).get(), !width.zero(),
+                        isMultiColumn(cell), isMultiRow(cell));
+       // cur can become invalid after paragraphs were merged
+       cur.fixIfBroken();
+       return true;
+}
+
+
+bool Tabular::toggleVarwidth(idx_type cell, bool const varwidth)
+{
+       column_info[cellColumn(cell)].varwidth = varwidth;
        return true;
 }
 
@@ -1291,7 +1379,7 @@ bool Tabular::columnRightLine(col_type c) const
                idx_type i = cellIndex(r, c);
                if (c == cellColumn(i) + columnSpan(i) - 1) {
                        ++total;
-                       bool left = (c + 1 < ncols() 
+                       bool left = (c + 1 < ncols()
                                && cellInfo(cellIndex(r, c + 1)).left_line)
                                || c + 1 == ncols();
                        if (cellInfo(i).right_line && left)
@@ -1306,7 +1394,7 @@ LyXAlignment Tabular::getAlignment(idx_type cell, bool onlycolumn) const
 {
        if (!onlycolumn && (isMultiColumn(cell) || isMultiRow(cell)))
                return cellInfo(cell).alignment;
-       
+
        return column_info[cellColumn(cell)].alignment;
 }
 
@@ -1320,6 +1408,27 @@ Tabular::getVAlignment(idx_type cell, bool onlycolumn) const
 }
 
 
+int Tabular::offsetVAlignment() const
+{
+       // for top-alignment the first horizontal table line must be exactly at
+       // the position of the base line of the surrounding text line
+       // for bottom alignment, the same is for the last table line
+       int offset_valign = 0;
+       switch (tabular_valignment) {
+       case Tabular::LYX_VALIGN_BOTTOM:
+               offset_valign = rowAscent(0) - height();
+               break;
+       case Tabular::LYX_VALIGN_MIDDLE:
+               offset_valign = (- height()) / 2 + rowAscent(0);
+               break;
+       case Tabular::LYX_VALIGN_TOP:
+               offset_valign = rowAscent(0);
+               break;
+       }
+       return offset_valign;
+}
+
+
 Length const Tabular::getPWidth(idx_type cell) const
 {
        if (isMultiColumn(cell))
@@ -1378,7 +1487,7 @@ int Tabular::textVOffset(idx_type cell) const
        int voffset = cellInfo(cell).voffset;
        if (isMultiRow(cell)) {
                row_type const row = cellRow(cell);
-               voffset += (cellHeight(cell) - rowAscent(row) - rowDescent(row))/2; 
+               voffset += (cellHeight(cell) - rowAscent(row) - rowDescent(row))/2;
        }
        return voffset;
 }
@@ -1443,7 +1552,7 @@ void Tabular::write(ostream & os) const
           << ">\n";
        // global longtable options
        os << "<features"
-          << write_attribute("rotate", convert<string>(rotate))
+          << write_attribute("rotate", rotate)
           << write_attribute("booktabs", use_booktabs)
           << write_attribute("islongtable", is_long_tabular)
           << write_attribute("firstHeadTopDL", endfirsthead.topDL)
@@ -1457,13 +1566,11 @@ void Tabular::write(ostream & os) const
           << write_attribute("lastFootBottomDL", endlastfoot.bottomDL)
           << write_attribute("lastFootEmpty", endlastfoot.empty);
        // longtables cannot be aligned vertically
-       if (!is_long_tabular) {
-          os << write_attribute("tabularvalignment", tabular_valignment);
-          os << write_attribute("tabularwidth", tabular_width);
-       }
+       if (!is_long_tabular)
+               os << write_attribute("tabularvalignment", tabular_valignment);
+       os << write_attribute("tabularwidth", tabular_width);
        if (is_long_tabular)
-          os << write_attribute("longtabularalignment",
-                                longtabular_alignment);
+               os << write_attribute("longtabularalignment", longtabular_alignment);
        os << ">\n";
        for (col_type c = 0; c < ncols(); ++c) {
                os << "<column"
@@ -1472,6 +1579,7 @@ void Tabular::write(ostream & os) const
                   os << write_attribute("decimal_point", column_info[c].decimal_point);
                os << write_attribute("valignment", column_info[c].valignment)
                   << write_attribute("width", column_info[c].p_width.asString())
+                  << write_attribute("varwidth", column_info[c].varwidth)
                   << write_attribute("special", column_info[c].align_special)
                   << ">\n";
        }
@@ -1517,6 +1625,9 @@ void Tabular::write(ostream & os) const
                        cell_info[r][c].inset->write(os);
                        os << "\n\\end_inset\n"
                           << "</cell>\n";
+                       // FIXME This can be removed again once the mystery
+                       // crash has been resolved.
+                       os << flush;
                }
                os << "</row>\n";
        }
@@ -1581,6 +1692,7 @@ void Tabular::read(Lexer & lex)
                getTokenValue(line, "valignment", column_info[c].valignment);
                getTokenValue(line, "width", column_info[c].p_width);
                getTokenValue(line, "special", column_info[c].align_special);
+               getTokenValue(line, "varwidth", column_info[c].varwidth);
        }
 
        for (row_type i = 0; i < nrows(); ++i) {
@@ -1651,7 +1763,7 @@ void Tabular::read(Lexer & lex)
 
 bool Tabular::isMultiColumn(idx_type cell) const
 {
-       return (cellInfo(cell).multicolumn == CELL_BEGIN_OF_MULTICOLUMN 
+       return (cellInfo(cell).multicolumn == CELL_BEGIN_OF_MULTICOLUMN
                || cellInfo(cell).multicolumn == CELL_PART_OF_MULTICOLUMN);
 }
 
@@ -1666,13 +1778,40 @@ bool Tabular::hasMultiColumn(col_type c) const
 }
 
 
-Tabular::CellData & Tabular::cellInfo(idx_type cell) const
+bool Tabular::hasVarwidthColumn() const
+{
+       for (col_type c = 0; c < ncols(); ++c) {
+               if (column_info[c].varwidth)
+                       return true;
+       }
+       return false;
+}
+
+
+bool Tabular::isVTypeColumn(col_type c) const
+{
+       for (row_type r = 0; r < nrows(); ++r) {
+               idx_type idx = cellIndex(r, c);
+               if (getRotateCell(idx) == 0 && useBox(idx) == BOX_VARWIDTH)
+                       return true;
+       }
+       return false;
+}
+
+
+Tabular::CellData const & Tabular::cellInfo(idx_type cell) const
 {
        return cell_info[cellRow(cell)][cellColumn(cell)];
 }
 
 
-Tabular::idx_type Tabular::setMultiColumn(idx_type cell, idx_type number,
+Tabular::CellData & Tabular::cellInfo(idx_type cell)
+{
+       return cell_info[cellRow(cell)][cellColumn(cell)];
+}
+
+
+Tabular::idx_type Tabular::setMultiColumn(Cursor & cur, idx_type cell, idx_type number,
                                          bool const right_border)
 {
        idx_type const col = cellColumn(cell);
@@ -1687,8 +1826,17 @@ Tabular::idx_type Tabular::setMultiColumn(idx_type cell, idx_type number,
        if (column_info[col].alignment != LYX_ALIGN_DECIMAL)
                cs.alignment = column_info[col].alignment;
        setRightLine(cell, right_border);
+       // non-fixed width multicolumns cannot have multiple paragraphs
+       if (getPWidth(cell).zero()) {
+               toggleFixedWidth(cur, cellInset(cell).get(),
+                                !getPWidth(cell).zero(), isMultiColumn(cell),
+                                isMultiRow(cell));
+               // cur can become invalid after paragraphs were merged
+               cur.fixIfBroken();
+       }
 
-       for (idx_type i = 1; i < number; ++i) {
+       idx_type lastcell = cellIndex(row, col + number - 1);
+       for (idx_type i = 1; i < lastcell - cell + 1; ++i) {
                CellData & cs1 = cellInfo(cell + i);
                cs1.multicolumn = CELL_PART_OF_MULTICOLUMN;
                cs.inset->appendParagraphs(cs1.inset->paragraphs());
@@ -1714,8 +1862,9 @@ bool Tabular::hasMultiRow(row_type r) const
        return false;
 }
 
-Tabular::idx_type Tabular::setMultiRow(idx_type cell, idx_type number,
-                                      bool const bottom_border)
+Tabular::idx_type Tabular::setMultiRow(Cursor & cur, idx_type cell, idx_type number,
+                                      bool const bottom_border,
+                                      LyXAlignment const halign)
 {
        idx_type const col = cellColumn(cell);
        idx_type const row = cellRow(cell);
@@ -1731,10 +1880,19 @@ Tabular::idx_type Tabular::setMultiRow(idx_type cell, idx_type number,
        // be changed for the whole table row,
        // support changing this only for the multirow cell can be done via
        // \multirowsetup
-       // this feature would be a fileformat change
-       // until LyX supports this, use the deault alignment of multirow
-       // cells: left
-       cs.alignment = LYX_ALIGN_LEFT;
+       if (getPWidth(cell).zero())
+               cs.alignment = halign;
+       else
+               cs.alignment = LYX_ALIGN_LEFT;
+
+       // Multirows cannot have multiple paragraphs
+       if (getPWidth(cell).zero()) {
+               toggleFixedWidth(cur, cellInset(cell).get(),
+                                !getPWidth(cell).zero(),
+                                isMultiColumn(cell), isMultiRow(cell));
+               // cur can become invalid after paragraphs were merged
+               cur.fixIfBroken();
+       }
 
        // set the bottom line of the last selected cell
        setBottomLine(cell, bottom_border);
@@ -1768,7 +1926,7 @@ Tabular::idx_type Tabular::rowSpan(idx_type cell) const
        col_type row = cellRow(cell) + 1;
        while (row < nrows() && isPartOfMultiRow(row, column))
                ++row;
-       
+
        return row - cellRow(cell);
 }
 
@@ -1826,7 +1984,7 @@ int Tabular::getRotateCell(idx_type cell) const
 
 bool Tabular::needRotating() const
 {
-       if (rotate)
+       if (rotate && !is_long_tabular)
                return true;
        for (row_type r = 0; r < nrows(); ++r)
                for (col_type c = 0; c < ncols(); ++c)
@@ -1848,7 +2006,7 @@ Tabular::idx_type Tabular::cellAbove(idx_type cell) const
 {
        if (cellRow(cell) == 0)
                return cell;
-       
+
        col_type const col = cellColumn(cell);
        row_type r = cellRow(cell) - 1;
        while (r > 0 && cell_info[r][col].multirow == CELL_PART_OF_MULTIROW)
@@ -1881,16 +2039,15 @@ void Tabular::setUsebox(idx_type cell, BoxType type)
 }
 
 
-// FIXME: Remove this routine because we cannot insert \parboxes when the user
-// adds line breaks, see bug 4886.
 Tabular::BoxType Tabular::getUsebox(idx_type cell) const
 {
-       if ((!column_info[cellColumn(cell)].p_width.zero() && !isMultiColumn(cell)) ||
-               (isMultiColumn(cell) && !cellInfo(cell).p_width.zero()))
+       if (getRotateCell(cell) == 0
+           && ((!column_info[cellColumn(cell)].p_width.zero() && !isMultiColumn(cell)) ||
+               (isMultiColumn(cell) && !cellInfo(cell).p_width.zero())))
                return BOX_NONE;
        if (cellInfo(cell).usebox > 1)
                return cellInfo(cell).usebox;
-       return useParbox(cell);
+       return useBox(cell);
 }
 
 
@@ -2019,11 +2176,11 @@ bool Tabular::haveLTLastFoot(bool withcaptions) const
 }
 
 
-Tabular::idx_type Tabular::setLTCaption(row_type row, bool what)
+Tabular::idx_type Tabular::setLTCaption(Cursor & cur, row_type row, bool what)
 {
        idx_type i = getFirstCellInRow(row);
        if (what) {
-               setMultiColumn(i, numberOfCellsInRow(row), false);
+               setMultiColumn(cur, i, numberOfCellsInRow(row), false);
                setTopLine(i, false);
                setBottomLine(i, false);
                setLeftLine(i, false);
@@ -2139,23 +2296,24 @@ bool Tabular::isPartOfMultiRow(row_type row, col_type column) const
 }
 
 
-void Tabular::TeXTopHLine(otexstream & os, row_type row, string const lang) const
+void Tabular::TeXTopHLine(otexstream & os, row_type row, string const & lang,
+                         list<col_type> columns) const
 {
        // we only output complete row lines and the 1st row here, the rest
        // is done in Tabular::TeXBottomHLine(...)
 
        // get for each column the topline (if any)
-       vector<bool> topline;
+       map<col_type, bool> topline;
        col_type nset = 0;
-       for (col_type c = 0; c < ncols(); ++c) {
-               topline.push_back(topLine(cellIndex(row, c)));
+       for (auto const & c : columns) {
+               topline[c] = topLine(cellIndex(row, c));
                // If cell is part of a multirow and not the first cell of the
                // multirow, no line must be drawn.
                if (row != 0)
                        if (isMultiRow(cellIndex(row, c))
                            && cell_info[row][c].multirow != CELL_BEGIN_OF_MULTIROW)
                                topline[c] = false;
-               if (topline[c])
+               if (topline.find(c) != topline.end() && topline.find(c)->second)
                        ++nset;
        }
 
@@ -2171,28 +2329,28 @@ void Tabular::TeXTopHLine(otexstream & os, row_type row, string const lang) cons
                        os << "\\hline ";
                }
        } else if (row == 0) {
-               for (col_type c = 0; c < ncols(); ++c) {
-                       if (topline[c]) {
+               for (auto & c : columns) {
+                       if (topline.find(c)->second) {
                                col_type offset = 0;
                                for (col_type j = 0 ; j < c; ++j)
                                        if (column_info[j].alignment == LYX_ALIGN_DECIMAL)
                                                ++offset;
-                               
+
                                //babel makes the "-" character an active one, so we have to suppress this here
                                //see http://groups.google.com/group/comp.text.tex/browse_thread/thread/af769424a4a0f289#
                                if (lang == "slovak" || lang == "czech")
-                                       os << "\\expandafter" << (use_booktabs ? "\\cmidrule" : "\\cline") 
+                                       os << "\\expandafter" << (use_booktabs ? "\\cmidrule" : "\\cline")
                                        << "\\expandafter{\\expandafter" << c + 1 + offset << "\\string-";
                                else
                                        os << (use_booktabs ? "\\cmidrule{" : "\\cline{") << c + 1 + offset << '-';
-                               
+
                                col_type cstart = c;
-                               for ( ; c < ncols() && topline[c]; ++c) {}
-                               
+                               for ( ; c < ncols() && topline.find(c)->second; ++c) {}
+
                                for (col_type j = cstart ; j < c ; ++j)
                                        if (column_info[j].alignment == LYX_ALIGN_DECIMAL)
                                                ++offset;
-                               
+
                                os << c + offset << "} ";
                        }
                }
@@ -2201,18 +2359,19 @@ void Tabular::TeXTopHLine(otexstream & os, row_type row, string const lang) cons
 }
 
 
-void Tabular::TeXBottomHLine(otexstream & os, row_type row, string const lang) const
+void Tabular::TeXBottomHLine(otexstream & os, row_type row, string const & lang,
+                            list<col_type> columns) const
 {
        // we output bottomlines of row r and the toplines of row r+1
        // if the latter do not span the whole tabular
 
        // get the bottomlines of row r, and toplines in next row
        bool lastrow = row == nrows() - 1;
-       vector<bool> bottomline, topline;
+       map<col_type, bool> bottomline, topline;
        bool nextrowset = true;
-       for (col_type c = 0; c < ncols(); ++c) {
-               bottomline.push_back(bottomLine(cellIndex(row, c)));
-               topline.push_back(!lastrow && topLine(cellIndex(row + 1, c)));
+       for (auto const & c : columns) {
+               bottomline[c] = bottomLine(cellIndex(row, c));
+               topline[c] =  !lastrow && topLine(cellIndex(row + 1, c));
                // If cell is part of a multirow and not the last cell of the
                // multirow, no line must be drawn.
                if (!lastrow)
@@ -2222,15 +2381,15 @@ void Tabular::TeXBottomHLine(otexstream & os, row_type row, string const lang) c
                                bottomline[c] = false;
                                topline[c] = false;
                                }
-               nextrowset &= topline[c];
+               nextrowset &= topline.find(c) != topline.end() && topline.find(c)->second;
        }
 
        // combine this row's bottom lines and next row's toplines if necessary
        col_type nset = 0;
-       for (col_type c = 0; c < ncols(); ++c) {
+       for (auto const & c : columns) {
                if (!nextrowset)
-                       bottomline[c] = bottomline[c] || topline[c];
-               if (bottomline[c])
+                       bottomline[c] = bottomline.find(c)->second || topline.find(c)->second;
+               if (bottomline.find(c)->second)
                        ++nset;
        }
 
@@ -2244,13 +2403,13 @@ void Tabular::TeXBottomHLine(otexstream & os, row_type row, string const lang) c
                else
                        os << "\\hline ";
        } else {
-               for (col_type c = 0; c < ncols(); ++c) {
-                       if (bottomline[c]) {
+               for (auto & c : columns) {
+                       if (bottomline.find(c)->second) {
                                col_type offset = 0;
                                for (col_type j = 0 ; j < c; ++j)
                                        if (column_info[j].alignment == LYX_ALIGN_DECIMAL)
                                                ++offset;
-                               
+
                                //babel makes the "-" character an active one, so we have to suppress this here
                                //see http://groups.google.com/group/comp.text.tex/browse_thread/thread/af769424a4a0f289#
                                if (lang == "slovak" || lang == "czech")
@@ -2258,14 +2417,14 @@ void Tabular::TeXBottomHLine(otexstream & os, row_type row, string const lang) c
                                        << "\\expandafter{\\expandafter" << c + 1 + offset << "\\string-";
                                else
                                        os << (use_booktabs ? "\\cmidrule{" : "\\cline{") << c + 1 + offset << '-';
-                               
+
                                col_type cstart = c;
-                               for ( ; c < ncols() && bottomline[c]; ++c) {}
-                               
+                               for ( ; c < ncols() && bottomline.find(c)->second; ++c) {}
+
                                for (col_type j = cstart ; j < c ; ++j)
                                        if (column_info[j].alignment == LYX_ALIGN_DECIMAL)
                                                ++offset;
-                               
+
                                os << c + offset << "} ";
                        }
                }
@@ -2275,7 +2434,8 @@ void Tabular::TeXBottomHLine(otexstream & os, row_type row, string const lang) c
 
 
 void Tabular::TeXCellPreamble(otexstream & os, idx_type cell,
-                             bool & ismulticol, bool & ismultirow) const
+                             bool & ismulticol, bool & ismultirow,
+                             bool const bidi) const
 {
        row_type const r = cellRow(cell);
        if (is_long_tabular && row_info[r].caption)
@@ -2285,38 +2445,47 @@ void Tabular::TeXCellPreamble(otexstream & os, idx_type cell,
        LyXAlignment align = getAlignment(cell, !isMultiColumn(cell));
        // figure out how to set the lines
        // we always set double lines to the right of the cell
+       // or left in bidi RTL, respectively.
        col_type const c = cellColumn(cell);
        col_type const nextcol = c + columnSpan(cell);
+       bool const decimal = column_info[c].alignment == LYX_ALIGN_DECIMAL;
        bool colright = columnRightLine(c);
        bool colleft = columnLeftLine(c);
        bool nextcolleft = nextcol < ncols() && columnLeftLine(nextcol);
-       bool nextcellleft = nextcol < ncols() 
+       bool nextcellleft = nextcol < ncols()
                && leftLine(cellIndex(r, nextcol));
        bool coldouble = colright && nextcolleft;
        bool celldouble = rightLine(cell) && nextcellleft;
 
-       ismulticol = isMultiColumn(cell)
-               || (c == 0 && colleft != leftLine(cell))
-               || ((colright || nextcolleft) && !rightLine(cell) && !nextcellleft)
-               || (!colright && !nextcolleft && (rightLine(cell) || nextcellleft))
-               || (coldouble != celldouble);
+       ismulticol = (isMultiColumn(cell)
+                     || (c == 0 && colleft != leftLine(cell))
+                     || ((colright || nextcolleft) && !rightLine(cell) && !nextcellleft)
+                     || (!colright && !nextcolleft && (rightLine(cell) || nextcellleft))
+                     || (coldouble != celldouble))
+                    && !decimal;
 
        // we center in multicol when no decimal point
-       if (column_info[c].alignment == LYX_ALIGN_DECIMAL) {
+       if (decimal) {
                docstring const align_d = column_info[c].decimal_point;
-               DocIterator const dit = separatorPos(cellInset(cell).get(), align_d);
-               ismulticol |= !dit;
+               DocIterator const dit = separatorPos(cellInset(cell), align_d);
+               bool const nosep = !dit;
+               ismulticol |= nosep;
+               celldouble &= nosep;
        }
 
        // up counter by 1 for each decimally aligned col since they use 2 latex cols
        int latexcolspan = columnSpan(cell);
-       for(col_type col = c; col < c + columnSpan(cell); ++col)
+       for (col_type col = c; col < c + columnSpan(cell); ++col)
                if (column_info[col].alignment == LYX_ALIGN_DECIMAL)
                        ++latexcolspan;
 
        if (ismulticol) {
                os << "\\multicolumn{" << latexcolspan << "}{";
-               if (c ==0 && leftLine(cell))
+               if (((bidi && c == getLastCellInRow(cellRow(0)) && rightLine(cell))
+                    || (!bidi && c == 0 && leftLine(cell))))
+                       os << '|';
+               if (bidi && celldouble)
+                       // add extra vertical line if we want a double one
                        os << '|';
                if (!cellInfo(cell).align_special.empty()) {
                        os << cellInfo(cell).align_special;
@@ -2363,9 +2532,9 @@ void Tabular::TeXCellPreamble(otexstream & os, idx_type cell,
                                }
                        } // end if else !getPWidth
                } // end if else !cellinfo_of_cell
-               if (rightLine(cell) || nextcellleft)
+               if ((bidi && leftLine(cell)) || (!bidi && rightLine(cell)) || nextcellleft)
                        os << '|';
-               if (celldouble)
+               if (!bidi && celldouble)
                        // add extra vertical line if we want a double one
                        os << '|';
                os << "}{";
@@ -2419,7 +2588,21 @@ void Tabular::TeXCellPreamble(otexstream & os, idx_type cell,
                }
                os << "]{" << from_ascii(getPWidth(cell).asLatexString())
                   << "}\n";
+       } else if (getRotateCell(cell) != 0 && getUsebox(cell) == BOX_VARWIDTH) {
+               os << "\\begin{varwidth}[";
+               switch (valign) {
+               case LYX_VALIGN_TOP:
+                       os << 't';
+                       break;
+               case LYX_VALIGN_MIDDLE:
+                       os << 'm';
+                       break;
+               case LYX_VALIGN_BOTTOM:
+                       os << 'b';
+                       break;
        }
+       os << "]{\\linewidth}\n";
+}
 }
 
 
@@ -2435,6 +2618,8 @@ void Tabular::TeXCellPostamble(otexstream & os, idx_type cell,
                os << '}';
        else if (getUsebox(cell) == BOX_MINIPAGE)
                os << breakln << "\\end{minipage}";
+       else if (getRotateCell(cell) != 0 && getUsebox(cell) == BOX_VARWIDTH)
+               os << breakln << "\\end{varwidth}";
        if (getRotateCell(cell) != 0)
                os << breakln << "\\end{turn}";
        if (ismultirow)
@@ -2445,7 +2630,8 @@ void Tabular::TeXCellPostamble(otexstream & os, idx_type cell,
 
 
 void Tabular::TeXLongtableHeaderFooter(otexstream & os,
-                                       OutputParams const & runparams) const
+                                      OutputParams const & runparams,
+                                      list<col_type> columns) const
 {
        if (!is_long_tabular)
                return;
@@ -2457,7 +2643,7 @@ void Tabular::TeXLongtableHeaderFooter(otexstream & os,
                        if (row_info[r].caption &&
                            !row_info[r].endfirsthead && !row_info[r].endhead &&
                            !row_info[r].endfoot && !row_info[r].endlastfoot)
-                               TeXRow(os, r, runparams);
+                               TeXRow(os, r, runparams, columns);
                }
        }
        // output first header info
@@ -2466,7 +2652,7 @@ void Tabular::TeXLongtableHeaderFooter(otexstream & os,
                        os << "\\hline\n";
                for (row_type r = 0; r < nrows(); ++r) {
                        if (row_info[r].endfirsthead)
-                               TeXRow(os, r, runparams);
+                               TeXRow(os, r, runparams, columns);
                }
                if (endfirsthead.bottomDL)
                        os << "\\hline\n";
@@ -2480,7 +2666,7 @@ void Tabular::TeXLongtableHeaderFooter(otexstream & os,
                        os << "\\hline\n";
                for (row_type r = 0; r < nrows(); ++r) {
                        if (row_info[r].endhead)
-                               TeXRow(os, r, runparams);
+                               TeXRow(os, r, runparams, columns);
                }
                if (endhead.bottomDL)
                        os << "\\hline\n";
@@ -2492,7 +2678,7 @@ void Tabular::TeXLongtableHeaderFooter(otexstream & os,
                        os << "\\hline\n";
                for (row_type r = 0; r < nrows(); ++r) {
                        if (row_info[r].endfoot)
-                               TeXRow(os, r, runparams);
+                               TeXRow(os, r, runparams, columns);
                }
                if (endfoot.bottomDL)
                        os << "\\hline\n";
@@ -2506,7 +2692,7 @@ void Tabular::TeXLongtableHeaderFooter(otexstream & os,
                        os << "\\hline\n";
                for (row_type r = 0; r < nrows(); ++r) {
                        if (row_info[r].endlastfoot)
-                               TeXRow(os, r, runparams);
+                               TeXRow(os, r, runparams, columns);
                }
                if (endlastfoot.bottomDL)
                        os << "\\hline\n";
@@ -2526,15 +2712,16 @@ bool Tabular::isValidRow(row_type row) const
 
 
 void Tabular::TeXRow(otexstream & os, row_type row,
-                    OutputParams const & runparams) const
+                    OutputParams const & runparams,
+                    list<col_type> columns) const
 {
        idx_type cell = cellIndex(row, 0);
-       shared_ptr<InsetTableCell> inset = cellInset(cell);
-       Paragraph const & par = inset->paragraphs().front();
-       string const lang = par.getParLanguage(buffer().params())->lang();
+       InsetTableCell const * cinset = cellInset(cell);
+       Paragraph const & cpar = cinset->paragraphs().front();
+       string const clang = cpar.getParLanguage(buffer().params())->lang();
 
        //output the top line
-       TeXTopHLine(os, row, lang);
+       TeXTopHLine(os, row, clang, columns);
 
        if (row_info[row].top_space_default) {
                if (use_booktabs)
@@ -2554,7 +2741,15 @@ void Tabular::TeXRow(otexstream & os, row_type row,
        }
        bool ismulticol = false;
        bool ismultirow = false;
-       for (col_type c = 0; c < ncols(); ++c) {
+
+       // The bidi package (loaded by polyglossia) reverses RTL table columns
+       bool const bidi_rtl =
+               runparams.local_font->isRightToLeft()
+               && runparams.use_polyglossia;
+       idx_type lastcell =
+               bidi_rtl ? getFirstCellInRow(row) : getLastCellInRow(row);
+
+       for (auto const & c : columns) {
                if (isPartOfMultiColumn(row, c))
                        continue;
 
@@ -2562,15 +2757,18 @@ void Tabular::TeXRow(otexstream & os, row_type row,
 
                if (isPartOfMultiRow(row, c)
                    && column_info[c].alignment != LYX_ALIGN_DECIMAL) {
-                       if (cell != getLastCellInRow(row))
-                               os << " & "; 
+                       if (cell != lastcell)
+                               os << " & ";
                        continue;
                }
 
-               TeXCellPreamble(os, cell, ismulticol, ismultirow);
-               shared_ptr<InsetTableCell> inset = cellInset(cell);
+               TeXCellPreamble(os, cell, ismulticol, ismultirow, bidi_rtl);
+               InsetTableCell const * inset = cellInset(cell);
 
                Paragraph const & par = inset->paragraphs().front();
+
+               os.texrow().forceStart(par.id(), 0);
+
                bool rtl = par.isRTL(buffer().params())
                        && !par.empty()
                        && getPWidth(cell).zero()
@@ -2598,22 +2796,43 @@ void Tabular::TeXRow(otexstream & os, row_type row,
 
                if (getAlignment(cell) == LYX_ALIGN_DECIMAL) {
                        // copy cell and split in 2
-                       InsetTableCell head = InsetTableCell(*cellInset(cell).get());
-                       head.setBuffer(buffer());
+                       InsetTableCell head = InsetTableCell(*cellInset(cell));
+                       head.setBuffer(const_cast<Buffer &>(buffer()));
                        DocIterator dit = cellInset(cell)->getText(0)->macrocontextPosition();
                        dit.pop_back();
                        dit.push_back(CursorSlice(head));
                        head.setMacrocontextPositionRecursive(dit);
                        bool hassep = false;
                        InsetTableCell tail = splitCell(head, column_info[c].decimal_point, hassep);
-                       head.latex(os, newrp);
                        if (hassep) {
-                               os << '&';
                                tail.setBuffer(head.buffer());
                                dit.pop_back();
                                dit.push_back(CursorSlice(tail));
                                tail.setMacrocontextPositionRecursive(dit);
-                               tail.latex(os, newrp);
+                       }
+                       if (bidi_rtl) {
+                               if (hassep) {
+                                       tail.latex(os, newrp);
+                                       os << '&';
+                               }
+                               head.latex(os, newrp);
+                       } else {
+                               head.latex(os, newrp);
+                               if (hassep) {
+                                       os << '&';
+                                       tail.latex(os, newrp);
+                               }
+                       }
+               } else if (ltCaption(row)) {
+                       // Inside longtable caption rows, we must only output the caption inset
+                       // with its content and omit anything outside of that (see #10791)
+                       InsetIterator it = inset_iterator_begin(*const_cast<InsetTableCell *>(inset));
+                       InsetIterator i_end = inset_iterator_end(*const_cast<InsetTableCell *>(inset));
+                       for (; it != i_end; ++it) {
+                               if (it->lyxCode() != CAPTION_CODE)
+                                       continue;
+                               it->latex(os, runparams);
+                               break;
                        }
                } else if (!isPartOfMultiRow(row, c)) {
                        if (!runparams.nice)
@@ -2626,7 +2845,7 @@ void Tabular::TeXRow(otexstream & os, row_type row,
                        os << '}';
 
                TeXCellPostamble(os, cell, ismulticol, ismultirow);
-               if (cell != getLastCellInRow(row)) { // not last cell in row
+               if (cell != lastcell) { // not last cell in row
                        if (runparams.nice)
                                os << " & ";
                        else
@@ -2649,7 +2868,7 @@ void Tabular::TeXRow(otexstream & os, row_type row,
        os << '\n';
 
        //output the bottom line
-       TeXBottomHLine(os, row, lang);
+       TeXBottomHLine(os, row, clang, columns);
 
        if (row_info[row].interline_space_default) {
                if (use_booktabs)
@@ -2671,35 +2890,59 @@ void Tabular::TeXRow(otexstream & os, row_type row,
 
 void Tabular::latex(otexstream & os, OutputParams const & runparams) const
 {
-       bool const is_tabular_star = !tabular_width.zero();
+       bool const is_tabular_star = !is_long_tabular && !tabular_width.zero()
+               && !hasVarwidthColumn();
+       bool const is_xltabular = is_long_tabular
+               && (hasVarwidthColumn() || !tabular_width.zero());
+       TexRow::RowEntry pos = TexRow::textEntry(runparams.lastid, runparams.lastpos);
 
        //+---------------------------------------------------------------------
        //+                      first the opening preamble                    +
        //+---------------------------------------------------------------------
 
        os << safebreakln;
-       if (runparams.lastid != -1)
-               os.texrow().start(runparams.lastid, runparams.lastpos);
+       if (!TexRow::isNone(pos))
+               os.texrow().start(pos);
 
-       if (rotate != 0)
-               os << "\\begin{turn}{" << convert<string>(rotate) << "}\n";
+       if (rotate != 0) {
+               if (is_long_tabular)
+                       os << "\\begin{landscape}\n";
+               else
+                       os << "\\begin{turn}{" << convert<string>(rotate) << "}\n";
+       }
 
        if (is_long_tabular) {
-               os << "\\begin{longtable}";
+               if (is_xltabular)
+                       os << "\\begin{xltabular}";
+               else
+                       os << "\\begin{longtable}";
                switch (longtabular_alignment) {
                case LYX_LONGTABULAR_ALIGN_LEFT:
                        os << "[l]";
                        break;
                case LYX_LONGTABULAR_ALIGN_CENTER:
+                       os << "[c]";
                        break;
                case LYX_LONGTABULAR_ALIGN_RIGHT:
                        os << "[r]";
                        break;
                }
+               if (is_xltabular) {
+                       if (tabular_width.zero())
+                               os << "{" << from_ascii("\\columnwidth") << "}";
+                       else
+                               os << "{" << from_ascii(tabular_width.asLatexString()) << "}";
+               }
        } else {
                if (is_tabular_star)
                        os << "\\begin{tabular*}{" << from_ascii(tabular_width.asLatexString()) << "}";
-               else
+               else if (hasVarwidthColumn()) {
+                       os << "\\begin{tabularx}{";
+                       if (tabular_width.zero())
+                               os << from_ascii("\\columnwidth") << "}";
+                       else
+                               os << from_ascii(tabular_width.asLatexString()) << "}";
+               } else
                        os << "\\begin{tabular}";
                switch (tabular_valignment) {
                case LYX_VALIGN_TOP:
@@ -2712,19 +2955,33 @@ void Tabular::latex(otexstream & os, OutputParams const & runparams) const
                        break;
                }
        }
-       
+
        os << "{";
 
        if (is_tabular_star)
                os << "@{\\extracolsep{\\fill}}";
 
-       for (col_type c = 0; c < ncols(); ++c) {
-               if (columnLeftLine(c))
+       // The bidi package (loaded by polyglossia) swaps the column
+       // order for RTL (#9686). Thus we use this list.
+       bool const bidi_rtl =
+               runparams.local_font->isRightToLeft()
+               && runparams.use_polyglossia;
+       list<col_type> columns;
+       for (col_type cl = 0; cl < ncols(); ++cl) {
+               if (bidi_rtl)
+                       columns.push_front(cl);
+               else
+                       columns.push_back(cl);
+       }
+
+       for (auto const & c : columns) {
+               if ((bidi_rtl && columnRightLine(c)) || (!bidi_rtl && columnLeftLine(c)))
                        os << '|';
                if (!column_info[c].align_special.empty()) {
                        os << column_info[c].align_special;
                } else {
                        if (!column_info[c].p_width.zero()) {
+                               bool decimal = false;
                                switch (column_info[c].alignment) {
                                case LYX_ALIGN_LEFT:
                                        os << ">{\\raggedright}";
@@ -2739,24 +2996,91 @@ void Tabular::latex(otexstream & os, OutputParams const & runparams) const
                                case LYX_ALIGN_BLOCK:
                                case LYX_ALIGN_LAYOUT:
                                case LYX_ALIGN_SPECIAL:
-                               case LYX_ALIGN_DECIMAL:
                                        break;
+                               case LYX_ALIGN_DECIMAL: {
+                                       if (bidi_rtl)
+                                               os << ">{\\raggedright}";
+                                       else
+                                               os << ">{\\raggedleft}";
+                                       decimal = true;
+                                       break;
+                               }
                                }
 
+                               char valign = 'p';
                                switch (column_info[c].valignment) {
                                case LYX_VALIGN_TOP:
-                                       os << 'p';
+                                       // this is the default
                                        break;
                                case LYX_VALIGN_MIDDLE:
-                                       os << 'm';
+                                       valign = 'm';
                                        break;
                                case LYX_VALIGN_BOTTOM:
-                                       os << 'b';
+                                       valign = 'b';
                                        break;
-                       }
-                               os << '{'
-                                  << from_ascii(column_info[c].p_width.asLatexString())
-                                  << '}';
+                               }
+                               os << valign;
+
+                               // Fixed-width cells with alignment at decimal separator
+                               // are output as two cells of half the width with the decimal
+                               // separator as column sep. This effectively puts the content
+                               // centered, which differs from the normal decimal sep alignment
+                               // and is not ideal, but we cannot do better ATM (see #9568).
+                               // FIXME: Implement proper decimal sep alignment, e.g. via siunitx.
+                               if (decimal) {
+                                       docstring const halffixedwith =
+                                               from_ascii(Length(column_info[c].p_width.value() / 2,
+                                                                 column_info[c].p_width.unit()).asLatexString());
+                                       os << '{'
+                                          << halffixedwith
+                                          << '}'
+                                          << "@{\\extracolsep{0pt}" << column_info[c].decimal_point << "}"
+                                          << valign
+                                          << '{'
+                                          << halffixedwith
+                                          << '}';
+                               } else
+                                       os << '{'
+                                          << from_ascii(column_info[c].p_width.asLatexString())
+                                          << '}';
+                       } else if (column_info[c].varwidth) {
+                               switch (column_info[c].alignment) {
+                               case LYX_ALIGN_LEFT:
+                                       os << ">{\\raggedright\\arraybackslash}";
+                                       break;
+                               case LYX_ALIGN_RIGHT:
+                                       os << ">{\\raggedleft\\arraybackslash}";
+                                       break;
+                               case LYX_ALIGN_CENTER:
+                                       os << ">{\\centering\\arraybackslash}";
+                                       break;
+                               case LYX_ALIGN_NONE:
+                               case LYX_ALIGN_BLOCK:
+                               case LYX_ALIGN_LAYOUT:
+                               case LYX_ALIGN_SPECIAL:
+                               case LYX_ALIGN_DECIMAL:
+                                       break;
+                               }
+                               os << 'X';
+                       } else if (isVTypeColumn(c)) {
+                               switch (column_info[c].alignment) {
+                               case LYX_ALIGN_LEFT:
+                                       os << ">{\\raggedright}";
+                                       break;
+                               case LYX_ALIGN_RIGHT:
+                                       os << ">{\\raggedleft}";
+                                       break;
+                               case LYX_ALIGN_CENTER:
+                                       os << ">{\\centering}";
+                                       break;
+                               case LYX_ALIGN_NONE:
+                               case LYX_ALIGN_BLOCK:
+                               case LYX_ALIGN_LAYOUT:
+                               case LYX_ALIGN_SPECIAL:
+                               case LYX_ALIGN_DECIMAL:
+                                       break;
+                               }
+                               os << "V{\\linewidth}";
                        } else {
                                switch (column_info[c].alignment) {
                                case LYX_ALIGN_LEFT:
@@ -2774,12 +3098,12 @@ void Tabular::latex(otexstream & os, OutputParams const & runparams) const
                                }
                        } // end if else !column_info[i].p_width
                } // end if else !column_info[i].align_special
-               if (columnRightLine(c))
+               if ((bidi_rtl && columnLeftLine(c)) || (!bidi_rtl && columnRightLine(c)))
                        os << '|';
        }
        os << "}\n";
 
-       TeXLongtableHeaderFooter(os, runparams);
+       TeXLongtableHeaderFooter(os, runparams, columns);
 
        //+---------------------------------------------------------------------
        //+                      the single row and columns (cells)            +
@@ -2787,7 +3111,7 @@ void Tabular::latex(otexstream & os, OutputParams const & runparams) const
 
        for (row_type r = 0; r < nrows(); ++r) {
                if (isValidRow(r)) {
-                       TeXRow(os, r, runparams);
+                       TeXRow(os, r, runparams, columns);
                        if (is_long_tabular && row_info[r].newpage)
                                os << "\\newpage\n";
                }
@@ -2797,17 +3121,29 @@ void Tabular::latex(otexstream & os, OutputParams const & runparams) const
        //+                      the closing of the tabular                    +
        //+---------------------------------------------------------------------
 
-       if (is_long_tabular)
-               os << "\\end{longtable}";
-       else {
+       if (is_long_tabular) {
+               if (is_xltabular)
+                       os << "\\end{xltabular}";
+               else
+                       os << "\\end{longtable}";
+       } else {
                if (is_tabular_star)
                        os << "\\end{tabular*}";
+               else if (hasVarwidthColumn())
+                       os << "\\end{tabularx}";
                else
                        os << "\\end{tabular}";
        }
 
-       if (rotate != 0)
-               os << breakln << "\\end{turn}";
+       if (rotate != 0) {
+               if (is_long_tabular)
+                       os << breakln << "\\end{landscape}";
+               else
+                       os << breakln << "\\end{turn}";
+       }
+
+       if (!TexRow::isNone(pos))
+               os.texrow().start(pos);
 }
 
 
@@ -2975,6 +3311,13 @@ docstring Tabular::xhtmlRow(XHTMLStream & xs, row_type row,
                        continue;
 
                stringstream attr;
+
+               Length const cwidth = column_info[c].p_width;
+               if (!cwidth.zero()) {
+                       string const hwidth = cwidth.asHTMLString();
+                       attr << "style =\"width: " << hwidth << ";\" ";
+               }
+
                attr << "align='";
                switch (getAlignment(cell)) {
                case LYX_ALIGN_LEFT:
@@ -3006,7 +3349,7 @@ docstring Tabular::xhtmlRow(XHTMLStream & xs, row_type row,
                else if (isMultiRow(cell))
                        attr << " rowspan='" << rowSpan(cell) << "'";
 
-               xs << html::StartTag(celltag, attr.str()) << html::CR();
+               xs << html::StartTag(celltag, attr.str(), true) << html::CR();
                ret += cellInset(cell)->xhtml(xs, runparams);
                xs << html::EndTag(celltag) << html::CR();
                ++cell;
@@ -3309,21 +3652,26 @@ void Tabular::plaintext(odocstringstream & os,
 }
 
 
-shared_ptr<InsetTableCell> Tabular::cellInset(idx_type cell) const
+shared_ptr<InsetTableCell> Tabular::cellInset(idx_type cell)
 {
        return cell_info[cellRow(cell)][cellColumn(cell)].inset;
 }
 
 
-shared_ptr<InsetTableCell> Tabular::cellInset(row_type row,
-                                              col_type column) const
+shared_ptr<InsetTableCell> Tabular::cellInset(row_type row, col_type column)
 {
        return cell_info[row][column].inset;
 }
 
 
+InsetTableCell const * Tabular::cellInset(idx_type cell) const
+{
+       return cell_info[cellRow(cell)][cellColumn(cell)].inset.get();
+}
+
+
 void Tabular::setCellInset(row_type row, col_type column,
-                             shared_ptr<InsetTableCell> ins) const
+                           shared_ptr<InsetTableCell> ins)
 {
        CellData & cd = cell_info[row][column];
        cd.inset = ins;
@@ -3335,31 +3683,58 @@ void Tabular::validate(LaTeXFeatures & features) const
        features.require("NeedTabularnewline");
        if (use_booktabs)
                features.require("booktabs");
-       if (is_long_tabular)
-               features.require("longtable");
+       if (is_long_tabular && !hasVarwidthColumn()) {
+               if (tabular_width.zero())
+                       features.require("longtable");
+               else
+                       features.require("xltabular");
+       }
+       if (rotate && is_long_tabular)
+               features.require("lscape");
        if (needRotating())
                features.require("rotating");
+       if (hasVarwidthColumn()) {
+               if (is_long_tabular)
+                       features.require("xltabular");
+               else
+                       features.require("tabularx");
+       }
        for (idx_type cell = 0; cell < numberofcells; ++cell) {
                if (isMultiRow(cell))
                        features.require("multirow");
+               if (getUsebox(cell) == BOX_VARWIDTH)
+                       features.require("varwidth");
                if (getVAlignment(cell) != LYX_VALIGN_TOP
-                   || !getPWidth(cell).zero())
+                   || !getPWidth(cell).zero()
+                   || isVTypeColumn(cellColumn(cell)))
                        features.require("array");
+               // Tell footnote that we need a savenote
+               // environment in non-long tables or
+               // longtable headers/footers
+               else if (!is_long_tabular && !features.inFloat())
+                       features.saveNoteEnv("tabular");
+               else if (!isValidRow(cellRow(cell)))
+                       features.saveNoteEnv("longtable");
+
                cellInset(cell)->validate(features);
+               features.saveNoteEnv(string());
        }
 }
 
 
-Tabular::BoxType Tabular::useParbox(idx_type cell) const
+Tabular::BoxType Tabular::useBox(idx_type cell) const
 {
        ParagraphList const & parlist = cellInset(cell)->paragraphs();
+       if (parlist.size() > 1)
+               return BOX_VARWIDTH;
+
        ParagraphList::const_iterator cit = parlist.begin();
        ParagraphList::const_iterator end = parlist.end();
 
        for (; cit != end; ++cit)
                for (int i = 0; i < cit->size(); ++i)
-                       if (cit->isNewline(i))
-                               return BOX_PARBOX;
+                       if (cit->isNewline(i) || cit->layout().isEnvironment())
+                               return BOX_VARWIDTH;
 
        return BOX_NONE;
 }
@@ -3373,13 +3748,13 @@ Tabular::BoxType Tabular::useParbox(idx_type cell) const
 
 InsetTableCell::InsetTableCell(Buffer * buf)
        : InsetText(buf, InsetText::PlainLayout), isFixedWidth(false),
-         contentAlign(LYX_ALIGN_CENTER)
+         isMultiColumn(false), isMultiRow(false), contentAlign(LYX_ALIGN_CENTER)
 {}
 
 
 bool InsetTableCell::forcePlainLayout(idx_type) const
 {
-       return !isFixedWidth;
+       return isMultiRow || (isMultiColumn && !isFixedWidth);
 }
 
 
@@ -3389,18 +3764,17 @@ bool InsetTableCell::allowParagraphCustomization(idx_type) const
 }
 
 
+bool InsetTableCell::forceLocalFontSwitch() const
+{
+       return isFixedWidth;
+}
+
+
 bool InsetTableCell::getStatus(Cursor & cur, FuncRequest const & cmd,
        FuncStatus & status) const
 {
        bool enabled = true;
        switch (cmd.action()) {
-       case LFUN_LAYOUT:
-               enabled = !forcePlainLayout();
-               break;
-       case LFUN_LAYOUT_PARAGRAPH:
-               enabled = allowParagraphCustomization();
-               break;
-
        case LFUN_MATH_DISPLAY:
                if (!hasFixedWidth()) {
                        enabled = false;
@@ -3413,7 +3787,7 @@ bool InsetTableCell::getStatus(Cursor & cur, FuncRequest const & cmd,
        return true;
 }
 
-docstring InsetTableCell::asString(bool intoInsets) 
+docstring InsetTableCell::asString(bool intoInsets)
 {
        docstring retval;
        if (paragraphs().empty())
@@ -3432,9 +3806,10 @@ docstring InsetTableCell::asString(bool intoInsets)
 }
 
 
-void InsetTableCell::addToToc(DocIterator const & di, bool output_active) const
+void InsetTableCell::addToToc(DocIterator const & di, bool output_active,
+                                                         UpdateType utype, TocBackend & backend) const
 {
-       InsetText::iterateForToc(di, output_active);
+       InsetText::iterateForToc(di, output_active, utype, backend);
 }
 
 
@@ -3446,6 +3821,33 @@ docstring InsetTableCell::xhtml(XHTMLStream & xs, OutputParams const & rp) const
 }
 
 
+void InsetTableCell::metrics(MetricsInfo & mi, Dimension & dim) const
+{
+       TextMetrics & tm = mi.base.bv->textMetrics(&text());
+
+       // Hand font through to contained lyxtext:
+       tm.font_.fontInfo() = mi.base.font;
+       mi.base.textwidth -= 2 * TEXT_TO_INSET_OFFSET;
+
+       // This can happen when a layout has a left and right margin,
+       // and the view is made very narrow. We can't do better than
+       // to draw it partly out of view (bug 5890).
+       if (mi.base.textwidth < 1)
+               mi.base.textwidth = 1;
+
+       // We tell metrics here not to expand on multiple pars
+       // This is the difference to InsetText::Metrics
+       if (hasFixedWidth())
+               tm.metrics(mi, dim, mi.base.textwidth, false);
+       else
+               tm.metrics(mi, dim, 0, false);
+       mi.base.textwidth += 2 * TEXT_TO_INSET_OFFSET;
+       dim.asc += TEXT_TO_INSET_OFFSET;
+       dim.des += TEXT_TO_INSET_OFFSET;
+       dim.wid += 2 * TEXT_TO_INSET_OFFSET;
+}
+
+
 
 /////////////////////////////////////////////////////////////////////
 //
@@ -3455,14 +3857,15 @@ docstring InsetTableCell::xhtml(XHTMLStream & xs, OutputParams const & rp) const
 
 InsetTabular::InsetTabular(Buffer * buf, row_type rows,
                           col_type columns)
-       : Inset(buf), tabular(buf, max(rows, row_type(1)), max(columns, col_type(1))), scx_(0), 
-       rowselect_(false), colselect_(false)
+       : Inset(buf), tabular(buf, max(rows, row_type(1)), max(columns, col_type(1))),
+         rowselect_(false), colselect_(false)
 {
 }
 
 
 InsetTabular::InsetTabular(InsetTabular const & tab)
-       : Inset(tab), tabular(tab.tabular),  scx_(0)
+       : Inset(tab), tabular(tab.tabular),
+         rowselect_(false), colselect_(false)
 {
 }
 
@@ -3501,7 +3904,7 @@ bool InsetTabular::insetAllowed(InsetCode code) const
 bool InsetTabular::allowsCaptionVariation(std::string const & newtype) const
 {
        return tabular.is_long_tabular &&
-               (newtype == "Standard" || newtype == "LongTableNoNumber");
+               (newtype == "Standard" || newtype == "Unnumbered");
 }
 
 
@@ -3549,7 +3952,7 @@ void InsetTabular::read(Lexer & lex)
 int InsetTabular::rowFromY(Cursor & cur, int y) const
 {
        // top y coordinate of tabular
-       int h = yo(cur.bv()) - tabular.rowAscent(0) + offset_valign_;
+       int h = yo(cur.bv()) - tabular.rowAscent(0) + tabular.offsetVAlignment();
        row_type r = 0;
        for (; r < tabular.nrows() && y > h; ++r)
                h += tabular.rowAscent(r) + tabular.rowDescent(r)
@@ -3585,25 +3988,27 @@ void InsetTabular::metrics(MetricsInfo & mi, Dimension & dim) const
                                // multicolumn or multirow cell, but not first one
                                continue;
                        idx_type const cell = tabular.cellIndex(r, c);
-                       Dimension dim;
+                       Dimension dim0;
                        MetricsInfo m = mi;
                        Length const p_width = tabular.getPWidth(cell);
                        if (!p_width.zero())
-                               m.base.textwidth = p_width.inPixels(mi.base.textwidth);
-                       tabular.cellInset(cell)->metrics(m, dim);
-                       if (!p_width.zero())
-                               dim.wid = m.base.textwidth;
-                       tabular.cellInfo(cell).width = dim.wid + 2 * WIDTH_OF_LINE 
+                               m.base.textwidth = mi.base.inPixels(p_width);
+                       else if (tabular.column_info[c].varwidth)
+                               m.base.textwidth = tabular.column_info[c].width;
+                       tabular.cellInset(cell)->metrics(m, dim0);
+                       if (!p_width.zero() || tabular.column_info[c].varwidth)
+                               dim0.wid = m.base.textwidth;
+                       tabular.cellInfo(cell).width = dim0.wid + 2 * WIDTH_OF_LINE
                                + tabular.interColumnSpace(cell);
 
                        // FIXME(?): do we need a second metrics call?
-                       TextMetrics const & tm = 
+                       TextMetrics const & tm =
                                mi.base.bv->textMetrics(tabular.cellInset(cell)->getText(0));
 
                        // determine horizontal offset because of decimal align (if necessary)
                        int decimal_width = 0;
                        if (tabular.getAlignment(cell) == LYX_ALIGN_DECIMAL) {
-                               InsetTableCell tail = InsetTableCell(*tabular.cellInset(cell).get());
+                               InsetTableCell tail = InsetTableCell(*tabular.cellInset(cell));
                                tail.setBuffer(tabular.buffer());
                                // we need to set macrocontext position everywhere
                                // otherwise we crash with nested insets (e.g. footnotes)
@@ -3634,69 +4039,56 @@ void InsetTabular::metrics(MetricsInfo & mi, Dimension & dim) const
                        int const lastpardes = tm.last().second->descent()
                                + TEXT_TO_INSET_OFFSET;
                        int offset = 0;
-                       switch (tabular.getVAlignment(cell)) { 
+                       switch (tabular.getVAlignment(cell)) {
                                case Tabular::LYX_VALIGN_TOP:
-                                       break; 
+                                       break;
                                case Tabular::LYX_VALIGN_MIDDLE:
-                                       offset = -(dim.des - lastpardes)/2; 
-                                       break; 
+                                       offset = -(dim0.des - lastpardes)/2;
+                                       break;
                                case Tabular::LYX_VALIGN_BOTTOM:
-                                       offset = -(dim.des - lastpardes); 
+                                       offset = -(dim0.des - lastpardes);
                                        break;
                        }
                        tabular.cell_info[r][c].voffset = offset;
-                       maxasc = max(maxasc, dim.asc - offset);
-                       maxdes = max(maxdes, dim.des + offset);
+                       maxasc = max(maxasc, dim0.asc - offset);
+                       maxdes = max(maxdes, dim0.des + offset);
                }
                int const top_space = tabular.row_info[r].top_space_default ?
-                       default_line_space :
-                       tabular.row_info[r].top_space.inPixels(mi.base.textwidth);
+                   default_line_space :
+                   mi.base.inPixels(tabular.row_info[r].top_space);
                tabular.setRowAscent(r, maxasc + ADD_TO_HEIGHT + top_space);
                int const bottom_space = tabular.row_info[r].bottom_space_default ?
-                       default_line_space :
-                       tabular.row_info[r].bottom_space.inPixels(mi.base.textwidth);
+                   default_line_space :
+                   mi.base.inPixels(tabular.row_info[r].bottom_space);
                tabular.setRowDescent(r, maxdes + ADD_TO_HEIGHT + bottom_space);
        }
 
-       // for top-alignment the first horizontal table line must be exactly at
-       // the position of the base line of the surrounding text line
-       // for bottom alignment, the same is for the last table line
-       switch (tabular.tabular_valignment) {
-       case Tabular::LYX_VALIGN_BOTTOM:
-               offset_valign_ = tabular.rowAscent(0) - tabular.height();
-               break;
-       case Tabular::LYX_VALIGN_MIDDLE:
-               offset_valign_ = (- tabular.height()) / 2 + tabular.rowAscent(0);
-               break;
-       case Tabular::LYX_VALIGN_TOP:
-               offset_valign_ = tabular.rowAscent(0);
-               break;
-       }
-
-       tabular.updateColumnWidths();
-       dim.asc = tabular.rowAscent(0) - offset_valign_;        
+       // We need to recalculate the metrics after column width calculation
+       // with xtabular (possibly multiple times, so the call is recursive).
+       if (tabular.updateColumnWidths(mi) && tabular.hasVarwidthColumn())
+               metrics(mi, dim);
+       dim.asc = tabular.rowAscent(0) - tabular.offsetVAlignment();
        dim.des = tabular.height() - dim.asc;
        dim.wid = tabular.width() + 2 * ADD_TO_TABULAR_WIDTH;
 }
 
 
-bool InsetTabular::isCellSelected(Cursor & cur, row_type row, col_type col) 
-       const
+bool InsetTabular::isCellSelected(Cursor & cur, row_type row, col_type col) const
 {
        if (&cur.inset() == this && cur.selection()) {
                if (cur.selIsMultiCell()) {
                        row_type rs, re;
                        col_type cs, ce;
                        getSelection(cur, rs, re, cs, ce);
-                       
+
                        idx_type const cell = tabular.cellIndex(row, col);
                        col_type const cspan = tabular.columnSpan(cell);
                        row_type const rspan = tabular.rowSpan(cell);
-                       if (col + cspan - 1 >= cs && col <= ce 
+                       if (col + cspan - 1 >= cs && col <= ce
                                && row + rspan - 1 >= rs && row <= re)
                                return true;
-               } else 
-                       if (col == tabular.cellColumn(cur.idx()) 
+               } else
+                       if (col == tabular.cellColumn(cur.idx())
                                && row == tabular.cellRow(cur.idx())) {
                        CursorSlice const & beg = cur.selBegin();
                        CursorSlice const & end = cur.selEnd();
@@ -3713,11 +4105,10 @@ bool InsetTabular::isCellSelected(Cursor & cur, row_type row, col_type col)
 
 void InsetTabular::draw(PainterInfo & pi, int x, int y) const
 {
-       x += scx_ + ADD_TO_TABULAR_WIDTH;
+       x += ADD_TO_TABULAR_WIDTH;
 
        BufferView * bv = pi.base.bv;
        Cursor & cur = pi.base.bv->cursor();
-       resetPos(cur);
 
        // FIXME: As the full background is painted in drawBackground(),
        // we have no choice but to do a full repaint for the Text cells.
@@ -3726,25 +4117,21 @@ void InsetTabular::draw(PainterInfo & pi, int x, int y) const
        bool const original_selection_state = pi.selected;
 
        idx_type idx = 0;
-       first_visible_cell = Tabular::npos;
 
-       int yy = y + offset_valign_;
+       int yy = y + tabular.offsetVAlignment();
        for (row_type r = 0; r < tabular.nrows(); ++r) {
                int nx = x;
                for (col_type c = 0; c < tabular.ncols(); ++c) {
                        if (tabular.isPartOfMultiColumn(r, c))
                                continue;
-                       
+
                        idx = tabular.cellIndex(r, c);
-                       
+
                        if (tabular.isPartOfMultiRow(r, c)) {
                                nx += tabular.cellWidth(idx);
                                continue;
                        }
 
-                       if (first_visible_cell == Tabular::npos)
-                               first_visible_cell = idx;
-
                        pi.selected |= isCellSelected(cur, r, c);
                        int const cx = nx + tabular.textHOffset(idx);
                        int const cy = yy + tabular.textVOffset(idx);
@@ -3757,7 +4144,7 @@ void InsetTabular::draw(PainterInfo & pi, int x, int y) const
                }
 
                if (r + 1 < tabular.nrows())
-                       yy += tabular.rowDescent(r) + tabular.rowAscent(r + 1) 
+                       yy += tabular.rowDescent(r) + tabular.rowAscent(r + 1)
                                + tabular.interRowSpace(r + 1);
        }
 }
@@ -3765,8 +4152,8 @@ void InsetTabular::draw(PainterInfo & pi, int x, int y) const
 
 void InsetTabular::drawBackground(PainterInfo & pi, int x, int y) const
 {
-       x += scx_ + ADD_TO_TABULAR_WIDTH;
-       y += offset_valign_ - tabular.rowAscent(0);
+       x += ADD_TO_TABULAR_WIDTH;
+       y += tabular.offsetVAlignment() - tabular.rowAscent(0);
        pi.pain.fillRectangle(x, y, tabular.width(), tabular.height(),
                pi.backgroundColor(this));
 }
@@ -3775,9 +4162,8 @@ void InsetTabular::drawBackground(PainterInfo & pi, int x, int y) const
 void InsetTabular::drawSelection(PainterInfo & pi, int x, int y) const
 {
        Cursor & cur = pi.base.bv->cursor();
-       resetPos(cur);
 
-       x += scx_ + ADD_TO_TABULAR_WIDTH;
+       x += ADD_TO_TABULAR_WIDTH;
 
        if (!cur.selection())
                return;
@@ -3804,7 +4190,7 @@ void InsetTabular::drawSelection(PainterInfo & pi, int x, int y) const
                                }
                                int const w = tabular.cellWidth(cell);
                                int const h = tabular.cellHeight(cell);
-                               int const yy = y - tabular.rowAscent(r) + offset_valign_;
+                               int const yy = y - tabular.rowAscent(r) + tabular.offsetVAlignment();
                                if (isCellSelected(cur, r, c))
                                        pi.pain.fillRectangle(xx, yy, w, h, Color_selection);
                                xx += w;
@@ -3814,7 +4200,7 @@ void InsetTabular::drawSelection(PainterInfo & pi, int x, int y) const
                                     + tabular.interRowSpace(r + 1);
                }
 
-       } 
+       }
        // FIXME: This code has no effect because InsetTableCell does not handle
        // drawSelection other than the trivial implementation in Inset.
        //else {
@@ -3825,48 +4211,55 @@ void InsetTabular::drawSelection(PainterInfo & pi, int x, int y) const
 }
 
 
+namespace {
+
+void tabline(PainterInfo const & pi, int x1, int y1, int x2, int y2,
+             bool drawline, bool heavy = false)
+{
+       ColorCode const col = drawline ? Color_tabularline : Color_tabularonoffline;
+       pi.pain.line(x1, y1, x2, y2, pi.textColor(col),
+                                drawline ? Painter::line_solid : Painter::line_onoffdash,
+                                (heavy ? 2 : 1) * Painter::thin_line);
+}
+
+}
+
+
 void InsetTabular::drawCellLines(PainterInfo & pi, int x, int y,
                                 row_type row, idx_type cell) const
 {
        y -= tabular.rowAscent(row);
        int const w = tabular.cellWidth(cell);
        int const h = tabular.cellHeight(cell);
-       Color const linecolor = pi.textColor(Color_tabularline);
-       Color const gridcolor = pi.textColor(Color_tabularonoffline);
 
        // Top
        bool drawline = tabular.topLine(cell)
                || (row > 0 && tabular.bottomLine(tabular.cellAbove(cell)));
-       pi.pain.line(x, y, x + w, y,
-               drawline ? linecolor : gridcolor,
-               drawline ? Painter::line_solid : Painter::line_onoffdash);
+       bool heavy = tabular.use_booktabs && row == 0 && tabular.rowTopLine(row);
+       tabline(pi, x, y, x + w, y, drawline, heavy);
 
        // Bottom
        drawline = tabular.bottomLine(cell);
-       pi.pain.line(x, y + h, x + w, y + h,
-               drawline ? linecolor : gridcolor,
-               drawline ? Painter::line_solid : Painter::line_onoffdash);
+       heavy = tabular.use_booktabs && row == tabular.nrows() - 1
+               && tabular.rowBottomLine(row);
+       tabline(pi, x, y + h, x + w, y + h, drawline, heavy);
 
        // Left
        col_type const col = tabular.cellColumn(cell);
        drawline = tabular.leftLine(cell)
                || (col > 0 && tabular.rightLine(tabular.cellIndex(row, col - 1)));
-       pi.pain.line(x, y, x, y + h,
-               drawline ? linecolor : gridcolor,
-               drawline ? Painter::line_solid : Painter::line_onoffdash);
+       tabline(pi, x, y, x, y + h, drawline);
 
        // Right
        x -= tabular.interColumnSpace(cell);
        col_type next_cell_col = col + 1;
-       while (next_cell_col < tabular.ncols() 
+       while (next_cell_col < tabular.ncols()
                && tabular.isMultiColumn(tabular.cellIndex(row, next_cell_col)))
                next_cell_col++;
        drawline = tabular.rightLine(cell)
                   || (next_cell_col < tabular.ncols()
                       && tabular.leftLine(tabular.cellIndex(row, next_cell_col)));
-       pi.pain.line(x + w, y, x + w, y + h,
-               drawline ? linecolor : gridcolor,
-               drawline ? Painter::line_solid : Painter::line_onoffdash);
+       tabline(pi, x + w, y, x + w, y + h, drawline);
 }
 
 
@@ -3874,7 +4267,6 @@ void InsetTabular::edit(Cursor & cur, bool front, EntryDirection)
 {
        //lyxerr << "InsetTabular::edit: " << this << endl;
        cur.finishUndo();
-       cur.setSelection(false);
        cur.push(*this);
        if (front) {
                if (isRightToLeft(cur))
@@ -3893,7 +4285,6 @@ void InsetTabular::edit(Cursor & cur, bool front, EntryDirection)
        }
        cur.setCurrentFont();
        // FIXME: this accesses the position cache before it is initialized
-       //resetPos(cur);
        //cur.bv().fitCursor();
 }
 
@@ -3924,13 +4315,37 @@ void InsetTabular::updateBuffer(ParIterator const & it, UpdateType utype)
 }
 
 
-void InsetTabular::addToToc(DocIterator const & cpit, bool output_active) const
+void InsetTabular::addToToc(DocIterator const & cpit, bool output_active,
+                                                       UpdateType utype, TocBackend & backend) const
 {
        DocIterator dit = cpit;
        dit.forwardPos();
        size_t const end = dit.nargs();
        for ( ; dit.idx() < end; dit.top().forwardIdx())
-               cell(dit.idx())->addToToc(dit, output_active);
+               cell(dit.idx())->addToToc(dit, output_active, utype, backend);
+}
+
+
+bool InsetTabular::hitSelectRow(BufferView const & bv, int x) const
+{
+       int const x0 = xo(bv) + ADD_TO_TABULAR_WIDTH;
+       return x < x0 || x > x0 + tabular.width();
+}
+
+
+bool InsetTabular::hitSelectColumn(BufferView const & bv, int y) const
+{
+       int const y0 = yo(bv) - tabular.rowAscent(0) + tabular.offsetVAlignment();
+       // FIXME: using ADD_TO_TABULAR_WIDTH is not really correct since
+       // there is no margin added vertically to tabular insets.
+       // However, it works for now.
+       return y < y0 + ADD_TO_TABULAR_WIDTH || y > y0 + tabular.height() - ADD_TO_TABULAR_WIDTH;
+}
+
+
+bool InsetTabular::clickable(BufferView const & bv, int x, int y) const
+{
+       return hitSelectRow(bv, x) || hitSelectColumn(bv, y);
 }
 
 
@@ -3942,45 +4357,46 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        Cursor & bvcur = cur.bv().cursor();
 
        FuncCode const act = cmd.action();
-       
+
        switch (act) {
 
        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()) {
+               if (hitSelectRow(cur.bv(), cmd.x())) {
                        row_type r = rowFromY(cur, cmd.y());
                        cur.idx() = tabular.getFirstCellInRow(r);
+                       cur.pit() = 0;
                        cur.pos() = 0;
                        cur.resetAnchor();
                        cur.idx() = tabular.getLastCellInRow(r);
+                       cur.pit() = cur.lastpit();
                        cur.pos() = cur.lastpos();
-                       cur.setSelection(true);
-                       bvcur = cur; 
+                       cur.selection(true);
+                       bvcur = cur;
                        rowselect_ = true;
                        break;
                }
                // select column
-               int const y0 = yo(cur.bv()) - tabular.rowAscent(0) + offset_valign_;
-               if (cmd.y() < y0 + ADD_TO_TABULAR_WIDTH 
-                       || cmd.y() > y0 + tabular.height()) {
+               if (hitSelectColumn(cur.bv(), cmd.y())) {
                        col_type c = columnFromX(cur, cmd.x());
                        cur.idx() = tabular.cellIndex(0, c);
+                       cur.pit() = 0;
                        cur.pos() = 0;
                        cur.resetAnchor();
                        cur.idx() = tabular.cellIndex(tabular.nrows() - 1, c);
+                       cur.pit() = cur.lastpit();
                        cur.pos() = cur.lastpos();
-                       cur.setSelection(true);
-                       bvcur = cur; 
+                       cur.selection(true);
+                       bvcur = cur;
                        colselect_ = true;
                        break;
                }
                // do not reset cursor/selection if we have selected
                // some cells (bug 2715).
                if (cmd.button() == mouse_button::button3
-                   && &bvcur.selBegin().inset() == this 
-                   && bvcur.selIsMultiCell()) 
+                   && &bvcur.selBegin().inset() == this
+                   && bvcur.selIsMultiCell())
                        ;
                else
                        // Let InsetTableCell do it
@@ -4004,7 +4420,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                                cur.pit() = 0;
                                cur.pos() = 0;
                                bvcur.setCursor(cur);
-                               bvcur.setSelection(true);
+                               bvcur.selection(true);
                                break;
                        }
                        // select (additional) column
@@ -4016,7 +4432,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                                cur.pit() = 0;
                                cur.pos() = 0;
                                bvcur.setCursor(cur);
-                               bvcur.setSelection(true);
+                               bvcur.selection(true);
                                break;
                        }
                        // only update if selection changes
@@ -4025,7 +4441,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                                cur.noScreenUpdate();
                        setCursorFromCoordinates(cur, cmd.x(), cmd.y());
                        bvcur.setCursor(cur);
-                       bvcur.setSelection(true);
+                       bvcur.selection(true);
                        // if this is a multicell selection, we just set the cursor to
                        // the beginning of the cell's text.
                        if (bvcur.selIsMultiCell()) {
@@ -4042,12 +4458,12 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
 
        case LFUN_CELL_BACKWARD:
                movePrevCell(cur);
-               cur.setSelection(false);
+               cur.selection(false);
                break;
 
        case LFUN_CELL_FORWARD:
                moveNextCell(cur);
-               cur.setSelection(false);
+               cur.selection(false);
                break;
 
        case LFUN_CHAR_FORWARD_SELECT:
@@ -4057,7 +4473,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_CHAR_RIGHT_SELECT:
        case LFUN_CHAR_RIGHT:
        case LFUN_CHAR_LEFT_SELECT:
-       case LFUN_CHAR_LEFT: 
+       case LFUN_CHAR_LEFT:
        case LFUN_WORD_FORWARD:
        case LFUN_WORD_FORWARD_SELECT:
        case LFUN_WORD_BACKWARD:
@@ -4066,14 +4482,14 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_WORD_RIGHT_SELECT:
        case LFUN_WORD_LEFT:
        case LFUN_WORD_LEFT_SELECT: {
-               // determine whether we move to next or previous cell, where to enter 
+               // determine whether we move to next or previous cell, where to enter
                // the new cell from, and which command to "finish" (i.e., exit the
                // inset) with:
                bool next_cell;
                EntryDirection entry_from = ENTRY_DIRECTION_IGNORE;
                FuncCode finish_lfun;
 
-               if (act == LFUN_CHAR_FORWARD 
+               if (act == LFUN_CHAR_FORWARD
                                || act == LFUN_CHAR_FORWARD_SELECT
                                || act == LFUN_WORD_FORWARD
                                || act == LFUN_WORD_FORWARD_SELECT) {
@@ -4087,7 +4503,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                        next_cell = false;
                        finish_lfun = LFUN_FINISHED_BACKWARD;
                }
-               // LEFT or RIGHT commands --- the interpretation will depend on the 
+               // LEFT or RIGHT commands --- the interpretation will depend on the
                // table's direction.
                else {
                        bool const right = act == LFUN_CHAR_RIGHT
@@ -4095,14 +4511,14 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                                || act == LFUN_WORD_RIGHT
                                || act == LFUN_WORD_RIGHT_SELECT;
                        next_cell = isRightToLeft(cur) != right;
-                       
+
                        if (lyxrc.visual_cursor)
                                entry_from = right ? ENTRY_DIRECTION_LEFT:ENTRY_DIRECTION_RIGHT;
 
                        finish_lfun = right ? LFUN_FINISHED_RIGHT : LFUN_FINISHED_LEFT;
                }
 
-               bool const select =     act == LFUN_CHAR_FORWARD_SELECT 
+               bool const select =     act == LFUN_CHAR_FORWARD_SELECT
                    || act == LFUN_CHAR_BACKWARD_SELECT
                    || act == LFUN_CHAR_RIGHT_SELECT
                    || act == LFUN_CHAR_LEFT_SELECT
@@ -4111,15 +4527,15 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                        || act == LFUN_WORD_BACKWARD_SELECT
                        || act == LFUN_WORD_LEFT_SELECT;
 
-               // If we have a multicell selection or we're 
+               // If we have a multicell selection or we're
                // not doing some LFUN_*_SELECT thing anyway...
                if (!cur.selIsMultiCell() || !select) {
                        col_type const c = tabular.cellColumn(cur.idx());
                        row_type const r = tabular.cellRow(cur.idx());
-                       // Are we trying to select the whole cell and is the whole cell 
+                       // Are we trying to select the whole cell and is the whole cell
                        // not yet selected?
                        bool const select_whole = select && !isCellSelected(cur, r, c) &&
-                               ((next_cell && cur.pit() == cur.lastpit() 
+                               ((next_cell && cur.pit() == cur.lastpit()
                                && cur.pos() == cur.lastpos())
                                || (!next_cell && cur.pit() == 0 && cur.pos() == 0));
 
@@ -4137,7 +4553,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                                break;
                        }
 
-                       // FIXME: When we support the selection of an empty cell, remove 
+                       // FIXME: When we support the selection of an empty cell, remove
                        // the !empty_cell from this condition. For now we jump to the next
                        // cell if the current cell is empty.
                        if (cur.result().dispatched() && !empty_cell)
@@ -4154,9 +4570,10 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                else
                        movePrevCell(cur, entry_from);
                // if we're exiting the table, call the appropriate FINISHED lfun
-               if (sl == cur.top())
+               if (sl == cur.top()) {
                        cmd = FuncRequest(finish_lfun);
-               else
+                       cur.undispatched();
+               } else
                        cur.dispatched();
 
                cur.screenUpdateFlags(Update::Force | Update::FitCursor);
@@ -4168,7 +4585,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_DOWN:
                if (!(cur.selection() && cur.selIsMultiCell()))
                        cell(cur.idx())->dispatch(cur, cmd);
-               
+
                cur.dispatched(); // override the cell's decision
                if (sl == cur.top()) {
                        // if our Text didn't do anything to the cursor
@@ -4243,55 +4660,21 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                cur.screenUpdateFlags(Update::Force | Update::FitCursor);
                break;
 
-//     case LFUN_SCREEN_DOWN: {
-//             //if (hasSelection())
-//             //      cur.selection() = false;
-//             col_type const col = tabular.cellColumn(cur.idx());
-//             int const t =   cur.bv().top_y() + cur.bv().height();
-//             if (t < yo() + tabular.getHeightOfTabular()) {
-//                     cur.bv().scrollDocView(t, true);
-//                     cur.idx() = tabular.cellBelow(first_visible_cell) + col;
-//             } else {
-//                     cur.idx() = tabular.getFirstCellInRow(tabular.rows() - 1) + col;
-//             }
-//             cur.par() = 0;
-//             cur.pos() = 0;
-//             break;
-//     }
-//
-//     case LFUN_SCREEN_UP: {
-//             //if (hasSelection())
-//             //      cur.selection() = false;
-//             col_type const col = tabular.cellColumn(cur.idx());
-//             int const t =   cur.bv().top_y() + cur.bv().height();
-//             if (yo() < 0) {
-//                     cur.bv().scrollDocView(t, true);
-//                     if (yo() > 0)
-//                             cur.idx() = col;
-//                     else
-//                             cur.idx() = tabular.cellBelow(first_visible_cell) + col;
-//             } else {
-//                     cur.idx() = col;
-//             }
-//             cur.par() = cur.lastpar();
-//             cur.pos() = cur.lastpos();
-//             break;
-//     }
-
        case LFUN_LAYOUT_TABULAR:
                cur.bv().showDialog("tabular");
                break;
 
-       case LFUN_INSET_MODIFY: {
-               string arg;
-               if (cmd.getArg(1) == "from-dialog")
-                       arg = cmd.getArg(0) + to_utf8(cmd.argument().substr(19));
+       case LFUN_INSET_MODIFY:
+               // we come from the dialog
+               if (cmd.getArg(0) == "tabular")
+                       tabularFeatures(cur, cmd.getLongArg(1));
                else
-                       arg = to_utf8(cmd.argument());
-               if (!tabularFeatures(cur, arg))
                        cur.undispatched();
                break;
-       }
+
+       case LFUN_TABULAR_FEATURE:
+               tabularFeatures(cur, to_utf8(cmd.argument()));
+               break;
 
        // insert file functions
        case LFUN_FILE_INSERT_PLAINTEXT_PARA:
@@ -4302,7 +4685,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                                FileName(to_utf8(cmd.argument())));
                        if (tmpstr.empty())
                                break;
-                       cur.recordUndoInset(INSERT_UNDO);
+                       cur.recordUndoInset();
                        if (insertPlaintextString(cur.bv(), tmpstr, false)) {
                                // content has been replaced,
                                // so cursor might be invalid
@@ -4317,7 +4700,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_CUT:
                if (cur.selIsMultiCell()) {
                        if (copySelection(cur)) {
-                               cur.recordUndoInset(DELETE_UNDO);
+                               cur.recordUndoInset();
                                cutSelection(cur);
                        }
                } else
@@ -4326,14 +4709,14 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
 
        case LFUN_SELF_INSERT:
                if (cur.selIsMultiCell()) {
-                       cur.recordUndoInset(DELETE_UNDO);
+                       cur.recordUndoInset();
                        cutSelection(cur);
                        BufferView * bv = &cur.bv();
                        docstring::const_iterator cit = cmd.argument().begin();
                        docstring::const_iterator const end = cmd.argument().end();
                        for (; cit != end; ++cit)
                                bv->translateAndInsert(*cit, getText(cur.idx()), cur);
-       
+
                        cur.resetAnchor();
                        bv->bookmarkEditPosition();
                } else
@@ -4343,7 +4726,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_CHAR_DELETE_BACKWARD:
        case LFUN_CHAR_DELETE_FORWARD:
                if (cur.selIsMultiCell()) {
-                       cur.recordUndoInset(DELETE_UNDO);
+                       cur.recordUndoInset();
                        cutSelection(cur);
                } else
                        cell(cur.idx())->dispatch(cur, cmd);
@@ -4369,7 +4752,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                // pass to InsertPlaintextString, but
                // only if we have multi-cell content
                if (clip.find_first_of(from_ascii("\t\n")) != docstring::npos) {
-                       cur.recordUndoInset(INSERT_UNDO);
+                       cur.recordUndoInset();
                        if (insertPlaintextString(cur.bv(), clip, false)) {
                                // content has been replaced,
                                // so cursor might be invalid
@@ -4391,7 +4774,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                        break;
                }
                if (theClipboard().isInternal()) {
-                       cur.recordUndoInset(INSERT_UNDO);
+                       cur.recordUndoInset();
                        pasteClipboard(cur);
                }
                break;
@@ -4410,6 +4793,7 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
        case LFUN_FONT_SIZE:
        case LFUN_FONT_UNDERLINE:
        case LFUN_FONT_STRIKEOUT:
+       case LFUN_FONT_CROSSOUT:
        case LFUN_FONT_UNDERUNDERLINE:
        case LFUN_FONT_UNDERWAVE:
        case LFUN_LANGUAGE:
@@ -4458,25 +4842,9 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
 }
 
 
-// function sets an object as defined in func_status.h:
-// states OK, Unknown, Disabled, On, Off.
-bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
-       FuncStatus & status) const
+bool InsetTabular::getFeatureStatus(Cursor & cur, string const & s,
+                      string const & argument, FuncStatus & status) const
 {
-       switch (cmd.action()) {
-       case LFUN_INSET_MODIFY: {
-               if (&cur.inset() != this || cmd.getArg(0) != "tabular") 
-                       break;
-
-               // FIXME: We only check for the very first argument...
-               string const s = cmd.getArg(1);
-               // We always enable the lfun if it is coming from the dialog
-               // because the dialog makes sure all the settings are valid,
-               // even though the first argument might not be valid now.
-               if (s == "from-dialog") {
-                       status.setEnabled(true);
-                       return true;
-               }
 
                int action = Tabular::LAST_ACTION;
                int i = 0;
@@ -4492,8 +4860,6 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        return true;
                }
 
-               string const argument = cmd.getLongArg(2);
-
                row_type sel_row_start = 0;
                row_type sel_row_end = 0;
                col_type sel_col_start = 0;
@@ -4506,6 +4872,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                switch (action) {
                case Tabular::SET_PWIDTH:
                case Tabular::SET_MPWIDTH:
+               case Tabular::TOGGLE_VARWIDTH_COLUMN:
                case Tabular::SET_SPECIAL_COLUMN:
                case Tabular::SET_SPECIAL_MULTICOLUMN:
                case Tabular::APPEND_ROW:
@@ -4521,7 +4888,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        return true;
 
                case Tabular::SET_TABULAR_WIDTH:
-                       status.setEnabled(!tabular.rotate &&  !tabular.is_long_tabular
+                       status.setEnabled(!tabular.rotate
                                && tabular.tabular_valignment == Tabular::LYX_VALIGN_MIDDLE);
                        break;
 
@@ -4608,9 +4975,13 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
 
                case Tabular::SET_LINE_TOP:
                case Tabular::SET_LINE_BOTTOM:
+                       status.setEnabled(!tabular.ltCaption(tabular.cellRow(cur.idx())));
+                       break;
+
                case Tabular::SET_LINE_LEFT:
                case Tabular::SET_LINE_RIGHT:
-                       status.setEnabled(!tabular.ltCaption(tabular.cellRow(cur.idx())));
+                       status.setEnabled(!tabular.use_booktabs
+                                         && !tabular.ltCaption(tabular.cellRow(cur.idx())));
                        break;
 
                case Tabular::TOGGLE_LINE_TOP:
@@ -4624,12 +4995,14 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        break;
 
                case Tabular::TOGGLE_LINE_LEFT:
-                       status.setEnabled(!tabular.ltCaption(tabular.cellRow(cur.idx())));
+                       status.setEnabled(!tabular.use_booktabs
+                                         && !tabular.ltCaption(tabular.cellRow(cur.idx())));
                        status.setOnOff(tabular.leftLine(cur.idx()));
                        break;
 
                case Tabular::TOGGLE_LINE_RIGHT:
-                       status.setEnabled(!tabular.ltCaption(tabular.cellRow(cur.idx())));
+                       status.setEnabled(!tabular.use_booktabs
+                                         && !tabular.ltCaption(tabular.cellRow(cur.idx())));
                        status.setOnOff(tabular.rightLine(cur.idx()));
                        break;
 
@@ -4638,22 +5011,25 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                // therefore allow always left but right and center only if there is no width
                case Tabular::M_ALIGN_LEFT:
                        flag = false;
+                       // fall through
                case Tabular::ALIGN_LEFT:
                        status.setOnOff(tabular.getAlignment(cur.idx(), flag) == LYX_ALIGN_LEFT);
                        break;
 
                case Tabular::M_ALIGN_RIGHT:
                        flag = false;
+                       // fall through
                case Tabular::ALIGN_RIGHT:
-                       status.setEnabled(!(tabular.isMultiRow(cur.idx()) 
+                       status.setEnabled(!(tabular.isMultiRow(cur.idx())
                                && !tabular.getPWidth(cur.idx()).zero()));
                        status.setOnOff(tabular.getAlignment(cur.idx(), flag) == LYX_ALIGN_RIGHT);
                        break;
 
                case Tabular::M_ALIGN_CENTER:
                        flag = false;
+                       // fall through
                case Tabular::ALIGN_CENTER:
-                       status.setEnabled(!(tabular.isMultiRow(cur.idx()) 
+                       status.setEnabled(!(tabular.isMultiRow(cur.idx())
                                && !tabular.getPWidth(cur.idx()).zero()));
                        status.setOnOff(tabular.getAlignment(cur.idx(), flag) == LYX_ALIGN_CENTER);
                        break;
@@ -4665,13 +5041,14 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        break;
 
                case Tabular::ALIGN_DECIMAL:
-                       status.setEnabled(!tabular.isMultiRow(cur.idx()) 
+                       status.setEnabled(!tabular.isMultiRow(cur.idx())
                                && !tabular.isMultiColumn(cur.idx()));
                        status.setOnOff(tabular.getAlignment(cur.idx(), true) == LYX_ALIGN_DECIMAL);
                        break;
 
                case Tabular::M_VALIGN_TOP:
                        flag = false;
+                       // fall through
                case Tabular::VALIGN_TOP:
                        status.setEnabled(!tabular.getPWidth(cur.idx()).zero()
                                && !tabular.isMultiRow(cur.idx()));
@@ -4681,6 +5058,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
 
                case Tabular::M_VALIGN_BOTTOM:
                        flag = false;
+                       // fall through
                case Tabular::VALIGN_BOTTOM:
                        status.setEnabled(!tabular.getPWidth(cur.idx()).zero()
                                && !tabular.isMultiRow(cur.idx()));
@@ -4690,6 +5068,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
 
                case Tabular::M_VALIGN_MIDDLE:
                        flag = false;
+                       // fall through
                case Tabular::VALIGN_MIDDLE:
                        status.setEnabled(!tabular.getPWidth(cur.idx()).zero()
                                && !tabular.isMultiRow(cur.idx()));
@@ -4698,6 +5077,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        break;
 
                case Tabular::SET_LONGTABULAR:
+               case Tabular::TOGGLE_LONGTABULAR:
                        // setting as longtable is not allowed when table is inside a float
                        if (cur.innerInsetOfType(FLOAT_CODE) != 0
                                || cur.innerInsetOfType(WRAP_CODE) != 0)
@@ -4718,30 +5098,30 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
 
                case Tabular::TABULAR_VALIGN_TOP:
                        status.setEnabled(tabular.tabular_width.zero());
-                       status.setOnOff(tabular.tabular_valignment 
+                       status.setOnOff(tabular.tabular_valignment
                                == Tabular::LYX_VALIGN_TOP);
                        break;
                case Tabular::TABULAR_VALIGN_MIDDLE:
                        status.setEnabled(tabular.tabular_width.zero());
-                       status.setOnOff(tabular.tabular_valignment 
+                       status.setOnOff(tabular.tabular_valignment
                                == Tabular::LYX_VALIGN_MIDDLE);
                        break;
                case Tabular::TABULAR_VALIGN_BOTTOM:
                        status.setEnabled(tabular.tabular_width.zero());
-                       status.setOnOff(tabular.tabular_valignment 
+                       status.setOnOff(tabular.tabular_valignment
                                == Tabular::LYX_VALIGN_BOTTOM);
                        break;
 
                case Tabular::LONGTABULAR_ALIGN_LEFT:
-                       status.setOnOff(tabular.longtabular_alignment 
+                       status.setOnOff(tabular.longtabular_alignment
                                == Tabular::LYX_LONGTABULAR_ALIGN_LEFT);
                        break;
                case Tabular::LONGTABULAR_ALIGN_CENTER:
-                       status.setOnOff(tabular.longtabular_alignment 
+                       status.setOnOff(tabular.longtabular_alignment
                                == Tabular::LYX_LONGTABULAR_ALIGN_CENTER);
                        break;
                case Tabular::LONGTABULAR_ALIGN_RIGHT:
-                       status.setOnOff(tabular.longtabular_alignment 
+                       status.setOnOff(tabular.longtabular_alignment
                                == Tabular::LYX_LONGTABULAR_ALIGN_RIGHT);
                        break;
 
@@ -4846,6 +5226,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        status.setOnOff(tabular.ltCaption(sel_row_start));
                        break;
 
+               case Tabular::TOGGLE_BOOKTABS:
                case Tabular::SET_BOOKTABS:
                        status.setOnOff(tabular.use_booktabs);
                        break;
@@ -4860,6 +5241,39 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        break;
                }
                return true;
+}
+
+
+// function sets an object as defined in FuncStatus.h:
+// states OK, Unknown, Disabled, On, Off.
+bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
+                             FuncStatus & status) const
+{
+       switch (cmd.action()) {
+       case LFUN_INSET_MODIFY:
+               if (cmd.getArg(0) != "tabular")
+                       break;
+               if (cmd.getArg(1) == "for-dialog") {
+                       // The dialog is asking the status of a command
+                       if (&cur.inset() != this)
+                               break;
+                       string action = cmd.getArg(2);
+                       string arg = cmd.getLongArg(3);
+                       return getFeatureStatus(cur, action, arg, status);
+               } else {
+                       // We always enable the lfun if it is coming from the dialog
+                       // because the dialog makes sure all the settings are valid,
+                       // even though the first argument might not be valid now.
+                       status.setEnabled(true);
+                       return true;
+               }
+
+       case LFUN_TABULAR_FEATURE: {
+               if (&cur.inset() != this)
+                       break;
+               string action = cmd.getArg(0);
+               string arg = cmd.getLongArg(1);
+               return getFeatureStatus(cur, action, arg, status);
        }
 
        case LFUN_CAPTION_INSERT: {
@@ -4876,7 +5290,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                }
                // check if there is already a caption
                bool have_caption = false;
-               InsetTableCell itc = InsetTableCell(*tabular.cellInset(cur.idx()).get());
+               InsetTableCell itc = InsetTableCell(*tabular.cellInset(cur.idx()));
                ParagraphList::const_iterator pit = itc.paragraphs().begin();
                ParagraphList::const_iterator pend = itc.paragraphs().end();
                for (; pit != pend; ++pit) {
@@ -4933,13 +5347,14 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                        status.setEnabled(false);
                        return true;
                }
-       case LFUN_NEWLINE_INSERT: {
-               if (tabular.getPWidth(cur.idx()).zero()) {
+               // fall through
+       case LFUN_NEWLINE_INSERT:
+               if ((tabular.isMultiColumn(cur.idx()) || tabular.isMultiRow(cur.idx()))
+                   && tabular.getPWidth(cur.idx()).zero()) {
                        status.setEnabled(false);
                        return true;
-               } else
-                       return cell(cur.idx())->getStatus(cur, cmd, status);
-       }
+               }
+               return cell(cur.idx())->getStatus(cur, cmd, status);
 
        case LFUN_NEWPAGE_INSERT:
                status.setEnabled(false);
@@ -4953,7 +5368,7 @@ bool InsetTabular::getStatus(Cursor & cur, FuncRequest const & cmd,
                                getSelection(cur, rs, re, cs, ce);
                                if (paste_tabular && paste_tabular->ncols() == ce - cs + 1
                                          && paste_tabular->nrows() == re - rs + 1)
-                                       status.setEnabled(true);        
+                                       status.setEnabled(true);
                                else {
                                        status.setEnabled(false);
                                        status.message(_("Selection size should match clipboard content."));
@@ -5046,13 +5461,7 @@ docstring InsetTabular::xhtml(XHTMLStream & xs, OutputParams const & rp) const
 void InsetTabular::validate(LaTeXFeatures & features) const
 {
        tabular.validate(features);
-       // FIXME XHTML
-       // It'd be better to be able to get this from an InsetLayout, but at present
-       // InsetLayouts do not seem really to work for things that aren't InsetTexts.
-       if (features.runparams().flavor == OutputParams::HTML)
-               features.addCSSSnippet(
-                       "table { border: 1px solid black; display: inline-block; }\n"
-                       "td { border: 1px solid black; padding: 0.5ex; }");
+       features.useInsetLayout(getLayout());
 }
 
 
@@ -5076,13 +5485,12 @@ void InsetTabular::cursorPos(BufferView const & bv,
        // y offset     correction
        y += cellYPos(sl.idx());
        y += tabular.textVOffset(sl.idx());
-       y += offset_valign_;
+       y += tabular.offsetVAlignment();
 
        // x offset correction
        x += cellXPos(sl.idx());
        x += tabular.textHOffset(sl.idx());
        x += ADD_TO_TABULAR_WIDTH;
-       x += scx_;
 }
 
 
@@ -5120,10 +5528,8 @@ int InsetTabular::dist(BufferView & bv, idx_type const cell, int x, int y) const
 Inset * InsetTabular::editXY(Cursor & cur, int x, int y)
 {
        //lyxerr << "InsetTabular::editXY: " << this << endl;
-       cur.setSelection(false);
        cur.push(*this);
        cur.idx() = getNearestCell(cur.bv(), x, y);
-       resetPos(cur);
        return cur.bv().textMetrics(&cell(cur.idx())->text()).editXY(cur, x, y);
 }
 
@@ -5157,7 +5563,7 @@ int InsetTabular::cellYPos(idx_type const cell) const
        row_type row = tabular.cellRow(cell);
        int ly = 0;
        for (row_type r = 0; r < row; ++r)
-               ly += tabular.rowDescent(r) + tabular.rowAscent(r + 1) 
+               ly += tabular.rowDescent(r) + tabular.rowAscent(r + 1)
                        + tabular.interRowSpace(r + 1);
        return ly;
 }
@@ -5173,36 +5579,6 @@ int InsetTabular::cellXPos(idx_type const cell) const
 }
 
 
-void InsetTabular::resetPos(Cursor & cur) const
-{
-       BufferView & bv = cur.bv();
-       int const maxwidth = bv.workWidth();
-
-       int const scx_old = scx_;
-       int const i = cur.find(this);
-       if (i == -1) {
-               scx_ = 0;
-       } else {
-               int const X1 = 0;
-               int const X2 = maxwidth;
-               int const offset = ADD_TO_TABULAR_WIDTH + 2;
-               int const x1 = xo(cur.bv()) + cellXPos(cur[i].idx()) + offset;
-               int const x2 = x1 + tabular.cellWidth(cur[i].idx());
-
-               if (x1 < X1)
-                       scx_ = X1 + 20 - x1;
-               else if (x2 > X2)
-                       scx_ = X2 - 20 - x2;
-               else
-                       scx_ = 0;
-       }
-
-       // only update if offset changed
-       if (scx_ != scx_old)
-               cur.screenUpdateFlags(Update::Force | Update::FitCursor);
-}
-
-
 void InsetTabular::moveNextCell(Cursor & cur, EntryDirection entry_from)
 {
        row_type const row = tabular.cellRow(cur.idx());
@@ -5237,7 +5613,6 @@ void InsetTabular::moveNextCell(Cursor & cur, EntryDirection entry_from)
        if (cur.selIsMultiCell()) {
                cur.pit() = cur.lastpit();
                cur.pos() = cur.lastpos();
-               resetPos(cur);
                return;
        }
 
@@ -5245,7 +5620,7 @@ void InsetTabular::moveNextCell(Cursor & cur, EntryDirection entry_from)
        cur.pos() = 0;
 
        // in visual mode, place cursor at extreme left or right
-       
+
        switch(entry_from) {
 
        case ENTRY_DIRECTION_RIGHT:
@@ -5260,7 +5635,6 @@ void InsetTabular::moveNextCell(Cursor & cur, EntryDirection entry_from)
 
        }
        cur.setCurrentFont();
-       resetPos(cur);
 }
 
 
@@ -5295,7 +5669,6 @@ void InsetTabular::movePrevCell(Cursor & cur, EntryDirection entry_from)
        if (cur.selIsMultiCell()) {
                cur.pit() = cur.lastpit();
                cur.pos() = cur.lastpos();
-               resetPos(cur);
                return;
        }
 
@@ -5303,7 +5676,7 @@ void InsetTabular::movePrevCell(Cursor & cur, EntryDirection entry_from)
        cur.pos() = cur.lastpos();
 
        // in visual mode, place cursor at extreme left or right
-       
+
        switch(entry_from) {
 
        case ENTRY_DIRECTION_RIGHT:
@@ -5318,27 +5691,26 @@ void InsetTabular::movePrevCell(Cursor & cur, EntryDirection entry_from)
 
        }
        cur.setCurrentFont();
-       resetPos(cur);
 }
 
 
-bool InsetTabular::tabularFeatures(Cursor & cur, string const & argument)
+void InsetTabular::tabularFeatures(Cursor & cur, string const & argument)
 {
+       cur.recordUndoInset(this);
+
        istringstream is(argument);
+       // limit the size of strings we read to avoid memory problems
+       is >> setw(65636);
        string s;
-       is >> s;
-       if (insetCode(s) != TABULAR_CODE)
-               return false;
-
        // Safe guard.
        size_t safe_guard = 0;
        for (;;) {
                if (is.eof())
-                       break;
+                       return;
                safe_guard++;
                if (safe_guard > 1000) {
                        LYXERR0("parameter max count reached!");
-                       break;
+                       return;
                }
                is >> s;
                Tabular::Feature action = Tabular::LAST_ACTION;
@@ -5360,7 +5732,6 @@ bool InsetTabular::tabularFeatures(Cursor & cur, string const & argument)
                LYXERR(Debug::DEBUG, "Feature: " << s << "\t\tvalue: " << val);
                tabularFeatures(cur, action, val);
        }
-       return true;
 }
 
 
@@ -5385,7 +5756,7 @@ static void checkLongtableSpecial(Tabular::ltType & ltt,
 
 bool InsetTabular::oneCellHasRotationState(bool rotated,
                row_type row_start, row_type row_end,
-               col_type col_start, col_type col_end) const 
+               col_type col_start, col_type col_end) const
 {
        for (row_type r = row_start; r <= row_end; ++r)
                for (col_type c = col_start; c <= col_end; ++c)
@@ -5433,10 +5804,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                break;
 
        case Tabular::ALIGN_DECIMAL:
-               if (tabular.column_info[tabular.cellColumn(cur.idx())].alignment == LYX_ALIGN_DECIMAL)
-                       setAlign = LYX_ALIGN_CENTER;
-               else
-                       setAlign = LYX_ALIGN_DECIMAL;
+               setAlign = LYX_ALIGN_DECIMAL;
                break;
 
        case Tabular::M_VALIGN_TOP:
@@ -5458,8 +5826,6 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                break;
        }
 
-       cur.recordUndoInset(ATOMIC_UNDO, this);
-
        getSelection(cur, sel_row_start, sel_row_end, sel_col_start, sel_col_end);
        row_type const row = tabular.cellRow(cur.idx());
        col_type const column = tabular.cellColumn(cur.idx());
@@ -5474,10 +5840,12 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::SET_PWIDTH: {
                Length const len(value);
-               tabular.setColumnPWidth(cur, cur.idx(), len);
-               if (len.zero()
-                   && tabular.getAlignment(cur.idx(), true) == LYX_ALIGN_BLOCK)
-                       tabularFeatures(cur, Tabular::ALIGN_CENTER, string());
+               for (col_type c = sel_col_start; c <= sel_col_end; ++c) {
+                       tabular.setColumnPWidth(cur, tabular.cellIndex(row, c), len);
+                       if (len.zero()
+                           && tabular.getAlignment(tabular.cellIndex(row, c), true) == LYX_ALIGN_BLOCK)
+                               tabularFeatures(cur, Tabular::ALIGN_CENTER, string());
+               }
                break;
        }
 
@@ -5485,6 +5853,13 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                tabular.setMColumnPWidth(cur, cur.idx(), Length(value));
                break;
 
+       case Tabular::TOGGLE_VARWIDTH_COLUMN: {
+               bool const varwidth = value == "on";
+               for (col_type c = sel_col_start; c <= sel_col_end; ++c)
+                       tabular.toggleVarwidth(tabular.cellIndex(row, c), varwidth);
+               break;
+       }
+
        case Tabular::SET_MROFFSET:
                tabular.setMROffset(cur, cur.idx(), Length(value));
                break;
@@ -5522,7 +5897,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                cur.idx() = tabular.cellIndex(sel_row_start, column);
                cur.pit() = 0;
                cur.pos() = 0;
-               cur.setSelection(false);
+               cur.selection(false);
                break;
 
        case Tabular::DELETE_COLUMN:
@@ -5545,7 +5920,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                cur.idx() = tabular.cellIndex(row, sel_col_start);
                cur.pit() = 0;
                cur.pos() = 0;
-               cur.setSelection(false);
+               cur.selection(false);
                break;
 
        case Tabular::COPY_ROW:
@@ -5635,6 +6010,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
        case Tabular::M_VALIGN_BOTTOM:
        case Tabular::M_VALIGN_MIDDLE:
                flag = false;
+               // fall through
        case Tabular::VALIGN_TOP:
        case Tabular::VALIGN_BOTTOM:
        case Tabular::VALIGN_MIDDLE:
@@ -5648,7 +6024,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                        // just multicol for one single cell
                        // check whether we are completely in a multicol
                        if (!tabular.isMultiColumn(cur.idx()))
-                               tabular.setMultiColumn(cur.idx(), 1,
+                               tabular.setMultiColumn(cur, cur.idx(), 1,
                                        tabular.rightLine(cur.idx()));
                        break;
                }
@@ -5657,11 +6033,11 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                idx_type const s_start = cur.selBegin().idx();
                row_type const col_start = tabular.cellColumn(s_start);
                row_type const col_end = tabular.cellColumn(cur.selEnd().idx());
-               cur.idx() = tabular.setMultiColumn(s_start, col_end - col_start + 1,
+               cur.idx() = tabular.setMultiColumn(cur, s_start, col_end - col_start + 1,
                                                   tabular.rightLine(cur.selEnd().idx()));
                cur.pit() = 0;
                cur.pos() = 0;
-               cur.setSelection(false);
+               cur.selection(false);
                break;
        }
 
@@ -5703,8 +6079,9 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                        // just multirow for one single cell
                        // check whether we are completely in a multirow
                        if (!tabular.isMultiRow(cur.idx()))
-                               tabular.setMultiRow(cur.idx(), 1,
-                                                   tabular.bottomLine(cur.idx()));
+                               tabular.setMultiRow(cur, cur.idx(), 1,
+                                                   tabular.bottomLine(cur.idx()),
+                                                   tabular.getAlignment(cur.idx()));
                        break;
                }
                // we have a selection so this means we just add all this
@@ -5712,11 +6089,12 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                idx_type const s_start = cur.selBegin().idx();
                row_type const row_start = tabular.cellRow(s_start);
                row_type const row_end = tabular.cellRow(cur.selEnd().idx());
-               cur.idx() = tabular.setMultiRow(s_start, row_end - row_start + 1,
-                                               tabular.bottomLine(cur.selEnd().idx()));
+               cur.idx() = tabular.setMultiRow(cur, s_start, row_end - row_start + 1,
+                                               tabular.bottomLine(cur.selEnd().idx()),
+                                               tabular.getAlignment(cur.selEnd().idx()));
                cur.pit() = 0;
                cur.pos() = 0;
-               cur.setSelection(false);
+               cur.selection(false);
                break;
        }
 
@@ -5755,6 +6133,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::SET_ALL_LINES:
                setLines = true;
+               // fall through
        case Tabular::UNSET_ALL_LINES:
                for (row_type r = sel_row_start; r <= sel_row_end; ++r)
                        for (col_type c = sel_col_start; c <= sel_col_end; ++c) {
@@ -5777,6 +6156,13 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                }
                break;
 
+       case Tabular::TOGGLE_LONGTABULAR:
+               if (tabular.is_long_tabular)
+                       tabularFeatures(cur, Tabular::UNSET_LONGTABULAR);
+               else
+                       tabular.is_long_tabular = true;
+               break;
+
        case Tabular::SET_LONGTABULAR:
                tabular.is_long_tabular = true;
                break;
@@ -5830,8 +6216,6 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                tabular.longtabular_alignment = Tabular::LYX_LONGTABULAR_ALIGN_RIGHT;
                break;
 
-               
-
        case Tabular::SET_ROTATE_CELL:
                for (row_type r = sel_row_start; r <= sel_row_end; ++r)
                        for (col_type c = sel_col_start; c <= sel_col_end; ++c)
@@ -5872,6 +6256,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::UNSET_LTFIRSTHEAD:
                flag = false;
+               // fall through
        case Tabular::SET_LTFIRSTHEAD:
                tabular.getRowOfLTFirstHead(row, ltt);
                checkLongtableSpecial(ltt, value, flag);
@@ -5880,6 +6265,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::UNSET_LTHEAD:
                flag = false;
+               // fall through
        case Tabular::SET_LTHEAD:
                tabular.getRowOfLTHead(row, ltt);
                checkLongtableSpecial(ltt, value, flag);
@@ -5888,6 +6274,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::UNSET_LTFOOT:
                flag = false;
+               // fall through
        case Tabular::SET_LTFOOT:
                tabular.getRowOfLTFoot(row, ltt);
                checkLongtableSpecial(ltt, value, flag);
@@ -5896,6 +6283,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::UNSET_LTLASTFOOT:
                flag = false;
+               // fall through
        case Tabular::SET_LTLASTFOOT:
                tabular.getRowOfLTLastFoot(row, ltt);
                checkLongtableSpecial(ltt, value, flag);
@@ -5904,6 +6292,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
 
        case Tabular::UNSET_LTNEWPAGE:
                flag = false;
+               // fall through
        case Tabular::SET_LTNEWPAGE:
                tabular.setLTNewPage(row, flag);
                break;
@@ -5911,24 +6300,26 @@ void InsetTabular::tabularFeatures(Cursor & cur,
        case Tabular::SET_LTCAPTION: {
                if (tabular.ltCaption(row))
                        break;
-               cur.idx() = tabular.setLTCaption(row, true);
+               cur.idx() = tabular.setLTCaption(cur, row, true);
                cur.pit() = 0;
                cur.pos() = 0;
-               cur.setSelection(false);
+               cur.selection(false);
                // If a row is set as caption, then also insert
                // a caption. Otherwise the LaTeX output is broken.
-               lyx::dispatch(FuncRequest(LFUN_INSET_SELECT_ALL));
+               // Select cell if it is non-empty
+               if (cur.lastpos() > 0 || cur.lastpit() > 0)
+                       lyx::dispatch(FuncRequest(LFUN_INSET_SELECT_ALL));
                lyx::dispatch(FuncRequest(LFUN_CAPTION_INSERT));
                break;
        }
-       
+
        case Tabular::UNSET_LTCAPTION: {
                if (!tabular.ltCaption(row))
                        break;
-               cur.idx() = tabular.setLTCaption(row, false);
+               cur.idx() = tabular.setLTCaption(cur, row, false);
                cur.pit() = 0;
                cur.pos() = 0;
-               cur.setSelection(false);
+               cur.selection(false);
                FuncRequest fr(LFUN_INSET_DISSOLVE, "caption");
                if (lyx::getStatus(fr).enabled())
                        lyx::dispatch(fr);
@@ -5943,6 +6334,10 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                break;
        }
 
+       case Tabular::TOGGLE_BOOKTABS:
+               tabular.use_booktabs = !tabular.use_booktabs;
+               break;
+
        case Tabular::SET_BOOKTABS:
                tabular.use_booktabs = true;
                break;
@@ -6100,6 +6495,7 @@ bool InsetTabular::pasteClipboard(Cursor & cur)
                        inset->setChange(Change(buffer().params().track_changes ?
                                                Change::INSERTED : Change::UNCHANGED));
                        cur.pos() = 0;
+                       cur.pit() = 0;
                }
        }
        return true;
@@ -6146,7 +6542,7 @@ bool InsetTabular::isRightToLeft(Cursor & cur) const
 }
 
 
-docstring InsetTabular::asString(idx_type stidx, idx_type enidx, 
+docstring InsetTabular::asString(idx_type stidx, idx_type enidx,
                                  bool intoInsets)
 {
        LASSERT(stidx <= enidx, return docstring());
@@ -6176,12 +6572,12 @@ void InsetTabular::getSelection(Cursor & cur,
        cs = tabular.cellColumn(beg.idx());
        ce = tabular.cellColumn(end.idx());
        if (cs > ce)
-               swap(cs, ce);
+               std::swap(cs, ce);
 
        rs = tabular.cellRow(beg.idx());
        re = tabular.cellRow(end.idx());
        if (rs > re)
-               swap(rs, re);
+               std::swap(rs, re);
 }
 
 
@@ -6220,7 +6616,7 @@ bool InsetTabular::allowParagraphCustomization(idx_type cell) const
 
 bool InsetTabular::forcePlainLayout(idx_type cell) const
 {
-       return !tabular.getPWidth(cell).zero();
+       return tabular.isMultiColumn(cell) && !tabular.getPWidth(cell).zero();
 }
 
 
@@ -6259,7 +6655,6 @@ bool InsetTabular::insertPlaintextString(BufferView & bv, docstring const & buf,
        if (usePaste) {
                paste_tabular.reset(new Tabular(buffer_, rows, maxCols));
                loctab = paste_tabular.get();
-               cols = 0;
                dirtyTabularStack(true);
        } else {
                loctab = &tabular;
@@ -6393,7 +6788,7 @@ bool InsetTabular::insertCompletion(Cursor & cur, docstring const & s, bool fini
 }
 
 
-void InsetTabular::completionPosAndDim(Cursor const & cur, int & x, int & y, 
+void InsetTabular::completionPosAndDim(Cursor const & cur, int & x, int & y,
                                    Dimension & dim) const
 {
        TextMetrics const & tm = cur.bv().textMetrics(cur.text());
@@ -6439,4 +6834,22 @@ string InsetTabular::params2string(InsetTabular const & inset)
 }
 
 
+void InsetTabular::setLayoutForHiddenCells(DocumentClass const & dc)
+{
+       for (Tabular::col_type c = 0; c < tabular.ncols(); ++c) {
+               for (Tabular::row_type r = 0; r < tabular.nrows(); ++r) {
+                       if (!tabular.isPartOfMultiColumn(r,c) &&
+                           !tabular.isPartOfMultiRow(r,c))
+                               continue;
+
+                       ParagraphList & parlist = tabular.cellInset(r,c)->paragraphs();
+                       ParagraphList::iterator it = parlist.begin();
+                       ParagraphList::iterator const en = parlist.end();
+                       for (; it != en; ++it)
+                                       it->setLayout(dc.plainLayout());
+               }
+       }
+}
+
+
 } // namespace lyx