|
|
|
|
|
by joshellington
5018 days ago
|
|
Great start for a first endeavor. Couple notes: - Would definitely recommend rethinking the header/banner area. Takes up a lot of vertical space and seems unnecessary for such a simple tool. - You're loading jQuery twice. Recommend removing that second version (line 14). - http://wordcount.is/js/jquery.center.js is returning 404 and doesn't seem needed (line 12). - It's best practice to cache your selectors for re-use in jQuery. Here's a slight rewrite that does that and cleans up your event binding: https://gist.github.com/3764620 Happy hacking! |
|
I went ahead and implemented all of your suggestions (I think). Seems to be a little bit faster! Thanks.