Re: Ticket #8: Colour coding for the backend lines in the server status window

Le mercredi 16 septembre 2009 à 10:31:58, Magnus Hagander a écrit :
> On Wed, Sep 16, 2009 at 10:22, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> > On Tue, Sep 15, 2009 at 10:18 PM, Guillaume Lelarge
> > [...]
> >> The patch is not complete because of these:
> >>
> >> * Shouldn't the colors be customizable? I can add a tab on frmOptions.
> >
> > I think so.
>
> Agreed.
>

So do I. This new patch adds a tab on frmOptions to allow the colors' change.

> I'd also think the default should be "not so colorful". Meaning the
> one that will show the most (in most cases I've looked, that's idle)
> should be the default gray color. Then the colors you suggested for
> the rest (or maybe blue for running, because it's often a "softer"
> color than green).
>

Changed. Not sure it's really better.

> >> * Where do I put the configuration of the time limit? on a toolbar of
> >> the frmStatus window, or in the frmOptions dialog?
> >
> > I think frmStatus. The time limits of interest may be different on
> > your DW server than your OLTP system, so it would make sense to allow
> > control on the window itself. I think a setting against the server is
> > probably going a little too far, but if you can find a nice interface
> > for it, I could be persuaded...
>
> Actually, for starters I think we'd be fine even without the time
> limit feature at all :-)
>

I didn't add the configuration of the time limit. I will probably work on this
after this patch. I'm not sure of the UI. I think I should add it to the
toolbar, but there's already a time configuration (refresh delay).

> Now, if you can make the table sortable by "query start time", *that*
> would be useful :-)
>

It would be useful for every table pgAdmin displays (statistics come to my
mind quite instantly).

I attach the new patch.

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