On Thu, 2012-07-26 at 12:37 -0300, Vinicius Santos wrote:
> I finished the patch. It works fine on Windows 7, but the
> SetLabel(wxString) doesn't work on my Ubuntu.
> I did make some simple tests with the wxWidgets on Ubuntu, but I can't do
> it to work.
> Can you help-me with the test on Linux?
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?
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
Another thing that bugs me. To show the full name of an object, you use
GetSchema()->GetName() + wxT(".") + GetName()
instead of the simpler:
Please change this.
Once all of this have been done, I'll review it one more time, and apply
it if ready.
In response to
pgadmin-hackers by date
|Next:||From: Guillaume Lelarge||Date: 2012-08-26 17:01:57|
|Subject: pgAdmin III commit: Update latvian translation|
|Previous:||From: Guillaume Lelarge||Date: 2012-08-26 16:40:27|
|Subject: Re: pgAdmin III commit: Lots of work on domains,
and check constraints|