Re: [pgadmin-hackers] [patch] upgrade slickgrid

Hi Dave

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? 

We haven't noticed any performance issues with this upgraded version of SlickGrid, nor our other changes to the query results grid on the branch that we will be submitting soon. Do you have an example of a large table that used to cause you performance problems? 

Also... would this be an appropriate time to remove the embedded libraries altogether, and move to npm/yarn installations of them? 

I think that would be a great idea. Moving the vendor-ed libraries into a package manager would help us manage upgrading of libraries. The package.json file is a more standard format for managing javascript dependencies, when compared to our libraries.txt file.