Thanks for taking a look at it and thank you for the quick feedback. I'll make those adjustments you suggested.

As far as excessive AJAX requests, it is doing them onChange of any attributes that exist...so I guess if you are looking at a page with a bunch of products each having a bunch of attributes it is going to make a separate AJAX call for every attribute during the initial page load. I might be able to alter it so that on initial load it only makes one call per product or so that an adjusted price is calculated right off the bat without AJAX calls.

I'll need to set up a larger sample cart to get a feel for what is practical.