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

[Abdulaziz A Alayed (via Code Review)]

Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3271

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/71/3271/1

fdo#55432  fix the context menu position for RTL sheet

Change-Id: I7961f7e47cabef6b06e66e5495a231b7b8b5f24b
---
M sc/source/ui/view/gridwin.cxx
1 file changed, 6 insertions(+), 2 deletions(-)



diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 145b887e..f7f7931 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 tabNo = pViewData->GetTabNo();
+            sal_Bool LayoutIsRTL = aDoc->IsLayoutRTL(tabNo);
 
             SCCOL nCurX = pViewData->GetCurX();
             SCROW nCurY = pViewData->GetCurY();
@@ -3002,8 +3005,9 @@
             long nSizeXPix;
             long nSizeYPix;
             pViewData->GetMergeSizePixel( nCurX, nCurY, nSizeXPix, nSizeYPix );
-            aMenuPos.X() += nSizeXPix;
-            aMenuPos.Y() += nSizeYPix;
+            // fdo#55432 take the correct position for RTL sheet
+            aMenuPos.X() = LayoutIsRTL ?  aMenuPos.X() -= nSizeXPix : aMenuPos.X() += nSizeXPix;
+            aMenuPos.Y() =  nSizeYPix;
 
             if (pViewData)
             {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7961f7e47cabef6b06e66e5495a231b7b8b5f24b
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Abdulaziz A Alayed <aalayed at kacst.edu.sa>