From: Juergen Spitzmueller Date: Fri, 28 Dec 2018 09:11:42 +0000 (+0100) Subject: Add tabular-features set-inner-lines and fix set-all-lines behavior X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=b6dc00dc99e7bbb1906ab8aa0a90ff4f41a8f57f;p=features.git 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 3816d5c727..0db9ad8cd9 100644 --- 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 index 0000000000..a8179e9559 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 b78d50d172..27bdbdd7c0 100644 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 index 0000000000..9ad978e1d7 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 index 0000000000..c9ea752710 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 4b20445ca0..2f0ef6b289 100644 --- 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 f1cf912de5..ed28241494 100644 --- 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 87114ec367..92ccdb6bc2 100644 --- a/src/insets/InsetTabular.h +++ b/src/insets/InsetTabular.h @@ -306,6 +306,8 @@ public: /// SET_TABULAR_WIDTH, /// + SET_INNER_LINES, + /// LAST_ACTION }; ///