[15/28] Пожалуйста, оцените код.

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

Пройдено на 100%, оцените мой код.

Посмотрел другие работы и там делали через “display:block;” Почему так? и удобнее ли делать этим способом?

И если есть пример “идеального” кода этого испытания, скиньте тему или свой вариант)

https://jsfiddle.net/4ed3urgk/4/

2

Обратите внимание на поведение курсора в Вашем меню: выбрать элемент меню можно только кликая по надписи.

https://htmlacademy.ru/courses/50/run/6
https://htmlacademy.ru/courses/50/run/7
А, здесь нас научили выбирать элемент не только по надписи, но и по всей площади от бордера до бордера.

CSS 
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;
    list-style: none;
    background: #fff;    
}

.main-menu a {
    display: block;
}

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

.main-menu > li > a:hover {
    background: #1abc9c;
}

.main-menu .active > a {
    /*background: #1abc9c;  /* РАСкомментировать для прохождения испытания */
}

.main-menu ul {
    display: none;         /* ЗАкомментировать для прохождения испытания */
    margin: 0;
    padding:10px 0;
    list-style: none;
    border-bottom: 2px solid #2c3e50;
}
.main-menu > li:hover ul {
    display: block;
}
.main-menu ul a {
    color: #2980b9;
    padding: 5px 15px;     
}

.main-menu ul a:hover {
    text-decoration: none;
}
1 лайк
3

Спасибо за ответ, буду знать, и в моем случае(при добавлении display: block; ) пришлось перенести внутренние отступы из “li” в “a”, иначе блок со ссылкой не занимал всю ширину “li”(он сжимался от внутренних отступов “li”).