[PATCH] fdo#55432 fix the context menu position for RTL sheet
Abdulaziz A Alayed (via Code Review)
gerrit at gerrit.libreoffice.org
Mon Apr 8 00:51:38 PDT 2013
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>
More information about the LibreOffice
mailing list