Print

[scifiguy]

In particular I find the following problematic in the powered by credit:

- Using a generic "footer" class causes it conflict with many site's existing stylesheets making site integration more difficult. I recommend using something unique and descriptive like "cpgcredit" or even "cpgfooter" so it can be styled independantly and is unlikely to conflict with other stylesheets.

- You have hardcoded inline styling on the div, including centering it and adding top padding. I am at a loss as to why you would do this on code that is inaccessible to the page designer. I would like to see the inline styling removed so that the external CSS file can fully determine its display.

Currently it is trivial to totally hide the powered by credit via css so I really don't know why you have made it so difficult to access the actual HTML portion of it in the first place. Regardless, as far as I can tell the above changes would make it vastly more user friendly without having any impact on whatever value you feel the current code obfuscation provides.

[Joachim Müller]

you're right, marking this thread as "valid" and moving to "approved" section.