Я так то тоже не спец, но всё же,
1. Не верстай на float, конечно это дело вкуса, но вобще сейчас верстают на flexbox ил grid, они удобнее.
2. Не давай блокам классы block, item, я на самом деле тоже так делал раньше, класс должен описывать блок. Напиши section_inner, shop_list_item.
3. Незнаю насколько ты новичок, но если вдруг незнаешь, прочитай про normalize.css и reset.css, они сбрасывают предустановленные стили браузера, которые обычно мешают. Лично я пользуюсь normalize, впринципе, любой из них настраиваешь под себя, и используешь в будущих проектах.
4. Старайся не использовать <br>, есть случаи, когда какой-то девиз, лозунг, что-то ещё должны переноситься на определенном слове, обычный же текст оставляй как есть, ты же потом будешь адаптировать сайт под телефоны и планшеты, переставлять <br>запаришься. Лучше поставь max-width, этого хватит.
Ну и по мелочам, указывай a:hover, cursor:pointer, люди должны понимать, что это ссылка. В комментариях css пиши не section номер, а там допустим header, main, aside, popup, такое. Ну и про БЭМ, это грубо говоря подсказка, как именовать классы, и структурировать html, тоже советую почитать.