Skip to content

Commit e84f17c

Browse files
committed
Refactor render function to improve button naming consistency and simplify event listener setup
1 parent 2b070dc commit e84f17c

1 file changed

Lines changed: 10 additions & 12 deletions

File tree

debugging/book-library/script.js

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -80,25 +80,23 @@ function render() {
8080
pagesCell.innerHTML = myLibrary[i].pages;
8181

8282
//add and wait for action for read/unread button
83-
let toggleReadBTN = document.createElement("button");
84-
toggleReadBTN.id = i;
85-
toggleReadBTN.className = "btn btn-success";
86-
wasReadCell.appendChild(toggleReadBTN);
83+
let toggleReadButton = document.createElement("button");
84+
toggleReadButton.className = "btn btn-success";
85+
wasReadCell.appendChild(toggleReadButton);
8786
let readStatus = myLibrary[i].check ? "Yes" : "No";
88-
toggleReadBTN.innerText = readStatus;
87+
toggleReadButton.innerText = readStatus;
8988

90-
toggleReadBTN.addEventListener("click", function () {
89+
toggleReadButton.addEventListener("click", function () {
9190
myLibrary[i].check = !myLibrary[i].check;
9291
render();
9392
});
9493

9594
//add delete button to every row and render again
96-
let deleteBtn = document.createElement("button");
97-
deleteBtn.id = i + 5;
98-
deleteCell.appendChild(deleteBtn);
99-
deleteBtn.className = "btn btn-warning";
100-
deleteBtn.innerHTML = "Delete";
101-
deleteBtn.addEventListener("click", function () {
95+
let deleteButton = document.createElement("button");
96+
deleteCell.appendChild(deleteButton);
97+
deleteButton.className = "btn btn-warning";
98+
deleteButton.innerHTML = "Delete";
99+
deleteButton.addEventListener("click", function () {
102100
alert(`You've deleted title: ${myLibrary[i].title}`);
103101
myLibrary.splice(i, 1);
104102
render();

0 commit comments

Comments
 (0)