[PATCH] Fixes fdo#46186, The Table Remains in the defined writing mo...

gokul s (via Code Review) gerrit at gerrit.libreoffice.org
Sat Feb 2 02:07:42 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1964

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/64/1964/1

Fixes fdo#46186, The Table Remains in the defined writing mode.

The table in impress is drawn under the rectangular are, on Selecting
the table it was only selecting the rectangular area and not the cells
inside, Making the layout to be as RTL which was mirroring the table.

But, We need the table to be in RTL Writing mode and not mirroring the
table. Now on applying my patch, If table is selected, it will select
the entire cells inside which is the actual table and the functionality
works fine.

Change-Id: I9d6bdde5019322488be66fa89a6488d348b2cf44
---
M svx/source/table/tablecontroller.cxx
1 file changed, 7 insertions(+), 0 deletions(-)



diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx
index 9368675..76a2fd7 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -366,6 +366,13 @@
         const SdrMarkList& rMarkList= mpView->GetMarkedObjectList();
         if( rMarkList.GetMarkCount() == 1 )
             bSelected = mxTableObj.get() == rMarkList.GetMark(0)->GetMarkedSdrObj();
+        /* Selecting the table means selecting the entire cells inturn*/
+        if(!hasSelectedCells())
+        {
+            maCursorFirstPos = pTableObj->getFirstCell();
+            maCursorLastPos = pTableObj->getLastCell();
+            mbCellSelectionMode=true;
+        }
     }
 
     if( bSelected )

-- 
To view, visit https://gerrit.libreoffice.org/1964
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9d6bdde5019322488be66fa89a6488d348b2cf44
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: gokul s <gswaminathan at kacst.edu.sa>


More information about the LibreOffice mailing list