On Thu, 15 Nov 2012, BALATON Zoltan escribió: > On Wed, 14 Nov 2012, Rodolfo García Peñas wrote: > >Then, the image can be resized using wIconValidateIconSize(). The resize > >should be done in get_rimage_icon_from_wm_hints(), not in the function > >get_wwindow_image_from_wmhints(). This is because the function > >get_wwindow_image_from_wmhints() is used in wIconStore() too. If we resize > >the image before save it to disk, then if we change the icon/dock size, then > >the image saved will have a different size than the curren icon size. Is > >better resize the image when is painted in the screen, not the image saved. > > Thanks a lot for fixing all this. I couldn't really follow all your > changes so I'm not sure how it all works now so here are some > questions/thoughts. I think you've thought about these already but > asking it just in case: > > - This does not mean that the image is resized every time it is > painted, does it? If yes then it may be faster to check the size of > the cached image before using it and remove it when the icon size > has been changed. > > - Images are also enlarged when they are too small? This may look > bad so probably there could be a limit (say no more than 2x) or only > scale down images if they are too big but never enlarge them. > > - Is it possible to change icons of applications without restarting > them (the Apply button in the inspector is working) and does the > ignore client supplied icon option work?
Hi, please, test the patches just I sent. Carlos, please, make a deep test before uploading it. I think the patches are ok, but I can make a deep checking at this moment. Feel free to rebase them or change the apply order, commit info,... The main change is that wIconUpdate can receive the icon. Then wIconChangeImageFile can set the image without using the icon database (Domain). Cheers. kix > Regards, > BALATON Zoltan -- To unsubscribe, send mail to [email protected].
