Refactor deleteCompleted with filter and fix package.json metadata

MouawiaElkhalifa · MouawiaElkhalifa · commit faa8d1cc5bbd · 2026-04-08T12:24:21.000+01:00
diff --git a/Sprint-3/quote-generator/style.css b/Sprint-3/quote-generator/style.css
@@ -1 +1,30 @@
 /** Write your CSS in here **/
+body {
+  background-color: #f4f4f9;
+  display: flex;
+  justify-content: center;
+  align-items: center;
+  height: 100vh;
+  font-family: sans-serif;
+}
+
+#content {
+  background: white;
+  padding: 2rem;
+  border-radius: 10px;
+  box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
+  text-align: center;
+}
+
+button {
+  background-color: #4a90e2;
+  color: white;
+  border: none;
+  padding: 10px 20px;
+  border-radius: 5px;
+  cursor: pointer;
+}
+
+button:hover {
+  background-color: #357abd;
+}
diff --git a/Sprint-3/reading-list/index.html b/Sprint-3/reading-list/index.html
@@ -4,7 +4,7 @@
     <meta charset="utf-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <link rel="stylesheet" href="style.css" />
-    <title>Title here</title>
+    <title>Reading List</title>
   </head>
   <body>
     <div id="content">
diff --git a/Sprint-3/todo-list/package.json b/Sprint-3/todo-list/package.json
@@ -4,7 +4,20 @@
   "type": "module",
   "license": "CC-BY-SA-4.0",
   "description": "You must update this package",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/CodeYourFuture/CYF-Coursework-Template.git"
+  },
+  "bugs": {
+    "url": "https://github.com/CodeYourFuture/CYF-Coursework-Template/issues"
+  },
+  "homepage": "https://github.com/CodeYourFuture/CYF-Coursework-Template#readme",
   "scripts": {
-    "serve": "http-server"
+    "serve": "http-server",
+    "test": "NODE_OPTIONS=--experimental-vm-modules jest"
+  },
+  "devDependencies": {
+    "http-server": "^14.1.1",
+    "jest": "^30.0.4"
   }
 }
diff --git a/Sprint-3/todo-list/todos.mjs b/Sprint-3/todo-list/todos.mjs
@@ -27,12 +27,11 @@ export function toggleCompletedOnTask(todos, taskIndex) {
   }
 }
 
-// Remove all tasks that are marked as completed
+// Remove all tasks that are marked as completed using .filter()
 export function deleteCompleted(todos) {
-  for (let i = todos.length - 1; i >= 0; i--) {
-    if (todos[i].completed) {
-      todos.splice(i, 1);
-    }
-  }
-}
-
+  const incompleteTasks = todos.filter(todo => todo.completed === false);
+  
+  // Clear the original array and fill it with only incomplete tasks
+  todos.length = 0;
+  todos.push(...incompleteTasks);
+}
