Jump to content

Rate my template


quackquack

Recommended Posts

And this is the code:

<!--Note: This is a template--><html><head><style type="text/css">body{background-color: #FFFFFE}a:link {color: #0000FF}a:visited {color: #00FF00}a:hover {color: #00FF00}a:active {color: #9966FF}</style></head><body><p><img align="center" src="http://i125.photobucket.com/albums/p42/hns_marcon/28.jpg" alt="" width="754" height="138"></p><div align="center"><a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a><a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a><a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a><a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a><a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a><a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a></div><table width="282" border="1" bordercolor="#00FF00">  <tr>	<td width="223" height="326" bgcolor="#00FFFF"><p><br>	  Text here.Text here. Content here <br>	  Text here.Text here. Content here <br>	  Text here.Text here. Content here <br>	  Text here.Text here. Content here<br>	  Text here.Text here. Content here <br>	  Text here.Text here. Content here <br>	  Text here.Text here. Content here <br>	  Text here.Text here. Content here</p>		<p>Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here<br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here<br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>		  Text here.Text here. Content here <br>	  </p></td>  </tr></table><table width="219" height="277" border="1" cellpadding="1">  <tr>	<td width="209" height="273" bgcolor="#99FF99">This is the part for Extras . . . </td>  </tr></table></body></html>

Link to comment
Share on other sites

first off, you should lose the borders around your buttons by adding border="0"e.g. replace:

<a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif"></a>

with:

<a href=""><img src="http://i125.photobucket.com/albums/p42/hns_marcon/Buttons/Orange%20buttons/Blank.gif" border="0"></a>

I like the header image, but overall i feel the colors on the site clash. When im planning a new site i experiment with different color schemes and present the client with a few options. when we agree on a color scheme i pretty much stick to that. this will give your design more consistency. when i cant come up with something i like, i sometimes visit: http://www.colourlovers.com/ they have great color pallettes there to choose from.hope this helps a bit.

Link to comment
Share on other sites

Depending on how many images are on your page, you can cut down on a lot of text if you declare the image borders in the CSS rather than in-line with the img tag.

<style type="text/css">img { border: 0px solid #000000; }</style>

Or, if you only want to turn off borders for images that are in links:

<style type="text/css">a img { border: 0px solid #000000; }</style>

Link to comment
Share on other sites

Depending on how many images are on your page, you can cut down on a lot of text if you declare the image borders in the CSS rather than in-line with the img tag.
<style type="text/css">img { border: 0px solid #000000; }</style>

Or, if you only want to turn off borders for images that are in links:

<style type="text/css">a img { border: 0px solid #000000; }</style>

Or to make your code even shorter :)
<style type="text/css">img { border-width: 0; }</style>

<style type="text/css">a img { border-width: 0; }</style>

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...