Thread: [pgAdmin4][Patch]: RM#1435 - "Move to last page" should also scroll to end
[pgAdmin4][Patch]: RM#1435 - "Move to last page" should also scroll to end
From
Surinder Kumar
Date:
Hi
Please find attached patch with fix.
Every time either a new row is added or pasted. Scroll to the bottom of the table div.
Please review
Thanks,
Surinder Kumar
Attachment
Hi On Wed, Jul 6, 2016 at 9:16 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote: > Hi > > Please find attached patch with fix. > Every time either a new row is added or pasted. Scroll to the bottom of the > table div. > > Please review It doesn't seem to work for me in either Safari or Chrome. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
Re: [pgAdmin4][Patch]: RM#1435 - "Move to last page" should also scroll to end
From
Surinder Kumar
Date:
On Thu, Jul 7, 2016 at 2:30 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi
On Wed, Jul 6, 2016 at 9:16 AM, Surinder Kumar
<surinder.kumar@enterprisedb.com> wrote:
> Hi
>
> Please find attached patch with fix.
> Every time either a new row is added or pasted. Scroll to the bottom of the
> table div.
>
> Please review
It doesn't seem to work for me in either Safari or Chrome.
Sorry, I missed this case, Now I have fixed it and it should work.
Please find the updated patch and review.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment
Thanks, applied. On Thu, Jul 7, 2016 at 11:07 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote: > On Thu, Jul 7, 2016 at 2:30 PM, Dave Page <dpage@pgadmin.org> wrote: >> >> Hi >> >> On Wed, Jul 6, 2016 at 9:16 AM, Surinder Kumar >> <surinder.kumar@enterprisedb.com> wrote: >> > Hi >> > >> > Please find attached patch with fix. >> > Every time either a new row is added or pasted. Scroll to the bottom of >> > the >> > table div. >> > >> > Please review >> >> It doesn't seem to work for me in either Safari or Chrome. > > Sorry, I missed this case, Now I have fixed it and it should work. > Please find the updated patch and review. >> >> >> -- >> 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