What do you think of my website ?

[JamesBurton]

What do you think of my website still under development but the style and design.

http://www.sitessl.co.uk

[JamesBurton]

Hello

[JamesBurton]

Hello What do you think of my website ?

[Paulie_D]

Seriously…have some patience.

But at the moment…there is really nothing to comment on.

[chrisburton]

@Paulie_D Haha.

@JamesBurton There’s nothing really to critique. You have a background and a box.

[JamesBurton]

OK could is the code right and what could i improve on ?
does the colors look right and etc

[Paulie_D]

@JamesBurton Are you brand new to web design..because there is a lot in the code that could be improved but if I started listing it it might discourage you?

At the very least:
I would suggest that you look at the HTML5 elements (header/nav/footer) that are available to you, learn how to structure a navigation etc. Also, you can get rid of the break tags between the divs and learn margins etc.

[JamesBurton]

@Paulie_D Yes a am a new web designer if you can list it a would be great

[chrisburton]

@Paulie_D I don’t think there’s anything wrong with break tags as long as they’re used properly. Such as breaking up sections in a sidebar. Of course this is a separate discussion.

[JamesBurton]

@ChristopherBurton Could you list what wrong with my code and i can improve it and be a better web designer

[Paulie_D]

We’re not saying that it’s wrong…it works after all but it’s not..erm…current.

Not everything needs to be in a div and some of your ID names are less than descriptive.

For instance, you have div with an ID of top whereas you could just use the ‘header’ element and do away with a lot of that.

See http://html5doctor.com/ for a list of useful HMTL5 elements. Perhaps take a look at some more tutorial sites (http://net.tutsplus.com/ or http://webdesign.tutsplus.com/) are some good starting points.

[chrisburton]

@JamesBurton I’m a letterer, man. I’m not sure I could be much help with that. And it seems @Paulie_D is helping you without doing the work for you.

[JamesBurton]

what Should a change ( top ) = header

in the code

Home

Contact
















Welcome to SiteSSL UK





























© 2012 SiteSSL UK Ltd

CSS for the code

body

{

font-family: sans-serif;

margin: auto;

padding: 0px;

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

}



#information2

{

width: 1000px;

margin: 0 auto;

}



#top

{

/*background-color: #474544;*/

background-color: #0072C6;

color: #fff;

height: 40px;

}



.toplink

{

color: #808080;

text-decoration: none;

font-size: 14px;

}



.toplink:hover

{

color: #fff;

}



#information

{

padding: 6px;

height: 400px;

background-color: #fff;

/*border-width: 10px;

border-style: solid;

border-color: transparent;*/

border: 8px solid transparent;

}



.text1

{

font-size: 20px;

}



#bottom

{

height: 40px;

font-size: 12px;

}



.Box1

{

float: left;

height: 20px;

width: 60px;

color: #fff;

padding: 10px;

text-decoration: none;

}



.Box1:hover

{

background-color: rgba(170, 170, 170, 0.34);

color: #fff;

}



#top2

{

height: 20px;

font-size: 20px;

}



#search

{

float: right;

}



#option2

{

width: 1000px;

margin: 0 auto;

}



#option

{

float: right;

}



.Color

{

/*border: 10px solid transparent;*/

border: 10px solid rgba(0, 0, 0, 0.10); /*rgba(255, 255, 255, 0.40);*/

}

[chrisburton]

This is how I would write it.

Home

Contact










Welcome to SiteSSL UK


















This is where you box would go







 © 2012 SiteSSL UK Ltd

[JamesBurton]

@ChristopherBurton I Have a look the the code and i think it is better because it has bin layout
better and easy to read

Would the CSS be smaller with the code you give ?

Thank you

[Author]

Posts