The Web Design Group

... Making the Web accessible to all.

Welcome Guest ( Log In | Register )

 
Reply to this topicStart new topic
> Review Please
Scoobs
post Oct 21 2011, 12:20 AM
Post #1


Newbie
*

Group: Members
Posts: 13
Joined: 16-July 11
Member No.: 14,948



www.vintageparlor.com

This is the first website I have created for a client and I am nearing the end of it's development. What do you think of it? I integrated a phpBB3 forum into my design to be used as an inventory categorizer. I thought it was pretty clever and it looks very nice.

Thanks! biggrin.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
Darin McGrew
post Oct 21 2011, 12:20 PM
Post #2


WDG Member
********

Group: Root Admin
Posts: 8,365
Joined: 4-August 06
From: Mountain View, CA
Member No.: 3



Our online validator reports some markup errors:
htmlhelp.com/cgi-bin/validate.cgi?url=http%3A%2F%2Fwww.vintageparlor.com%2F&warnings=yes

You use images of text without alt text. It's better to use real text, but if you're going to use images of text, then you need to put the text itself in the alt attribute. See:
http://htmlhelp.com/feature/art3.htm

You specify color without specifying background color. Your color is displayed against the user's default color, and there may not be sufficient contrast. The general rule is, if you specify one color/background, then you have to specify them all.

Speaking of sufficient contrast, the black-on-red menu bar is very hard to read.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
Scoobs
post Oct 22 2011, 08:22 PM
Post #3


Newbie
*

Group: Members
Posts: 13
Joined: 16-July 11
Member No.: 14,948



Thanks! I added the alt attributes and found some hiccups in the code.

As for the background, should I just set that in a body tag?
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
pandy
post Oct 23 2011, 04:21 AM
Post #4


🌟Computer says no🌟
********

Group: WDG Moderators
Posts: 20,716
Joined: 9-August 06
Member No.: 6



In your style sheet is better. Besides, you don't have body tag. wink.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
newwebseo
post Oct 24 2011, 04:36 AM
Post #5


Advanced Member
****

Group: Members
Posts: 125
Joined: 9-October 09
Member No.: 9,984



Site is good but there is one thing that i must bring to your notice.

You have used dark brown color in menu strip and on that you have used "black font" for menu titles.

This is not good.
On a website you should use contrast font color so that visitor can easily read what is written on your site.

If you are using dark background then use light color for text.
I hope this will help you.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post

Reply to this topicStart new topic
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:

 



- Lo-Fi Version Time is now: 29th March 2024 - 12:07 AM