Dave Page a écrit :
> On Sat, Jun 14, 2008 at 4:21 PM, Guillaume Lelarge
> <guillaume@[EMAIL PROTECTED]
> 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.
>
At least, it didn't work on GTK+.
>> 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?
AFAIK, yes, it does.
According to the manual
(http://docs.wxwidgets.org/2.8.6/wx_commondialogsoverview.html),
"Some
dialogs have both platform-dependent and platform-independent
implementations, so that if underlying windowing systems do not provide
the required functionality, the generic cl***** and functions can stand
in."
> 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 :-) ).
>
hehe :)
>> Comments?
>
> You spelt colour wrong :-). Other than that (and the possible issue
> above), looks good.
>
Yeah, my first (unsent) mail used color. I changed all color with colour
because of the spelling in pgAdmin :)
--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@[EMAIL PROTECTED]
)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
|
