Начало хорошие :)
Советы на будущие:
Не используй
position: absolute без необходимости.
top, right, left, bottom - Это очень плохо.
margin - это отступ от блока.
Например есть горизонтальный список и необходимо у каждого элемента списка сделать отступ с лева и права.
...
nav a:first-child {
padding-left: 19px;
margin-left: 128px;
}
К примеру у
bootstrap есть класс
container который центрует содержание страницы.
...
.hero-shirt {
margin: 67px 0 0 78px;
}
Здесь нужно было использовать
padding, так как по логике элемент должен идти внуторь, а не изнутри.
...
Так же очень важно использовать
модульную сетку grid. Она помогает позиционировать элементы на страницы.
...
nav a:after {
content: "|";
padding-left: 18px;
}
Лучше было бы использовать как то так:
nav {
/* ... */
}
nav ul {
text-align: center;
letter-spacing: -.3em; /* убирает пробелы между inline */
}
nav li {
display: inline-block;
letter-spacing: normal; /* нормализует пробелы */
}
nav li + li {
border-left: 1px solid white;
}
nav a {
display: block;
color: white;
padding: 0 10px; /* Но лучше использовать EM */
}
Не используй локальные шрифты!
Ну для начало, это не легально..
Есть специальные сервисы по хранению шрифтов.
Они бесплатны и не нарушаешь права/лицензию и т.д.
Google Fonts,
fonts.com
А вообще ты умничка и у тебя очень здорово получилась первая работа, желаю дальнейших успехов ))
Простой
Средний
Простой
