|
|
|||||||||
|
|||||||||
| |||||||||
|
|
|
| |||||||||
![]() |
|
|
«
Previous Thread
|
Next Thread
»
|
Thread Tools | Search this Thread | Rate Thread | Display Modes |
|
#1
|
|||
|
|||
|
Anyone willing...
Would anyone be able to rate my website for me please (http://www.harrymills.co.uk). If you do can you please rate it under the following categories:
>Code >Looks >Content >Accessibility Thanks Pyromancer |
|
#2
|
||||
|
||||
|
Code -> html validate, so thumbs up -> though gallery script doesn't work in Firebird (Mozilla)
Looks -> OK if you like green, bit bland generally Content -> Apart from the blog (not my thing), just seems to be a collection of links. Accessibility -> 508 OK, needs repair with full priority checking according to Bobby http://bobby.watchfire.com/bobby/bo...lth=Bobby%2F3.3Overall, not too shabby ![]()
__________________
Cheers, Jamie # skiFFie | Home of the 'accessibility module' for Drupal # Jamie Burns [me] Accessibility Module [drupal] # guidelines | search | wap resources | not getting help | fold to cure # Any form of employment is strictly prohibited ...... __________________ Let the might of your compassion arise to bring a quick end to the flowing stream of the blood and tears ..... Please hear my anguished words of truth. __________________ |
|
#3
|
||||
|
||||
|
when using xhtml1.1 you should serve the page as application/xhtml+xml, yours is served as text/html,
you have to set this serverside, it doesn't help with the meta-tag. http://www.w3.org/TR/xhtml-media-types/ http://www.greytower.net/en/archive...tmlcontent.html you have a bit illogical order of headings, it would be better if you had the topbanner as h1, then the headers in navigation as h2, and the first p in maincontet also as a h2, then use css to controll the look just an example, <div id="topbanner"><h1>Welcome to harrymills.co.uk</h1></div> <div id="leftcontent"> <h2>Navigation Links</h2> [..] <div id="maincontent"> <h2>On this site I aim to provide:</h2> http://www.webaim.org/techniques/structure/ in lower screen resolution the elements overlap each other, one of the problem with position:absolute, and a set height. would be better if the only navigation was positoned, then use margin-left on the other two elements, something like this. (not tested) div#topbanner { margin-left: 22%; width: 75%; padding: 1%; border: thin dashed #009900; background-color: #006600; font-size: 1.8em; font-family: Verdana, Tahoma, Arial; font-weight: bold; color: #ffffff; text-align: center; } div#maincontent { margin-left: 22%; width: 75%; min-height: 80%; padding: 1%; border: thin solid #009900; background-color: #003300; font-size: 0.8em; color: #FFFFFF; } and remember to set a background color to the body, if you want a white background then set one, not everyone have white as default background color. otherwise a clean and simple design, |
![]() |
| Viewing: Dev Shed Forums > Web Design > Website Critiques > Anyone willing... |
| Thread Tools | Search this Thread |
| Display Modes | Rate This Thread |
|
|
|
|