New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minify CSS #7

Closed
Reinmar opened this Issue Apr 6, 2017 · 2 comments

Comments

Projects
None yet
1 participant
@Reinmar
Copy link
Member

Reinmar commented Apr 6, 2017

I noticed we don't minify CSS here :D

@Reinmar

This comment has been minimized.

@Reinmar Reinmar closed this in ae2da32 May 3, 2017

@Reinmar

This comment has been minimized.

Copy link
Member Author

Reinmar commented May 3, 2017

Interestingly, even though the build to which I compared the size was done on a month old codebase (with less features), after enabling CSS minification the overal size decreased so there was quite a lot to gain.

Old buid:

-rw-r--r--   1 p  staff   743783 20 Apr 13:34 ckeditor.compat.js
-rw-r--r--   1 p  staff   418135 20 Apr 13:34 ckeditor.js

New build:

-rw-r--r--   1 p  staff   740860  3 May 18:48 ckeditor.compat.js
-rw-r--r--   1 p  staff   412210  3 May 18:47 ckeditor.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment