15/28, Испытание: многоуровневое меню. Интересно стороннее мнение, может можно сделать лучше/красивее.

Курсы §13. Мастерская: создаём меню
1

Сделано на 100%, но, так как сколько людей - столько мнений, то интересно Ваши мнения. Может можно сделать лучше.

html,
body {
margin: 0;
padding: 0;
}

body {
width: 280px;
min-height: 280px;
padding: 10px;

font-size: 14px;
line-height: 1.4;
font-family: “Arial”, sans-serif;

background-color: #ecf0f1;
}

.main-menu {
padding: 0;
margin: 0;
width: 200px;
}

.main-menu a {
display: block;
text-decoration: none;
}

.main-menu .active > a {
padding: 10px 15px;
background-color: #1abc9c;
color: #ffffff;
}

.main-menu ul {
list-style: none;
background-color: #ffffff;
padding: 10px 0;
margin: 0;
border-bottom: 2px solid #2c3e50;
}

.main-menu .active > ul a {
padding: 5px 15px;
background-color: #ffffff;
color: #2980b9;
text-decoration: underline;
}

.main-menu > li > a {
padding: 10px 15px;
color: #ffffff;
background-color: #2980b9;
border-bottom: 2px solid #2c3e50;
}

2

Могу предложить следующее:

.main-menu .active > ul a {
padding: 5px 15px;
background-color: #ffffff;
color: #2980b9;
text-decoration: underline;
}
…это цвет определён в .main-menu ul

В main-menu (не знаю на что это влияет) маркеры тоже лучше скрыть – добавить list-style: none;

.main-menu .active > a {
padding: 10px 15px;
background-color: #1abc9c;
color: #ffffff;
}
…эти свойства у Вас в .main-menu > li > a

text-decoration: none; я бы перенёс из .main-menu a в .main-menu > li > a ,
тогда в .main-menu .active > ul a можно убрать text-decoration: underline;

3

Отличное замечание, видно, что Вы уже набили руку, исправляя чужой код. Судя по форуму Вы самый активный житель)

вот тут сомнение. а надо ли? по сути это нам тут ничем не поможет.

не поспоришь.

и опять 5 баллов! спасибо, красиво)