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
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 |