search

MarcelHarabagiu / Todo-list

create elements in a todo list
0 stars 0 forks source link

dom and model (will pair program for this) #7

Open HelloMihai opened 3 years ago

HelloMihai commented 3 years ago

you are adding data to the dom then using it to manipulate local storage

this is a bad concept

you need a data model in your JS that is represented in the DOM, not use the DOM to store your data model

MarcelHarabagiu commented 3 years ago

related to issue #5
reliance on text match is too error prone https://github.com/MarcelHarabagiu/Todo-list/issues/5#issue-786282863 now data is added to an array as multiple objects