comctl32: toolbar[2/3]: test standard buttons size (resend)
Mikołaj Zalewski
mikolaj at zalewski.pl
Tue Mar 20 05:09:32 CDT 2007
There were no tests for button sizes after setting a standard bitmap.
I've also removed the calls to TB_SETBUTTONSIZE as they are not needed
and the call for small buttons with 22x22 is misleading.
-------------- next part --------------
From b2c46acd5c5222c346d406d021fe7b24058ca9e4 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Miko=C5=82aj_Zalewski?= <mikolaj at zalewski.pl>
Date: Mon, 19 Mar 2007 18:47:27 +0100
Subject: [PATCH] comctl32: toolbar: test standard buttons size
---
dlls/comctl32/tests/toolbar.c | 6 +++++-
dlls/comctl32/toolbar.c | 14 ++++----------
2 files changed, 9 insertions(+), 11 deletions(-)
diff --git a/dlls/comctl32/tests/toolbar.c b/dlls/comctl32/tests/toolbar.c
index e8def41..43c93dd 100644
--- a/dlls/comctl32/tests/toolbar.c
+++ b/dlls/comctl32/tests/toolbar.c
@@ -350,12 +350,14 @@ static void test_add_bitmap(void)
CHECK_IMAGELIST(4, 20, 15);
ok(SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&bmp128) == 0, "TB_ADDBITMAP - unexpected return\n");
CHECK_IMAGELIST(10, 20, 15);
- /* however TB_SETBITMAPSIZE/add std bitmap won't change the image size (the button size does change!) */
+ /* however TB_SETBITMAPSIZE/add std bitmap won't change the image size (the button size does change) */
ok(SendMessageA(hToolbar, TB_SETBITMAPSIZE, 0, MAKELONG(8, 8)) == TRUE, "TB_SETBITMAPSIZE failed\n");
UpdateWindow(hToolbar);
+ compare((int)SendMessageA(hToolbar, TB_GETBUTTONSIZE, 0, 0), MAKELONG(15, 14), "%x");
CHECK_IMAGELIST(10, 20, 15);
ok(SendMessageA(hToolbar, TB_ADDBITMAP, 0, (LPARAM)&stdsmall) == 1, "TB_SETBITMAPSIZE failed\n");
UpdateWindow(hToolbar);
+ compare((int)SendMessageA(hToolbar, TB_GETBUTTONSIZE, 0, 0), MAKELONG(23, 22), "%x");
CHECK_IMAGELIST_TODO_COUNT(22, 20, 15);
/* check standard bitmaps */
@@ -364,10 +366,12 @@ static void test_add_bitmap(void)
rebuild_toolbar(&hToolbar);
ok(SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&addbmp) == 0, "TB_ADDBITMAP - unexpected return\n");
CHECK_IMAGELIST(15, 16, 16);
+ compare((int)SendMessageA(hToolbar, TB_GETBUTTONSIZE, 0, 0), MAKELONG(23, 22), "%x");
addbmp.nID = IDB_STD_LARGE_COLOR;
rebuild_toolbar(&hToolbar);
ok(SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&addbmp) == 0, "TB_ADDBITMAP - unexpected return\n");
CHECK_IMAGELIST(15, 24, 24);
+ compare((int)SendMessageA(hToolbar, TB_GETBUTTONSIZE, 0, 0), MAKELONG(31, 30), "%x");
addbmp.nID = IDB_VIEW_SMALL_COLOR;
rebuild_toolbar(&hToolbar);
diff --git a/dlls/comctl32/toolbar.c b/dlls/comctl32/toolbar.c
index ba8ea80..c40ced0 100644
--- a/dlls/comctl32/toolbar.c
+++ b/dlls/comctl32/toolbar.c
@@ -2775,19 +2775,13 @@ TOOLBAR_AddBitmap (HWND hwnd, WPARAM wParam, LPARAM lParam)
/* Windows resize all the buttons to the size of a newly added standard image */
if (lpAddBmp->nID & 1)
{
- /* large icons */
- SendMessageW (hwnd, TB_SETBITMAPSIZE, 0,
- MAKELPARAM((WORD)24, (WORD)24));
- SendMessageW (hwnd, TB_SETBUTTONSIZE, 0,
- MAKELPARAM((WORD)31, (WORD)30));
+ /* large icons: 24x24. Will make the button 31x30 */
+ SendMessageW (hwnd, TB_SETBITMAPSIZE, 0, MAKELPARAM(24, 24));
}
else
{
- /* small icons */
- SendMessageW (hwnd, TB_SETBITMAPSIZE, 0,
- MAKELPARAM((WORD)16, (WORD)16));
- SendMessageW (hwnd, TB_SETBUTTONSIZE, 0,
- MAKELPARAM((WORD)22, (WORD)22));
+ /* small icons: 16x16. Will make the buttons 23x22 */
+ SendMessageW (hwnd, TB_SETBITMAPSIZE, 0, MAKELPARAM(16, 16));
}
TOOLBAR_CalcToolbar (hwnd);
--
1.4.4.2
More information about the wine-patches
mailing list