From: Laurent R. <ro...@cl...> - 2003-12-02 21:00:13
|
Hi, 1.=20 =20 Aldo make a change in GUI.pm for correct XP color but don't work for = me on Win98Se. In Win32::GUI::Class new # figure out the correct background color=20 # (to avoid the "white background" syndrome on XP) if(not exists $args{-color}) { my($undef, $major, $minor) =3D Win32::GetOSVersion(); if($major =3D=3D 5 && $minor > 0) { =20 $args{-color} =3D Win32::GUI::constant("COLOR_BTNFACE", 0)+1; } else { $args{-color} =3D Win32::GUI::constant("COLOR_WINDOW", 0)+1; } } I remove red +1 and now it's correct for me. 2. 3. 4. New bug to investigate ;o) 5.=20 If -style need to be deprecated, i don't think it's a good idea to a = add -forcestyle. We keep -style without deprecated warning message.=20 Or, we remove it and force people to use registered class for fix a = default style. For me, -style don't need to be deprecated. It's a programmer choice (and supposed know what he's doing ;o). -popstyle/-pushstyle for set/reset a specific control style. -style are usefull for set a fixed style.=20 Laurent. ----- Original Message -----=20 Hello. I'm going to look into this list of bugs to see what I can fix, but = I'm quite sure none of them are to do with my patches :) and maybe you = have some answers: 1. Default DialogBox background is white on Windows 2000, rather than = the correct colour for a buttonface/3dface. I assume this is known = because it's blatantly obvious. 2. NEM event onResize seems to be called in the constructor for = Windows and possibly all other widgets. This is very frustrating because = if you use onResize to handle your scaling, then often it tries to scale = widgets that have not yet been defined, causing an error.=20 3. Timers no longer seem to work as documented. In fact they don't = work at all (for me). This could be something that only manifests itself = when the NEM is used, but I haven't tested that. 4. NEM events seem to be triggered tons of times. For example when I = click a mouse button I get about 60 onMouseDown calls. 5. There are times when -style is appropriate. I admit that -style = SHOULD be deprecated, but there should be a new argument such as = -forcestyle for when you know what you're doing. I can see that default = styles could be changed in future, and hardcoding -popstyle for each = default style seems rather crude to me. Steve |