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

From Murtuza Zabuawala
Subject Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewportto first row in gird - Query tool
Date
Msg-id CAKKotZRWj5-nBJydkL57dyXBhcVCUF5yw-bQ3jsq-KAvAHfMww@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewportto first row in gird - Query tool  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch]: Switching between Tabs resets table viewportto first row in gird - Query tool  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
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

Attachment

pgadmin-hackers by date:

Previous
From: Harshal Dhumal
Date:
Subject: Fix for RM2845: Make "Save changes" prompt configurable
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Fix bundle creation on Windows which was failingdue