21 ms·
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 unne
by joshellington 14y 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 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 https://gist.github.com/3764620
Happy hacking!
- joelrunyon 14y agoHey Josh, I went ahead and implemented all of your suggestions (I think). Seems to be a little bit faster! Thanks.