You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In order to improve performance of the web applications, support for minification for JS and CSS files was added in e74b390, using flask-assets, cssmin, and jsmin.
Since then the application has started using SASS for CSS minification (see #2808), meaning only JS is being minified. Only 2 files are currently being minified, static/js/{source,journalist}.js, with savings of <1k and <2k respectively.
To reduce code complexity at the cost of 1-2k in page size, it would make sense to drop JS minification as well. Alternatively, the files could be minified during the build process (tho IMO it's overkill given the minimal gains).
One other disadvantage of server-side minification is that the Apache user needs to be able to write the minified files somewhere on the webroot. Removing it would allow for tightening up the relevant apparmor profile, removing the related write perms
User Research Evidence
n/a
User Stories
n/a
The text was updated successfully, but these errors were encountered:
So +1 to getting rid of this extra complexity. The jsmin package is also unmaintained (c.f. tikitu/jsmin#33 (comment)) and problematic because the version we're on uses a now-removed setuptools option, which I had to workaround in #6358 (comment).
Description
In order to improve performance of the web applications, support for minification for JS and CSS files was added in e74b390, using
flask-assets
,cssmin
, andjsmin
.Since then the application has started using SASS for CSS minification (see #2808), meaning only JS is being minified. Only 2 files are currently being minified,
static/js/{source,journalist}.js
, with savings of <1k and <2k respectively.To reduce code complexity at the cost of 1-2k in page size, it would make sense to drop JS minification as well. Alternatively, the files could be minified during the build process (tho IMO it's overkill given the minimal gains).
One other disadvantage of server-side minification is that the Apache user needs to be able to write the minified files somewhere on the webroot. Removing it would allow for tightening up the relevant apparmor profile, removing the related write perms
User Research Evidence
n/a
User Stories
n/a
The text was updated successfully, but these errors were encountered: