If I have reviewed correctly, the current patch allows to select the colour for background color for server, and its children.
I believe - we should allow to select text color too for better readability.
Yes Ashesh, After our discussion I tried that but with all the different colours in browser tree it looks cartoonish.
I am not sure - what looked like cartoonish to you (totally depend on the color choices made by the users). :-)
If I choose any dark color as background color, text in nodes won't be clear to the user, and that will make the tree nodes completely unusable.
If we allow to choose the foreground/text color, those dark colors will become usable by choosing suitable light color as text colors.
I don't want to make any decision here, just asking for opinions from others.
So - Again ball is back in Dave's courtyard :-).
pgAdmin 3 didn't allow selection of the foreground colour, and I don't recall anyone ever asking for it. It's clearly up to the user to not select colours that won't work.
I'm fine with just setting the background colour to satisfy this requirement, though if it's trivial to do, I have no objection to allowing changes to the foreground colour.
