Re: A fix and a new functionnality for the colour patch

On Sat, Jun 14, 2008 at 4:21 PM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info> wrote:
> Hi,
>
> The colour patch commited by Dave during pgCon2008 allows a user to add a
> background color to the treeview for each specific registered server.
>
> When a user deletes the colour in the server's property, pgAdmin will use a
> black (#000000) background. The treeview node's text will be unreadable.
> What the patch does is impliying white colour when the user doesn't spécify
> a colour.

Hmm, thought I covered that cased. Thanks for catching.

> The new functionality adds a colour button that opens the standard colour
> dialog. The user doesn't new to know the HTML colour coding.

Is the colour dialog available on all platforms? I did look for one
when writing the original patch but couldn't find it. It's possible I
was hungover from the Jagermeister the night before though (EDB party
at pgCon :-) ).

> Comments?

You spelt colour wrong :-). Other than that (and the possible issue
above), looks good.

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