Le 12/04/2010 14:14, Dave Page a écrit :
> On Sun, Apr 11, 2010 at 10:36 PM, Guillaume Lelarge
> <guillaume(at)lelarge(dot)info> wrote:
>> I have an almost working patch. I still have one bug: if someone adds
>> the first server in a group, the group and the server don't appear in
>> the treeview. After quite some tests and debugging, the program adds
>> them in the treeview, but after the CreateObjects call, they disappear.
> I didn't see that here. It seemed to work fine - though it sure would
> be nice if it would dynamically move the nodes around. Restarting
> pgAdmin to see the changes doesn't feel nice.
Sure, it's way nicer. Done.
> It also feels like I
> should be able to right-click a group node, though I appreciate you
> cannot do that with the Servers node either.
What do you want with a right click in a group node? access to the
> Finally (I think!) - it seems weird that the Servers node lists the
> total number of Servers, even those that aren't direct children.
> Y'know - as I type I can't help thinking that the correct way to do
> this is to consider the root node to be the group, in which we
> currently have a single, fixed group called 'Servers'. Additional
> groups would then be additional root nodes... but I don't know if you
> can do that on all operating systems.
Done. Seems to work everywhere I checked.
So, now, when you launch pgAdmin, it behaves exactly as before (ie you
still have one one group, Servers). But you can add more groups and
affects servers into those groups).
>> BTW, it is not the final patch, some work remain to be done (the group
>> combobox is only used as a textbox right now, it's badly displayed, some
>> code are redundant, etc.)
> Yeah, I saw that.
They are all fixed by now.
> BTW - I noticed another issue while I was there. The default colour
> shown by the colour select button seems to be random, so for servers
> for which I never previously chose a colour updating them to set a
> group is also changing them to a random colour, unless I manually
> choose that too.
New patch attached. Seems good to go for me.
In response to
pgadmin-hackers by date
|Next:||From: pgAdmin Trac||Date: 2010-04-19 01:50:36|
|Subject: [pgAdmin III] #169: SET search_path = value|
|Previous:||From: pgAdmin Trac||Date: 2010-04-17 15:20:46|
|Subject: Re: [pgAdmin III] #39: Guru Hints are empty|