Где ошибка в LocalStorage для ToDo List на JS?

Question

[Roman Jolud]

Сделал LocalStorage для своего To do list , но информация слетает при обновлении странички.
Что делать , уже около 2 часов сижу , код снизу:

HTML код:

<div class="header">
        <h1>To do list</h1>
        <h4></h4>
        <input type="text" placeholder="Your task here" id="toDoEl">
        <span class="addBtn" onclick="newElement()">Add</span>
    </div>
    <ul id="list">

    </ul>

var list = document.querySelector('ul');
var todos;
function toLocal(){
    todos = list.innerHTML;
    localStorage.setItem('todos',todos);
}
list.addEventListener('click', function (ev) {
    if(ev.target.tagName === "LI") {
       ev.target.classList.toggle('checked');
        toLocal();
    } else if(ev.target.tagName === "SPAN") {
       var div = ev.target.parentNode;
       div.remove();
        toLocal();
    }
}, false);

function newElement() {
    var li = document.createElement('li');
    var inputValue = document.getElementById('toDoEl').value;
    var t = document.createTextNode(inputValue);
    li.appendChild(t);
    if(inputValue == "") {
       alert("Введите ваше дело!");
    } else {
       document.getElementById('list').appendChild(li);
    }
    document.getElementById('toDoEl').value = "";
    var span = document.createElement('SPAN');
    var txt = document.createTextNode("X");
    span.className = "close";
    span.appendChild(txt);
    li.appendChild(span);
 toLocal();
}

if(localStorage.getItem('todos')){
    list.innerHTML = localStorage.setItem('todos',todos);
}

Answer 1

[Aleksandr]

if(localStorage.getItem('todos')){
    // у тебя
    // list.innerHTML = localStorage.setItem('todos',todos);
    // должно быть
    list.innerHTML = localStorage.getItem('todos');
}

Вместо setItem надо getItem

ps: Отдыхать надо :)

Comments

[Андрей Буров]

у getItem один аргумент.

Answer 2

[Андрей Буров]

list.innerHTML = localStorage.setItem('todos',todos);

Может все же getItem?