![]() | |
![]() |
| | Thread Tools | Display Modes |
#1
| |||
| |||
|
#2
| |||
| |||
|
|
Hi Please could you critique my site. I've finally ditched tables and this is css buisness is new to me! www.paulwatt.info Overall a great start! |
#3
| |||
| |||
|
|
Paul Watt wrote: Hi Please could you critique my site. I've finally ditched tables and this is css buisness is new to me! www.paulwatt.info Overall a great start! General: - Remove the period at the end of the main header ("Welcome ..."). - Remove the phrase "is able to" from the text. It reduces the dynamism of the sentences, i.e., you are able to, not that you do. (E.g.: "provides" instead of "is able to provide.") - There is no need for a <br /> in the first paragraph. - "Its impossible" should be "It's impossible." - "Its impossible" and the following paragraph should be combined. They say basically the same thing. Layout: - Use header element, <h1>, for the header, not <p><strong>. Style <h1> to taste. - Some margin for maincontent would make the text a bit easier to read. - "<p> </p>" for line spacing. Ack! If you want extra spacing at the bottom of maincontent, use padding-bottom. |
#4
| |||
| |||
|
|
"Jim Moe" <jmm-list.AXSPAMGN (AT) sohnen-moe (DOT) com> wrote in message news:GY-dnTfivOczU7fZnZ2dneKdnZydnZ2d (AT) giganews (DOT) com... Paul Watt wrote: Hi Please could you critique my site. I've finally ditched tables and this is css buisness is new to me! www.paulwatt.info Overall a great start! General: - Remove the period at the end of the main header ("Welcome ..."). - Remove the phrase "is able to" from the text. It reduces the dynamism of the sentences, i.e., you are able to, not that you do. (E.g.: "provides" instead of "is able to provide.") - There is no need for a <br /> in the first paragraph. - "Its impossible" should be "It's impossible." - "Its impossible" and the following paragraph should be combined. They say basically the same thing. Layout: - Use header element, <h1>, for the header, not <p><strong>. Style <h1> to taste. - Some margin for maincontent would make the text a bit easier to read. - "<p> </p>" for line spacing. Ack! If you want extra spacing at the bottom of maincontent, use padding-bottom. Thanks Jim, it looks much better now. However when I tried to apply padding to the maincontent div to push the text from the edge, it pushed the other side of the div out of line with the header graphic. would setting max-width do the trick? cheers Paul |
#5
| |||
| |||
|
|
Thanks Jim, it looks much better now. However when I tried to apply padding to the maincontent div to push the text from the edge, it pushed the other side of the div out of line with the header graphic. would setting max-width do the trick? Did you set "padding" or "padding-bottom"? Since all you seem to need is |
![]() |
| Thread Tools | |
| Display Modes | |
| |