RusRogo / MyFirstRep

Здесь хранятся проекты
GNU General Public License v3.0
0 stars 0 forks source link

Code review #1

Open ZvezdocheTag opened 6 years ago

ZvezdocheTag commented 6 years ago

html

По поводу наименования, для начало норм, но зачастую все проекты используют какуют то методологию, одна из самых популярных БЭМ. Наименования также должны быть осмысленные.

Также при написании кода, неважно html, css, js важна последовательность (code style). К примеру у тебя header описан нативным тегом

https://github.com/RusRogo/MyFirstRep/blob/a05c10cfe84733389cec0c514b242af3525fac7a/git/index.html#L11, а footer https://github.com/RusRogo/MyFirstRep/blob/a05c10cfe84733389cec0c514b242af3525fac7a/git/index.html#L47 у тебя объявлен через класс, можно было бы использовать тег <footer>

css

  1. в папке /css лежит reset.css почему ты его не подключил ?
  2. использование float: left|right, и display: inline-block сейчас не актуально для разметки старницы, посмотри flexbox проще и удобней инструмент.
  3. font-family: Myriad Pro; хорошая практика добавлять еще тип шрифта -> font-family: Myriad Pro, sans-serif, чутка подробнее можешь тут глянуть (http://htmlbook.ru/css/font-family)
  4. При :hover можешь добавить css transition будет смотреться приятней

В общем норм для старту 👍 Если есть вопросы по теме/этой верстке пиши тут.

RusRogo commented 6 years ago

Исправил под Flexbox, применил БЭМ (постарался применить)

RusRogo commented 6 years ago

папка с исправлением - "HTML Module 3 -- corrected 22.,09"

ZvezdocheTag commented 6 years ago

Если работаешь с гитом, можно не менять название дирректорий и папок, все трекается гитом по коммитам, и видно что где ))