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 <aala...@kacst.edu.sa> _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice