Thanks Neo, 
I updated my pages to include the following:

<style type="text/css">
<!--
a:link       { color: #333399; text-decoration: none; }
a:visited    { color: #333399; text-decoration: none; }
a:hover      { color: purple; text-decoration: underline; }
-->
</style>

I will work on the Font portion in the very near future, if time allows! :-)

-------------------
Peter Szaro




-----Original Message-----
From: Neo [mailto:2nerotik(at)home.com]
Sent: Saturday, June 02, 2001 12:48 PM
To: hwg-critique(at)hwg.org
Subject: Re: http://peterszaro.com


****** Please Note: This message came from the INTERNET. ******

The only thing I can really say, Oh man you used Frontpage :o

J/K actually it might explain why you have 3 sets of Styles
when you only need one for the links.

ie. you have:
<style type="text/css">
<!--
a            { text-decoration: none }
a:link       { color: #000000 }
a:visited    { color: #333399 }
-->
</style>
<style type="text/css">
<!--
a:hover      { color: purple }
a:link       { color: #333399 }
-->
</style>
<style type="text/css">
<!--
a:link       { text-decoration: none }
a:visited    { text-decoration: none }
a:hover      { text-decoration: underline }
-->
</style>

When all you need is:
<style type="text/css">
<!--
a:link       { color: #333399; text-decoration: none; }
a:visited    { color: #333399; text-decoration: none; }
a:hover      { color: purple; text-decoration: underline; }
-->
</style>

You are also missing the a:active style as well you could replace all
the font tags and make your styles look like this:

<style type="text/css">
<!--
a:link    { font : normal 2 arial, helvetica;
             text-decoration: none;
             color: #333399; }
a:active  { font : normal 2 arial, helvetica;
             text-decoration: none;
             color: #050C42; }
a:visited { font : normal 2 arial, helvetica;
             text-decoration: none;
             color: #333399; }
a:hover   { font : normal 2 arial, helvetica;
             text-decoration: underline;
             color: purple; }
-->
</style>


That's about all I can really say, I personally prefer using HomeSite
so I'm always working with the code and keep it nice and slim while
making sure it works with NN4+ and IE5+.

Other suggestions would be to change the font from Arial to:
Verdana, Geneva, Arial, Helvetica, sans-serif.

I find Verdana much easier to read on screen.

As well as creating one external style sheet and link all your pages
to it, you wouldn't believe the amount of time you save and the ease
of use. Imagine having 5-10 pages and then having to go in and change
every font tag, now imagine changing that same font in an external
sheet and having it automatically change on all the pages.

It may be a pain initially to change over but once done you never go
back and I've never had any problems. I also externally link my
JavaScripts whenever possible.


.oOo.
|| The early bird gets the worm but
|| the second mouse gets the cheese.
||
|| PhotoShop Tutorial List --> http://members.home.com/ps-tutorials/
|| Addicted 2 Swing --> http://members.home.com/addicted2swing/
|| Fontaholic ? --> http://www.fontsanon.com


*****************************************************************************
This message may contain privileged and confidential information. If you
think, for any reason, that this message may have been addressed to you in
error, you must not disseminate or copy this message, or use its content
as the basis for taking any action. Further, we would ask you to notify
us immediately by return e-mail, telephone or facsimile.
*****************************************************************************

HTML: hwg-critique mailing list archives, maintained by Webmaster @ IWA