Gevik Babakhani a écrit :
> Hi Dave,
>
> Thank you for reviewing :)
>
>> - The buttons should not become active when viewing an existing type.
> Done.
>
They're still active for me.
>> - The background of the button area needs to be transparent.
>> See the attached screenshot and you'll see why.
>
> Making the background transparent (for wxPanel) proved to be very hard in
> wxWidgets. After a lot of googling and testing I managed to imitate the
> background gradient. I do this only for wxWIN. It should work without this
> hack on wxGTK and wxMAC.
>
I just checked on GTK. It works.
>> - Please follow existing naming conventions - eg. use
>> ctlListItemArranger, per the other custom controls.
>
> Done. (I hope I did it right)
>
> Please find the changes in a new patch to review. Please also run embed-xrc
> again.
>
> I have also modified the .mk files. But at this moment I have no way of
> testing on wxGTK and wxMAC.
>
include/images/ctllistitemarrangerFirst.xpm should be renamed
include/images/ctlListItemArrangerFirst.xpm (notice the upper case in
the filename).
I still need to read the code but it seems a good addition. Thanks, Gevik.
--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com