[28/28] Что можно подправить? [100%]

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

Добрый вечер. Оцените код, пожалуйста.

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

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

.main-menu {
    padding: 0;
    margin: 0;
    background-color: #2980b9;
    width: 200px;
}

.main-menu li {
    position: relative;
    padding: 10px 0 10px 15px;
    border-bottom: 2px solid #2c3e50;
}

.main-menu li a {
    color: white;
    text-decoration: none;    
}

.main-menu .sub-menu {
    position: absolute;
    left: 180px;
    list-style: none;
    background-color: #1abc9c;
    z-index: 10;
    top: 0;
    padding: 0;
}

.main-menu .sub-menu li {
    border-bottom: 2px solid #2ecc71;
    width: 185px;
}


.main-menu .opened {
    background-color: #1abc9c;
}
2

Старайтесь использовать однообразные стили. А в целом - хорошо.

color: white;

3

Всё же есть замечания. Внутренние отступы лучше задавать ссылкам, а ширину для списка подменю, а не пунктов li.

4

Со вторым замечанием соглашусь, а вот с первым разрешите поспорить (ну или проясните, пожалуйста) - разве не удобнее делать отступы в блочном по дефолту элементе, чем делать ссылку блочной, а потом уже в ней делать отступ?

5

Спасибо!

6

когда мы задаем отступы ссылке - мы увеличиваем “кликабельную” область ссылки, т.е. делаем её удобней для пользователя

1 лайк
7

Спасибо за объяснение!