]> git.lyx.org Git - lyx.git/blobdiff - src/insets/InsetTabular.cpp
Circumvent odd stmary font metrics (part of #9990).
[lyx.git] / src / insets / InsetTabular.cpp
index e827bfd27e5aafad2d153404095b694ddeca0c7d..6d8f7d0737138f45d5a1b6183e0290c2f96b66d9 100644 (file)
@@ -110,6 +110,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,6 +153,7 @@ 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 },
@@ -177,6 +180,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 },
@@ -2676,14 +2680,16 @@ 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();
+       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";
@@ -2812,6 +2818,9 @@ void Tabular::latex(otexstream & os, OutputParams const & runparams) const
 
        if (rotate != 0)
                os << breakln << "\\end{turn}";
+
+       if (!TexRow::isNone(pos))
+               os.texrow().start(pos);
 }
 
 
@@ -3960,9 +3969,11 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                        || cmd.x() > xo(cur.bv()) + tabular.width()) {
                        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;
@@ -3975,9 +3986,11 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                        || cmd.y() > y0 + tabular.height()) {
                        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;
@@ -4291,16 +4304,17 @@ void InsetTabular::doDispatch(Cursor & cur, FuncRequest & cmd)
                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:
@@ -4467,25 +4481,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;
@@ -4501,8 +4499,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;
@@ -4617,9 +4613,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:
@@ -4633,12 +4633,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;
 
@@ -4707,6 +4709,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)
@@ -4855,6 +4858,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;
@@ -4869,6 +4873,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: {
@@ -5296,23 +5333,21 @@ void InsetTabular::movePrevCell(Cursor & cur, EntryDirection entry_from)
 }
 
 
-bool InsetTabular::tabularFeatures(Cursor & cur, string const & argument)
+void InsetTabular::tabularFeatures(Cursor & cur, string const & argument)
 {
+       cur.recordUndoInset(this);
+
        istringstream is(argument);
        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;
@@ -5334,7 +5369,6 @@ bool InsetTabular::tabularFeatures(Cursor & cur, string const & argument)
                LYXERR(Debug::DEBUG, "Feature: " << s << "\t\tvalue: " << val);
                tabularFeatures(cur, action, val);
        }
-       return true;
 }
 
 
@@ -5429,8 +5463,6 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                break;
        }
 
-       cur.recordUndoInset(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());
@@ -5750,6 +5782,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;
@@ -5918,6 +5957,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;