> If you guys wouldn't mind, could I get a site check on this?
>
> http://www.cgraytaylor.net
>
> It seems to check out okay for me in Firefox, IE 7, and Opera. I had
> to lower the font size in the gallery photo descriptions for Opera to
> lay out the text correctly which makes it a touch small for my tastes,
> but is better than having orphan words. It's my first CSS site, so
> some critiques would be much appreciated.
>
> Phoebe
Hi Phoebe,
One quick thing I saw was in the gallery. When the cursor rolls over
the image, there is a slight "jump" when the the border around the
image changes between 1px and 2px.
.thumbnail img{
border: 1px solid white;
margin: 0 5px 5px 0;
}
If you make the white border about the image 2px the jump should
disappear.
.thumbnail img{
border: 2px solid white;
margin: 0 5px 5px 0;
}
Another little thing was (this is simply subjective on my part) the
menu up top. A little more space between each item might be nice.
Right now you have the following:
#slantedmenu ul li{
display: inline;
}
#slantedmenu ul li a{
color: #494949;
padding: 5px 0;
padding-right: 35px;
margin: 0;
text-decoration: none;
background-image: url(graphics/buttons/divider.jpg);
background-repeat: no-repeat;
background-position: right;
}
I would do the following to separate them a bit and make them easier
to read. First, add a margin-right: 30px to the "#slantedmenu ul li"
as such:
#slantedmenu ul li{
display: inline;
margin-right: 30px;
}
Then shrink the padding-right in "#slantedmenu ul li a" from 35px to
25px as such:
#slantedmenu ul li a{
color: #494949;
padding: 5px 0;
padding-right: 25px;
margin: 0;
text-decoration: none;
background-image: url(graphics/buttons/divider.jpg);
background-repeat: no-repeat;
background-position: right;
}
You could also combine the "padding: 5px 0" and "padding-right: 25px"
to a single line like this:
padding: 5px 25px 5px 0;
Just a thought. :)
You've done a good job with your first sight. Be encouraged.
Mark
______________________________________________________________________
css-discuss [EMAIL PROTECTED]
http://www.css-discuss.org/mailman/listinfo/css-d
List wiki/FAQ -- http://css-discuss.incutio.com/
List policies -- http://css-discuss.org/policies.html
Supported by evolt.org -- http://www.evolt.org/help_support_evolt/
