On 8/12/07, Tim Offenstein <[EMAIL PROTECTED]> wrote: > > I'm looking at your site in Firefox 2 on a Mac (10.4.10). Everything > seems to behave reasonably well. > > Comments: > - I can't read the quote at top unless I bump the font size way up. > - The backgrounds on links in the main column is a bit intrusive. My > preference is background on hover only. > - The contrast on both the side menus is too weak (IMHO). I like the > rollover effect though - very well done. > - The page holds together nicely on resizing. > - You have 5 header 1 tags. The recommended best practice is no more > than two and preferably only one which should be a duplicate of the > page title. I recommend reducing the h1's. Header markup should > reflect a hierarchical layout of the page. > - Your page validates XHTML Strict but there's a <i></i> tag in > there. <i> is deprecated and should be replaced with <em>. I don't > know how the W3C validator is saying this page is okay with that <i> > in there?!? > - Good job of commenting your XHTML. > - I recommend dropping the <br /> tags where possible and instead > style your CSS to compensate. Strictly speaking, you're using HTML > markup for stylistic presentation. (Nitpicky I know.) > > Overall, very good job. > > -Tim > > >DUH! > >Sorry I forgot to add the link been a long night.. > > > >Can be seen here. <http://home.comcast.net/%7Es.oravec/index.html>
Thanks Tim I changed I think all the things you had mentioned . Thanks again.. ______________________________________________________________________ css-discuss [EMAIL PROTECTED] http://www.css-discuss.org/mailman/listinfo/css-d IE7 information -- http://css-discuss.incutio.com/?page=IE7 List wiki/FAQ -- http://css-discuss.incutio.com/ Supported by evolt.org -- http://www.evolt.org/help_support_evolt/
