Первый макет верстки, какие ошибки?

Question

[Артем Масиков]

https://codepen.io/sport_defense/pen/xxKwByP
Добрый вечер, в дальнейшем планирую перенести сайт на хостинг, как будет полностью готов (работы много). Интересуют такие вопросы по моей верстке,
Правильность позиционирование элементов
Правильность указания ссылок на картинки
Количество написанного кода (может можно в каких-то местах сделать короче?) или по другому.

Answer 1

[Артем]

1. На 92 строке display: block; можно удалить, так как за ним сразу идет display: flex; который перекрывает его.

2. Желательно подровнять отступы, чтобы они были более менее одинаковыми, чтобы гармоничнее смотрелось, т.е. если между "Комплексы упражнений с пользователями" и секцией с текстом 35px, то между секцией с текстом и div-map-full тоже лучше сделать 35px. Если с начала блока color-cart до "Комплексы упражнений с пользователями" 117px, то с div-map-full до конца блока лучше тоже сделать 117px. У ваc немного странно проставлены отступы в начале и конце, padding-bottom проставлен в color-cart, а padding-top в section-training-complex-general-text, лучше сразу тогда прописать в color-cart padding-top: 117px; и padding-bottom: 117px; или просто padding: 117px 0;

3. У вас текст везде обернут в div'ы, это неправильно, для них есть свои теги, для обычного текста - тег p, для заголовков h1, h2, h3 и т.д.

4. Чтобы не было лишних отступов, как в футере например, пропишите в начале *{margin: 0; padding: 0;} или можно просто подключить файл с reset.css, подробнее про это например тут: https://habr.com/ru/post/45296/

Comments

[Артем Масиков]

Спасибо за развернутый комментарий, у меня подключен normolize, остальное из вышеуказанного исправлю завтра. И покажу что получилось. Сейчас начал изучать адаптив и понял, что для моего кода не подойдет, так как он очень плохо написано верно?

[Артем]

Артем Масиков, просто оборачиваем содержимое сайта в div с class="container" и в медиа запросах меняем ширину контейнера в зависимости от ширины экрана и меняем свойства определенных блоков, которые должны как-то изменяться, чтобы адекватно отображаться при заданной ширине контейнера.

[Максим]

Артем Масиков, глобально обнулять внешние и внутренние отступы это плохая практика ..

[Артем Масиков]

Максим Ленский, Потому что поедет весь сайт?)

[Максим]

Артем Масиков, ни куда он не поедет ..но разработчики браузеров вкрутили стили для тегов т.е то каким образом по дефолту они должны выглядеть а вы эти стили собъёте на ноль ...
сбрасывать глобально можно только в том случае если это действительно требуется

[Артем Масиков]

Артем, Обновил код, можете посмотреть.