Skip to content

Commit 7a8d508

Browse files
committed
book library
1 parent d3c51c0 commit 7a8d508

File tree

1 file changed

+16
-9
lines changed

1 file changed

+16
-9
lines changed

debugging/book-library/script.js

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,14 +31,21 @@ function submit() {
3131
if (
3232
title.value == null ||
3333
title.value == "" ||
34+
author.value == null ||
35+
author.value == "" ||
3436
pages.value == null ||
3537
pages.value == ""
3638
) {
3739
alert("Please fill all fields!");
3840
return false;
3941
} else {
40-
let book = new Book(title.value, title.value, pages.value, check.checked);
41-
library.push(book);
42+
let book = new Book(title.value, author.value, pages.value, check.checked);
43+
myLibrary.push(book);
44+
// Clear the form after successful submission
45+
title.value = "";
46+
author.value = "";
47+
pages.value = "";
48+
check.checked = false;
4249
render();
4350
}
4451
}
@@ -54,7 +61,7 @@ function render() {
5461
let table = document.getElementById("display");
5562
let rowsNumber = table.rows.length;
5663
//delete old table
57-
for (let n = rowsNumber - 1; n > 0; n-- {
64+
for (let n = rowsNumber - 1; n > 0; n--) {
5865
table.deleteRow(n);
5966
}
6067
//insert updated row and cells
@@ -76,7 +83,7 @@ function render() {
7683
changeBut.className = "btn btn-success";
7784
wasReadCell.appendChild(changeBut);
7885
let readStatus = "";
79-
if (myLibrary[i].check == false) {
86+
if (myLibrary[i].check == true) {
8087
readStatus = "Yes";
8188
} else {
8289
readStatus = "No";
@@ -90,11 +97,11 @@ function render() {
9097

9198
//add delete button to every row and render again
9299
let delButton = document.createElement("button");
93-
delBut.id = i + 5;
94-
deleteCell.appendChild(delBut);
95-
delBut.className = "btn btn-warning";
96-
delBut.innerHTML = "Delete";
97-
delBut.addEventListener("clicks", function () {
100+
delButton.id = i + 5;
101+
deleteCell.appendChild(delButton);
102+
delButton.className = "btn btn-warning";
103+
delButton.innerHTML = "Delete";
104+
delButton.addEventListener("click", function () {
98105
alert(`You've deleted title: ${myLibrary[i].title}`);
99106
myLibrary.splice(i, 1);
100107
render();

0 commit comments

Comments
 (0)