Hi Guys,

I'm not sure if this is the correct place to ask this question.

I'm trying to create a javascript file (.js) that contains the adsense code that should be placed on my static html file. This way, I can reference the same adsense block in all my pages and whenever I decide to change the colors, dimensions .. etc, I will just go there and make my updates.

The javascript file looks like this:
document.writeln('<script type="text/javascript"><!--');
document.writeln('google_ad_client = " <Adsense ID removed> ";');
document.writeln('google_ad_width = 728;');
document.writeln('google_ad_height = 90;');
document.writeln('google_ad_format = "728x90_as";');
document.writeln('google_ad_type = "text_image";');
document.writeln('google_ad_channel ="7100825086";');
document.writeln('google_color_border = "FFDDDD";');
document.writeln('google_color_bg = "FFDDDD";');
document.writeln('google_color_link = "000000";');
document.writeln('google_color_url = "000000";');
document.writeln('google_color_text = "000000";');
document.writeln('//--></script>');
document.writeln('<script type="text/javascript" src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>');

This works great. The problem happens when I try to surround this code by a table (using script code like document.writeln('<table><tr><td>'); )

when I do that, I found that the table and the ad block are generated as 2 blocks even if the script should be placed inside the table cell (1 block).

Anyone has some ideas?

Moderators, if this is the incorrect place for this question, please feel free to move it ;-)

moving to javascript

Thanks Chris. I thought it maybe the wrong place to ask this question

I'm not necessarily sure that the answer is a Javascript one. I just put my adsense code into an include file (usually a layout file) and adjust it from there.

In other words, SSI may hold your answer, not necessarily Javascript.

BillyRandyFreddieJoeFloydBob,

can you please post a sample? of how you can do it. I don't necessarily need this to be using Javascript.

nice name BTW

<!--#include virtual="/folder/file_with_adsense_code.ext" -->

What Hirst said. You can use a regular HTML file for the include as long as it's part of an SSI file (e.g. something.shtml, something.asp, something.php).

Thanks for noticing my name. I was just feeling goofy one night so I stuck it up.

That's the same thing I did with mine. Yours isn't working because it's not correct.

On your page add the JS call to your js file (made from the adsense code) like this:

<script type="text/javascript" src="/path/to/your/adsense.js"></script>

Now, in the JS file you're going to save, you need to do this:

<!--

google_ad_client = " <Adsense ID removed> ";
google_ad_width = 728;
google_ad_height = 90;
google_ad_format = "728x90_as";
google_ad_type = "text_image";
google_ad_channel ="7100825086";
google_color_border = "FFDDDD";
google_color_bg = "FFDDDD";
google_color_link = "000000";
google_color_url = "000000";
google_color_text = "000000";

//-->

Then, you're all set.

Chris,

thanks, but I have static html files and the statement:

<!--#include virtual="/folder/file_with_adsense_code.ext" -->

didn't work for me.


Deb,
I successfully managed to put the google adsense code in a javascript and it's working just fine.

The problem is when I try to put a table around the adsense code (to make the adsense block look wider than 728 px). I try to do it like this:

in the adsense.js file:

 
/* Begin Google Adsense code */
document.writeln('<TABLE bgcolor="FFDDDD" WIDTH=777 BORDER=1 bordercolor="728FB1" CELLPADDING=0 CELLSPACING=0><tr><td><center>');
document.writeln('<script type="text/javascript"><!--');
document.writeln('google_ad_client = "publisher code";');
document.writeln('google_ad_width = 728;');
document.writeln('google_ad_height = 90;');
document.writeln('google_ad_format = "728x90_as";');
document.writeln('google_ad_type = "text_image";');
document.writeln('google_ad_channel ="7100825086";');
document.writeln('google_color_border = "FFDDDD";');
document.writeln('google_color_bg = "FFDDDD";');
document.writeln('google_color_link = "000000";');
document.writeln('google_color_url = "000000";');
document.writeln('google_color_text = "000000";');
document.writeln('//--></script>');
document.writeln('<script type="text/javascript" src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>');
document.writeln('</center></td></tr></table>');
/* End Google Adsense code */

In the .html file:

 
<!-- Begin Google Adsense code -->
<script TYPE="text/javascript" src="script/adsense.js"></script>
<!-- End Google Adsense code -->

Please notice that if I put the <table> code inside the html file it works fine. But I need to put it in the .js file to make sure I can change it whenever I want easily.

To take a look at the problem, just see this page http://www.smartbro.com/index1.html

What I need to accomplish is this http://www.smartbro.com/index.html (the adsense block at the top)

Any ideas why this happen?

Change the writeIn to just
write

It should be
document.write

See if it helps.

If it still doesn't work, give me about a hour and I'll tweak the codes on my site and post it here for you. OK?

Thanks for the quick reply. I've tried to replace writeln with write, but the google adsense just disappears ... that's weird!!

I would be grateful if you could take a look at the code on my website. thanks in advance :-)

doh. The double quotes need to be escaped. I have a major headache today with all of the thunder and rain ... it's why I didn't catch this earlier.

Also, you had that as writeln (lowecase "L")
That's not even correct.

It's document.write(' this line blah blah blah ');

OK, here's the corrected code. Try it now and see if we strike pay dirt.

document.write('<TABLE bgcolor=\"FFDDDD\" WIDTH=777 BORDER=1 bordercolor=\"728FB1\" CELLPADDING=0 CELLSPACING=0><tr><td><center>');
document.write('<script type=\"text/javascript\"><!--');
document.write('google_ad_client = \"publisher code\";');
document.write('google_ad_width = 728;');
document.write('google_ad_height = 90;');
document.write('google_ad_format = \"728x90_as\";');
document.write('google_ad_type = \"text_image\";');
document.write('google_ad_channel =\"7100825086\";');
document.write('google_color_border = \"FFDDDD\";');
document.write('google_color_bg = \"FFDDDD\";');
document.write('google_color_link = \"000000\";');
document.write('google_color_url = \"000000\";');
document.write('google_color_text = \"000000\";');
document.write('//--></script>');
document.write('<script type=\"text/javascript\" src=\"http://pagead2.googlesyndication.com/pagead/show_ads.js\"></script>');
document.write('</center></td></tr></table>');

Deb,

thanks for your reply, but I don't think you understand my problem correctly. The code that I've sent you earlier with (writeln) IS working. The only problem with it (which I'm trying to solve here) is that I get 2 boxes. See attached screenshot (screenshot #1). When I don't write the table code inside the .js file, and write it inside the html file, the output comes as (screenshot #2) which is what I want.

I'm trying to find out why the table is not displayed correctly when written inside the .js file.

BTW, I tried your updated code and still doesn't work.

OK, beats me then. I'm not real clear today anyway (have migraine )

LOOOL! The problem is quite confusing. Isn't it?

Yeah, it's confusing when headache stops the thought process altogether.
I'll look at this tomorrow when my brain isn't so foggy.

ok, I'll wait then.

thank you very much Deb!!

You're welcome, Tony.

Anyone can help me with this guys?