Add tabular-features set-inner-lines and fix set-all-lines behavior

Patch by Daniel Ramöller (racoon)

Fixes: #11382

diff --git a/lib/Makefile.am b/lib/Makefile.am
index 3816d5c7279e763d19a0056a372e20dac6558efa..0db9ad8cd95f181cd72bde47982120136430fa91 100644 (file)
--- a/lib/Makefile.am
+++ b/lib/Makefile.am
@@ -569,6 +569,7 @@ dist_images_DATA1X = \
        images/tabular-feature_set-longtabular.svgz \
        images/tabular-feature_set-rotate-cell.svgz \
        images/tabular-feature_set-border-lines.svgz \
+       images/tabular-feature_set-inner-lines.svgz \
        images/tabular-feature_set-rotate-tabular.svgz \
        images/tabular-feature_toggle-line-bottom.svgz \
        images/tabular-feature_toggle-line-left.svgz \
@@ -1848,6 +1849,7 @@ dist_imagesoxygen_DATA1X = \
        images/oxygen/tabular-feature_multirow.svgz \
        images/oxygen/tabular-feature_set-all-lines.svgz \
        images/oxygen/tabular-feature_set-border-lines.svgz \
+       images/oxygen/tabular-feature_set-inner-lines.svgz \
        images/oxygen/tabular-feature_set-longtabular.svgz \
        images/oxygen/tabular-feature_toggle-line-bottom.svgz \
        images/oxygen/tabular-feature_toggle-line-left.svgz \
@@ -2045,6 +2047,7 @@ dist_imagesclassic_DATA = \
        images/classic/tabular-feature_multirow.png  \
        images/classic/tabular-feature_set-all-lines.png  \
        images/classic/tabular-feature_set-border-lines.png  \
+       images/classic/tabular-feature_set-inner-lines.png  \
        images/classic/tabular-feature_set-longtabular.png  \
        images/classic/tabular-feature_set-rotate-cell.png  \
        images/classic/tabular-feature_set-rotate-tabular.png  \

diff --git a/lib/images/classic/tabular-feature_set-inner-lines.png b/lib/images/classic/tabular-feature_set-inner-lines.png
new file mode 100644 (file)
index 0000000..a8179e9
Binary files /dev/null and b/lib/images/classic/tabular-feature_set-inner-lines.png differ

diff --git a/lib/images/classic/tabular-feature_unset-all-lines.png b/lib/images/classic/tabular-feature_unset-all-lines.png
index b78d50d17209906a8b10feadc47e29fc513011d2..27bdbdd7c09b0249224abec55d34c99c3fa88e12 100644 (file)
Binary files a/lib/images/classic/tabular-feature_unset-all-lines.png and b/lib/images/classic/tabular-feature_unset-all-lines.png differ

diff --git a/lib/images/oxygen/tabular-feature_set-inner-lines.svgz b/lib/images/oxygen/tabular-feature_set-inner-lines.svgz
new file mode 100644 (file)
index 0000000..9ad978e
Binary files /dev/null and b/lib/images/oxygen/tabular-feature_set-inner-lines.svgz differ

diff --git a/lib/images/tabular-feature_set-inner-lines.svgz b/lib/images/tabular-feature_set-inner-lines.svgz
new file mode 100644 (file)
index 0000000..c9ea752
Binary files /dev/null and b/lib/images/tabular-feature_set-inner-lines.svgz differ

diff --git a/lib/ui/stdtoolbars.inc b/lib/ui/stdtoolbars.inc
index 4b20445ca07ed25bfc8e6b1b76489e70e6d39cf2..2f0ef6b2892f125a9a52fc471976bd5c5f603120 100644 (file)
--- a/lib/ui/stdtoolbars.inc
+++ b/lib/ui/stdtoolbars.inc
@@ -159,6 +159,7 @@ ToolbarSet
                Item "Set right line" "tabular-feature toggle-line-right"
                Item "Set border lines" "tabular-feature set-border-lines"
                Item "Set all lines" "tabular-feature set-all-lines"
