Skip site navigation (1) Skip section navigation (2)

Re: Info on non supported Features

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Jasmin Dizdarevic <jasmin(dot)dizdarevic(at)gmail(dot)com>, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Info on non supported Features
Date: 2011-01-10 09:55:33
Message-ID: 4D2AD795.10709@lelarge.info (view raw or flat)
Thread:
Lists: pgadmin-hackers
Le 10/01/2011 10:53, Dave Page a écrit :
> On Mon, Jan 10, 2011 at 1:22 AM, Jasmin Dizdarevic
> <jasmin(dot)dizdarevic(at)gmail(dot)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.
> 

I don't see why it couldn't work for radio buttons and checkboxes. They
also have text. We dont' use that now, but it could be use for this.


-- 
Guillaume
 http://www.postgresql.fr
 http://dalibo.com

In response to

Responses

pgadmin-hackers by date

Next:From: Dave PageDate: 2011-01-10 10:01:29
Subject: Re: Info on non supported Features
Previous:From: Dave PageDate: 2011-01-10 09:53:27
Subject: Re: Info on non supported Features

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group