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

Revisit JavaScript graphing libraries #2921

Closed
robocoder opened this issue Feb 13, 2012 · 2 comments
Closed

Revisit JavaScript graphing libraries #2921

robocoder opened this issue Feb 13, 2012 · 2 comments
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. worksforme The issue cannot be reproduced and things work as intended.

Comments

@robocoder
Copy link
Contributor

This is a planning ticket.

Phase 1

  • ~~We're currently using jqplot (to replace OpenFlashCharts). There is ~460K of unminified JavaScript in libs/jqplot.~~
  • We should use a concatenated & minified version, instead of deferring this to the AssetManager. (YUIcompressed to about 180K.)
  • We should sync up with upstream version of jqplot

Phase 2

  • greg has indicated that the new map implementation will use Raphal (v2.0.2 is 86K minified).
  • Since Raphal partially overlaps jqplot, I propose we investigate replacing jqplot with gRaphal (v0.5.0 would add 29K minified). (A net reduction of 65K.)
@robocoder
Copy link
Contributor Author

(In [5845]) fixes #2934, refs #2921 - pre-concatenate & minify jqplot library and plugins

  • a single 164K file (yuicompressed) vs 460K spread over 18 files
  • this will improve initial Dashboard loading time by reducing the AssetManager's workload
  • deferring sync up with upstream jqplot to next milestone

@mattab
Copy link
Member

mattab commented Dec 14, 2012

If we need to update jqplot, let's crete a new ticket, work seems done on this one (Change of plans, the SVG Maps use https://github.com/gka/piwik-map-widget/blob/master/js/vendor/raphael-min.js ) -- please reopen if you can think of any problem here. Thanks!

@robocoder robocoder added this to the Future releases milestone Jul 8, 2014
@robocoder robocoder mentioned this issue Feb 15, 2012
Closed
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. worksforme The issue cannot be reproduced and things work as intended.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mattab @robocoder