Re: More information about the selected objects

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Vinicius Santos <vinicius(dot)santos(dot)lista(at)gmail(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: More information about the selected objects
Date: 2012-09-11 17:47:50
Message-ID: 1347385670.1945.90.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, 2012-09-11 at 12:20 -0300, Vinicius Santos wrote:
>
>
> Got some time to work on this. SetLabel is useful for widgets,
> not for
> windows. Change your call to the SetLabel() method with a call
> to
> SetTitle(). Works great for me on Linux. Don't know on
> Windows, maybe
> you could check?
>
> OK. I changed it.
>
>
> About your patch, it seems good so far. My only issue is that,
> when I
> click on the root node, or on a servers group node, it still
> displays
> the old connection string. It shouldn't. It should get back to
> "pgAdmin
> III".
>
> Here, I would like the opinion of you.
>
> When the user clicks on the "Database" node, the pgAdmin is connected
> to the database "postgres".
> So, would not it be interesting to show this information?
>

Well, as far as I know, pgadmin could be connected to many databases. It
should only display the connection it used to get the informations on
the selected object.

>
>
> Another thing that bugs me. To show the full name of an
> object, you use
> this code:
>
> GetSchema()->GetName() + wxT(".") + GetName()
>
> instead of the simpler:
>
> GetFullIdentifier()
>
> GetFullIdentifier() doesn't show the same information.
>
>
> We can't change it, because it is used in several other places.

Are you sure?

--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Vinicius Santos 2012-09-11 18:49:02 Re: More information about the selected objects
Previous Message Guillaume Lelarge 2012-09-11 17:44:26 Re: Information for added functionalities