[15/28] [100%] Требуется оценка


#1

Требуется конструктивная критика с аргументированием своего мнения.

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 {
    padding: 0;
    margin: 0;
    width: 200px;   
}

.main-menu li {
    list-style: none;
}

.main-menu > li > a {
    border-bottom: 2px solid #2c3e50;  
}

.main-menu a {
    display: block;
    color: white;
    text-decoration: none;
    padding: 10px 15px;
    background: #2980b9;
}

.main-menu .active {
    border-bottom: 2px solid #2c3e50;
}

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

.main-menu li ul {
    padding: 10px 0; 
    background-color: white;
}

.main-menu li li a {
    background: white;
    color: #2980b9;
    text-decoration: underline;
    padding: 5px 15px;
}

#2

Много лишнего написано, можно все это уложить в более короткий вариант.


#4

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

}
.main-menu > li {
border-bottom: 2px solid #2c3e50;
list-style: none;

}

.main-menu a {
display: block;
padding: 10px 15px;
color: #ffffff;
text-decoration: none;
}
.main-menu > .active > a {
border-bottom: 2px solid #2c3e50;
background: #1abc9c;
}
.main-menu ul {
padding: 10px 0px;
background: #ffffff;
}
.main-menu li li a {
padding: 5px 0;
padding-left: 15px;
color: #2980b9;
text-decoration: underline;
}


#5

Правда, я еще обратил внимание, что если убрать лист-стаил нон, то визуально ничего в меню не меняется. Это ошибка?


#6

Меняется, только вы не видите. Добавьте внешний отступ слева списку, чтобы разглядеть маркеры)


#7

Точно, сам же вчера проверял и заметил это. За ночь забыл, обидно.