Winner Computing

therubbad · #1 Apr 15th, 2007, 23:30

Hey, made this, been designing for about a month, need your opinions on what to remove/add

Lchad · #2 Apr 15th, 2007, 23:56

Very nice basic site. I would add more space between columns and more space on the left and right margin. It seems tight!

You might want to consider shortening the text a bit and try to summarize it a bit using bullets. People have really short attention spans and need you to get to the point. We also need visuals to break up all the text. Photos, bullets, headers, subheaders, different colors etc.

Now that the basics are there, jazz it up!

abrewitt · #3 Apr 25th, 2007, 07:48

For designing for a month you have done the right thing and kept it simple - good work, add some good quality photos in their and maybe some vector graphics to make ot distinguishable.

karinne · #4 Apr 25th, 2007, 12:05

Not bad for a first site ... it's pretty bland in the content area. You'll need to spice that up

Have a look at what others are doing by browsing through the various css showcase sites out there Get some inspiration from them.

The code is not bad really ... just a few things you should be doing in the way of semantics. You're not using the proper elements for the job.

Example:

Code: Select all

<div id="navbar">&nbsp; &nbsp;<span class="style2"> <a href="/index.html"> HOME</a> &nbsp; &nbsp;<a href="../about/index.html">ABOUT US </a> &nbsp; &nbsp; <a href="../pcteam/index.html"> PC ASSISTANCE TEAM </a> &nbsp; &nbsp; <a href="../design/index.html">WEB DESIGN </a> &nbsp; &nbsp; <a href="../contact/index.html">CONTACT US </a></span></div>

A menu is basically a list of items so ... put it in an unordered list - CSS Design: Taming Lists

Code: Select all

<div align="center"><span class="style3">Let Us Help You Enjoy Your Computer </span></div>

and

Code: Select all

<span class="style3">Welcome</span>

Should be headings instead ... like

Code: Select all

<h1>Let Us Help You Enjoy Your Computer</h1>

and so on ... you can style the headings in your css

Code: Select all

h1 {
   color: #090;
   margin: 3px 0 5px 4px;
}

etc...

and don't wrap paragraphs in <span> tags... use the <p> element ..

Code: Select all

<span class="style2">&nbsp;&nbsp;&nbsp;&nbsp; Here at Winner Computing, our primary goal is for you to enjoy your computer and use it to its full extent. We are an innovative developer of websites and a source of learning for the general public. From our certified technicians to our professional web designers we can offer you complete computer training, design your personal or professional web sites and offer you the best hosting packages out there today. We strive to offer you the best possible service and support.
<br><br>
   &nbsp;&nbsp;&nbsp;&nbsp;   Want to learn more about your computer? Sure, you can take classes at your local college but by the time you learn the basics this can run into several hundred dollars or more and you have only scratched the surface.
We at Winner Computing, can help you learn only those areas you wish to learn and at a more affordable rate. Just want to learn how to install software? Or, add home networking? We can help you with all of this and more.[
</span>

should be

Code: Select all

<p>Here at Winner Computing, our primary goal is for you to enjoy your computer and use it to its full extent. We are an innovative developer of websites and a source of learning for the general public. From our certified technicians to our professional web designers we can offer you complete computer training, design your personal or professional web sites and offer you the best hosting packages out there today. We strive to offer you the best possible service and support.</p>

<p>Want to learn more about your computer? Sure, you can take classes at your local college but by the time you learn the basics this can run into several hundred dollars or more and you have only scratched the surface.
We at Winner Computing, can help you learn only those areas you wish to learn and at a more affordable rate. Just want to learn how to install software? Or, add home networking? We can help you with all of this and more.</p>

and to get that indent instead of all those   ... you use the text-indent property

Code: Select all

p {
  text-indent: 5px;
}

mikew2479 · #5 Apr 25th, 2007, 14:01

the initial design bit at the top ie the green lines look ok and the text colour sort of goes, i'm not saying i'm a great designer but i can critique on what appeals to me, the text as i slightly remember looked all over the place.it didn't look professional youre' trying to create a site so honest critism i think helps, but i seemed there was some talent in there somhow although perhaps not my taste there may have been an element.
I hope i didn't offend you.
mikew.

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Computing A-Level	Jack Franklin	Webforumz Cafe	4	Feb 3rd, 2008 17:13
Computing question	MetallicWarfare	Webforumz Cafe	5	Oct 30th, 2007 18:51
Janauarys winner!	benbramz	Hall of Fame	4	Jan 30th, 2006 20:31
January's Winner	Webforumz Staff	Hall of Fame	1	Feb 9th, 2004 10:10

		Webforumz.com > Main Forums > Design Your Website > Free Web Site Critique
Winner Computing