I have build a to-do list with JavaScript can you please check my script file?

Code Feedback
1 
let taskArray=[];


function deletetask(currentitem,button){
    button.remove();
    currentitem.remove();  
}

function addTask(){

    let task=document.getElementById('task').value;

    if(task!= ''){
        
        taskArray.push(task);
    
        const ul=document.getElementById('list');
        const li=document.createElement('li');
        const button=document.createElement('input');
        
        button.setAttribute('type','button');
        button.setAttribute('value','Delete task');
        button.setAttribute('onclick','deletetask(this.previousElementSibling,this)');
    
        li.innerText=task;
        ul.append(li);
        ul.append(button);
    
        li.classList.add('list-items');
    }
    else{
        alert("Please Enter Task");
    }
    
}

HTML FILE

<body>
    <div class="container">
      <h1>To-Do List</h1>
      <form action="">
        <input type="text" id="task" />
        <input type="button" value="Add Task" onclick="addTask()" />

        <ul id="list"></ul>
      </form>
    </div>
  </body>
4

I made changes as you suggested but was unable to clear the last item typed in input field.
I used task.innerText=" " ; but it’s not working

const ul=document.getElementById('list');

function deletetask(currentitem){
    currentitem.remove();
}

function addTask(){

    let task=document.getElementById('task').value;

    if(task!= ''){
        
        const li=document.createElement('li');
        const button=document.createElement('input');
        
        button.setAttribute('type','button');
        button.setAttribute('value','Delete task');
        
        button.setAttribute('onclick','deletetask(this.parentElement)');
    
        li.innerText=task;
        ul.append(li);
        li.append(button);
   
        li.classList.add('list-items');
    
        task.innerText=``;

    }
    else{
        alert("Please Enter Task");
    }

}
6

Thank you for your help.

document.getElementById('task').value=``;

this worked.

8

Thanks for the solution :smile:

9

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.