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

Re: Dialogs review

From: "Dave Page" <dpage(at)pgadmin(dot)org>
To: "Guillaume Lelarge" <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Dialogs review
Date: 2008-07-10 07:39:14
Message-ID: 937d27e10807100039w478dd5d2r7c3a32905d4792f1@mail.gmail.com (view raw or flat)
Thread:
Lists: pgadmin-hackers
On Wed, Jul 9, 2008 at 10:51 PM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info> wrote:
> Guillaume Lelarge a écrit :
>>
>> [...]
>> I tend to prefer the "one patch for each dialog" approach, but I would
>> like to know your thoughts on this.
>>
>
> Here is the first one, dlgAggregate.xrc.

Cool. Testing on Mac, there are a couple of minor issues.

- There needs to be more of a border around the panes and between the
contorls. The controls are very tight to the top, bottom and sides of
the dialog (to the extend that the "I'm active" glow around controls
is cut off on the top and right hand sides for instance. Similar
eclipsing can be seen between controls. We probably just need an extra
2d or so everywhere.

- When resizing on the definition pane, the height of the list box
increases, but doesn't decrease, so if you shrink the dialogue again,
the controls at the bottom of the tab get hidden. The same does not
happen on the properties page where the comments textbox resizes
correctly.

Aside from that, it does look good. Resizing of labels in different
languages works nicely, and everything looks nice and consistent.

Nice work :-)

-- 
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com

In response to

Responses

pgadmin-hackers by date

Next:From: Zdenek KotalaDate: 2008-07-10 08:25:28
Subject: Re: patch to fix pgadmin1.8.4 build on Solaris 10.
Previous:From: Guillaume LelargeDate: 2008-07-09 21:51:51
Subject: Re: Dialogs review

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