petewarden/openheatmap

Issue #9

Closed this issue · 1 comments

This is my fix for Issue #9, apologies if something is wrong I never used github before :-)

#9

Thanks for that, I've merged it in. Funnily enough I noticed something
similar performance-wise when I reused some of the code in another project,
and ported it over to the prototype method, but didn't backport the changes:
https://github.com/petewarden/flxjs/blob/master/flxjs.js

I really appreciate the patch, and great to hear you've been getting use
out of the component!

cheers,
Pete

On Fri, Feb 3, 2012 at 9:50 AM, galund-mintel <
reply@reply.github.com

wrote:

This is my fix for Issue #9, apologies if something is wrong I never used
github before :-)

#9

You can merge this Pull Request by running:

git pull https://github.com/galund-mintel/openheatmap master

Or you can view, comment on it, or merge it online at:

#10

-- Commit Summary --

  • Performance fixes: create class methods using Class.prototype, rather
    than

-- File Changes --

M static/scripts/jquery.openheatmap.js (536)

-- Patch Links --

https://github.com/petewarden/openheatmap/pull/10.patch
https://github.com/petewarden/openheatmap/pull/10.diff


Reply to this email directly or view it on GitHub:
#10