Hola.
Bueno para empezar no veo la estructura del HTML, pero ya te puedo adelantar que para un menú mejor usa una lista en vez de parrafo
En segundo lugar estas juntando selectores, pero no se si para aplicarles las misma regla (entonces les faltaría una coma entre cada selector) o porque son selectores descendientes, lo cual no se si es necesario del todo...
Ayudaría que pusiera el código HTML para ver sobre que aplicas esas reglas.