What do you think about this?
0058-More-information-about-the-selected-objetc.patch
Description: Binary data
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
> It doesn't return the schema when the schema belongs to your
> search_path. That's the right way to do it.
>
OK. But we must always show the scheme in this context.
We can create another function too.
On Tue, 2012-09-11 at 15:49 -0300, Vinicius Santos wrote:
> 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.
>
> Thinking so, I think y
>
> 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.
>
Thinking so, I think you have reason.
>
> Are you sure?
>
Yes. GetFullIdentifier() doesn't return the schema.
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 k
> 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.
>
> 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?
>
Yes. Of course.
>
> About your patch, it seems good so far.
On Thu, 2012-07-26 at 12:37 -0300, Vinicius Santos wrote:
> Hello,
>
> 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 t
Hello,
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?
Thanks.
0001-More-information-about-the-selected-objetc.p
>
>
> I much prefer what you call "on caption" (I call it the title bar).
>
I called it as "on caption" because of my habit with Delphi.
It is the title bar really
> That's already what we do with the query tool.
>
So, I'll send the patch in the title bar, for you to give a look.
On Tue, 2012-07-24 at 01:02 -0300, Vinicius Santos wrote:
> We can also do so. On caption.
>
>
I much prefer what you call "on caption" (I call it the title bar).
That's already what we do with the query tool.
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com
--
Sent vi
We can also do so. On caption.
<>
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
On Wed, 2012-07-18 at 10:41 -0300, Vinicius Santos wrote:
>
> One alternative option might be to have an extra wxAUI menubar
> panel,
> with a couple of textlabels on it. That could be moved around
> and
> hidden or shown in the same way we do for th
>
> One alternative option might be to have an extra wxAUI menubar panel,
> with a couple of textlabels on it. That could be moved around and
> hidden or shown in the same way we do for the menu bar (or the
> database selection bar on the query tool).
>
Good ideia.
I can do. Then we see if it impr
On Wed, Jul 18, 2012 at 2:13 PM, Vinicius Santos
wrote:
>> Not really. The panel you've used for the server name is fixed width.
>> As I mentioned before, on some, if not all platforms, you can only
>> have 1 variable width panel. That's one of the reasons we've avoided
>> putting extra info in th
>
> Not really. The panel you've used for the server name is fixed width.
> As I mentioned before, on some, if not all platforms, you can only
> have 1 variable width panel. That's one of the reasons we've avoided
> putting extra info in there before, that isn't a clearly defined
> width.
>
It's t
On Wed, Jul 18, 2012 at 1:50 PM, Vinicius Santos
wrote:
>
> Really. Your servers have very long names. This should happen only at lower
> resolutions.
Not really. The panel you've used for the server name is fixed width.
As I mentioned before, on some, if not all platforms, you can only
have 1 va
>
> I can test the PPAS stuff, but not the Greenplum objects. But... they
> shouldn't be an issue - if an object is derived from pgSchema, you can
> (and should use GetSchema(). If it's derived from a pgTable, you
> probably want the table name too. If it's derived from pgObject,
> pgServer, pgData
I fixed the problems.
I removed the implementation on EnterpriseDB items, because I don't have
how to test it. Can you help to test it?
I tested on Windows 7 and Ubuntu.
0001-More-information-about-the-selected-objects.patch
Description: Binary data
--
Sent via pgadmin-hackers mailing list (p
>
> Which is entirely expected as both public synonyms and casts are
> database level objects and thus don't have pgSchema as a parent class.
>
>
I forgot to check the pgCast. Sorry
I'll fix it and resubmit.
On Tue, Jul 17, 2012 at 5:24 PM, Vinicius Santos
wrote:
> Attached.
>
> I think the information "user@database ( server )" is important.
>
> I tested on Windows 7.
> I'll test on Ubuntu tonight.
I have no idea how you managed to get that to compile on Windows:
./schema/edbSynonym.cpp: In member
Attached.
I think the information "user@database ( server )" is important.
I tested on Windows 7.
I'll test on Ubuntu tonight.
0001-More-information-about-the-selected-objects.patch
Description: Binary data
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make chan
On Mon, Jul 16, 2012 at 10:22 PM, Guillaume Lelarge
wrote:
> On Sat, 2012-07-14 at 20:48 +0100, Dave Page wrote:
>>
>>
>> On Saturday, July 14, 2012, Guillaume Lelarge wrote:
>> On Sat, 2012-07-14 at 18:57 +0100, Dave Page wrote:
>> >
>> >
>> > On Saturday, July 14,
On Sat, 2012-07-14 at 20:48 +0100, Dave Page wrote:
>
>
> On Saturday, July 14, 2012, Guillaume Lelarge wrote:
> On Sat, 2012-07-14 at 18:57 +0100, Dave Page wrote:
> >
> >
> > On Saturday, July 14, 2012, Guillaume Lelarge wrote:
> > On Sat, 2012-07
On Saturday, July 14, 2012, Guillaume Lelarge wrote:
> On Sat, 2012-07-14 at 18:57 +0100, Dave Page wrote:
> >
> >
> > On Saturday, July 14, 2012, Guillaume Lelarge wrote:
> > On Sat, 2012-07-14 at 11:28 -0300, Vinicius Santos wrote:
> > >
> > > Not sure I like the
On Sat, 2012-07-14 at 18:57 +0100, Dave Page wrote:
>
>
> On Saturday, July 14, 2012, Guillaume Lelarge wrote:
> On Sat, 2012-07-14 at 11:28 -0300, Vinicius Santos wrote:
> >
> > Not sure I like the point either. We don't have that
> notation
> >
On Saturday, July 14, 2012, Guillaume Lelarge wrote:
> On Sat, 2012-07-14 at 11:28 -0300, Vinicius Santos wrote:
> >
> > Not sure I like the point either. We don't have that notation
> > on
> > PostgreSQL. I think I would prefer something like "Retrieving
> > detail
>
> I think I like the second one better, but this is just my opinion.
>
Well, I'll start working on the second, then.
If new opinions, we changed.
On Sat, 2012-07-14 at 11:28 -0300, Vinicius Santos wrote:
>
> Not sure I like the point either. We don't have that notation
> on
> PostgreSQL. I think I would prefer something like "Retrieving
> details on
> table schema.table" and not say the database. Beca
> Not sure I like the point either. We don't have that notation on
> PostgreSQL. I think I would prefer something like "Retrieving details on
> table schema.table" and not say the database. Because if you add the
> database, someone will probably want also the server. It may not be
> usefull to you
On Sat, 2012-07-14 at 10:59 -0300, Vinicius Santos wrote:
> Forwarding, sorry.
>
> 2012/7/14 Vinicius Santos
> I see some value on it, but I don't like the arrow.
>
>
> Any suggestions?
> I could put the point rather than the arrow or other
>
Forwarding, sorry.
2012/7/14 Vinicius Santos
> I see some value on it, but I don't like the arrow.
>>
>
> Any suggestions?
> I could put the point rather than the arrow or other something. What do
> you think?
>
On Fri, 2012-07-13 at 21:47 -0300, Vinicius Santos wrote:
> > So, this information would solve the problem. What do you think?
> >
>
> Any comment? Can I develop the patch?
You can certainly work on a patch. Shouldn't be hard to code.
I see some value on it, but I don't like the arrow.
--
Gui
> So, this information would solve the problem. What do you think?
>
Any comment? Can I develop the patch?
Hello,
When you click on a table, in the status bar appears: "Retrieving details
on table ..."
I'd like to put more information and leave it like this: "Retrieving
details on table ->., For example:
"Retrieving details on table db_developers->public.table..."
The same for all objects, like func
35 matches
Mail list logo
