Quote:
Originally Posted by tommylogic
Not a fan of the site, generally. Specificly, the menu bar uses images of text instead of actual text... bad practice... Could be done better with css. Next, the content boxes are "one size fits all". And finally, it has programming issues.. on the "what we do" page..
Code:
<!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[if !supportLists]--><!--[endif]--><!--[if !supportLists]--><!--[if !supportLists]--><!--[if !supportLists]--><!--[endif]-->
Then.. wtf is this??? SHODDY!!!
And I wont even get into the general Markup issues.. but needless to say... FAILED Validation 75 Errors
**EDIT**
Sorry Jordon, didn't realize this was yours. But none the less, it needs some work. I understand its a charity piece, but really needs to be tightened up. I've seen your work and I know you know better. The artwork on this (and most of your others) is spot on... clean up the code and use proper techniques.. (like the menu bar, thats just plain lazy) .. Anyhow, nice to see you doing some work for a good cause.
It's a template.. and a REALLY sh*tty one at best !!! |
Thanks for the feedback.
The reason why the invalid code is included is that all of the content on the site is included via a content management system (Joomla) so the weird endif multiple statements have been entered by the client through the WYSIWYG editor.
I admit its bad practice to not run it through a validator before launch, something I am still getting used to.
I will be validating it and trying to solve the issue of the fixed size content box. The main problem with that is I dont know how much content the client will be inputting and each page uses the same code so if I am using a fixed size.
Could anyone recommend a fix for this based on my code?