[15/28] Оцените работу пожалуйста.

Курсы §13. Мастерская: создаём меню
1 
html,
body {
    margin: 0;
    padding: 0;
}

body {
    width: 280px;
    min-height: 280px;
    padding: 10px;
    font-family: "Arial", sans-serif;
    font-size: 14px;
    line-height: 1.4;
    background: #ecf0f1;
}

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

ul { list-style-type: none; } /* убирает маркер у ссылки Курсы */

.active ul {
    border-bottom: 2px solid #2c3e50;
    background-color: #fff;
    padding: 10px 0;
    list-style-type: none; 
}

.active li a {
    display: block;
    color: #2980b9;
    text-decoration: underline;
    padding: 5px 15px;
}

.active > a {
    display: block;
    background-color: #1abc9c;
    color: white;
    padding: 10px 15px;
    text-decoration: none;
    border-bottom: 2px solid #2c3e50;
}

.active ~ li {
    background-color: #2980b9;
    padding: 10px 15px;
    border-bottom: 2px solid #2c3e50;
}

.active ~ li a {
    display: block;
    color: #fff;
    text-decoration: none;
}
2

Усложнили вы себе задачу…
Можно гораздо короче. Учитесь выделять общие для элементов стили, например, все ссылки в этом задании у вас блочные. Об этом можно было написать всего лишь раз. Вы об этом браузеру сказали трижды, мало ли вдруг забудет…
Смотрите свой вариант и сравнивайте. Задавайте вопросы, если что-то непонятно.
html,
body {
margin: 0;
padding: 0;
}

body {
    width: 280px;
    min-height: 280px;
    padding: 10px;
    font-family: "Arial", sans-serif;
    font-size: 14px;
    line-height: 1.4;
    background: #ecf0f1;
}

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

ul { list-style-type: none; } /* убирает маркер у ссылки Курсы (не только, у всех)*/

.active ul {
    border-bottom: 2px solid #2c3e50;
    background-color: #fff;
    padding: 10px 0;
   /* list-style-type: none; */
}

.active li a {
   /*display: block;*/
     color: #2980b9;
     text-decoration: underline;
     padding: 5px 15px;
     background-color: #fff; /*цвет ссылок подменю*/
     border: none;/*в подменю ссылки не подчеркнуты*/
}

a /*общие стили для всех ссылок: главного меню и подменю*/ {
    display: block;
    /*background-color: #1abc9c;*/
    background-color: #2980b9;
    color: #fff;
    padding: 10px 15px;
    text-decoration: none;
    border-bottom: 2px solid #2c3e50;
}
.active a {
    background-color: #1abc9c;/*фон активного пункта*/
}

/*.active ~ li {
    background-color: #2980b9;
    padding: 10px 15px;
    border-bottom: 2px solid #2c3e50;
}*/

/*.active ~ li a {
    display: block;
    color: #fff;
    text-decoration: none;
}*/
1 лайк
3

Благодарю, так и знал, что навертел чёрти чего. И чего мне не пришло в голову, задать общий стиль для ссылок.

4

Теперь будет приходить)

1 лайк