HighDots Forums  

please critique site

Websites/HTML pages critique & reviews Discuss and review existing WWW material (alt.html.critique)


Discuss please critique site in the Websites/HTML pages critique & reviews forum.



Reply
 
Thread Tools Display Modes
  #1  
Old   
Paul Watt
 
Posts: n/a

Default please critique site - 03-29-2006 , 10:43 AM






Hi Please could you critique my site. I've finally ditched tables and this
is css buisness is new to me!

the url is :

www.paulwatt.info

Many Thanks

Paul Watt



Reply With Quote
  #2  
Old   
Jim Moe
 
Posts: n/a

Default Re: please critique site - 03-29-2006 , 01:15 PM






Paul Watt wrote:
Quote:
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>&nbsp;</p>" for line spacing. Ack! If you want extra spacing at the
bottom of maincontent, use padding-bottom.

--
jmm (hyphen) list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)


Reply With Quote
  #3  
Old   
Paul Watt
 
Posts: n/a

Default Re: please critique site - 03-29-2006 , 02:17 PM




"Jim Moe" <jmm-list.AXSPAMGN (AT) sohnen-moe (DOT) com> wrote

Quote:
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>&nbsp;</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




Reply With Quote
  #4  
Old   
Nije Nego
 
Posts: n/a

Default Re: please critique site - 03-29-2006 , 04:14 PM



On Wed, 29 Mar 2006 20:17:32 +0100, Paul Watt wrote:

Quote:
"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>&nbsp;</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
Decrease width for the value of padding.
--
o'tom po'tom


Reply With Quote
  #5  
Old   
Jim Moe
 
Posts: n/a

Default Re: please critique site - 03-29-2006 , 07:25 PM



Paul Watt wrote:
Quote:
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
extra space at the bottom, padding-bottom is what to use.

--
jmm (hyphen) list (at) sohnen-moe (dot) com
(Remove .AXSPAMGN for email)


Reply With Quote
Reply




Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off



Powered by vBulletin Version 3.5.4
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.