On Mon, Jan 10, 2011 at 1:22 AM, Jasmin Dizdarevic <jasmin.dizdare...@gmail.com> wrote: > Hi Guys, > as I started working on email notification Dave told me, that I've to > disable all email notification relevant GUI elements, if the required > pgAgent version is not installed. > I did that of course, but I thought why we should not tell the user, why a > control is disabled. > My idea is not to disable the control, but hide it and place an explaining > text on it's position. In code we know which version is required because of > the call pgConn::BackendMinimumVersion(major, minor). > I've extended this method to pgConn::BackendMinimumVersion(major,wxControl* > control) and every time this is called, the desired control will be hidden > and a wxStaticText control will be added. > The text is for example: Supported in PostgreSQL %i.%i, major, minor > => Supported in PostgreSQL 9.1. See also the attached screenshots. > I know this is not applicable in all situations, but it's a useful > information. > What you think about it? Shall I finalize this feature?
Changing the type of control doesn't seem like a good idea. It can make the dialogue look different depending on the server that is connected, which may confuse the user. If you simply set the text in the existing control, I think that would be acceptable. Of course, it wouldn't work for radio buttons/checkboxes, but I think that's a small price to pay. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
