search

boknows / cFIREsim-open

An attempt to re-write the popular retirement calculator (cFIREsim.com) in mainly javascript, and begin the journey toward open-source.
Apache License 2.0
154 stars 52 forks source link

Updated Jquery and Dygraphs #51

Closed breadtk closed 9 years ago

breadtk commented 9 years ago

Additionally moved the end of file scripts to be ontop of </body> since this is technically against HTML convention to be outside of <html> tag.

breadtk commented 9 years ago

This pull now includes moving Dygraph from Dygraph.com to Cloudflare's hosted solution which is much faster.

breadtk commented 9 years ago

One other thing, I would also recommend that you take a look at your <script> usage in head. A lot of Cfiresim's load time could be reduced significantly if you used async attributes on <script> tags that need to wait for the Dom. Prime candidates include /js/cFIREsimOpen.js, /js/marketData.js (due to its size), and /js/validation.js.

A longer and detailed explanation of why this is good can be found in this StackExchange answer.