Skip to content
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

Jump to bottom

Compile piwik.js with Closure Compiler to save a few bytes #4337

Closed
anonymous-matomo-user opened this issue Nov 24, 2013 · 4 comments
Closed

Compile piwik.js with Closure Compiler to save a few bytes #4337

anonymous-matomo-user opened this issue Nov 24, 2013 · 4 comments
Labels
c: Performance For when we could improve the performance / speed of Matomo. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Milestone
Backlog (Help wan...

Comments

@anonymous-matomo-user
Copy link

Compile piwik tracking code piwik.js with following tool:
http://closure-compiler.appspot.com/home

My results:
Original Size: 8.61KB gzipped (21.39KB uncompressed)
Compiled Size: 8.01KB gzipped (19.04KB uncompressed)
Saved 7.02% off the gzipped size (11.02% without gzip)
@mattab
Copy link
Member

mattab commented Nov 25, 2013

Thanks for the suggestion. Interesting but not high impact so moving to requests

@julienmoumne
Copy link
Member

If this is implemented, it would be nice to also apply it to the other js files, see #4373.

@tsteur
Copy link
Member

tsteur commented May 7, 2014

In a3c70f2: refs #4337 some more test fixes

@anonymous-matomo-user anonymous-matomo-user added this to the Future releases milestone Jul 8, 2014
@mattab mattab removed the P: normal label Aug 3, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@tsteur
refs matomo-org#4337 some more test fixes
a3c70f2
@mattab mattab modified the milestones: Long term, Mid term Dec 23, 2015
@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016
@Findus23
Copy link
Member

piwik.js has already been minified for a long time.
Further size improvements could be done in Matomo 4 by removing legacy support: #6093

@mattab mattab added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label Jan 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: Performance For when we could improve the performance / speed of Matomo. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Milestone
Backlog (Help wanted)
Development

No branches or pull requests
5 participants
@tsteur @mattab @julienmoumne @Findus23 @anonymous-matomo-user