HTML in JavaScript Error

Question

I'm tryng to put html into javascript code:

       document.write("<a class=\"twitter-timeline\"href=\"https://twitter.com/test\"          data-widget-    id=\"248798161076883457\">Tweets by @test</a>
             <script>!function(d,s,id){
              var js,fjs=d.getElementsByTagName(s)[0];
             if(!d.getElementById(id))
              {js=d.createElement(s);
          js.id=id;js.src=\"//platform.twitter.com/widgets.js\";
              fjs.parentNode.insertBefore(js,fjs);
        }
       }(document,\"script\",\"twitter-wjs\");
       </script>
        ");

}

The error message is: Unexpected token ILLEGAL what's wrong? Thx!

change //platform.twitter.com to either http://platform.twitter.com or platform.twitter.com — karthikr
– karthikr, Commented Sep 20, 2012 at 15:28
@karthikr: No need to do that, that's a valid protocol-relative URL. — user1233508
– user1233508, Commented Sep 20, 2012 at 15:30
document.write is a bad function to use for several reasons including security issues. Consider instead using an element's innerHTML property to set HTML into your document, which works better and does not have the same security issues. — Ed Bayiates
– Ed Bayiates, Commented Sep 20, 2012 at 15:30
document.write("<a class='twitter-timeline' href='twitter.com/test' data-widget- id='248798161076883457'>Tweets by @test</a><script>!function(d,s,id){var js,fjs=d.getElementsByTagName(s)[0];if(!d.getElementById(id)){js=d.createElement(s);js.id=id;js.src='platform.twitter.com/widgets.js'; fjs.parentNode.insertBefore(js,fjs);}}(document,'script','twitter-wjs');</script>"); — Kuldeep Daftary
– Kuldeep Daftary, Commented Sep 20, 2012 at 15:30
Glad it works! Which script you used in the end? the 1 that I shared or the 1 shared by user1666993 . Thanks — Kuldeep Daftary
– Kuldeep Daftary, Commented Sep 20, 2012 at 15:38

svennergr · Accepted Answer · 2012-09-20 15:30:01Z

4

A string in javascript must not go over multiple lines without the right format:

   document.write("<a class=\"twitter-timeline\"href=\"https://twitter.com  /test\"          data-widget-    id=\"248798161076883457\">Tweets by @test</a>\
         <script>!function(d,s,id){\
          var js,fjs=d.getElementsByTagName(s)[0];\
         if(!d.getElementById(id))\
          {js=d.createElement(s);\
      js.id=id;js.src=\"//platform.twitter.com/widgets.js\";\
          fjs.parentNode.insertBefore(js,fjs);\
    }\
   }(document,\"script\",\"twitter-wjs\");\
   </script>\
    ");

In general:

string = "Multiple\
Lines\
supported."

answered Sep 20, 2012 at 15:30

svennergr

8102 gold badges10 silver badges28 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Kuldeep Daftary · Accepted Answer · 2012-09-20 15:30:57Z

0

document.write("<a class='twitter-timeline' href='https://twitter.com/test'  data-widget- id='248798161076883457'>Tweets by @test</a><script>!function(d,s,id){var js,fjs=d.getElementsByTagName(s)[0];if(!d.getElementById(id)){js=d.createElement(s);js.id=id;js.src='platform.twitter.com/widgets.js'; fjs.parentNode.insertBefore(js,fjs);}}(document,'script','twitter-wjs');</script>");

You can use single quotes instead of double quotes inside script! It makes it easier to read and it works perfectly

answered Sep 20, 2012 at 15:30

Kuldeep Daftary

6113 gold badges16 silver badges31 bronze badges

Collectives™ on Stack Overflow

HTML in JavaScript Error

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related