[Libreoffice-commits] core.git: Branch 'libreoffice-6-3' - svx/source
Tor Lillqvist (via logerrit)
logerrit at kemper.freedesktop.org
Fri Nov 1 08:44:51 UTC 2019
svx/source/table/tablecontroller.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit 76c0741d1377274e5347c586ada4061ee157527b
Author: Tor Lillqvist <tml at collabora.com>
AuthorDate: Wed Oct 30 17:30:14 2019 +0200
Commit: Tor Lillqvist <tml at collabora.com>
CommitDate: Fri Nov 1 09:44:13 2019 +0100
tdf#128454: Make it possible to delete a table's single row or column
It is already possible to delete all rows or all columns when there
are multiple ones. So why it should be impossible to delete the single
row or column when there is just one I can't understand.
(This change is just one way to provide a workaround for the bug; the
ideal would be if it was possible to specifically delete a whole table
also in Impress documents, not just Writer ones.)
(The change is only for the LibreOfficeKit use case, but maybe it
should be in all cases.)
Change-Id: If9221d248d59e515a110ff94cfb96fdd76aecafb
Reviewed-on: https://gerrit.libreoffice.org/81801
Reviewed-by: Tor Lillqvist <tml at collabora.com>
Tested-by: Tor Lillqvist <tml at collabora.com>
(cherry picked from commit f4594ae70f3bcde7a87ec606030ec5217ba53788)
Reviewed-on: https://gerrit.libreoffice.org/81868
Tested-by: Jenkins
diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx
index be279e41a8e9..57c9c1dbbeac 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -463,11 +463,11 @@ void SvxTableController::GetState( SfxItemSet& rSet )
break;
}
case SID_TABLE_DELETE_ROW:
- if( !mxTable.is() || !hasSelectedCells() || (mxTable->getRowCount() <= 1) )
+ if( !mxTable.is() || !hasSelectedCells() || (!comphelper::LibreOfficeKit::isActive() && mxTable->getRowCount() <= 1) )
rSet.DisableItem(SID_TABLE_DELETE_ROW);
break;
case SID_TABLE_DELETE_COL:
- if( !mxTable.is() || !hasSelectedCells() || (mxTable->getColumnCount() <= 1) )
+ if( !mxTable.is() || !hasSelectedCells() || (!comphelper::LibreOfficeKit::isActive() && mxTable->getColumnCount() <= 1) )
rSet.DisableItem(SID_TABLE_DELETE_COL);
break;
case SID_TABLE_MERGE_CELLS:
More information about the Libreoffice-commits
mailing list