The design looks good. I'm liking the navigation menu, quite a unique way of doing it.
Looking at the source of the website I noticed the CSS was not in a separate file, to clean it up she should place it in a file like style.css and just link to that in the <head> tag of the html.
Also to make the markup valid, she needs to put an 'alt' attribute in the img tags.
The naming of the CSS IDs is also a little hard to follow, instead of naming them 'style12' or 'appDiv12' she could use names such as 'footer' and 'content_left' etc. Which would be easier to edit if she or someone else needs to in the future.
Finally I'm not really sure about this myself but she is using the div align in the XHTML, which should probably be set in the styling. Using the margin:auto and float left or right should achieve the same result. However from my limited experience working with floats, it can be a pain so keeping in the alignment in the div tags should be fine!
This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
By continuing to use this site, you are consenting to our use of cookies.