To save execution time, wouldn't it be better to do this upon "saving" a template to the database? That way it has already been stripped, and it doesn't need to be done upon page load, i know it won't make much of a performance difference, but if i page uses a LOT of HTML and a LOT of templates combined, it might take longer to load by a little bit.
Also seen as how vBulletin saves the template to the database in 2 forms, you just strip the whitespaces and such for the actual template shown on the site, while the default remains the same for when you edit it via the ACP?
Nice addition none the less, just thought that implementation could make it better
- Zero Tolerance