search

ascent2009 / repair-design-project

First student project during the course SkillFactory training
0 stars 0 forks source link

Comments #1

Open sffr-dev opened 4 years ago

sffr-dev commented 4 years ago

  1. https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L140 https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L147 Это лишнее. У вас задана ширина и высота. А текст у кнопки центруется по-умолчанию. Паддинги только мешают вертикальной центровке

  2. https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L192 Из-за этого едет верстка при наведении на средний и последний пункты. Либо убрать, либо перенести в основной стиль .project-list__item a

  3. https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L304 https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L475 https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L835 Вот это один и тот же стиль

  4. Нет класса demo-container. Из-за этого на широком мониторе расползается блок.

  5. https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/index.html#L415-L416 Класс можно повесить сразу на форму, а див удалить. Плюс ограничить ширину формы. См. макет

  6. Так как хедер у вас стики (это круто!), то футер можно не делать. Он тут лишний

  7. https://github.com/ascent2009/repair-design-project/blob/81185c950c49d68b86fd25e42506802240aacc2a/CSS/style.css#L1110 Нужно сделать либо доп. стили от 1000 до 1440, либо поменять брейкпойнт на 1440. См. верстку на разрешении 1024px

  8. Стилиация чекбокса: http://dimox.name/custom-checkboxes-and-radio-buttons-using-css-only/

В остальном - все круто! Вы молодец!

ascent2009 commented 4 years ago

Подправил верстку и стили в соответствие с рекомендациями ментора, не получилось внедрить интерактивную карту Goggle - она вставляется статичной, как картинка