I’ve been unable to get your code to work. Would really appreciate it if you could whip up a quick demo page. Thanks!

Thanks for the post Luke.

I would also like to share my experience with using .change() for checkboxes in IE. I think it’s better to use .click() for checkboxes and jquery. Cuz .change() seems to be having problems in IE6 & IE7!!!

Hope this tip will save you guys time…

I think it’d be more correct when js is disabled it degrades to radio buttons so the 3 states are still available. Thus the html starts out as radio buttons, the js rewrites it into the multistate checkbox.

[quote post=”2353″]There is a lot to say about code modularity, but if I’m wasting time hunting down a 5 line function, I’m going to be mad.[/quote]

Heh, been there, done that. Bonus points if it’s a global function called a() which gets overloaded in 7 different places, and the real function is buried under many levels of dependencies (ie. A imports B which imports C and etc – and a() is conveniently located in Z on the bottom of this linked list).

Spaghetti code FTW. :mrgreen:

The second version is more readable. I’ve been finding compactness is more and more relevant to readability than I ever thought. That thought has reduced my commenting by a large amount. When I’m reading someone else’s code (which is most of the time I’m reading code now) and I want to see a function – scrolling, and searching is the worst way. I just want to read it through like a book, and the more of the code I can see at once, the faster I can read it. There is a lot to say about code modularity, but if I’m wasting time hunting down a 5 line function, I’m going to be mad.

Excellent work. If I had written both iterations, I’d be awfully proud of the second block.