Change in core[libreoffice-4-0]: fdo#44582 fixing horizontal scroll bar in sub windows

[Lior Kaplan (via Code Review)]

Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1908

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/08/1908/1

fdo#44582 fixing horizontal scroll bar in sub windows

Change-Id: Ifea57dec77c5b23771684a1e46267f8431478075
Reviewed-on: https://gerrit.libreoffice.org/1806
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
---
M vcl/unx/gtk/gdi/salnativewidgets-gtk.cxx
1 file changed, 6 insertions(+), 1 deletion(-)



diff --git a/vcl/unx/gtk/gdi/salnativewidgets-gtk.cxx b/vcl/unx/gtk/gdi/salnativewidgets-gtk.cxx
index 736834a..e15fbfa 100644
--- a/vcl/unx/gtk/gdi/salnativewidgets-gtk.cxx
+++ b/vcl/unx/gtk/gdi/salnativewidgets-gtk.cxx
@@ -1041,7 +1041,12 @@
             rNativeContentRegion.Right() = rNativeContentRegion.Left() + 1;
         if (!rNativeContentRegion.GetHeight())
             rNativeContentRegion.Bottom() = rNativeContentRegion.Top() + 1;
-        returnVal = sal_True;
+
+        returnVal = true;
+
+        //See fdo#44582, Horizontal scrollbar in navigator window is broken
+        if ((nPart==PART_BUTTON_LEFT || nPart==PART_BUTTON_RIGHT) && Application::GetSettings().GetLayoutRTL())
+            returnVal = false;
     }
     if( (nType == CTRL_MENUBAR) && (nPart == PART_ENTIRE_CONTROL) )
     {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifea57dec77c5b23771684a1e46267f8431478075
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Lior Kaplan <kaplanlior at gmail.com>
Gerrit-Reviewer: Abdulaziz A Alayed <aalayed at kacst.edu.sa>
Gerrit-Reviewer: Caolán McNamara <caolanm at redhat.com>