[Libreoffice-commits] core.git: vcl/source

[Tomaž Vajngerl (via logerrit)]

 vcl/source/gdi/FileDefinitionWidgetDraw.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 9e7a0fc1573a95a1542207ac86a18e630c1fc835
Author:     Tomaž Vajngerl <tomaz.vajngerl at collabora.co.uk>
AuthorDate: Mon Nov 18 00:34:43 2019 +0100
Commit:     Tomaž Vajngerl <quikee at gmail.com>
CommitDate: Mon Nov 18 07:15:00 2019 +0100

    widget theme: correct sizes of combobox/listbox
    
    Combobox/listbox was clipped on the right side. Fixed this by
    increasing the size of the widget by one and moving the button to
    the right by one pixel.
    
    Change-Id: If73e93d0c99c1daff9c0bab095169dba3c619450
    Reviewed-on: https://gerrit.libreoffice.org/83043
    Tested-by: Jenkins
    Reviewed-by: Tomaž Vajngerl <quikee at gmail.com>

diff --git a/vcl/source/gdi/FileDefinitionWidgetDraw.cxx b/vcl/source/gdi/FileDefinitionWidgetDraw.cxx
index 3d650719eb79..d6f827de3938 100644
--- a/vcl/source/gdi/FileDefinitionWidgetDraw.cxx
+++ b/vcl/source/gdi/FileDefinitionWidgetDraw.cxx
@@ -898,8 +898,8 @@ bool FileDefinitionWidgetDraw::getNativeControlRegion(
             if (ePart == ControlPart::ButtonDown)
             {
                 Point aPoint(aLocation.X() + rBoundingControlRegion.GetWidth()
-                                 - aComboButtonSize.Width(),
-                             aLocation.Y() + 1);
+                                 - aComboButtonSize.Width() - 1,
+                             aLocation.Y());
                 rNativeContentRegion = tools::Rectangle(aPoint, aComboButtonSize);
                 rNativeBoundingRegion = rNativeContentRegion;
                 return true;
@@ -917,7 +917,7 @@ bool FileDefinitionWidgetDraw::getNativeControlRegion(
                 Size aSize(rBoundingControlRegion.GetWidth(), aComboButtonSize.Height());
                 rNativeContentRegion = tools::Rectangle(aLocation, aSize);
                 rNativeBoundingRegion = rNativeContentRegion;
-                rNativeBoundingRegion.expand(1);
+                rNativeBoundingRegion.expand(2);
                 return true;
             }
         }