Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewportto first row in gird - Query tool - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewportto first row in gird - Query tool
Date
Msg-id CA+OCxowGGw4y8YyasZd=2S35vcn0m9p+6Z0=_-D1mvQ4jd-SCQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewportto first row in gird - Query tool  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers
Thanks, applied.

On Thu, Nov 23, 2017 at 8:03 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Dave,

PFA updated patch.


On Wed, Nov 22, 2017 at 10:26 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Nov 22, 2017 at 3:18 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Ensure the scroll location is retains in the queryto
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Make the "Save Changes" prompts in the query toolopt