Re: [pgadmin-hackers] Keywords in Uppercase

2011-11-01 Thread Vinicius Santos
Patch attached. 0002-Keywords-in-Uppercase.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

Re: [pgadmin-hackers] Option "Nulls First" in the ordering screen.

2011-11-01 Thread Vinicius Santos
Hello, I implemented a function for the two buttons "Asc" and "Desc", so that the maintenance is easier. Take a look and tell me what you think. Thanks. 0003-Implementation-Nulls-First-option-on-dlgEditGridOpti.patch Description: Binary data -- Sent via pgadmin-hackers mailing list (pgadmin-

[pgadmin-hackers] pgAdmin III commit: Change location of the "Keywords in Uppercase" chec

2011-11-01 Thread Guillaume Lelarge
Change location of the "Keywords in Uppercase" checkbox It didn't belong to the Colours item. A better location would be the "Query editor" item. Branch -- master Details --- http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=f9a5b05d667e952694b972abd749298f5eb6c8b5 Author: V

Re: [pgadmin-hackers] Keywords in Uppercase

2011-11-01 Thread Guillaume Lelarge
On Tue, 2011-11-01 at 08:41 -0200, Vinicius Santos wrote: > Patch attached. Thanks, I pushed it. -- Guillaume http://blog.guillaume.lelarge.info http://www.dalibo.com -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://ww

Re: [pgadmin-hackers] Option "Nulls First" in the ordering screen.

2011-11-01 Thread Guillaume Lelarge
On Tue, 2011-11-01 at 08:47 -0200, Vinicius Santos wrote: > Hello, > > I implemented a function for the two buttons "Asc" and "Desc", so that > the maintenance > is easier. > > Take a look and tell me what you think. > Well, I don't like the UI. It takes too much space for only one information.

[pgadmin-hackers] [PATCH] Improve autocompletion for SELECT statements

2011-11-01 Thread Wander Winkelhorst
Hello, A big part of my job is creating simple select statements with a few joins in them. Unfortunately, the existing autocomplete functionality of pgadmin was a bit too limited for my use case. A relatively simple query like: select * from person p join company c on p.company_id = c.id where p.n

Re: [pgadmin-hackers] [PATCH] Improve autocompletion for SELECT statements

2011-11-01 Thread Guillaume Lelarge
Hi, On Mon, 2011-10-31 at 20:23 +0100, Wander Winkelhorst wrote: > Hello, > > A big part of my job is creating simple select statements with a few joins > in them. Unfortunately, the existing autocomplete functionality of pgadmin > was a bit too limited for my use case. A relatively simple query

Re: [pgadmin-hackers] Option "Nulls First" in the ordering screen.

2011-11-01 Thread Vinicius Santos
> Well, I don't like the UI. It takes too much space for only one > information. I would better like a combobox, with the two choices. > I thought so too. I agree with you. > Moreover, it simply doesn't work on Linux. I've got an assertion failure > on OnPaint for the list. I actually don't unde

Re: [pgadmin-hackers] [PATCH] Improve autocompletion for SELECT statements

2011-11-01 Thread Wander Winkelhorst
Hi, On Tue, Nov 1, 2011 at 2:50 PM, Guillaume Lelarge wrote: > > Line 482 is when you freed the buffer. Not sure it's relevant to this > issue, but you have a bug in your code. > Sorry; seems like the allocation was off by one. Attached is the updated patch. Regards, Wander Winkelhorst. diff -

Re: [pgadmin-hackers] Copy/Paste table(s) functions - git context patch

2011-11-01 Thread Guillaume Lelarge
On Thu, 2011-10-13 at 06:22 +0200, Vladimir Kokovic wrote: > Hi, > > I finally found the time to make a new version ! > > > * When I copy a table, and then paste it, I can not paste it once more. > > Why is that? > Fixed > OK. > > * If I paste a table and there is an error (for example, the ta

Re: [pgadmin-hackers] Option "Nulls First" in the ordering screen.

2011-11-01 Thread Guillaume Lelarge
On Tue, 2011-11-01 at 12:07 -0200, Vinicius Santos wrote: > > Well, I don't like the UI. It takes too much space for only one > > information. I would better like a combobox, with the two choices. > > > > I thought so too. I agree with you. > OK. > > > Moreover, it simply doesn't work on Linux