Diven i menyn ger endast en margin som man skulle kunna ta bort och lägga samma css på länken.

Du har lagt in 4 olika divar för ett och samma element. Finns absolut ingen anledning för detta. Experimentera runt så du får bort all onödig kod.

Jaja, jag får tacka för tips och kritik.
Har försökt förut men får försöka igen, men daniel om jag ska ha en länk i texten senare kommer den sitta helt konstigt.

Om du bara sätter margin och float: left; på a-elementet så behöver du inte ha en div i din li.

Din kod
<li> (bakgrundsbild, bredd, höjd)
<div> (margin, padding)
<a> (färg, textstil, etc)

Det du kan göra
<li> (bakgrundsbild, bredd, höjd)
<a> (färg, textstil, margin, padding, float)

Samma resultat.

Sedan har du, som föregående nämner, flertalet onödiga divvar.

Vad är syftet med sidan? Och varför?