Peer-to-Peer Code Review

[okoyecharles]

Your Project looks great so far, Although there are some changes I believe would make it look better.

Changes Required ♻

• [ ] As you can see below, I noticed you used this whereas there was a cleaner way to do renew the indexes. https://github.com/piesieomane/To-do-list/blob/ad357ecd4133d9e844e141efa40a0bd62afa6709/src/clear.js#L17-L23

I reccomend you make use of the index functionality in the forEach loop, something like this.

 newArr.forEach((element, i) => { 
   element.index = i; 
 }); 
 task = newArr; 

---

• [ ] SUGGESTION: I also tried checking the todos on your list, although they worked they did not give the todos the corresponding style. A checked todo should look like this instead of as it looked.

• [ ] As you see below I believe that its better HTML practice to use the main tags and section tag instead of nested divs https://github.com/piesieomane/To-do-list/blob/ad357ecd4133d9e844e141efa40a0bd62afa6709/src/index.html#L14-L37

The document flow should look like this for better readability.

<main class="container">
     <!--- First Section--->
    <section>

    </section>

     <!--- Second Section--->
    <section>

    </section>

     <!--- Third Section--->
    <section>

    </section>
</main>

[piesieomane]

Thanks for the review, I will make the necessary changes to the available issues you made