Re: Server Status window work

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Server Status window work
Date: 2009-02-13 11:42:19
Message-ID: 49955C9B.9060408@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Dave Page a écrit :
> On Thu, Feb 12, 2009 at 11:09 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>
>> The patch attached takes care of this. It's not really finished but it
>> gives an idea of what could be done. There's also the combo stuff. I
>> would love to get comments on this, to see if I go further.
>
> I immediately got an assertion (well, four):
> [...]
> I suspect, that was because of the existing refresh rate(s) being
> stored in the registry.
>

Sorry, I should have warned you...

> Aside from that, I quite like this layout :-) A few thoughts, in no
> particular order:
>
> - The layout is not being restored when the window is re-opened.
> Should be easy to do - the code already exists in other forms.
>

The part of this code is commented on the patch. I should have better
looked at the patch before sending it.

> - The main pane on the right should perhaps be the log pane, as thats
> the one that will usually have lots of info in it, even on a developer
> system with no other users.
>

Done.

> - Can we move the per-pane controls to a single toolbar at the top of
> the form, and have that display only the controls for the active pane?
> That would be a lot tidier I think, and would largely solve the
> pane-too-small-tools-truncated-or-obscured problem. There is a flag to
> the AUI manager to have it highlight the top bar of the active pane
> iirc, so it should be obvious which pane a user would be modifying.
>

I'll look at this.

Thanks for your comments, and sorry for the assertion.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message svn 2009-02-16 09:22:22 SVN Commit by dpage: r7589 - trunk/pgadmin3/docs/en_US
Previous Message Dave Page 2009-02-13 11:38:53 Re: Patch: when select bit varying type, column length can be inputted