|
|
|||||||||
|
|||||||||
| |||||||||
|
|
|
| |||||||||
![]() |
|
|
«
Previous Thread
|
Next Thread
»
|
Thread Tools | Search this Thread | Rate Thread | Display Modes |
|
#1
|
||||
|
||||
|
http://cratersoft.wefixtech.co.uk/inadequacy/about.php
Soon to be inadequacy.org
Im going to buy the domain when i finish the site, but that is the only finished page right now. Any comments/suggestions? Thanks |
|
#2
|
||||
|
||||
|
Everything seems nicely valid, which is always a good sign
The site scales nicely, with font-size and window size. I have to say I'm a little confused about why you've used a <table> for you main menu <ul> it up! Also, you seem have some inline styling, this is never a good thing. The design is clean, clear, simple etc. (although the Tux is a little confusing, not sure how it relates) Can I also suggest using an image replacement technique to get your header into a <h1> tag. Hope that helps.
__________________
Control post quality through the power of rep' ( )
|
|
#3
|
||||
|
||||
|
Im working on getting it all fixed before I drop ALL inline styling ;p
I had some issues using a list in place of the table, but will give it another go. There will be a few images next to tux. A nice row perhaps. Will have to see how it plays out. Could you explain your last comment please. Thanks for the review ;p |
|
#4
|
||||
|
||||
|
Quote:
I will try. To make your code more semantic (meaningful), the image saying "inadequacy - News for Trolls" at the top of your page could be set as the background image to a <h1> element. For example: <h1 style="background: url('img/logo.gif') top left no-repeat; text-indent: -3000px; height: 60px; width: 468px">Inadequacy - News for Trolls</h1> That should look like exactly what you have at the moment, however, there is now some text for a screen reader to find, and also you've identified the title of the document. There are many different techniques for doing it, and the above is just one which I've yet to be told of a bug with (although some will probably post with one shortly). Again, I hope that helps ![]() |
![]() |
| Viewing: Dev Shed Forums > Web Design > Website Critiques > http://cratersoft.wefixtech.co.uk/inadequacy/about.php |
| Thread Tools | Search this Thread |
| Display Modes | Rate This Thread |
|
|
|
|