comdlg: Fix usage of constants

André Hentschel nerv at dawncrow.de
Mon Oct 29 18:30:12 CDT 2012


both are 4, so it doesn't matter, but vspacing should be used with vertical values like bottom and top, while hspacing should be used with left and right

---
 dlls/comdlg32/itemdlg.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c
index e5c8927..0fdfe0d 100644
--- a/dlls/comdlg32/itemdlg.c
+++ b/dlls/comdlg32/itemdlg.c
@@ -1214,17 +1214,17 @@ static SIZE update_layout(FileDialogImpl *This)
     }
 
     /* The custom controls */
-    customctrls_rc.left = dialog_rc.left + vspacing;
-    customctrls_rc.right = dialog_rc.right - vspacing;
-    customctrls_rc.bottom = filename_rc.top - hspacing;
+    customctrls_rc.left = dialog_rc.left + hspacing;
+    customctrls_rc.right = dialog_rc.right - hspacing;
+    customctrls_rc.bottom = filename_rc.top - vspacing;
     customctrls_rc.top = customctrls_rc.bottom -
         ctrl_container_resize(This, customctrls_rc.right - customctrls_rc.left);
 
     /* The ExplorerBrowser control. */
-    ebrowser_rc.left = dialog_rc.left + vspacing;
+    ebrowser_rc.left = dialog_rc.left + hspacing;
     ebrowser_rc.top = toolbar_rc.bottom + vspacing;
     ebrowser_rc.right = dialog_rc.right - hspacing;
-    ebrowser_rc.bottom = customctrls_rc.top - hspacing;
+    ebrowser_rc.bottom = customctrls_rc.top - vspacing;
 
     /****
      * Move everything to the right place.
-- 
1.8.0


-- 

Best Regards, André Hentschel


More information about the wine-patches mailing list