[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