On Mon, Oct 20, 2008 at 11:59 PM, Gevik Babakhani <pgdev(at)xs4all(dot)nl> wrote:
> Hereby a v1.0 patch implementing basic ordering for a wxListCtrl:
> - The patch is developed using MSVC 2008.
> - 2 new source files ctl\ListItemArranger.cpp and include\ListItemArranger.h
> are added.
> - 4 new image file are added in include\images (wxListItemArranger*.xpm)
> - For demo the dlgType.xrc and dlgType.cpp are modified.
> - 2 mk files modified manually.
> - After patching run embed-xrc
> - For compiling with MSVC the two source files must be added manually to ctl
> and include filters inside VS2005/2008.
> - For testing: Goto "add new composite type" then the definition tab.
> Please let me know what you think.
Sorry for the delay in looking at this. A few (minor comments)
- The buttons should not become active when viewing an existing type.
- The background of the button area needs to be transparent. See the
attached screenshot and you'll see why.
- Please follow existing naming conventions - eg. use
ctlListItemArranger, per the other custom controls.
- Please retain the comments and grouping on the includes - it helps
prevent people mistakenly re-ordering things they shouldn't - eg.
// Main app header
// wxWindows headers
// App headers
They should almost always be in that order (one exception being
wx/regex.h which seems to need to be last usually). I appreciate that
we have some inonsistencies at the moment which do need improvement...
EnterpriseDB UK: http://www.enterprisedb.com
Description: image/png (15.2 KB) (inlined above)
In response to
pgadmin-hackers by date
|Next:||From: Dave Page||Date: 2008-10-27 13:23:46|
|Subject: Re: Buglets in dlgFunction|
|Previous:||From: Guillaume Lelarge||Date: 2008-10-24 08:33:58|
|Subject: Re: [HACKERS] Hot Standby utility and administrator