| From: | Erwin Brandstetter <brandstetter(at)falter(dot)at> |
|---|---|
| To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
| Cc: | pgadmin-hackers(at)postgresql(dot)org, Dave Page <dpage(at)pgadmin(dot)org>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
| Subject: | Re: Changed default ordering in tables |
| Date: | 2011-08-08 12:47:36 |
| Message-ID: | 4E3FDAE8.7070008@falter.at |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
Thanks to both of you!
Looking forward to the next release!
--
Erwin Brandstetter
On 06.08.2011 14:49, Guillaume Lelarge wrote:
> On Fri, 2011-07-29 at 10:33 +0100, Dave Page wrote:
>> On Fri, Jul 29, 2011 at 9:14 AM, Erwin Brandstetter
>> <brandstetter(at)falter(dot)at> wrote:
>>> So, that's two distinct issue:
>>> - "Last 100 rows" broken for multi-column pkeys.
>>> - Default sort order DESC for edit grid, where it should be ASC.
>> That one I can reproduce, and fix. Patch attached.
>>
>> Ashesh, can you review and test please? We're getting close to release
>> so I want to be extra careful.
>>
> I checked your patch. Seems good to me. So I applied it. I also applied
> another patch to fix Erwin's bug (missing initialization of the
> pkAscending variable).
>
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Guillaume Lelarge | 2011-08-08 20:07:52 | pgAdmin III commit: Fix "Current log" selection in the log combobox |
| Previous Message | Vladimir Kokovic | 2011-08-08 04:24:56 | Re: Copy/Paste table(s) functions - git context patch |