From 04b7eef479f083d313019017228efa5a0b240879 Mon Sep 17 00:00:00 2001
From: Elisabeth-Matulian <elisabethmatulian@gmail.com>
Date: Sun, 15 Mar 2026 20:54:37 +0000
Subject: [PATCH 1/2] completed todo list app

---
 Sprint-3/todo-list/index.html |  1 +
 Sprint-3/todo-list/script.mjs |  7 ++++++-
 Sprint-3/todo-list/style.css  | 11 +++++++++++
 Sprint-3/todo-list/todos.mjs  | 10 ++++++++++
 4 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/Sprint-3/todo-list/index.html b/Sprint-3/todo-list/index.html
index 4d12c4654..7e4685cdd 100644
--- a/Sprint-3/todo-list/index.html
+++ b/Sprint-3/todo-list/index.html
@@ -20,6 +20,7 @@ <h1>My ToDo List</h1>
 
     <ul id="todo-list" class="todo-list">      
     </ul>
+    <button id="delete-completed-btn" class="del-completed-btn">Delete completed tasks</button>
 
     <!-- 
       This is a template for the To-do list item. 
diff --git a/Sprint-3/todo-list/script.mjs b/Sprint-3/todo-list/script.mjs
index ba0b2ceae..4c82fc477 100644
--- a/Sprint-3/todo-list/script.mjs
+++ b/Sprint-3/todo-list/script.mjs
@@ -73,4 +73,9 @@ function createListItem(todo, index) {
   });
 
   return li;
-}
\ No newline at end of file
+}
+
+document.getElementById("delete-completed-btn").addEventListener("click", () => {
+  Todos.deleteCompleted(todos);
+  render();
+});
\ No newline at end of file
diff --git a/Sprint-3/todo-list/style.css b/Sprint-3/todo-list/style.css
index 535e91227..3c5808b78 100644
--- a/Sprint-3/todo-list/style.css
+++ b/Sprint-3/todo-list/style.css
@@ -101,6 +101,17 @@ h1 {
   color: red;
 }
 
+.del-completed-btn {
+  width: 100%;
+  height: 40px;
+  font-size: 16px;
+  background-color: #ef2121;
+  color: rgb(255, 255, 255);
+  border: none;
+  border-radius: 6px;
+  cursor: pointer;
+}
+
 .todo-item.completed .description {
   text-decoration: line-through;
   color: gray;
diff --git a/Sprint-3/todo-list/todos.mjs b/Sprint-3/todo-list/todos.mjs
index f17ab6a25..7c0396fd0 100644
--- a/Sprint-3/todo-list/todos.mjs
+++ b/Sprint-3/todo-list/todos.mjs
@@ -26,4 +26,14 @@ export function toggleCompletedOnTask(todos, taskIndex) {
   if (todos[taskIndex]) {
     todos[taskIndex].completed = !todos[taskIndex].completed;
   }
+}
+
+// Delete all completed tasks from the list
+export function deleteCompleted(todoList) {
+  // creating filtered object that holds not completed tasks
+  let filteredList = todoList.filter((element) => element.completed === false);
+  // removing all properties from todoList(todos)
+  todoList.length = 0;
+  // adding filtered tasks to todoList(todos)
+  filteredList.forEach((item) => todoList.push(item));
 }
\ No newline at end of file

From cc88f4359a7e3b0f23779c751348392a7840dcf1 Mon Sep 17 00:00:00 2001
From: Elisabeth-Matulian <elisabethmatulian@gmail.com>
Date: Thu, 26 Mar 2026 22:28:10 +0000
Subject: [PATCH 2/2] a test added / code refactoring

---
 Sprint-3/todo-list/todos.mjs      |  2 +-
 Sprint-3/todo-list/todos.test.mjs | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/Sprint-3/todo-list/todos.mjs b/Sprint-3/todo-list/todos.mjs
index 7c0396fd0..0b7b9f840 100644
--- a/Sprint-3/todo-list/todos.mjs
+++ b/Sprint-3/todo-list/todos.mjs
@@ -35,5 +35,5 @@ export function deleteCompleted(todoList) {
   // removing all properties from todoList(todos)
   todoList.length = 0;
   // adding filtered tasks to todoList(todos)
-  filteredList.forEach((item) => todoList.push(item));
+  todoList.push(...filteredList);
 }
\ No newline at end of file
diff --git a/Sprint-3/todo-list/todos.test.mjs b/Sprint-3/todo-list/todos.test.mjs
index bae7ae491..b3f52904d 100644
--- a/Sprint-3/todo-list/todos.test.mjs
+++ b/Sprint-3/todo-list/todos.test.mjs
@@ -130,3 +130,16 @@ describe("toggleCompletedOnTask()", () => {
   });
 });
 
+
+describe("deleteCompletedTasks()", () => {
+
+  test("Delete 2 Completed task", () => {
+    const todos = createMockTodos();
+    const todosBeforeDeletion = createMockTodos();
+    const lengthBeforeDeletion = todos.length;
+    Todos.deleteCompleted(todos);
+
+    expect(todos).toHaveLength(lengthBeforeDeletion - 2);
+  });
+
+});