Re: PATCH: Fix Explain data output column not auto-sizing

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: "J(dot)F(dot) Oster" <jinfroster(at)mail(dot)ru>, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Fix Explain data output column not auto-sizing
Date: 2014-10-30 05:41:00
Message-ID: CAG7mmoyAbHpAHYbbMtFiFQ0-pUrvVoafPa=AV2-ioxNZ4ZDQ5g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Committed to the master branch.
Thanks for the patch

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com>

*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>

On Mon, Oct 27, 2014 at 6:29 PM, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com
> wrote:

> Hi,
>
> I have reviewed the patch.
> It looks good to me.
>
>
> Regards,
> Sanket Mehta
> Sr Software engineer
> Enterprisedb
>
> On Mon, Oct 27, 2014 at 4:14 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Ashesh, can you review/commit this please?
>>
>> On Fri, Oct 24, 2014 at 6:02 PM, J.F. Oster <jinfroster(at)mail(dot)ru> wrote:
>> > Hello!
>> >
>> > There is a small bug with Explain data output.
>> >
>> > Steps to reproduce:
>> >
>> > 1. Open a new Query window.
>> > 2. Perform Explain of any query. Switch to "Data output" tab. Column
>> > was auto-sized, ok.
>> > 3. Execute the query.
>> > 4. Perform Explain again. Switch to "Data output" tab. Column wasn't
>> > auto-sized this time, it is too narrow and user has to resize it
>> > manually.
>> >
>> > Here is a fix, please see attachment.
>> >
>> > --
>> > Best regards,
>> > J.F.
>> >
>> > --
>> > Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> > To make changes to your subscription:
>> > http://www.postgresql.org/mailpref/pgadmin-hackers
>> >
>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2014-10-30 05:51:22 pgAdmin III commit: Added support for Slony-I 2.2+ with PostgreSQL 9.3+
Previous Message Ashesh Vashi 2014-10-30 05:38:57 Re: PATCH: Clear undo/redo history when a file is opened in Query editor