Hello Jason,
Good work on your first skin and glad to hear you have found DNN C. useful.
On each of the text pages it may help to introduce an image along with the text, such as at:
http://www.stchome.com/Services/Nee...fault.aspxI see that for each page you have created a separate container with an image for the title, you could have implemented this with sIFR meaning that for any future updates you do to the site all you have to do is enter the title and it will automatically be styled in the font required.
Have a look at:
Incorporating sIFR (Scalable Inman Flash Replacement) with DotNetNuke skinsAlongside this you can then implement Header tags - these help the search engines to categorise the content and therefore improve search engine results. - www.skinningtoolkit.com implements the header tag structure. There is a 40page pdf on this here:
Semantic Code and Header Tags in DotNetNuke SkinsIt may also be worth implementing text re-sizing for IE as some users may find it hard to read the smaller font size:
How to create resizable textFor instance on this page the table text at the bottom re-sizes to large, but the main body of text at the top does not resize.
http://www.stchome.com/AboutUs/WebD...fault.aspxThese are things to consider, but overall looks good, thanks,