Re: [patch] upgrade slickgrid

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Joao Pedro De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [patch] upgrade slickgrid
Date: 2017-05-17 12:59:50
Message-ID: CA+OCxozd7t-rhNetnT_edWnZtDKGY-fidWGxDWQi4VM4yjer=g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Mon, May 15, 2017 at 7:35 PM, Joao Pedro De Almeida Pereira <
jdealmeidapereira(at)pivotal(dot)io> wrote:

> Hello Hackers
>
> We upgraded SlickGrid to the latest version + 53ee34. We upgraded since we
> are working on some changes to the query results functionality, including
> improvements to column selection and shortcuts.
>
> This SlickGrid upgrade includes a PR we wrote that simplifies some of
> these changes.
>
> Please note that we are not requesting this be included in the 1.5 release.
>

Have you confirmed that removing the changes we made to shorten the class
names used for rows and cells doesn't affect the performance or memory
utilisation of the grid?

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-05-17 13:01:43 Re: [patch] upgrade slickgrid
Previous Message Murtuza Zabuawala 2017-05-17 10:41:02 Re: [pgAdmin4][PATCH] To fix the issue with Node rename