Great
Good
Fair
Poor
Horrible
Welcome to our happy home, JasonP!
I took a fairly long look at your site and gave it a "fair" rating.
Things I like about it:
- It's simple and relatively clean overall.
- You've managed to find a way to use the Cooper font that doesn't suck.
- The blue background outside the page is calming and doesn't distract the eye from the content.
Things I feel could be improved on:
- The menu buttons are too big and aren't rounded off nicely. You may wish to consider a "pill" shape for them instead.
- The product list should be in a list format and there should be a bit more of a text explanation of each product (possibly with a picture corresponding with each.) You may also want to look at putting each product on its own page for search engine and ease of reading purposes.
- The Gallery. Your client (at least from what I can gather from the site) specializes in custom work, so this section should be made more prominent, with a combination of thumbnails and short descriptions and larger pages with longer descriptions. I've done something very similar for a fireplace client of mine, and he's had numerous people call because of the work that they've seen (a lot of them will even call looking for something specific they saw in this section that they liked.) Click here to see the "Our Work" section for my client.
Hope this helps!
If I've helped you out in any way, please pay it forward. My wife and I are walking for Autism Speaks. Please donate, and thanks.
If someone helped you out, be sure to "Like" their post and/or help them in kind. The "Like" link is on the bottom right of each post, beside the "Share" link.
My stuff (well, some of it): My bowling alley site | Canadian Postal Code Info (beta)
I agree with what TheGame has said. I like the simple, clean look. In addition to what he has said, I notice this:
On an 800x600 resolution, your header and navigation takes up roughly half of the screen, leaving little room for content to be viewed "above the fold" (before scrolling).
Another minor addition that would look nice would be putting your navigation at the bottom of your page as well, in text form. That would allow ppl to navigate to a new page w/o scrolling back up.
Crisp....Clean... and to the point!
I like it
I agree with the GAme and Slufdaddy too,the NAV is too big and the mouse over color should be another color so your page will look better
I like the new mouseovers, much better. The text nav at the bottom is exactly what I had in mind, a nice added effect would be to give the links a CSS mouseover effect, like a different color or an underline.
I agree; it looks much nicer now.
If I've helped you out in any way, please pay it forward. My wife and I are walking for Autism Speaks. Please donate, and thanks.
If someone helped you out, be sure to "Like" their post and/or help them in kind. The "Like" link is on the bottom right of each post, beside the "Share" link.
My stuff (well, some of it): My bowling alley site | Canadian Postal Code Info (beta)
Thanks guys.
I've been trying to put in a CSS mouseover effect where the underline disappears from the text when you move your mouse over it.
Here is the CSS code that I used:
a.nav:link {
color: #FFFFFF;
text-decoration: underline;
background-color: #0099CC;
}
a.nav:hover {
color: #FFFFFF;
text-decoration: none;
background-color: #0099CC;
}
a.nav:active {
color: #FFFFFF;
text-decoration: none;
background-color: #0099CC;
}
a.nav:visited {
color: #FFFFFF;
text-decoration: underline;
background-color: #0099CC;
}
--------
<p>
<a href="services.htm" class="nav">Services</a> |
<a href="productlist.htm" class="nav">Product List</a> |
<a href="rawmaterial.htm" class="nav">Raw Material</a> |
<a href="commonquestions.htm" class="nav">Common Questions</a> |
<a href="gallery.htm" class="nav">Gallery</a> |
<a href="links.htm" class="nav">Links</a>
</p>
Only a few of them work sometimes. What is wrong w/ my coding?
You dont need to wrtie that long in your site just this:
<style type="text/css">
<!--
a:link {color: black; text-decoration: none; }
a:visited {color: #0066CC; text-decoration: none; }
a:hover {color: grey; text-decoration: none; }
a:active {color: red; text-decoration: none; }
-->
</style>
That enough for your mouse over effect
If you want to use class for your mouse over,your way is ok? i found nothing wrong in your code?