Skip to content

Commit b756833

Browse files
committed
Fix: delete task function
1 parent 090c430 commit b756833

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

script.js

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,11 @@ const editTaskText = (task) => {
4040
editTaskText.focus();
4141
}
4242

43+
// delete the task when clicked on deleteIcon
44+
const deleteTask = (task) => {
45+
task.remove();
46+
};
47+
4348
// add the task when click
4449
taskAddBtn.addEventListener("click", () => {
4550
// get input field value
@@ -61,6 +66,8 @@ taskAddBtn.addEventListener("click", () => {
6166
else
6267
alert("Task must be of at least 5 characters to be registered.");
6368

69+
taskInput.focus();
70+
6471
});
6572

6673
// check which task is clicked and which button is clicked
@@ -70,21 +77,25 @@ taskContainer.addEventListener("click", (e) => {
7077

7178
if(taskElement.classList.contains("task"))
7279
{
80+
e.stopPropagation(); // Stop immediate propagation of the click event
81+
7382
// get the data value of clicked element
7483
const clickedElement = e.target;
75-
const data = clickedElement.getAttribute("data");;
84+
const data = clickedElement.getAttribute("data");
7685

7786
switch(data)
7887
{
7988
case 'check':
8089
completedTask(taskElement);
8190
break;
8291
case 'task':
92+
// does nothing for now
8393
break;
8494
case 'edit':
8595
editTaskText(taskElement);
8696
break;
8797
case 'delete':
98+
deleteTask(taskElement);
8899
break;
89100
}
90101
}

0 commit comments

Comments
 (0)