Jump to content

help


jbingman

Recommended Posts

The header banner doesn't look very professional, looks like it was just slapped together quickly. The links are very small and don't really stand out much.

Basically the site looks pretty bland, there is a lot more creativity that can go into it.

Link to comment
https://forums.phpfreaks.com/topic/76603-help/#findComment-387968
Share on other sites

attached image

 

--> Mostly padding issues

 

--> Can't tell which text are links and which aren't (use different color!)

 

--> Logo needs padding, so does the title

 

--> I don't know what this is for, but having advertisements for a CPR class doesn't really work for me. I don't know about you but CPR trainging doesn't have many relevant ads - maybe I'm wrong

 

[attachment deleted by admin]

Link to comment
https://forums.phpfreaks.com/topic/76603-help/#findComment-388677
Share on other sites

Your DTD:

 

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Frameset//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-frameset.dtd">

 

Why frameset? You aren't using frames, do XHTML 1.1 or 1.0 Strict

 

Also, why are you using tables/div tags. It's as though you couldn't decide to go all out CSS or table layout.

Link to comment
https://forums.phpfreaks.com/topic/76603-help/#findComment-389523
Share on other sites

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.