[warren buffet]

Все у него правильно, кроме этого

this.classList.toggle('done');

Зачем this, если передается элемент. Его и бери

function toToggle(el) {el.classList.toggle('done');}

Ну, и смысл этой функции не ясен. Тогда уж toggleDone(){}