In message <[EMAIL PROTECTED]> on 17 Feb 2003 22:30:07 +0100, Mikael 
Cardell <[EMAIL PROTECTED]> said:

mc> When 3.6 was released, I looked around in the code and found a new
mc> pixmap, ":xpm:cross", that I found neat and used in my window
mc> decorations. I also happened to change TitleButtonShadowDepth to 1 and
mc> noticed that :xpm:cross and the old :xpm:resize didn't follow the
mc> value of this variable.
mc> 
mc> If you look in the code at
mc> 
mc>   util.c:Create3DResizeImage()
mc> 
mc> you see that the call to Draw3DBorder() uses a constant 2 instead of
mc> Scr->TitleButtonShadowDepth. In, for instance, Create3DBarImage(),
mc> TitleButtonShadowDepth is used, but not here. Why?
mc> 
mc> The same goes for the new Create3DCrossImage().
mc> 
mc> I changed the constants and recompiled and I think the result is much
mc> nicer:
mc> 
mc>   http://hack.org/~mc/images/mcscreen.png
mc> 
mc> What do you all think?

I entirely agree, and just committed the change.  Incidently, there
were a total of 8 routines with this error (or so it appeared to me):

Create3DCrossImage()
Create3DIconifyImage()
Create3DSunkenResizeImage()
Create3DBoxImage()
Create3DResizeImage()
Create3DResizeAnimation()
Create3DMenuAnimation()
Create3DZoomAnimation()

-- 
Richard Levitte     | http://richard.levitte.org/ | Spannv. 38, I
Levitte Programming | http://www.lp.se/           | S-168 35 Bromma
T: +46-708-26 53 44 |                             | SWEDEN
     "Price, performance, quality...  choose the two you like"

Reply via email to