Widzę, że dużo poprawiłeś, zaczyna to jakoś wyglądać (przynajmniej od strony kodu). Poprawiłeś błędy techniczne? Duży plus. Jest jeszcze strona semantyczna (takie modne tu ostatnio słowo - semantyka), np. masz w kodzie:
Kod
<ul class="nav">
<li class="nav">cos</li>
<li class="nav">ktos</li>
<li class="nav">jakos</li>
</ul>
Po co ta sama klasa dla każdego elementu listy? Miałoby to sens, jakbyś chciał formatować kazdy element list oddzielnie - wtedy dajesz:
Kod
<ul class="nav">
<li class="nav1">cos</li>
<li class="nav2">ktos</li>
<li class="nav3">jakos</li>
</ul>
itd.
W TWOIM wypadku jeśli chcesz sformatować <li> w <ul> to w arkuszu stylów dajesz:
Kod
ul.nav li {
}
A jeśli masz tylko jedną listę na stronie to wystarczy:
Kod
li {
}
Warto jednak nadać ID dla unikatowych, jednorazowych bloków strony oraz klasy dla tych powtarzających się - w ten sposób ułatwiasz sobie późniejszą rozbudowę (np. dodanie drugiego menu). Jednak klasa dla każdego elementu to przesada - należy zachować równowagę