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

Keith Curtis keithcu at gmail.com
Fri Mar 21 04:27:04 PDT 2014


 vcl/source/window/toolbox.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 509441038ab95dd3a60efd1b6c302bf22bfbc631
Author: Keith Curtis <keithcu at gmail.com>
Date:   Mon Mar 17 20:16:57 2014 -0400

    Hopefully fix Windows HiDPI toolbar layout bug
    
    On Windows HiDPI, toolbar buttons are cut off. This may fix the problem.
    
    Here is a screenshot:
    http://i.imgur.com/NADAvYi.png
    
    I can't prove this fixes anything on Windows because I can't see this
    on Linux and don't really understand the surrounding code. On the other
    hand, it is easy to prove this is reasonable code.
    
    Change-Id: I69c19ad46844bead942ce63883d163cb9d0690c9
    Reviewed-on: https://gerrit.libreoffice.org/8637
    Tested-by: LibreOffice gerrit bot <gerrit at libreoffice.org>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/vcl/source/window/toolbox.cxx b/vcl/source/window/toolbox.cxx
index 3cb7322..61204c9 100644
--- a/vcl/source/window/toolbox.cxx
+++ b/vcl/source/window/toolbox.cxx
@@ -1736,8 +1736,8 @@ bool ToolBox::ImplCalcItem()
     long            nDropDownArrowWidth = TB_DROPDOWNARROWWIDTH;
 
     // set defaults if image or text is needed but empty
-    nDefWidth       = GetDefaultImageSize().Width();
-    nDefHeight      = GetDefaultImageSize().Height();
+    nDefWidth       = GetDefaultImageSize().Width() * GetDPIScaleFactor();
+    nDefHeight      = GetDefaultImageSize().Height() * GetDPIScaleFactor();
 
     mnWinHeight = 0;
     // determine minimum size necessary in NWF


More information about the Libreoffice-commits mailing list