🚀 go-pugleaf

> 3. Ponadto z nazwy w/w funkcji wnioskuję iż naruszasz strukturę DOM by 
> coś zanimować. Nie wiem jak konkretnie animujesz więc tylko zgaduję. 
> Jeśli np. używasz innerHTML gdzieś dalej do zapisu, to wtedy takie cuda 
> mogą się dziać. Nie należy używać do tego celu innerHTML bo jest 
> strasznie wolne gdyż wymusza odświeżanie _CAŁEJ_ struktury DOM. Wtedy 
> okresowo możesz tracić dostęp do DOM mimo iż pozornie wydaje Ci się, że 
> nic się w niej nie zmienia. To dość częsty błąd początkujących 
> programistów JS a i zaawansowani czasem też o tym nie wiedzą.
> 

// Animuj literkę w napisie tytułowym:

var letterIndex = -1;
var mainTitle;

function animateMainTitle() {
     'use strict';
     if (mainTitle === undefined) {
         mainTitle = document.getElementById("main_title").innerHTML;
     }
     letterIndex = (letterIndex + 1) % mainTitle.length;
     var s = mainTitle.substr(0, letterIndex) +
             '<span class="selected_letter">' +
             mainTitle.substr(letterIndex, 1) +
             '</span>' +
             mainTitle.substr(letterIndex + 1);
     document.getElementById("main_title").innerHTML = s;
     setTimeout(animateMainTitle, 100);
}

Czy gdybym zamiast innerHTML umieścił wszystkie literki w span'ach i 
ustawiał tylko atrybut class odpowiedniej literki to byłoby lepiej?