+               Item "Set inner lines" "tabular-feature set-inner-lines"
                Item "Unset all lines" "tabular-feature unset-all-lines"
                Separator
                Item "Align left" "command-alternatives tabular-feature m-align-left;tabular-feature align-left"

diff --git a/src/insets/InsetTabular.cpp b/src/insets/InsetTabular.cpp
index f1cf912de53537894b3ac55e8baea48079f66dc8..ed2824149479fe2337ff05b94a3063b0e20ffe06 100644 (file)
--- a/src/insets/InsetTabular.cpp
+++ b/src/insets/InsetTabular.cpp
@@ -199,6 +199,7 @@ TabularFeature tabularFeature[] =
        { Tabular::LONGTABULAR_ALIGN_RIGHT, "longtabular-align-right", false },
        { Tabular::SET_DECIMAL_POINT, "set-decimal-point", true },
        { Tabular::SET_TABULAR_WIDTH, "set-tabular-width", true },
+       { Tabular::SET_INNER_LINES, "set-inner-lines", false },
        { Tabular::LAST_ACTION, "", false }
 };
 
@@ -4969,6 +4970,7 @@ bool InsetTabular::getFeatureStatus(Cursor & cur, string const & s,
 
                case Tabular::SET_ALL_LINES:
                case Tabular::UNSET_ALL_LINES:
+               case Tabular::SET_INNER_LINES:
                case Tabular::SET_BORDER_LINES:
                        status.setEnabled(!tabular.ltCaption(tabular.cellRow(cur.idx())));
                        break;
@@ -5779,6 +5781,7 @@ void InsetTabular::tabularFeatures(Cursor & cur,
        row_type sel_row_start;
        row_type sel_row_end;
        bool setLines = false;
+       bool setLinesInnerOnly = false;
        LyXAlignment setAlign = LYX_ALIGN_LEFT;
        Tabular::VAlignment setVAlign = Tabular::LYX_VALIGN_TOP;
 
@@ -6131,6 +6134,9 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                break;
        }
 
+       case Tabular::SET_INNER_LINES:
+               setLinesInnerOnly = true;
+               // fall through
        case Tabular::SET_ALL_LINES:
                setLines = true;
                // fall through
@@ -6138,10 +6144,16 @@ void InsetTabular::tabularFeatures(Cursor & cur,
                for (row_type r = sel_row_start; r <= sel_row_end; ++r)
                        for (col_type c = sel_col_start; c <= sel_col_end; ++c) {
                                idx_type const cell = tabular.cellIndex(r, c);
-                               tabular.setTopLine(cell, setLines);
-                               tabular.setBottomLine(cell, setLines);
-                               tabular.setRightLine(cell, setLines);
-                               tabular.setLeftLine(cell, setLines);
+                               if (!setLinesInnerOnly || r != sel_row_start)
+                                       tabular.setTopLine(cell, setLines);
+                               if ((!setLinesInnerOnly || r != sel_row_end)
+                                   && (!setLines || r == sel_row_end))
+                                       tabular.setBottomLine(cell, setLines);
+                               if ((!setLinesInnerOnly || c != sel_col_end)
+                                   && (!setLines || c == sel_col_end))
+                                       tabular.setRightLine(cell, setLines);
+                               if ((!setLinesInnerOnly || c != sel_col_start))
+                                       tabular.setLeftLine(cell, setLines);
                        }
                break;
 

diff --git a/src/insets/InsetTabular.h b/src/insets/InsetTabular.h
index 87114ec367bb23cf0d668c2f28a8ff44c59e71ef..92ccdb6bc2a6475b57db667f904c6f80ba17fb16 100644 (file)
--- a/src/insets/InsetTabular.h
+++ b/src/insets/InsetTabular.h
@@ -306,6 +306,8 @@ public:
                ///
                SET_TABULAR_WIDTH,
                ///
+               SET_INNER_LINES,
+               ///
                LAST_ACTION
        };
        ///