[pgadmin-hackers] pgAdmin 4 commit: Minimise memory requirements for rendered cells andr - Mailing list pgadmin-hackers

From Dave Page
Subject [pgadmin-hackers] pgAdmin 4 commit: Minimise memory requirements for rendered cells andr
Date
Msg-id E1cK7PE-0005PV-Ey@gothos.postgresql.org
Whole thread Raw
List pgadmin-hackers
Minimise memory requirements for rendered cells and rows in SlickGrid. Fixes #2035

- The CSS class 'slick-row' has been renamed to 'sr'

- The CSS class 'slick-cell' has been renamed to 'sc'

- The SQL Editor included an unused 'normal_row' class on each row which has been removed as it was unreferenced in the
codeor CSS. 

The intent of these changes is to minimise memory usage by the grid, by saving a few bytes per row/cell.
It's questionable how much difference this actually makes given that SlickGrid updates the document so it
only includes the visible rows on the fly, but every little helps.

Branch
------
master

Details
-------
http://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=87d49377261c9a6fa92bd19dc045740bc68eb701

Modified Files
--------------
.../static/css/slickgrid/slick-default-theme.css   | 20 +++++++++---------
web/pgadmin/static/css/slickgrid/slick.grid.css    | 12 +++++------
web/pgadmin/static/js/slickgrid/README             |  9 ++++++++
web/pgadmin/static/js/slickgrid/slick.grid.js      | 24 +++++++++++-----------
.../tools/sqleditor/static/css/sqleditor.css       | 14 ++++++-------
.../sqleditor/templates/sqleditor/js/sqleditor.js  |  4 ++--
6 files changed, 46 insertions(+), 37 deletions(-)


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgadmin-hackers] PATCH: To fix the issue in Notes control (pgAdmin4)
Next
From: Surinder Kumar
Date:
Subject: Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fix RM1790 - [Web] Supportsetting a field's value to "null"