[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