scrimba
Bootcamp code reviews
πŸ‘©πŸΌβ€πŸ« Review of Choreslist for Biljana
Go Pro!Bootcamp

Bootcamp

Study group

Collaborate with peers in your dedicated #study-group channel.

Code reviews

Submit projects for review using the /review command in your #code-reviews channel

πŸ‘©πŸΌβ€πŸ« Review of Choreslist for Biljana
AboutCommentsNotes
πŸ‘©πŸΌβ€πŸ« Review of Choreslist for Biljana
Expand for more info
readme.md
run
preview
console
# Code Review: ChoresList
### _Code by: Biljana ,_
### _Reviewer: Felicity Evans_

## What you did well
- had fun!
- used google fonts
- consistent class names - Tailwind inspired?

## Questions
I had alot of fun on this challenge, I was able to practice and solidify my understanding of manipulating the DOM pushing items into an array rendering a list of items with html via js. I learned how to use the Set object to eliminate duplicates. I attempted to delete each item individually but still didnt get it to work but I will try and figure it out after I have a better understanding on how to use the for loop. Overall I'm really happy with what i was able to achieve :)

## What could be improved?
- use flexbox
- add form container with display: flex
- set choresArray to be what is in local storage
let choresArray = JSON.parse(localStorage.getItem("chores"))
- when deleting clear localStorage also
- trying to set localstorage with an empty array - move code down
- not getting chores from local storage: localStorage.getItem()
- move addEventListenersToListItems() out of addBtn.addEventListener
- function to remove individual items
const ulClassList = document.querySelectorAll(".ul-el")

ulClassList.forEach(function(item) {
item.addEventListener("click", function(event){
console.log(event.target.innerText)
const newArray = choresArray.filter(function(chore) {
return chore !== event.target.innerText
} )
choresArray = newArray
renderChoreListItems()
})
})




Console
/index.html
-9:47