From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewport to first row in gird - Query tool |
Date: | 2017-11-23 09:07:57 |
Message-ID: | CA+OCxowGGw4y8YyasZd=2S35vcn0m9p+6Z0=_-D1mvQ4jd-SCQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, applied.
On Thu, Nov 23, 2017 at 8:03 AM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> PFA updated patch.
>
>
> On Wed, Nov 22, 2017 at 10:26 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Wed, Nov 22, 2017 at 3:18 PM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> PFA patch to fix the issue where while switching the tabs in query tool
>>> resets the position of the table viewport to the first row.
>>> RM#2875
>>>
>>> *Steps:*
>>> 1) Open query tool
>>> 2) Execute:
>>> select * from generate_series(1, 1000) as "column-1";
>>> 3) Scroll till the End OR till ~250 row
>>> 4) Click on 'Explain' Tab
>>> 5) Go back to 'Data output' Tab
>>>
>>
>> Hmm, minor issue; each time I return the to data tab, the data moves up
>> two rows! It's almost like it's calculating the viewport taking into
>> account the row header (e.g. row 250 is visible immediate below the
>> header), but then re-positioning without taking it into account (e.g. row
>> 250 is aligned with the top of the row header, thus making row 252 the
>> first visible one).
>>
> Fixed.
>
>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2017-11-23 09:21:03 | pgAdmin 4 commit: Make the "Save Changes" prompts in the query tool opt |
Previous Message | Dave Page | 2017-11-23 09:07:50 | pgAdmin 4 commit: Ensure the scroll location is retains in the query to |