Skip to content
This repository was archived by the owner on Oct 26, 2020. It is now read-only.

Orhan-week-7 #990

Open
wants to merge 1 commit into
base: manchester3
Choose a base branch
from
Open

Conversation

OBakir90
Copy link

Your Details

Your Name:
Your City:
Your Slack Name:

Homework Details

Module:
Week:

Copy link

@Ashaghel Ashaghel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work


## 3. What should you do if you get stuck?

<!-- Write your answer here -->
I use debugging. if still doesnt work i do some search and still cant sort it out ask someone else

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect answers

delBut.id = i + 5;
cell5.appendChild(delBut);
delBut.className = "btn btn-warning";
delBut.innerHTML = "Delete";
delBut.addEventListener("clicks", function () {
delBut.addEventListener("click", function () {
alert(`You've deleted title: ${myLibrary[i].title}`);
myLibrary.splice(i, 1);
render();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect work again !

@Ashaghel Ashaghel added the reviewed A mentor has reviewed this PR label Jun 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
reviewed A mentor has reviewed this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants