Open VadimZhiltsov opened 8 years ago
CSS очень избыточен, очень много не нужных стилей: 1)
.top_menu ul, .bottom_menu ul{ display: table-row; } .top_menu ul li, .bottom_menu ul li{ display: table-cell; width: auto; vertical-align: middle; text-align: center; text-align: left; padding-right: 48px; }
это можно преобразовать в
.top_menu ul li, .bottom_menu ul li{ display: inline-block; padding-right: 48px; }
2) Здесь стили с height ни на что не влияют
.content_wrapper { height: 100%;- min-height: 600px; margin: 40px auto; }
3) Дублирующийся стиль
.main_wrapper{ margin: 0 auto; width: 77.6%; max-width: 1060px; min-width: 796px;
4) Можно использовать более семантичные теги, чем div. nav - для навигационного меню aside - боковое меню
5) Ибыточные теги. Старайтесь не создавать лишний раз тег, это усложняет поддержку кода. лишний div
<div class="photo_item_wrapper"> <div class="img>">** <img src="img/bycicle.jpg" alt="bycicle"> </div> </div>
Либо лишний p, либо div:
<div class="item_description"> <p> .... </p> </div>
лишний div:
<footer> <div class="bottom_menu"> <ul>
6) Лишние класы на li тегах в
Зачёт
CSS очень избыточен, очень много не нужных стилей: 1)
это можно преобразовать в
2) Здесь стили с height ни на что не влияют
3) Дублирующийся стиль
4) Можно использовать более семантичные теги, чем div. nav - для навигационного меню aside - боковое меню
5) Ибыточные теги. Старайтесь не создавать лишний раз тег, это усложняет поддержку кода. лишний div
Либо лишний p, либо div:
лишний div:
6) Лишние класы на li тегах в
Зачёт