jag kollade lite snabbt på din kod och insåg att du gör saker onödigt komplicerat, eller rättare sagt du gör det "så lätt så det blir fel"
tex så har du en klass för varje linje mellan meny-alternativen, varför då?
alla ser ju likadana ut, så där hade du enkelt kunnat ha enbart <div class="linje">
Samma "dumhet" har du på själva "meny-elementen", en klass för varje alternativ men de ser alla likadana ut. Du skulle med andra ord kunna lägga den css-koden på ETT ställe istället för 4
Bäst hade ju iofs varit om menyn var en lista (<ul>) och linjen löses med en border på <li> elementet och en special-klass för sista elementet så att den inte får en border