[PATCH] fdo#55432 fix the context menu position for RTL sheet

David Tardon (via Code Review) gerrit at gerrit.libreoffice.org
Sat Apr 20 04:41:42 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3497

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/97/3497/1

fdo#55432  fix the context menu position for RTL sheet

Change-Id: I023215646b9aeadb4909c9f90f84f9e8b0dbd173
---
M sc/source/ui/view/gridwin.cxx
1 file changed, 5 insertions(+), 1 deletion(-)



diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 145b887e..1a66c97 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -2995,6 +2995,9 @@
         else if ( !bMouse )
         {
             //  non-edit menu by keyboard -> use lower right of cell cursor position
+            ScDocument* aDoc = pViewData->GetDocument();
+            SCTAB nTabNo = pViewData->GetTabNo();
+            sal_Bool bLayoutIsRTL = aDoc->IsLayoutRTL(nTabNo);
 
             SCCOL nCurX = pViewData->GetCurX();
             SCROW nCurY = pViewData->GetCurY();
@@ -3002,7 +3005,8 @@
             long nSizeXPix;
             long nSizeYPix;
             pViewData->GetMergeSizePixel( nCurX, nCurY, nSizeXPix, nSizeYPix );
-            aMenuPos.X() += nSizeXPix;
+            // fdo#55432 take the correct position for RTL sheet
+            aMenuPos.X() += bLayoutIsRTL ? -nSizeXPix : nSizeXPix;
             aMenuPos.Y() += nSizeYPix;
 
             if (pViewData)

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I023215646b9aeadb4909c9f90f84f9e8b0dbd173
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: David Tardon <dtardon at redhat.com>
Gerrit-Reviewer: Abdulaziz A Alayed <aalayed at kacst.edu.sa>



More information about the LibreOffice mailing list