Hi! I'm new to this forum and I really hope to find some help here. I almost finished my new website and would love to know what you think about it... www.ukgec.com

I'm learning myself through the books and Internet and I'm sure there are lots of things that need changing. I value your opinions. Thanks!

take out the <center></center> tags as this is the wrong way to center a site instead in the #wrap of your css add:

 

margin:0 auto;

 

 

Also I don't understand why you have two different footer styles on the site for your copyright information, I know you need to have the powered by cubecart stuff but it would be better if it was held within the same style as the rest of the pages footers

 

 

 

Other than that its good for a first site you might like http://htmldog.com/ if you haven't been on that site yet, it is very good for tutorials

Great first attempt!

 

Size and colour scheme are fine. Not 100% sure about the background. Seems a bit fussy. Also suggest alining your menu to the left.

 

Code, wise, just had a quick look so cant comment too much but recommend swapping out any images displaying text, with actual text links.

take out the <center></center> tags as this is the wrong way to center a site instead in the #wrap of your css add:

 

margin:0 auto;

 

 

Also I don't understand why you have two different footer styles on the site for your copyright information, I know you need to have the powered by cubecart stuff but it would be better if it was held within the same style as the rest of the pages footers

 

 

 

Other than that its good for a first site you might like http://htmldog.com/ if you haven't been on that site yet, it is very good for tutorials

Thanks for the quick reply! I used <center></center> in my code as using CSS centering (margin: auto;) doesn't really work in other browsers (like Firefox). Althogh I'm not sure should I really bother with Firefox?

As to the different footer styles you are right it does look wrong. To be honest I didn't pay attention to it as I hope to get rid of cubecart staff in couple of days after I buy the license key.

Thanks a lot again for your help and for the link as well.

Thanks for the quick reply! I used <center></center> in my code as using CSS centering (margin: auto;) doesn't really work in other browsers (like Firefox). Althogh I'm not sure should I really bother with Firefox?

As to the different footer styles you are right it does look wrong. To be honest I didn't pay attention to it as I hope to get rid of cubecart staff in couple of days after I buy the license key.

Thanks a lot again for your help and for the link as well.

 

You should definitely bother with firefox!

 

Its an extremely popular browser now.

Great first attempt!

 

Size and colour scheme are fine. Not 100% sure about the background. Seems a bit fussy. Also suggest alining your menu to the left.

 

Code, wise, just had a quick look so cant comment too much but recommend swapping out any images displaying text, with actual text links.

Thanks!!! I will consider another background. As to alining menu to the left I think you are right, I will try it tomorrow.

I didn't really understand which images displaying text you ment?

You should definitely bother with firefox!

 

Its an extremely popular browser now.

I'm not alone!!! It makes me feel better Thank you!

Im confused, you said;

 

Althogh I'm not sure should I really bother with Firefox?

 

I said you should bother

Not too bad for a first site. The background must go though (in my opinion), it really reminds me of being back in 1996, when the web was full with horrible 'porridge' style backgrounds. *shudders*

you will find that margin 0 auto works on most modern browsers as it is the css way of centering you can get around older browsers too by adding text-align: center to the body and then just aligning it back to the left in the wrapper But IE6 + will center using margin 0 auto.

you will find that margin 0 auto works on most modern browsers as it is the css way of centering you can get around older browsers too by adding text-align: center to the body and then just aligning it back to the left in the wrapper But IE6 + will center using margin 0 auto.

Thanks for advise but I have tried this before. It's not working in Firefox. This is a part of my CSS code, could you have a look please, what do I do wrong?

 

body {

text-align:center;

background-image:url(../image/bg.png);

background-repeat:repeat-x;

margin: auto;

}

/* Make background white for main part, width 779, text to the left*/

#main {

background-color:#FFFFFF;

width:779px;

text-align:left;

margin:0;

padding:0;

}

 

Thank you!

Yeah that's not at all how you center in css. Get rid of the margin: auto for the body. You can't center the body, because body contains everything on the page.

 

Give #wrap margin: 0 auto; , that should work. You are trying to center your content, which is in #wrap. So center it! Then you can get rid of <center></center>

• I dont like the font of the links (About us...) EDIT: And the mouse-over color is just horrible
  
• You should left-align the links (About us...)
  
• You should give the footer more space to the top (below the <hr>)
  
• The background should not be a repeated image. It simply looks rooky.
  
• A mouse-over effect on the top-right buttons would be nice.
  

 

My constructive feedback

Enjoy

Give #wrap margin: 0 auto; , that should work.

Thanks for that!!! It worked indeed.

take out the <center></center> tags as this is the wrong way to center a site instead in the #wrap of your css add:

 

margin:0 auto;

 

 

Also I don't understand why you have two different footer styles on the site for your copyright information, I know you need to have the powered by cubecart stuff but it would be better if it was held within the same style as the rest of the pages footers

 

 

 

Other than that its good for a first site you might like http://htmldog.com/ if you haven't been on that site yet, it is very good for tutorials

 

 

For a first attempt… Good work!

 

But if I look at it from a visitor’s point of view, it was not that pleasant an experience.

 

To start with, the homepage could have been a lot better.

The menu list should be made more prominent and stand apart from the rest of the content.

The alignment of the website should be re-done

Fonts like Arial, Tahoma and Verdana should be used instead.

The header image of your website looks shrunken

The rounded corners are not proper

 

That’s for now. Lets see how the website looks once you make these changes..

 

Thanks,

 

Web designer services

I have learned so much while trying to change this website... I feel like redoing this site from scratch. I hope you all will tell me your opinion on my new one. Thanks!