|
Kelly2
|
 |
« Reply #20 on: August 01, 2009, 12:33:10 AM » |
|
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #21 on: August 01, 2009, 12:47:20 AM » |
|
Oh wow thats insane... maybe I should listen to those few people that helped me... man now I have to restart....... 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Bob Hoil
|
 |
« Reply #22 on: August 01, 2009, 01:47:44 AM » |
|
Sorry about that little problem! It has all been fixed! I have re-coded the entire thing from scratch this morning and it should be up and working now. I kept some of the same features mainly just so I didn't have to completely restart like the rating, and affiliates area are still there!
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #23 on: August 01, 2009, 03:05:03 AM » |
|
I would not suggest adding some "padding" between your "boxes" and your Adsense ad, however a simple <br /> will do the trick too. Another thing you should do is improve on your banner image, if you would like (for a small fee) I could make you one. - Some of my work - 
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #24 on: August 01, 2009, 04:26:48 AM » |
|
Ok I have updated the site again! I think it looks ten times better. Looks much cleaner. So tell me what you think. I would love to have you make a banner but I am a little low on the cash right now! 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #25 on: August 01, 2009, 05:23:37 AM » |
|
I really like the header, but there are still lots you can improve on.
``~ Your text is 'touching' you 'walls' of your content wrapper, you should add a PADDING:10PX; rule to prevent this. (padding:XXXpx). ``~ Your background is quite dark, and it jumps from black to white, this somewhat hurts the eyes. Widen your content wrapper. ``~ ^^ I suggest making it 650PX to 800PX wide (width:XXXpx).
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #26 on: August 01, 2009, 05:40:19 AM » |
|
Ok I took all your suggestions into account! I really appreciate all your help! Also thanks for pointing out that other site! 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #27 on: August 01, 2009, 12:38:40 PM » |
|
It looks so much better then it used to.  .
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #28 on: August 01, 2009, 02:09:12 PM » |
|
Thanks! I worked hard on it! I am having one problem though it isn't centered in IE! I have tried to fix it but its not working! 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #29 on: August 01, 2009, 02:41:58 PM » |
|
I have centered your website in IE and fixed at least ten errors. Both <img src="image"> and </br> are incorrect for two important reasons. BR is a single tag, it does not open and it does not close so there is no need to open it with a forward slash. IMG and BR are both self closing tags, meaning you do not need another tag to close ( <p> Text </p>). The correct forms are <img src="image" /> and <br /><html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en"> <head> <meta http-equiv="Content-type" content="text/html;charset=UTF-8"/> <link rel="stylesheet" type="text/css" href="style/style.css"/> <title>The Ad Place</title> </head> <body> <div align="center"> <div id="wrapper"> <div id="header"> <img src="style/images/header.png" id="logo" alt="Ad Place"/> <div id="navbar"> <a href="index.php">Home </a> <a href="prov.php">Providers </a> <a href="adv.php">Advertisers </a> <a href="abo.php">About Us </a> </div> <br/> <div align="center"> <script type="text/javascript"><!-- google_ad_client = "pub-9821635504472704"; google_ad_slot = "5060944755"; google_ad_width = 468; google_ad_height = 15; //--> </script> <script type="text/javascript" src="http://pagead2.googlesyndication.com/pagead/show_ads.js"> </script> </div> </div> <div id="content"> <h2>Welcome! </h2> <p> Welcome to Ad Place! We provide free information on what ad providers there are and what they will do for you! We hope this information will help you make the best of your site and make some money! </p> <p> Please feel free to browse the site and check out all our different areas of the site. We hope the information provided is helpful!You can view my contact information below! </p> <h2>Rating Sites </h2> <p>On the site we also have a rating system! You are free to vote for each site once! So if you have used one of the sites on here please rate it on your experience you had with them! </p> <h2>Contact </h2> <p> Email: <a href="mailto:greg.froning@bobhoil.com">greg.froning@bobhoil.com </a> </p> </div> <div id="affiliates"> <ul id="aff"> <li> <a href="http://bobhoil.com/"> <img src="style/images/bobhoil.png" alt="BobHoil.com!"/> </a> </li> <li> <a href="https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=7042524"> <img src="http://bobhoil.com/donatebuts/donate.png" alt="Donate"/> </a> </li> <li> <a href="http://www.110mb.com/aff.php?un=bobhoil"> <img src="style/images/110.png" alt="110mb.com Free Hosting!"/> </a> </li> </ul> </div> <br/> <div align="center"> <script type="text/javascript"><!-- google_ad_client = "pub-9821635504472704"; google_ad_slot = "3500610466"; google_ad_width = 468; google_ad_height = 60; //--> </script> <script type="text/javascript" src="http://pagead2.googlesyndication.com/pagead/show_ads.js"> </script> </div> <div id="footer"> <p>© Ad Place 2008 Design Created by: <a href="http://bobhoil.com/">bobhoil.com </a> </p> </div> </div> </div> </body> </html>
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #30 on: August 01, 2009, 03:08:43 PM » |
|
Wow thanks! I have updated the rest of the pages so now the site displays good! Thanks for the help KellyAX! Hopefully this will help my site become a good source for web developers! Hopefully my advertisements pay off to so I can grab a domain at some point. Luckily this isn't my only website. 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
EpicCyndaquil
|
 |
« Reply #31 on: August 01, 2009, 04:44:19 PM » |
|
One recommendation - separate the links at the top more, looks like one giant link at first. 
|
|
|
|
|
Logged
|
|
|
|
|
Kelly2
|
 |
« Reply #32 on: August 01, 2009, 11:33:46 PM » |
|
One recommendation - separate the links at the top more, looks like one giant link at first.  Do you mean for him to remove his AdSense ad or physically separate his links? <div id="navbar"><a href="index.php">Home</a> <a href="prov.php">Providers</a> <a href="adv.php">Advertisers</a> <a href="abo.php">About Us</a></div>
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #33 on: August 02, 2009, 02:22:29 AM » |
|
Probably the menu. Only thing I could think of. They are close together now.
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #34 on: August 02, 2009, 03:44:08 AM » |
|
I might even go as far as suggesting to remove the first adsense code but that is completely up to you.
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #35 on: August 02, 2009, 04:02:24 AM » |
|
Do you think it is in a bad place? I thought maybe it was a good spot to help optimize the money I do get. Been trying to promote the site some. Hopefully I can get some good affiliates which will help! Also a domain name once I get the money for that.
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #36 on: August 02, 2009, 04:50:08 AM » |
|
I don't think it is a bad place but I rarely see the need for more then one ad per page.
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #37 on: August 02, 2009, 05:35:16 AM » |
|
oh ok so I should just keep the banner at the bottom? I guess it would make the site look better if I only had the one at the bottom also. I am planning on integrating an smf forum directly into my theme. Might take me a while but I have done it before so hopefully I can get it! 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|
Kelly2
|
 |
« Reply #38 on: August 02, 2009, 12:35:58 PM » |
|
Why do you have an empty div in your source code?  . <div align="center"> </div>
|
|
|
|
|
Logged
|
|
|
|
|
Bob Hoil
|
 |
« Reply #39 on: August 02, 2009, 12:57:44 PM » |
|
Oh must have forgot to take them out! My bad  I will remove them and reupload here soon. Thanks for pointing that out. 
|
|
|
|
|
Logged
|
 If you need information on ad programs check out this site!
|
|
|
|