Question

I was going through the solutions of Uncaught SyntaxError: missing ) after argument list, and tried every other way to sort it out but I am still getting this error with this code

$("input").on("keypress",function(event){

if(event.which === 13)
{
    var ToDotext=($(this).val());   

$("ul").append("<li><span><i class="fa fa-trash"></span> "  +  ToDotext  + "</li>");
$(this).val("");
}

} );

whenever I put  I am getting this error, without  things are working fine.

Thanks, @marekful It was written with double quotes on font awesome website. But with single quotes, it's working great. thanks again:) — shaykoo, Jan 12 '18 at 05:04

score 2 · Answer 1 · edited Jun 20 '20 at 09:12

Change the appending line to this.

$("ul").append("<li><span><i class='fa fa-trash'></i></span> "  +  ToDotext  + "</li>");

Changes:

fa fa-trash is in single quotes as you are using double quotes outside.
close the i tag

Suggestions:

Instead of using  inside , apply the class on span or remove span altogether and keep i tag.
Use jQuery element creation methods. like this.

$('ul li ').html($('', {class: 'fa fa-trash'}));

Negi Rox · Answer 2 · 2018-01-12T05:12:06.947

1

Use this code

   $("input").on("keypress",function(event){
    if(event.which === 13)
       {
           var ToDotext=($(this).val());   

        $("ul").append("<li><span><i class='fa fa-trash'></i></span>"+ ToDotext  + "</li>");

         $(this).val("");
     }
    });

edited Jan 12 '18 at 05:12

answered Jan 12 '18 at 05:02

Negi Rox

3,828
1
11
18

score 1 · Answer 3 · edited Jan 12 '18 at 06:03

I found the error in your script.

$("ul").append("<li><span><i class="fa fa-trash"></span>"+ToDotext+"</li>");

If you add component, with this code, the actual components are added as follows.

<li><span><i class=fa fa-trash></span>ToDoText</li>

As you see above, the class names of the  tag must be quoted with " or ' but it is not like that. It is because you used the same quotes adding the components. If you change "fa fa-trash" -> 'fa fa-trash', the problem will be solved.

score 0 · Answer 4 · answered Jan 12 '18 at 07:20

You can't have double quotations inside double quotations, simple example:

open your console:

var newElem = "<h1 class="rock">Hello There !!</h1>";
newElem // Uncaught SyntaxError: Unexpected identifier

You will have to enclose you class and other html attributes inside a single quotations, like so:

var newElem = "<h1 class='rock'>Hello There !!</h1>";

The below line:

$("ul").append("<li><span><i class='fa fa-trash'></span> "  +  ToDotext  + "</li>");

Would have to change to:

$("ul").append("<li><span><i class="fa fa-trash"></span> "  +  ToDotext  + "</li>");

Additional ::- if you writing long concatenations you probably want to SPLIT IT UP INTO ADDITIONAL LINK

is not working in

4 Answers4